- 01 May, 2016 3 commits
-
-
Sudip Mukherjee authored
It is not the kernel coding style to assign values to some variable in if statement. Split them up into different statements. Signed-off-by:
Sudip Mukherjee <sudip.mukherjee@codethink.co.uk> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Sudip Mukherjee authored
checkpatch warns that return is not a function and as such the brace after it is not required. Signed-off-by:
-
Parth Sane authored
Fixed alignment to match open parenthesis. Signed-off-by:
Parth Sane <laerdevstudios@gmail.com> Signed-off-by:
-
- 30 Apr, 2016 37 commits
-
-
Stuart Yoder authored
Add Stuart Yoder as additional maintainer of fsl-mc bus driver. Signed-off-by:
Stuart Yoder <stuart.yoder@nxp.com> Acked-by:
German Rivera <german.rivera@nxp.com> Signed-off-by:
-
Matthias Brugger authored
When updating the irq_chip and msi_domain_ops, the code checkes for already present functions. When more then one ITS controller are present in the system, irq_chip and msi_domain_ops got already set and a warning is invoked. This patch deletes the warning, as the funtions are just already set to the needed callbacks. Signed-off-by:
Matthias Brugger <mbrugger@suse.com> Signed-off-by:
-
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:
-
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:
-
