Re: [PATCH] sound: soc: soc-pcm: Remove useless varialbe 'i'

2015-01-03 Thread Chen Gang S
On 1/3/15 17:59, Takashi Iwai wrote: > At Sat, 03 Jan 2015 17:45:57 +0800, > Chen Gang wrote: >> >> 'i' is useless in soc_pcm_set_msb(), so remove it, the related warning >> (with allmodconfig under s390): >> >> CC [M] sound/soc/soc-pcm.o >> sound/soc/soc-pcm.c: In function 'soc_pcm_set_msb'

Re: [PATCH] s390: boot: compressed: misc: Express the return value of _sclp_print_early()

2015-01-05 Thread Chen Gang S
On 1/5/15 17:21, Heiko Carstens wrote: > On Sat, Jan 03, 2015 at 05:29:07PM +0800, Chen Gang wrote: >> _sclp_print_early() has return value: at present, return 0 for OK, 1 for >> failure. It returns '%r2', so use 'long' as return value (upper caller >> can check '%r2' directly). The related warning

Re: [PATCH] s390: include: timex: Use macro CLOCK_STORE_SIZE instead of hard code number

2015-01-07 Thread Chen Gang S
On 01/05/2015 04:59 PM, Martin Schwidefsky wrote: > On Sat, 03 Jan 2015 11:44:04 +0800 > Chen Gang wrote: > >> >> Thank you for your work. >> >> In honest, originally, I was not sure whether it would cause bug (do not >> know gcc would generic incorrect code for it). :-) > > Even if the code hap

Re: [PATCH] s390: include: timex: Use macro CLOCK_STORE_SIZE instead of hard code number

2015-01-09 Thread Chen Gang S
On 1/7/15 23:36, Martin Schwidefsky wrote: > On Wed, 07 Jan 2015 22:45:11 +0800 > Chen Gang S wrote: > >> On 01/05/2015 04:59 PM, Martin Schwidefsky wrote: >>> On Sat, 03 Jan 2015 11:44:04 +0800 >>> Chen Gang wrote: >>> >>>> >>>&g

Re: [PATCH] drm: Kconfig: Let all DRM_GEM_CMA_HELPER related macros depend on HAVE_DMA_ATTRS

