removed braces from single statement conditonals.
if, else, while opening brace placement was moved to last
on first line of each, and closing brace moved to first on
new line to follow the linux kernel coding style thus removing
checkpatch errors respectfully.
Signed-off-by: Gary Alan Rookard
On Monday, December 16, 2013 2:51 PM, Wei Yongjun wrote:
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return negative error code -EIO from the error handling
case instead of 0.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
The following patch series fixes some bugs and races in the imx-drm
code, which should probably be applied to an -rc kernel.
drivers/staging/imx-drm/imx-drm-core.c | 21 -
drivers/staging/imx-drm/ipu-v3/ipu-common.c | 32 +-
We must call drm_vblank_cleanup() on the error cleanup and unload paths
after we've had a successful call to drm_vblank_init(). Ensure that
the calls are in the reverse order to the initialisation order.
Signed-off-by: Russell King rmk+ker...@arm.linux.org.uk
---
Enable lock claims that it is serializing tve_enable/disable calls.
However, DRM already serialises mode sets with a mutex, which prevents
encoder/connector functions being called concurrently. Secondly,
holding a spinlock while calling clk_prepare_enable() is wrong; it
will cause a might_sleep()
imx_drm_crtc_register() doesn't clean up the CRTC upon failure, which
leaves the CRTC attached to the DRM device. Also, it does setup after
attaching the CRTC to the DRM device.
Fix this by reordering the function such that we do the setup before
drm_crtc_init(): this fixes both issues.
We have this definition, there's no reason not to use it.
Signed-off-by: Russell King rmk+ker...@arm.linux.org.uk
---
drivers/staging/imx-drm/imx-drm-core.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/imx-drm/imx-drm-core.c
We must not add more CRTCs than we have declared to the vblank
helpers, otherwise we overflow their arrays. Force failure if we
exceed the number of CRTCs.
Signed-off-by: Russell King rmk+ker...@arm.linux.org.uk
---
drivers/staging/imx-drm/imx-drm-core.c |9 +
1 files changed, 9
On Mon, Dec 16, 2013 at 11:33:01AM +, Russell King - ARM Linux wrote:
The following patch series fixes some bugs and races in the imx-drm
code, which should probably be applied to an -rc kernel.
drivers/staging/imx-drm/imx-drm-core.c | 21 -
Thanks for reporting this bug. The same bug exists in the driver for OF
(xillybus_of.c).
As for the choice of -EIO, I've surveyed several drivers calling
of_iomap() and pci_iomap(), and it turns out that some fail on -ENOMEM,
some on -EIO and some on -EBUSY.
Personally, I'm inclined towards
On Fri, 27 Sep 2013, Joseph Salisbury wrote:
commit b1a1442a23776756b254b69786848a94d92445ba
Author: Jiri Kosina jkos...@suse.cz
Date: Mon Jun 3 11:27:48 2013 +0200
HID: core: fix reporting of raw events
Reverting this commit in v3.12-rc2 prevents the system from locking up,
Bug fix: The error code was not set, so the error condition wasn't reflected
in the return value.
Reported-by: Wei Yongjun yongjun_...@trendmicro.com.cn
Signed-off-by: Eli Billauer eli.billa...@gmail.com
---
drivers/staging/xillybus/xillybus_of.c |2 +-
The spin_lock_irqsave() macro is not nestable. The second call will
overwrite the first record of flags so the IRQs will not be enabled
correctly at the end of the function.
I haven't looked at all the callers but it could be that this function
is always called with interrupts disabled and the
Signed-off-by: Matthias Wirth matthias.wi...@gmail.com
Signed-off-by: Lukas Senger lu...@fridolin.com
---
drivers/staging/rtl8188eu/include/osdep_service.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8188eu/include/osdep_service.h
This patchset cleans up the 2 files having the most checkpatch warnings/errors
in the rtl8188eu driver. The intention is to improve the coding style according
to
guidelines.
The 2 remaining warnings in osdep_service.h are macros requiring structural
change accommpanied by modifying every use of
From: Micky Ching micky_ch...@realsil.com.cn
Add card reader rtl8042, rtl8402 is much like rtl8411, so just add it to
rtl8411.c
Signed-off-by: Micky Ching micky_ch...@realsil.com.cn
---
drivers/mfd/rtl8411.c | 28
drivers/mfd/rtsx_pcr.c |5 +
From: Micky Ching micky_ch...@realsil.com.cn
With the recent added support request of yet another device, the
burden of duplicated code was becoming a little messy. To rectify is,
we init rtl8411-like chips to 8411 param first, then modify the
different values according each chip. And rtl8402 is
From: Micky Ching micky_ch...@realsil.com.cn
in order to remove duplicated code in rtl8411, we make 8411 as the base
init params, and other like-8411 chips will just change the different
value with 8411, this can save some source code.
Signed-off-by: Lee Jones lee.jo...@linaro.org
Signed-off-by:
From: Wei Yongjun yongjun_...@trendmicro.com.cn
module_platform_driver() makes the code simpler by eliminating
boilerplate code.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/staging/android/ion/tegra/tegra_ion.c | 13 +
1 file changed, 1 insertion(+), 12
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fixes the following sparse warnings:
drivers/staging/android/ion/tegra/tegra_ion.c:23:19: warning:
symbol 'idev' was not declared. Should it be static?
drivers/staging/android/ion/tegra/tegra_ion.c:24:19: warning:
symbol 'tegra_user_mapper' was
This patch fixes ERROR: trailing whitespace found by
checkpatch.pl, within dgap_driver.h
Signed-off-by: Masanari Iida standby2...@gmail.com
---
drivers/staging/dgap/dgap_driver.h | 22 +++---
1 file changed, 11 insertions(+), 11 deletions(-)
diff --git
This patch fixes ERROR: trailing whitespace found by
checkpatch.pl, within dgap_downld.h and dgap_sysfs.h.
Signed-off-by: Masanari Iida standby2...@gmail.com
---
drivers/staging/dgap/dgap_downld.h | 4 ++--
drivers/staging/dgap/dgap_sysfs.h | 2 +-
2 files changed, 3 insertions(+), 3
This patch fixes ERROR: trailing whitespace found by
checkpatch.pl, within dgap_fep5.h
Signed-off-by: Masanari Iida standby2...@gmail.com
---
drivers/staging/dgap/dgap_fep5.h | 45
1 file changed, 22 insertions(+), 23 deletions(-)
diff --git
This patch fixes ERROR: trailing whitespace found by
checkpatch.pl, within dgap_parse.c
Signed-off-by: Masanari Iida standby2...@gmail.com
---
drivers/staging/dgap/dgap_parse.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/drivers/staging/dgap/dgap_parse.c
This patch fixes ERROR: trailing whitespace found by
checkpatch.pl, within dgap_driver.c
Signed-off-by: Masanari Iida standby2...@gmail.com
---
drivers/staging/dgap/dgap_driver.c | 40 +++---
1 file changed, 20 insertions(+), 20 deletions(-)
diff --git
This patch fixes ERROR: trailing whitespace found by
checkpatch.pl, within digi.h
Signed-off-by: Masanari Iida standby2...@gmail.com
---
drivers/staging/dgap/digi.h | 24
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/drivers/staging/dgap/digi.h
On Mon, Dec 16, 2013 at 11:34:05AM +, Russell King wrote:
Trying to build a modular imx-drm results in a number of missing symbol
exports, caused by the recent changes to this driver. Add the necessary
exports, and the missing MODULE_LICENSE() tag.
Since commit 9c74360 (staging: imx-drm:
On Mon, Dec 16, 2013 at 12:38:23PM +, Russell King - ARM Linux wrote:
Russell King (8):
imx-drm: imx-drm-core: fix error cleanup path for imx_drm_add_crtc()
imx-drm: imx-drm-core: fix DRM cleanup paths
imx-drm: fix missing symbol exports
Except the little doubt I replied
On Mon, Dec 16, 2013 at 11:33:01AM +, Russell King - ARM Linux wrote:
The following patch series fixes some bugs and races in the imx-drm
code, which should probably be applied to an -rc kernel.
drivers/staging/imx-drm/imx-drm-core.c | 21 -
On Tue, Dec 17, 2013 at 10:36:58AM +0800, micky_ch...@realsil.com.cn wrote:
diff --git a/drivers/mfd/rtsx_pcr.h b/drivers/mfd/rtsx_pcr.h
index 947e79b..26b52ec 100644
--- a/drivers/mfd/rtsx_pcr.h
+++ b/drivers/mfd/rtsx_pcr.h
@@ -63,4 +63,12 @@ static inline u8 map_sd_drive(int idx)
#define
30 matches
Mail list logo