Age | Commit message (Collapse) | Author |
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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
|
|
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
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|