2015-02-24 Thread Chen Gang S
> On 2/3/15 05:26, Chen Gang S wrote: > > [...] > >> >> - Xtensa gcc5 cross compiler has issues: >> >>it causes more than 10 broken areas with allmodconfig (but no issues >>with defconfig). I guess, in fact, it is only 1 real issue (which can &

[PATCH v3] net: bluetooth: hci_sock: Use 'const u32 *' instead of 'void *' for 2nd parameter of hci_test_bit()

2015-02-07 Thread Chen Gang S
hci_test_bit() does not modify 2nd parameter, so it is better to let it be constant, or may cause build warning. The related warning (with allmodconfig under xtensa): net/bluetooth/hci_sock.c: In function 'hci_sock_sendmsg': net/bluetooth/hci_sock.c:955:8: warning: passing argument 2 of 'hci_t

Re: [PATCH v3] net: bluetooth: hci_sock: Use 'const u32 *' instead of 'void *' for 2nd parameter of hci_test_bit()

2015-02-07 Thread Chen Gang S
On 2/8/15 03:52, Joe Perches wrote: > On Sat, 2015-02-07 at 21:24 +0800, Chen Gang S wrote: >> hci_test_bit() does not modify 2nd parameter, so it is better to let it >> be constant, or may cause build warning. The related warning (with >> allmodconfig under xtensa):

Re: [PATCH v3] net: bluetooth: hci_sock: Use 'const u32 *' instead of 'void *' for 2nd parameter of hci_test_bit()

2015-02-08 Thread Chen Gang S
On 2/8/15 08:00, Chen Gang S wrote: > On 2/8/15 03:52, Joe Perches wrote: >> On Sat, 2015-02-07 at 21:24 +0800, Chen Gang S wrote: >>> hci_test_bit() does not modify 2nd parameter, so it is better to let it >>> be constant, or may cause build warning. The related war

Re: [PATCH v3] net: bluetooth: hci_sock: Use 'const u32 *' instead of 'void *' for 2nd parameter of hci_test_bit()

2015-02-08 Thread Chen Gang S
On 2/9/15 04:17, Marcel Holtmann wrote: > Hi Chen, > > hci_test_bit() does not modify 2nd parameter, so it is better to let it > be constant, or may cause build warning. The related warning (with > allmodconfig under xtensa): > > net/bluetooth/hci_sock.c: In function 'hci_sock

Re: [PATCH v3] net: bluetooth: hci_sock: Use 'const u32 *' instead of 'void *' for 2nd parameter of hci_test_bit()

2015-02-09 Thread Chen Gang S
On 2/10/15 00:57, David Laight wrote: > From: Marcel Holtmann >> Hi David, >> So at present, in kernel part, we can only say the original authors intended to do like this. And only within kernel part, it can not cause issue. I guess, original authors originally knew what we talk abou

Re: [PATCH v2] net: bluetooth: hci_sock: Use 'const void *' instead of 'void *' for 2nd parameter of hci_test_bit()

2015-02-04 Thread Chen Gang S
On 2/5/15 05:09, Marcel Holtmann wrote: > Hi Sergei, > -static inline int hci_test_bit(int nr, void *addr) +static inline int hci_test_bit(int nr, const void *addr) { return *((__u32 *) addr + (nr >> 5)) & ((__u32) 1 << (nr & 31)); } >> >>> Is there a 'standard' funct

Re: [PATCH v2] net: bluetooth: hci_sock: Use 'const void *' instead of 'void *' for 2nd parameter of hci_test_bit()

2015-02-05 Thread Chen Gang S
On 2/5/15 18:14, David Laight wrote: > From: Chen Gang S [mailto:gang.c...@sunrus.com.cn] >> On 2/5/15 05:09, Marcel Holtmann wrote: >>> Hi Sergei, >>> >>>>>> -static inline int hci_test_bit(int nr, void *addr) >>>>&g

[PATCH] drm: Kconfig: Let all DRM_GEM_CMA_HELPER related macros depend on HAVE_DMA_ATTRS

2015-02-01 Thread Chen Gang S
DRM_GEM_CMA_HELPER is depend on HAVE_DMA_ATTRS, or it will break the building. The related error (with allmodconfig under xtensa): CC [M] drivers/gpu/drm/drm_gem_cma_helper.o drivers/gpu/drm/drm_gem_cma_helper.c: In function 'drm_gem_cma_create': drivers/gpu/drm/drm_gem_cma_helper.c:110:1

[PATCH] rc: st_rc: Use CONFIG_PM_SLEEP instead of CONFIG_PM for st_rc_suspend() and st_rc_resume()

2015-02-01 Thread Chen Gang S
st_rc_suspend() and st_rc_resume() depend on CONFIG_PM_SLEEP finally, so they need CONFIG_PM_SLEEP instead of CONFIG_PM, or they will cause build warning (with allmodconfig under xtensa): CC [M] drivers/media/pci/smipcie/smipcie.o drivers/media/rc/st_rc.c:338:12: warning: 'st_rc_suspend' de

[PATCH] net: usb: sr9700: Use 'SR_' prefix for the common register macros

2015-02-02 Thread Chen Gang S
The commone register macors (e.g. RSR) is too commont to drivers, it may be conflict with the architectures (e.g. xtensa, sh). The related warnings (with allmodconfig under xtensa): CC [M] drivers/net/usb/sr9700.o In file included from drivers/net/usb/sr9700.c:24:0: drivers/net/usb/sr970

[PATCH v2] net: bluetooth: hci_sock: Use 'const void *' instead of 'void *' for 2nd parameter of hci_test_bit()

2015-02-02 Thread Chen Gang S
hci_test_bit() does not modify 2nd parameter, so it is better to let it be constant, or may cause build warning. The related warning (with allmodconfig under xtensa): net/bluetooth/hci_sock.c: In function 'hci_sock_sendmsg': net/bluetooth/hci_sock.c:955:8: warning: passing argument 2 of 'hci_t

Re: [PATCH] drm: Kconfig: Let all DRM_GEM_CMA_HELPER related macros depend on HAVE_DMA_ATTRS

2015-02-02 Thread Chen Gang S
). I guess, in fact, it is only 1 real issue (which can cause all breaks), and I shall try to fix it within this month. Welcome any ideas, suggestions, or completions. Thanks. On 2/1/15 22:08, Chen Gang S wrote: > DRM_GEM_CMA_HELPER is depend on HAVE_DMA_ATTRS, or it will break the > bu

Re: [PATCH v2] net: bluetooth: hci_sock: Use 'const void *' instead of 'void *' for 2nd parameter of hci_test_bit()

2015-02-02 Thread Chen Gang S
On 2/3/15 05:20, Joe Perches wrote: > On Tue, 2015-02-03 at 05:14 +0800, Chen Gang S wrote: >> hci_test_bit() does not modify 2nd parameter, so it is better to let it >> be constant, or may cause build warning. The related warning (with >> allmodconfig under xtensa): >

Re: [PATCH v2] net: bluetooth: hci_sock: Use 'const void *' instead of 'void *' for 2nd parameter of hci_test_bit()

2015-02-02 Thread Chen Gang S
On 2/3/15 10:32, Chen Gang S wrote: > On 2/3/15 05:20, Joe Perches wrote: >> On Tue, 2015-02-03 at 05:14 +0800, Chen Gang S wrote: >>> hci_test_bit() does not modify 2nd parameter, so it is better to let it >>> be constant, or may cause build warning. The related war

Re: [PATCH] drm: Kconfig: Let all DRM_GEM_CMA_HELPER related macros depend on HAVE_DMA_ATTRS

