summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2014-01-16fsl_dce: Fix tester(s) build warningsEmil Medve
drivers/staging/fsl_dce/tests/common/test_frame_helpers.c: In function ‘print_dce_data_list’: drivers/staging/fsl_dce/tests/common/test_frame_helpers.c:272:4: warning: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘size_t’ [-Wformat=] pr_info(" cpumem = %p, size = %lu\n", ^ drivers/staging/fsl_dce/tests/common/test_frame_helpers.c:283:3: warning: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘size_t’ [-Wformat=] pr_info(" cpumem = %p, size = %lu\n", ^ drivers/staging/fsl_dce/tests/performance_simple/dce_perf_simple.c: In function ‘copy_bman_output_to_buffer’: drivers/staging/fsl_dce/tests/performance_simple/dce_perf_simple.c:388:4: warning: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 2 has type ‘size_t’ [-Wformat=] pr_info("total frame length != calulated length (%lu)" ^ drivers/staging/fsl_dce/tests/performance_simple/dce_perf_simple.c:398:4: warning: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 2 has type ‘size_t’ [-Wformat=] pr_info("sg length != frame output length (%lu) (%u)\n", ^ drivers/staging/fsl_dce/tests/performance_simple/dce_perf_simple.c: In function ‘write_file’: drivers/staging/fsl_dce/tests/performance_simple/dce_perf_simple.c:522:3: warning: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘size_t’ [-Wformat=] pr_info("Error %d, data_len %lu\n", rc, data_len); ^ drivers/staging/fsl_dce/tests/performance_simple/dce_perf_simple.c: In function ‘dce_loopback_shutdown’: drivers/staging/fsl_dce/tests/performance_simple/dce_perf_simple.c:694:77: warning: comparison of distinct pointer types lacks a cast [enabled by default] do_div(dce_freq, 2); ^ drivers/staging/fsl_dce/tests/performance_simple/dce_perf_simple.c:694:2: warning: right shift count >= width of type [enabled by default] do_div(dce_freq, 2); ^ drivers/staging/fsl_dce/tests/performance_simple/dce_perf_simple.c:694:2: warning: passing argument 1 of ‘__div64_32’ from incompatible pointer type [enabled by default] In file included from arch/powerpc/include/asm/div64.h:1:0, from include/linux/kernel.h:111, from drivers/staging/fsl_dce/tests/performance_simple/dce_perf_simple.c:33: include/asm-generic/div64.h:35:17: note: expected ‘uint64_t *’ but argument is of type ‘uint32_t *’ extern uint32_t __div64_32(uint64_t *dividend, uint32_t divisor); ^ Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: If682239365c0d59bf01d7fc712cbfc410de0854c Reviewed-on: http://git.am.freescale.net:8181/8003 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Jeffrey Ladouceur <Jeffrey.Ladouceur@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-16crypto: caam - Add define for NULL encryptionAndrei Varvara
This define is required for making descriptors with NULL encryption. drivers/staging/fsl_dpa_offload/dpa_ipsec.c:45:355: error: ‘OP_PCL_IPSEC_NULL_ENC’ undeclared here (not in a function) struct ipsec_alg_suite ipsec_algs[] = IPSEC_ALGS; drivers/staging/fsl_dpa_offload/dpa_ipsec.c: In function ‘check_sa_params’: drivers/staging/fsl_dpa_offload/dpa_ipsec.c:2991:17: warning: comparison between pointer and integer [enabled by default] if (cipher_alg != OP_PCL_IPSEC_NULL_ENC && ^ Dropped during: b7c81aa Merge remote-tracking branch 'linus/master' into merge Signed-off-by: Andrei Varvara <andrei.varvara@freescale.com> Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: I61ef1a7d6bcf115f99f5160f76154c09e5f7f7da Reviewed-on: http://git.am.freescale.net:8181/7991 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-14fsl_qman: EXPORT_SYMBOL()s used by the debugfs support moduleEmil Medve
ERROR: "qman_ceetm_query_cq" [drivers/staging/fsl_qbman/qman_debugfs_interface.ko] undefined! ERROR: "qman_ceetm_get_xsfdr" [drivers/staging/fsl_qbman/qman_debugfs_interface.ko] undefined! ERROR: "qman_ceetm_query_ccgr" [drivers/staging/fsl_qbman/qman_debugfs_interface.ko] undefined! Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: I1c35c3a3d9228f46482e4a4cf771571251e848de Reviewed-on: http://git.am.freescale.net:8181/7930 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-14fsl_usdpaa: Remove UIO artifactsEmil Medve
Left behind by 'e9e4910 Cleanup USDPAA resources when a process exits' Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: If5237b8c583aa797e20247480bc4ffb54efee32b Reviewed-on: http://git.am.freescale.net:8181/7929 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-14fsl_qman: Remove errata testerEmil Medve
It's been disabled and unused since before SDK v1.0 and last time seems to have been used on Simics Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: I1237057ac0b719fb07ca6b5afe362c467b163316 Reviewed-on: http://git.am.freescale.net:8181/7928 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-14fsl_qman: Remove commented-out codeEmil Medve
Unused since before SDK v1.0 Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: I8a41de52a77ac1741cc0787e063b7bad22a30f00 Reviewed-on: http://git.am.freescale.net:8181/7927 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-14fsl_bman: Remove commented-out codeEmil Medve
Unused since before SDK v1.0 Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: I6584920ff290cda570ad66c3c771d8f1fd4a09d5 Reviewed-on: http://git.am.freescale.net:8181/7926 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-14fixup! fsl_qbman: Fix the checkpatch warning type PREFER_SEQ_PUTSEmil Medve
WARNING:PREFER_SEQ_PUTS: Prefer seq_puts to seq_printf checkpatch didn't catch this Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: Ifb55cd896b584ecf290a6ba386c3fefda4125f18 Reviewed-on: http://git.am.freescale.net:8181/7925 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-13fmd: fix a few spelling errorsMadalin Bucur
Fix some spelling errors reported by codespell. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: Ife8a41b1439798191140e2b3abc24f75ac208553 Reviewed-on: http://git.am.freescale.net:8181/7659 Reviewed-by: Cristian-Constantin Sovaiala <Cristian.Sovaiala@freescale.com> Reviewed-by: Jose Rivera <German.Rivera@freescale.com> Tested-by: Jose Rivera <German.Rivera@freescale.com> (cherry picked from commit 7ef92c5cb08d9b09436daf52f5f359294995832c) Reviewed-on: http://git.am.freescale.net:8181/7825
2014-01-13dpaa_eth: simplify adjust_link()Madalin Bucur
Remove awareness of macless + proxy scenario from adjust_link(). Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: Ie6e033480436634f12a504d5c3dd4d4ebed8851a Reviewed-on: http://git.am.freescale.net:8181/7792 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Marian Cristian Rotariu <marian.rotariu@freescale.com> Reviewed-on: http://git.am.freescale.net:8181/7816 Reviewed-by: Cristian-Constantin Sovaiala <Cristian.Sovaiala@freescale.com> Reviewed-by: Jose Rivera <German.Rivera@freescale.com> (cherry picked from commit 26b6123a5ad1c3da918b51d55453bf5bf0de0741) Reviewed-on: http://git.am.freescale.net:8181/7826
2014-01-10asf_XFRM: Removing compilation errors while ASF is enabled.Sandeep Malik
This patch removes the compilation errors when ASF is enabled. The errors were coming because the out_exit label has been changed to out whereas ASF code was still using out_exit. Signed-off-by: Sandeep Malik <Sandeep.Malik@freescale.com> --- CR:ENGR293944 --- Change-Id: I95ef5527001da4958dc54ab800800280d9479c20 Reviewed-on: http://git.am.freescale.net:8181/7643 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Rajan Gupta <rajan.gupta@freescale.com> Reviewed-by: Jose Rivera <German.Rivera@freescale.com>
2014-01-10asf: Removing compilation errors while ASF is enabled.Sandeep Malik
This patch removes the compilation errors when ASF is enabled. The errors were coming because the dev_hard_xmit function has been updated to have new argument. The code is stale and not required when ASF is enabled so has been removed. Signed-off-by: Sandeep Malik <Sandeep.Malik@freescale.com> Change-Id: Ia354528f63bf2c03685deaae90dad94fcc78c77c --- CR:ENGR293607 --- Reviewed-on: http://git.am.freescale.net:8181/7642 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Rajan Gupta <rajan.gupta@freescale.com> Reviewed-by: Jose Rivera <German.Rivera@freescale.com>
2014-01-10esdhc: Enable DMA err bit for eSDHC hostHaijun Zhang
eSDHC host had bit eSDHC_IRQSTAT[3] to indicate that DMA (SDMA or ADMA) transfer has failed. So enable this bit detecting and its interrupt. Signed-off-by: Haijun Zhang <Haijun.Zhang@freescale.com> Change-Id: If2abc2718d4bf0d020077848804a9ac59d5c2010 Reviewed-on: http://git.am.freescale.net:8181/5927 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Xiaobo Xie <X.Xie@freescale.com> Reviewed-by: Jose Rivera <German.Rivera@freescale.com>
2014-01-10esdhc: Add new models to errata A-005055Haijun Zhang
A-005055: A glitch is generated on the card clock due to software reset or a clock change. Impact list: T4240-4160-R1.0 B4860-4420-R1.0-R2.0 P3041-R1.0-R1.1-R2.0 P2041-2040-R1.0-R1.1-R2.0 P1010-1014-R1.0 P5020-5010-R1.0-R2.0 P5040-5021-R1.0-R2.0-R2.1 Signed-off-by: Haijun Zhang <haijun.zhang@freescale.com> Change-Id: Idf4e1fd52ec4b5748cf9cbf86365a70c9952d1be Reviewed-on: http://git.am.freescale.net:8181/5925 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Xiaobo Xie <X.Xie@freescale.com> Reviewed-by: Jose Rivera <German.Rivera@freescale.com>
2014-01-10powerpc/fsl-booke: Use SPRN_SPRGn rather than mfsprg/mtsprgScott Wood
This fixes a build break that was probably introduced with the removal of -Wa,-me500 (commit f49596a4cf4753d13951608f24f939a59fdcc653), where the assembler refuses to recognize SPRG4-7 with a generic PPC target. Signed-off-by: Scott Wood <scottwood@freescale.com> Cc: Dongsheng Wang <dongsheng.wang@freescale.com> Cc: Anton Vorontsov <avorontsov@mvista.com> Change-Id: I386dd50ee6718c12e8a6d8fc261200d0dfddd6bd Reviewed-on: http://git.am.freescale.net:8181/7610 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Dongsheng Wang <dongsheng.wang@freescale.com> Reviewed-by: Jose Rivera <German.Rivera@freescale.com>
2014-01-10esdhc: Workaround for eSDHC clock glitch issueHaijun Zhang
A-003980: SDHC: Glitch is generated on the card clock with software reset or clock divider change Description: A glitch may occur on the SDHC card clock when the software sets the RSTA bit (software reset) in the system control register. It can also be generated by setting the clock divider value. The glitch produced can cause the external card to switch to an unknown state. The occurrence is not deterministic. Workaround: A simple workaround is to disable the SD card clock before the software reset, and enable it when the module resumes normal operation. The Host and the SD card are in a master-slave relationship. The Host provides clock and control transfer across the interface. Therefore, any existing operation is discarded when the Host controller is reset. The recommended flow is as follows: 1. Software disable bit[3], SDCLKEN, of the System Control Register 2. Trigger software reset and/or set clock divider 3. Check bit[3], SDSTB, of the Present State Register for stable clock 4. Enable bit[3], SDCLKEN, of the System Control Register Using the above method, the eSDHC cannot send command or transfer data when there is a glitch in the clock line, and the glitch does not cause any issue. Signed-off-by: Haijun Zhang <haijun.zhang@freescale.com> Change-Id: I43a6ef8fdffeeeb13bfef215825d417778ce0bf3 Reviewed-on: http://git.am.freescale.net:8181/5916 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Xiaobo Xie <X.Xie@freescale.com> Reviewed-by: Jose Rivera <German.Rivera@freescale.com>
2014-01-10fsl_qbman: Fix the checkpatch warning type EXPORTED_WORLD_WRITABLEEmil Medve
WARNING:EXPORTED_WORLD_WRITABLE: Exporting world writable files is usually an error. Consider more restrictive permissions. Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: I61561d1a79f4119d91a45dd8788a819e70705a3d Reviewed-on: http://git.am.freescale.net:8181/7788 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-10fsl_qbman: Fix the checkpatch warning type STATIC_CONST_CHAR_ARRAYEmil Medve
WARNING:STATIC_CONST_CHAR_ARRAY: static const char * array should probably be static const char * const Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: I7cc81cbcf8ea5ce286111c2ab0f407d6d4d68bd1 Reviewed-on: http://git.am.freescale.net:8181/7787 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-10fsl_qbman: Fix the checkpatch warning type CONSIDER_KSTRTOEmil Medve
WARNING:CONSIDER_KSTRTO: strict_strtoul is obsolete, use kstrtoul instead Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: I482d39ab8f6aea1ba7a36f693709e4b1596722cb Reviewed-on: http://git.am.freescale.net:8181/7786 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-10fsl_qbman: Fix the checkpatch warning type SUSPECT_CODE_INDENTEmil Medve
WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 8) Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: I788edb028891ef1e92a818262b56e45c3eb13aa5 Reviewed-on: http://git.am.freescale.net:8181/7785 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-10fsl_qbman: Fix the checkpatch warning type LONG_LINEEmil Medve
WARNING:LONG_LINE: line over 80 characters Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: I2437493f24859184397a666b7e7375749c639229 Reviewed-on: http://git.am.freescale.net:8181/7784 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-10fsl_qbman: Fix the checkpatch warning type STORAGE_CLASSEmil Medve
WARNING:STORAGE_CLASS: storage class should be at the beginning of the declaration Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: I1fe009e7a44bbb4125e1b200265f4f8d8eb24565 Reviewed-on: http://git.am.freescale.net:8181/7783 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-10fsl_qbman: Fix the checkpatch warning type SINGLE_STATEMENT_DO_WHILE_MACROEmil Medve
WARNING:SINGLE_STATEMENT_DO_WHILE_MACRO: Single statement macros should not use a do {} while (0) loop Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: Ifc31cefb5ff0c0030516c24171d80a0816c3d809 Reviewed-on: http://git.am.freescale.net:8181/7782 Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com> Tested-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-10fsl_qbman: Fix the checkpatch warning type NEEDLESS_IFEmil Medve
WARNING:NEEDLESS_IF: debugfs_remove_recursive(NULL) is safe this check is probably not required WARNING:NEEDLESS_IF: kfree(NULL) is safe this check is probably not required Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: I4cc50940f601b6159f2e0b7b6305a44b72e2a8af Reviewed-on: http://git.am.freescale.net:8181/7781 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-10fsl_qbman: Fix the checkpatch warning type HOTPLUG_SECTIONEmil Medve
WARNING:HOTPLUG_SECTION: Using __devinit is unnecessary Also get rid of __devexit[_p] Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: I369aa7cc51a8ce34af717b013619d70f293bd82c Reviewed-on: http://git.am.freescale.net:8181/7780 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-10fsl_qbman: Fix the checkpatch warning type PREFER_ALIGNEDEmil Medve
WARNING:PREFER_ALIGNED: __aligned(size) is preferred over __attribute__((aligned(size))) Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: I5b36d98b0775ab5f5268a120d81553c00f05ba81 Reviewed-on: http://git.am.freescale.net:8181/7779 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-10fsl_qbman: Fix the checkpatch warning type PREFER_SEQ_PUTSEmil Medve
WARNING:PREFER_SEQ_PUTS: Prefer seq_puts to seq_printf Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: I2727672f82bfcd3dfa4972c34c39e2be38978268 Reviewed-on: http://git.am.freescale.net:8181/7778 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-10fsl_qbman: Fix the checkpatch warning type PREFER_PR_LEVELEmil Medve
WARNING:PREFER_PR_LEVEL: Prefer pr_warn(... to pr_warning(... Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: Ie1f3ba88a41b0e01284e7788a6bd512ca4f3c2e6 Reviewed-on: http://git.am.freescale.net:8181/7777 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-10fsl_qbman: Fix the checkpatch warning type LEADING_SPACEEmil Medve
WARNING:LEADING_SPACE: please, no spaces at the start of a line Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: Ia3601f0da6a798401b52c4591872de21e5e2dca5 Reviewed-on: http://git.am.freescale.net:8181/7776 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-10fsl_qbman: Fix the checkpatch warning type SPACE_BEFORE_TABEmil Medve
WARNING:SPACE_BEFORE_TAB: please, no space before tabs Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: I7298554a569ddf51b07760e87f9a1701706d8916 Reviewed-on: http://git.am.freescale.net:8181/7775 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-10fsl_qbman: Fix the checkpatch warning type SPACINGEmil Medve
WARNING:SPACING: space prohibited before semicolon Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: I367a7cd215d47bfc9abe4390b1f2a38c2694fb83 Reviewed-on: http://git.am.freescale.net:8181/7774 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-10fsl_qbman: Fix the checkpatch error type SPACINGEmil Medve
ERROR:SPACING: space required before the open parenthesis '(' ERROR:SPACING: space prohibited before that close parenthesis ')' ERROR:SPACING: space required after that ',' (ctx:VxV) Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: Ifa798bf9edbe187fe9ce21a6c96b70ba58598888 Reviewed-on: http://git.am.freescale.net:8181/7773 Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com> Tested-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-10fsl_qbman: Fix the checkpatch error type COMPLEX_MACROEmil Medve
ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parenthesis Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: I31e2455ca46408f7b87c1aa7006a8a803eada12f Reviewed-on: http://git.am.freescale.net:8181/7772 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-10fsl_qbman: Fix the checkpatch error type CODE_INDENTEmil Medve
ERROR:CODE_INDENT: code indent should use tabs where possible Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: I5d0a805810f249821ef0bfffd168896aac4156cf Reviewed-on: http://git.am.freescale.net:8181/7771 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-10fsl_qbman: Fix the checkpatch error type ELSE_AFTER_BRACEEmil Medve
ERROR:ELSE_AFTER_BRACE: else should follow close brace '}' Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: Idbad0a1de7ebcfaca3a7163573823ac6599d7e9c Reviewed-on: http://git.am.freescale.net:8181/7770 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-10fsl_qbman: Fix the checkpatch error type TRAILING_WHITESPACEEmil Medve
ERROR:TRAILING_WHITESPACE: trailing whitespace Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: I4d2a46862edc2066a9aabf236cb22e71bfa0e375 Reviewed-on: http://git.am.freescale.net:8181/7769 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-10fsl_qbman: Fix the checkpatch error type RETURN_PARENTHESESEmil Medve
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: I10ad551ab9607994560dda64ca088f0c23190e1a Reviewed-on: http://git.am.freescale.net:8181/7768 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-08iommu/fsl: Sync with the upstream PAMU driverEmil Medve
Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: Icc807d9e8c1a15e19e1dc5f09245384c19560457 Reviewed-on: http://git.am.freescale.net:8181/7709 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-08fsl_qbman: Sync with the upstream PAMU driverEmil Medve
Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: I034a28ca135f6e07a2419c83db1b2e63be37763f Reviewed-on: http://git.am.freescale.net:8181/7708 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-08fsl_qbman: Keep common header files in dpa_sys.hEmil Medve
Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: Ib8207e7821935666352822c45a46e0d26e7a773f Reviewed-on: http://git.am.freescale.net:8181/7706 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-08fsl_dce: Update the kfifo API usageEmil Medve
As per '498d319 kfifo API type safety' drivers/staging/fsl_dce/flib/dce_flow.c: In function ‘_pre_commit_cb’: drivers/staging/fsl_dce/flib/dce_flow.c:404:2: error: invalid initializer if (unlikely(kfifo_put(&fifo_arg->flow->fifo, fifo_arg->token) == 0)) ^ Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: Ifa1795cd44f18c4246ef3a65d4f3cf99e902f888 Reviewed-on: http://git.am.freescale.net:8181/7729 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-08fsl_qbman: fix a few spelling errorsMadalin Bucur
Fix some spelling errors reported by codespell. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: I4c2f89d218389b0019ae4ff80206e05b94d715e4 Reviewed-on: http://git.am.freescale.net:8181/7707 Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Jose Rivera <German.Rivera@freescale.com>
2014-01-07fsl_dce: Include missing OF header filesEmil Medve
drivers/staging/fsl_dce/dce_driver.c: In function ‘fsl_dce_probe’: drivers/staging/fsl_dce/dce_driver.c:154:2: error: implicit declaration of function ‘of_iomap’ [-Werror=implicit-function-declaration] topregs = of_iomap(nprop, 0); ^ drivers/staging/fsl_dce/dce_driver.c:164:2: error: implicit declaration of function ‘of_irq_to_resource’ [-Werror=implicit-function-declaration] ctrlpriv->err_irq = of_irq_to_resource(nprop, 0, NULL); ^ drivers/staging/fsl_dce/dce_driver.c:164:2: error: implicit declaration of function ‘of_irq_to_resource’ [-Werror=implicit-function-declaration] ctrlpriv->err_irq = of_irq_to_resource(nprop, 0, NULL); ^ drivers/staging/fsl_dce/dce_debugfs.c: In function ‘init_ccsrmempeek’: drivers/staging/fsl_dce/dce_debugfs.c:77:2: error: implicit declaration of function ‘of_get_address’ [-Werror=implicit-function-declaration] regaddr_p = of_get_address(dn, 0, &dce_ccsr_size, NULL); ^ drivers/staging/fsl_dce/dce_debugfs.c:82:2: error: implicit declaration of function ‘of_translate_address’ [-Werror=implicit-function-declaration] dce_ccsr_start = of_translate_address(dn, regaddr_p); ^ Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: Ia8f4acb61349ae8a5edb87f5767784b328993d46 Reviewed-on: http://git.am.freescale.net:8181/7721 Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com> Tested-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-07powerpc/e6500: Use CORENET_GENERICEmil Medve
As per '9e096757 powerpc/85xx: use one kernel option for all the CoreNet_Generic boards' drivers/edac/mpc85xx_edac.c: In function ‘mpc85xx_mc_clear_rfxe’: drivers/edac/mpc85xx_edac.c:1228:103: error: ‘HID1_RFXE’ undeclared (first use in this function) mtspr(SPRN_HID1, (orig_hid1[smp_processor_id()] & ~HID1_RFXE)); ^ Signed-off-by: Chunhe Lan <Chunhe.Lan@freescale.com> Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: Icedce355da55a2b1f32e3ba1d323b00fec2ca922 Reviewed-on: http://git.am.freescale.net:8181/7626 Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com> Tested-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-07powerpc/83xx: fix pci build breakScott Wood
Apparently part of upstream commit b37e161388ac3980d5dfb73050e85874b84253eb ("powerpc/pci: Fix boot panic on mpc83xx (regression)") got lost when merging 3.13-rc3. arch/powerpc/sysdev/fsl_pci.c: In function 'mpc83xx_add_bridge': arch/powerpc/sysdev/fsl_pci.c:879:3: error: implicit declaration of function 'fsl_setup_indirect_pci' Signed-off-by: Scott Wood <scottwood@freescale.com> Change-Id: Id1ecb29be87fe814a1dda5ffba61eb2a342e2669 Reviewed-on: http://git.am.freescale.net:8181/7611 Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com> Tested-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-07dma/fsl_raid: DMA unmapping is now handled in the DMA core codeEmil Medve
As per '54f8d50 dmaengine: remove DMA unmap from drivers' and '0776ae7 dmaengine: remove DMA unmap flags' drivers/dma/fsl_raid.c: In function ‘re_jr_unmap_dest_src’: drivers/dma/fsl_raid.c:135:16: error: ‘DMA_COMPL_SKIP_DEST_UNMAP’ undeclared (first use in this function) if (!(flags & DMA_COMPL_SKIP_DEST_UNMAP)) { ^ drivers/dma/fsl_raid.c:135:16: note: each undeclared identifier is reported only once for each function it appears in drivers/dma/fsl_raid.c:147:16: error: ‘DMA_COMPL_SKIP_SRC_UNMAP’ undeclared (first use in this function) if (!(flags & DMA_COMPL_SKIP_SRC_UNMAP)) { ^ Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: Ibfae0ded94c550ec83c300ffbbc9055803f2f7d2 Reviewed-on: http://git.am.freescale.net:8181/7705 Reviewed-by: Xuelin Shi <b29237@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com> Tested-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-07dma/fsl_raid: Include missing OF header filesEmil Medve
drivers/dma/fsl_raid.c: In function ‘re_jr_probe’: drivers/dma/fsl_raid.c:668:2: error: implicit declaration of function ‘irq_of_parse_and_map’ [-Werror=implicit-function-declaration] jr->irq = irq_of_parse_and_map(np, 0); ^ drivers/dma/fsl_raid.c: In function ‘raide_probe’: drivers/dma/fsl_raid.c:779:2: error: implicit declaration of function ‘of_iomap’ [-Werror=implicit-function-declaration] repriv->re_regs = of_iomap(ofdev->dev.of_node, 0); ^ drivers/dma/fsl_raid.c:779:18: warning: assignment makes pointer from integer without a cast [enabled by default] repriv->re_regs = of_iomap(ofdev->dev.of_node, 0); ^ Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Change-Id: If8bbbe08b2e4ab81852dbca00a9301491b82f5b6 Reviewed-on: http://git.am.freescale.net:8181/7704 Reviewed-by: Scott Wood <scottwood@freescale.com> Reviewed-by: Xuelin Shi <b29237@freescale.com> Reviewed-by: Emilian Medve <Emilian.Medve@freescale.com> Tested-by: Emilian Medve <Emilian.Medve@freescale.com>
2014-01-06dma: fix build breakage in s3c24xx-dmaDan Williams
This driver missed the dma unmap conversion. Replace s3c24xx_dma_unmap_buffers with dma_descriptor_unmap. Signed-off-by: Dan Williams <dan.j.williams@intel.com> Signed-off-by: Vinod Koul <vinod.koul@intel.com>
2014-01-06kbuild: Fix silent builds with make-4Emil Medve
make-4 changed the way/order it presents the command line options into MAKEFLAGS In make-3.8x, '-s' would always be first into a group of options with the '-'/hyphen removed $ make -p -s 2>/dev/null | grep ^MAKEFLAGS MAKEFLAGS = sp In make-4, '-s' seems to always be last into a group of options with the '-'/hyphen removed $ make -s -p 2>/dev/null | grep ^MAKEFLAGS MAKEFLAGS = ps Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com> Signed-off-by: Michal Marek <mmarek@suse.cz>
2014-01-06PPC: POWERNV: move iommu_add_device earlierAlexey Kardashevskiy
The current implementation of IOMMU on sPAPR does not use iommu_ops and therefore does not call IOMMU API's bus_set_iommu() which 1) sets iommu_ops for a bus 2) registers a bus notifier Instead, PCI devices are added to IOMMU groups from subsys_initcall_sync(tce_iommu_init) which does basically the same thing without using iommu_ops callbacks. However Freescale PAMU driver (https://lkml.org/lkml/2013/7/1/158) implements iommu_ops and when tce_iommu_init is called, every PCI device is already added to some group so there is a conflict. This patch does 2 things: 1. removes the loop in which PCI devices were added to groups and adds explicit iommu_add_device() calls to add devices as soon as they get the iommu_table pointer assigned to them. 2. moves a bus notifier to powernv code in order to avoid conflict with the notifier from Freescale driver. iommu_add_device() and iommu_del_device() are public now. Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru> Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>