- 30 Apr, 2016 40 commits
-
-
Horia Geantă authored
There are some error paths that allow for a NULL new_mc_io and err = 0 return code. Return -EINVAL instead. Signed-off-by:
Horia Geantă <horia.geanta@nxp.com> Signed-off-by:
Stuart Yoder <stuart.yoder@nxp.com> Acked-by:
German Rivera <german.rivera@nxp.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Itai Katz authored
The dpmcp driver supports dpmcp version 3.0 and above. This patch adds the code to check the version. Signed-off-by:
Itai Katz <itai.katz@nxp.com> Signed-off-by:
-
Horia Geanta authored
dpseci objects < 4.0 are not coherent-- in spite of the fact that the MC reports them to be coherent in certain versions. Add a special case to set the no shareability flag for dpseci objects < 4.0. Signed-off-by:
-
Itai Katz authored
The dprc driver supports dprc version 5.0 and above. This patch adds the code to check the version. Signed-off-by:
-
Itai Katz authored
The root dprc is discovered as a platform device in the device tree. The version of that dprc was previously set using hardcoded values from the API header in the kernel). This patch removes the use of the hardcoded version numbers and instead reads the actual dprc version from the hardware. Signed-off-by:
-
Itai Katz authored
Some DPAA2 devices have mmio regions that should be mapped as cacheable by drivers. Set IORESOURCE_CACHEABLE in the region's flags if applicable. Signed-off-by:
-
Stuart Yoder authored
Unless discovered devices have the no shareability flag set, set up coherent dma ops for them. Signed-off-by:
-
Itai Katz authored
Up until now if the object version expected by a driver (in the API header file) did not match the actual object version in the MC hardware the bus driver refused to bind the object to the driver or printed out WARN_ON dumps. This patch removes those checks, and the responsibility of object version checking should now be done in the object drivers themselves. If the actual version discovered is not supported, the driver's probe function should fail. Drivers should use version checks to support new features and provide backwards compatibility if at all possible. This patch also removes the checks that caused bus driver probing to fail if the overall MC version discovered did not match the firmware version from the API header...this was too strict. The overall MC version is informational like a release number, and continues to be printed in the boot log. Signed-off-by:
-
Stuart Yoder authored
The meaning of the "status" parameter in dprc_get_irq_status has changed, and this patch updates the flib and caller of the API. Signed-off-by:
-
Stuart Yoder authored
Signed-off-by:
-
Stuart Yoder authored
Signed-off-by:
-
Stuart Yoder authored
incorporated feedback from review comments, other misc cleanup/tweaks Signed-off-by:
-
Stuart Yoder authored
remove 3 of the remaining TODO items: -multiple root fsl-mc buses-- done in patch series starting with commit 14f92805 ("staging: fsl-mc: abstract test for existence of fsl-mc bus") -interrupt support-- done in patch series starting with commit 9b1b282c ("irqdomain: Added domain bus token DOMAIN_BUS_FSL_MC_MSI") -MC command serialization-- done in commit 63f2be5c ("staging: fsl-mc: Added serialization to mc_send_command()") Signed-off-by: -
Nikita Eshkeev authored
This patch fixes the checkpatch.pl warning: WARNING: line over 80 characters + .domain = "/system-controller@e6180000/pm-domains/c5/a4lc@1" Signed-off-by:
Nikita Eshkeev <kastolom@gmail.com> Signed-off-by:
-
han tai authored
Issue found by checkpatch.pl CHECK: Alignment should match open parenthesis Signed-off-by:
han tai <hantai1248@hotmail.com> Signed-off-by:
-
Clifton Barnes authored
fix checkpatch.pl warning about 'Missing a blank line after declarations' Signed-off-by:
Clifton Barnes <clifton.a.barnes@gmail.com> Signed-off-by:
-
Clifton Barnes authored
fix checkpatch.pl warning about 'Prefer 'unsigned int' to bare use of 'unsigned'' Signed-off-by:
-
Manav Batra authored
Fixes CHECK: Alignment should match open parenthesis Signed-off-by:
Manav Batra <batmanav10@gmail.com> Signed-off-by:
-
Nicholas Sim authored
Added braces on if arm of if statement where else arm already needs braces as suggested for clarity in Documentation/CodingStyle (several) Signed-off-by:
Nicholas Sim <nicholassimws@gmail.com> Signed-off-by:
-
Dan Carpenter authored
The NULL checking here doesn't make sense, so it causes a static checker warning. It turns out that p->mm can't be NULL so the inconsistency is harmless and we should just remove the check. Signed-off-by:
Dan Carpenter <dan.carpenter@oracle.com> Acked-by:
Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp> Signed-off-by:
-
Cihangir Akturk authored
Remove mutex_locked variables which are used to determine whether mutex is locked, instead add another label to unlock mutex on premature exits due to an error. This patch also addresses the folowing warnings reported by coccinelle: drivers/staging/fsl-mc/bus/mc-allocator.c:237:1-7: preceding lock on line 204 drivers/staging/fsl-mc/bus/mc-allocator.c:89:1-7: preceding lock on line 57 drivers/staging/fsl-mc/bus/mc-allocator.c:157:1-7: preceding lock on line 124 Signed-off-by:
Cihangir Akturk <cakturk@gmail.com> Signed-off-by:
-
Ben Hutchings authored
There's no net_device stashed in skb->cb, there's a net_device * there. To make it *really* clear, also change the write of the dev pointer into skb->cb from a memcpy() to an assignment. Fixes: 3fe56324 ("staging: rtl8192u: r8192U_core.c: Cleaning up ...") Signed-off-by:
Ben Hutchings <ben@decadent.org.uk> Signed-off-by:
-
Nicholas Sim authored
When testing pointers, it is not necessary to explicitly compare to NULL. Rewrite if condition as (!ptr) or (ptr) as suggested in Documentation/CodingStyle Signed-off-by:
-
Nicholas Sim authored
Add a blank line after function/struct/union/enum declarations for readability, as suggested in Documentation/CodingStyle Signed-off-by:
-
Nicholas Sim authored
Remove unneeded blank lines appearing after opening braces as suggested by checkpatch.pl Signed-off-by:
-
Nicholas Sim authored
Remove unneeded blank lines occuring before closing braces Signed-off-by:
-
Geliang Tang authored
This patch drops the local definition of LIST_CONTAINOR(), and uses container_of() instead of it. Signed-off-by:
Geliang Tang <geliangtang@163.com> Signed-off-by:
-
Manu Kumar authored
Aligned parentheses to conform to the coding style. Signed-off-by:
Manu Kumar <maraku@gmx.com> Signed-off-by:
-
Manu Kumar authored
Added white space between operators and operands. Because this sometimes maxed out the column width, some expressions were broken up into multiple lines, and comments were moved appropriately. Signed-off-by:
-
Daeseok Youn authored
The tty_alloc_driver() can allocate memory for ttys and termios. And also allocated memory will be released easily with put_tty_driver() call. Signed-off-by:
Daeseok Youn <daeseok.youn@gmail.com> Signed-off-by:
-
Daeseok Youn authored
There were already checking NULL about channel_t / un_t before calling dgnc_maxcps_room(). Signed-off-by:
-
Daeseok Youn authored
The kzalloc can be failed when memory is not enough to allocate. When kzalloc failed, it need to return error code with ENOMEM Signed-off-by:
-
Daeseok Youn authored
fix checkpatch.pl warning: Blank lines aren't necessary after an open brace '{' Signed-off-by: -
Daeseok Youn authored
remove parenthesis around the CONST | CONST. Signed-off-by:
-
Juanma de Hoyos authored
Coding rules request placing bitwise OR operators between empty spaces for better readibility. This patch applies this format to a line in dgnc_tty.c file. Signed-off-by:
Juanma de Hoyos <juanmahv@gmail.com> Signed-off-by:
-
Claudiu Beznea authored
This patch frees memory allocated inside mkimage() in case mkimage() or any other subsequent calls inside prism2_fwapply() from prism2fw.c file fails. To fix this I introduces goto labels where the free operation is done in case some operations fails. After the introduction of goto labels has been done, in order to use the same return path, "return x" instuctions were replaced with "goto" instuctions. Signed-off-by:
Claudiu Beznea <claudiu.beznea@gmail.com> Signed-off-by:
-
Nicholas Sim authored
It is not necessary to compare explicitly to NULL. Rewrite if condition as (!dev) or (dev) as suggested in Documentation/CodingStyle Signed-off-by:
-
Sudip Mukherjee authored
ion_device_create() can fail and if it fails then it returns the error value in ERR_PTR. Signed-off-by:
Sudip Mukherjee <sudip.mukherjee@codethink.co.uk> Reviewed-by:
Laura Abbott <labbott@redhat.com> Signed-off-by:
-
Ben Marsh authored
Modify memory allocation style in order to silence a checkpatch.pl warning. Signed-off-by:
Ben Marsh <bmarsh94@gmail.com> Signed-off-by:
-
Ben Marsh authored
Modifies the memory allocation style ion_test.c in order to remove a checkpatch.pl warning Signed-off-by:
-