2015-02-03 Thread Chen Gang S
On 02/03/2015 06:45 PM, Laurent Pinchart wrote: > Hi Chen, > > Thank you for the patch. > > On Sunday 01 February 2015 22:08:33 Chen Gang S wrote: >> DRM_GEM_CMA_HELPER is depend on HAVE_DMA_ATTRS, or it will break the >> building. The related error (with allmodconfig

Fwd: [PATCH] c6x: Makefile: Add -D__linux__

2015-03-02 Thread Chen Gang S
Just forward mail. Forwarded Message Subject: [PATCH] c6x: Makefile: Add -D__linux__ Date: Tue, 03 Mar 2015 06:33:42 +0800 From: Chen Gang <762976...@qq.com> To: Mark Salter , a-jacqu...@ti.com CC: linux-c6x-...@linux-c6x.org , linux-kernel@vger.kernel.org For gcc5 c6x raw c

Fwd: [PATCH] mm: memcontrol: Let mem_cgroup_move_account() have effect only if MMU enabled

2015-03-02 Thread Chen Gang S
Just forward mail. Forwarded Message Subject: [PATCH] mm: memcontrol: Let mem_cgroup_move_account() have effect only if MMU enabled Date: Tue, 03 Mar 2015 06:42:01 +0800 From: Chen Gang <762976...@qq.com> To: han...@cmpxchg.org, mho...@suse.cz CC: cgro...@vger.kernel.org, linux

Fwd: [PATCH] c6x: asm: pgtable: Define dummy pgprot_writecombine for !MMU

2015-03-02 Thread Chen Gang S
Forwarded Message Subject: [PATCH] c6x: asm: pgtable: Define dummy pgprot_writecombine for !MMU Date: Tue, 03 Mar 2015 07:06:25 +0800 From: Chen Gang <762976...@qq.com> To: msal...@redhat.com, a-jacqu...@ti.com CC: kirill.shute...@linux.intel.com, Andrew Morton , linux-c6x-...

Fwd: [PATCH] c6x: asm: Add default flat.h according to xtensa architecture

2015-03-02 Thread Chen Gang S
Just forward the original mail. Forwarded Message Subject: [PATCH] c6x: asm: Add default flat.h according to xtensa architecture Date: Tue, 03 Mar 2015 07:24:26 +0800 From: Chen Gang <762976...@qq.com> To: msal...@redhat.com, a-jacqu...@ti.com CC: linux-c6x-...@linux-c6x.org ,

Re: [PATCH] rc: st_rc: Use CONFIG_PM_SLEEP instead of CONFIG_PM for st_rc_suspend() and st_rc_resume()

2015-02-12 Thread Chen Gang S
Hello Maintainers: Please help to process this patch, when you have time. Thanks. On 2/2/15 15:54, Patrice Chotard wrote: > Hi Chen > > On 02/01/2015 03:42 PM, Chen Gang S wrote: >> st_rc_suspend() and st_rc_resume() depend on CONFIG_PM_SLEEP finally, so >> they need CONFI

Re: [PATCH] drm: Kconfig: Let all DRM_GEM_CMA_HELPER related macros depend on HAVE_DMA_ATTRS

2015-02-22 Thread Chen Gang S
On 2/3/15 05:26, Chen Gang S wrote: [...] > > - Xtensa gcc5 cross compiler has issues: > >it causes more than 10 broken areas with allmodconfig (but no issues >with defconfig). I guess, in fact, it is only 1 real issue (which can >cause all breaks), and I s

Fwd: [PATCH] c6x: kernel: setup: Include "linux/console.h" if both VT and DUMMY_CONSOLE enabled

2015-03-01 Thread Chen Gang S
This patch below is sent from my other email address (762976...@qq.com), but is rejected by linux-kernel@vger.kernel.org, so I forward it by my original email address. Excuse me, I will leave current company soon (sunrus.com.cn), so I have to change my email address too. At present, gmail in Chi

Fwd: [PATCH] c6x: kernel: setup: Type cast 'fdt' to "void *" for early_init_dt_scan() in machine_init()

2015-03-01 Thread Chen Gang S
Just forward to linux kernel mailing list. Thanks. Forwarded Message Subject: [PATCH] c6x: kernel: setup: Type cast 'fdt' to "void *" for early_init_dt_scan() in machine_init() Date: Sun, 01 Mar 2015 16:11:29 +0800 From: Chen Gang <762976...@qq.com> To: msal...@redhat.com, a-j

Fwd: [PATCH] c6x: kernel: setup: Type cast 'fdt' to "void *" for early_init_dt_scan() in machine_init()

2015-03-01 Thread Chen Gang S
Just forward to linux kernel mailing list. Thanks. Forwarded Message Subject: [PATCH] c6x: kernel: setup: Type cast 'fdt' to "void *" for early_init_dt_scan() in machine_init() Date: Sun, 01 Mar 2015 16:31:29 +0800 From: Chen Gang <762976...@qq.com> To: msal...@redhat.com, a-