[PATCH v3 27/27] devres: kill devm_ioremap_nocache

2017-12-23 Thread Yisheng Xie
Now, nobody use devm_ioremap_nocache anymore, can it can just be removed. After this patch the size of devres.o will be reduced from 20304 bytes to 18992 bytes. Suggested-by: Greg KH Signed-off-by: Yisheng Xie --- Documentation/driver-model/devres.txt | 1 - include/linux/io.h

[PATCH] drivers: lustre: obdclass: simplify unregister_shrinker() usage

2017-12-23 Thread Aliaksei Karaliou
lu_global_fini() explicitly uses knowledge about shrinker's internals to make decision about calling of unregister_shrinker(). Now this check was integrated into unregister_shrinker(), so it is safe to call it against unregistered shrinker. Signed-off-by: Aliaksei Karaliou --- drivers/staging/lu

[PATCH v3 03/27] staging: replace devm_ioremap_nocache with devm_ioremap

2017-12-23 Thread Yisheng Xie
Default ioremap is ioremap_nocache, so devm_ioremap has the same function with devm_ioremap_nocache, which can just be killed to save the size of devres.o. This patch is to use use devm_ioremap instead of devm_ioremap_nocache, which should not have any function change but prepare for killing devm_

[PATCH v3 00/27] kill devm_ioremap_nocache

2017-12-23 Thread Yisheng Xie
Hi all, When I tried to use devm_ioremap function and review related code, I found devm_ioremap and devm_ioremap_nocache is almost the same with each other, except one use ioremap while the other use ioremap_nocache. While ioremap's default function is ioremap_nocache, so devm_ioremap_nocache also

RE: PROBLEM: 4.15.0-rc3 APIC causes lockups on Core 2 Duo laptop

2017-12-23 Thread Thomas Gleixner
On Sat, 23 Dec 2017, Dexuan Cui wrote: > > From: Alexandru Chirvasitu [mailto:achirva...@gmail.com] > > Sent: Friday, December 22, 2017 14:29 > > > > The output of that precise command run just now on a freshly-compiled > > copy of that commit is attached. > > > > On Fri, Dec 22, 2017 at 09:31:2

Re: [PATCH v3 27/27] devres: kill devm_ioremap_nocache

2017-12-23 Thread Greg KH
On Sat, Dec 23, 2017 at 07:02:59PM +0800, Yisheng Xie wrote: > --- a/lib/devres.c > +++ b/lib/devres.c > @@ -44,35 +44,6 @@ void __iomem *devm_ioremap(struct device *dev, > resource_size_t offset, > EXPORT_SYMBOL(devm_ioremap); > > /** > - * devm_ioremap_nocache - Managed ioremap_nocache() > -

Re: [PATCH v3 00/27] kill devm_ioremap_nocache

2017-12-23 Thread Greg KH
On Sat, Dec 23, 2017 at 06:55:25PM +0800, Yisheng Xie wrote: > Hi all, > > When I tried to use devm_ioremap function and review related code, I found > devm_ioremap and devm_ioremap_nocache is almost the same with each other, > except one use ioremap while the other use ioremap_nocache. For all a

[PATCH] staging: vc04_services: Prefer BUG_ON instead of if condition followed by BUG.

2017-12-23 Thread kishore . p
From: Kishore KP Use BUG_ON instead of if condition followed by BUG. Pointed out by Coccinelle. Signed-off-by: Kishore KP Signed-off-by: Suniel Mahesh --- Note: - Patch was compile tested and built(ARCH=arm) on linux-next (latest). - No build issues reported. --- drivers/staging/vc04_servic

[PATCH] staging: vc04_services: Fix platform_no_drv_owner.cocci warnings.

2017-12-23 Thread kishore . p
From: Kishore KP Removed .owner field initialization, platform core does it automatically. Pointed out by Coccinelle. Signed-off-by: Kishore KP Signed-off-by: Suniel Mahesh --- Note: - Patch was compile tested and built(ARCH=arm) on linux-next (latest). - No build issues reported. --- drive

[PATCH] Staging: wlan-ng: hfa384x_usb: fixed two line limit coding style issues

2017-12-23 Thread Andy Pusch
Fixed two coding style issues. Signed-off-by: Andy Pusch --- drivers/staging/wlan-ng/hfa384x_usb.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/staging/wlan-ng/hfa384x_usb.c b/drivers/staging/wlan-ng/hfa384x_usb.c index 197f5a914e8f..d6df35e1745c 100644 ---

Re: [PATCH v3 00/27] kill devm_ioremap_nocache

2017-12-23 Thread Guenter Roeck
On 12/23/2017 05:48 AM, Greg KH wrote: On Sat, Dec 23, 2017 at 06:55:25PM +0800, Yisheng Xie wrote: Hi all, When I tried to use devm_ioremap function and review related code, I found devm_ioremap and devm_ioremap_nocache is almost the same with each other, except one use ioremap while the other

Re: [PATCH] staging: vc04_services: Prefer BUG_ON instead of if condition followed by BUG.

2017-12-23 Thread Stefan Wahren
Hi Kishore, > kishor...@techveda.org hat am 23. Dezember 2017 um 16:06 geschrieben: > > > From: Kishore KP > > Use BUG_ON instead of if condition followed by BUG. > Pointed out by Coccinelle. > > Signed-off-by: Kishore KP > Signed-off-by: Suniel Mahesh > --- > Note: > - Patch was compile te

Re: PROBLEM: 4.15.0-rc3 APIC causes lockups on Core 2 Duo laptop

2017-12-23 Thread Alexandru Chirvasitu
On Sat, Dec 23, 2017 at 02:32:52PM +0100, Thomas Gleixner wrote: > On Sat, 23 Dec 2017, Dexuan Cui wrote: > > > > From: Alexandru Chirvasitu [mailto:achirva...@gmail.com] > > > Sent: Friday, December 22, 2017 14:29 > > > > > > The output of that precise command run just now on a freshly-compiled

Re: PROBLEM: 4.15.0-rc3 APIC causes lockups on Core 2 Duo laptop

2017-12-23 Thread Dou Liyang
Hi Thomas, At 12/23/2017 09:32 PM, Thomas Gleixner wrote: [...] The BUG_ON panic happens at line 147: BUG_ON(!test_and_clear_bit(bit, cm->alloc_map)); I'm sure Thomas and Dou know it better than me. I'll have a look after the holidays. Merry Christmas! :-) I am tryin

[PATCH] staging: pi433: Cleanup codestyle, indent statements after case labels

2017-12-23 Thread Michael Panzlaff
To be applied on: git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git Branch: staging-next This patch changes the indentation of the statements after case labels. The linux coding guidelines do not explicitly mentiond this but pretty much all existing code doesn't put any statements i