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.kern

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 c

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 msal...@redhat.com, a-jacqu...@ti.com CC: linux-c6x-...@linux-c6x.org linux-c6x-...@linux-c6x.org, linux

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

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 a...@linux

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: ms

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: ms

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

2015-03-01 Thread Chen Gang S
: [PATCH] c6x: kernel: setup: Include "linux/console.h" if both VT and DUMMY_CONSOLE enabled Date: Sun, 01 Mar 2015 15:59:19 +0800 From: Chen Gang <762976...@qq.com> To: msal...@redhat.com, a-jacqu...@ti.com, r...@kernel.org CC: linux-c6x-...@linux-c6x.org , linux-kernel@vger.ker

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

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

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

2015-03-01 Thread Chen Gang S
: [PATCH] c6x: kernel: setup: Include linux/console.h if both VT and DUMMY_CONSOLE enabled Date: Sun, 01 Mar 2015 15:59:19 +0800 From: Chen Gang 762976...@qq.com To: msal...@redhat.com, a-jacqu...@ti.com, r...@kernel.org CC: linux-c6x-...@linux-c6x.org linux-c6x-...@linux-c6x.org, linux-kernel

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 ca

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 cause all breaks), and I shall try to fix it within

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

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 shall try to fix it within

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 CO

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 CONFIG_PM_SLEEP instead

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
any other members to send the fix patch for it. And for me, if "hci_u*filter" is related with the modules outside kernel (I guess, it should be), I recommend to put it to the related "uapi" header. Thanks. -- Chen Gang Open, share, and attitude like air, water, and l

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
guess, I am not the suitable member to make the related fix patch for this issue. Welcome any other members to send the fix patch for it. And for me, if hci_u*filter is related with the modules outside kernel (I guess, it should be), I recommend to put it to the related uapi header. Thanks. -- Chen

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
el members can not give a suitable test: - It is an API, but we only know kernel part implementation, and we also know that the kernel part implementation intends to use "unsigned long event_mask[2]" and "u32 *" type cast. - We don't know the other part implementation (

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/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 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
are not the suitable members to continue analyzing. So at present, for me, we can accept this patch. Thanks. -- Chen Gang Open, share, and attitude like air, water, and life which God blessed -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord

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):

[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
nter, so the caller need not use '&' for the 2 demotion array, or it reports warning: 'const unsigned int (*)[4]' is different with 'const unsigned int *'. Signed-off-by: Chen Gang --- net/bluetooth/hci_sock.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/net/bluetooth/

[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
the caller need not use '' for the 2 demotion array, or it reports warning: 'const unsigned int (*)[4]' is different with 'const unsigned int *'. Signed-off-by: Chen Gang gang.chen.5...@gmail.com --- net/bluetooth/hci_sock.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

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): net/bluetooth/hci_sock.c

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

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) +static inline int hci_test_bit(int nr, const void *addr) { return *((__u32 *) addr + (nr

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
ak the userspace API. > So might want to write unit tests to ensure working HCI filter before even > considering touching this. > For me, we have to remain hci_test_bit(), it is for "__u32 *" (which we can not change). The common test_bit() is for "unsigned long *", in

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
can not change). The common test_bit() is for unsigned long *, in this case, I guess it may cause issue under 64-bit environments. Thanks. -- Chen Gang Open, share, and attitude like air, water, and life which God blessed -- To unsubscribe from this list: send the line unsubscribe linux-kernel

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

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 under xtensa): CC [M] drivers

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 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] 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

[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
c inline int hci_test_bit(int nr, void *addr) ^ Signed-off-by: Chen Gang --- net/bluetooth/hci_sock.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/bluetooth/hci_sock.c b/net/bluetooth/hci_sock.c index 80c5a79..858b53a 100644 --- a/net/bluetooth/hc

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

2015-02-02 Thread Chen Gang S
x/list.h:8, from include/linux/module.h:9, from drivers/net/usb/sr9700.c:13: ./arch/xtensa/include/asm/processor.h:190:0: note: this is the location of the previous definition #define RSR(v,sr) __asm__ __volatile__ ("rsr %0,"__stringify(sr) : "=a"(v));

[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
inline int hci_test_bit(int nr, void *addr) ^ Signed-off-by: Chen Gang gang.chen.5...@gmail.com --- net/bluetooth/hci_sock.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/bluetooth/hci_sock.c b/net/bluetooth/hci_sock.c index 80c5a79..858b53a 100644

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 building

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

2015-02-02 Thread Chen Gang S
include/linux/module.h:9, from drivers/net/usb/sr9700.c:13: ./arch/xtensa/include/asm/processor.h:190:0: note: this is the location of the previous definition #define RSR(v,sr) __asm__ __volatile__ (rsr %0,__stringify(sr) : =a(v)); ^ Signed-off-by: Chen Gang gang.chen.5

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 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 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): [] diff --git a/net/bluetooth

[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
' defined but not used [-Wunused-function] static int st_rc_suspend(struct device *dev) ^ drivers/media/rc/st_rc.c:359:12: warning: 'st_rc_resume' defined but not used [-Wunused-function] static int st_rc_resume(struct device *dev) ^ Signed-off-by: Chen Gang

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

2015-02-01 Thread Chen Gang S
dma_mmap_writecombine' [-Werror=implicit-function-declaration] ret = dma_mmap_writecombine(cma_obj->base.dev->dev, vma, ^ Signed-off-by: Chen Gang --- drivers/gpu/drm/Kconfig | 3 ++- drivers/gpu/drm/atmel-hlcdc/Kconfig | 2 +- drivers/gpu/drm/imx/Kconfig | 2

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

2015-02-01 Thread Chen Gang S
=implicit-function-declaration] ret = dma_mmap_writecombine(cma_obj-base.dev-dev, vma, ^ Signed-off-by: Chen Gang gang.chen.5...@gmail.com --- drivers/gpu/drm/Kconfig | 3 ++- drivers/gpu/drm/atmel-hlcdc/Kconfig | 2 +- drivers/gpu/drm/imx/Kconfig | 2 +- drivers/gpu

[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
' defined but not used [-Wunused-function] static int st_rc_suspend(struct device *dev) ^ drivers/media/rc/st_rc.c:359:12: warning: 'st_rc_resume' defined but not used [-Wunused-function] static int st_rc_resume(struct device *dev) ^ Signed-off-by: Chen Gang

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] 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 gang.c...@sunrus.com.cn wrote: On 01/05/2015 04:59 PM, Martin Schwidefsky wrote: On Sat, 03 Jan 2015 11:44:04 +0800 Chen Gang gang.c...@sunrus.com.cn wrote: Thank you for your work. In honest

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

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 gang.c...@sunrus.com.cn 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

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' d

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] 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/s

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

2015-01-03 Thread Chen Gang
Our s390 can pass allmodconfig at present (although it is still has various warnings). Thanks. On 01/03/2015 05:29 PM, 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 >

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

2015-01-03 Thread Chen Gang
; ^ Signed-off-by: Chen Gang --- sound/soc/soc-pcm.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/soc-pcm.c b/sound/soc/soc-pcm.c index d6e8d6b..6b0136e 100644 --- a/sound/soc/soc-pcm.c +++ b/sound/soc/soc-pcm.c @@ -304,7 +304,7 @@ static bool

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

2015-01-03 Thread Chen Gang
defaults to 'int' in declaration of '_sclp_print_early' [-Wimplicit-int] extern _sclp_print_early(const char *); ^ At present, _sclp_print_early() is only used by puts(), so can still remain its declaration in 'misc.c' file. Signed-off-by: Chen Gang --- arch/s390/boot/compressed/misc.c

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

2015-01-03 Thread Chen Gang
Our s390 can pass allmodconfig at present (although it is still has various warnings). Thanks. On 01/03/2015 05:29 PM, 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

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

2015-01-03 Thread Chen Gang
defaults to 'int' in declaration of '_sclp_print_early' [-Wimplicit-int] extern _sclp_print_early(const char *); ^ At present, _sclp_print_early() is only used by puts(), so can still remain its declaration in 'misc.c' file. Signed-off-by: Chen Gang gang.chen.5...@gmail.com --- arch/s390

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

2015-01-03 Thread Chen Gang
; ^ Signed-off-by: Chen Gang gang.chen.5...@gmail.com --- sound/soc/soc-pcm.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/soc-pcm.c b/sound/soc/soc-pcm.c index d6e8d6b..6b0136e 100644 --- a/sound/soc/soc-pcm.c +++ b/sound/soc/soc-pcm.c @@ -304,7 +304,7 @@ static

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': sound

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

2015-01-02 Thread Chen Gang
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). :-) Thanks. On 01/02/2015 05:46 PM, Heiko Carstens wrote: > On Thu, Jan 01, 2015 at 10:27:32PM +0800, Chen Gang wrote: >> For C language,

Re: [PATCH v2] drivers: scsi: megaraid: Add "megaraid_sas_internal.h" for internal shared functions using

2015-01-02 Thread Chen Gang
Could you provide more detail for it, then I can make patch v3 (or I have no any idea about it). Thanks 发自我的 iPad > 在 2014年12月30日,下午8:45,Christoph Hellwig 写道: > >> On Mon, Dec 15, 2014 at 12:05:22AM +0800, Chen Gang wrote: >> For shared inline functions crossed source fi

Re: [PATCH v2] drivers: scsi: megaraid: Add megaraid_sas_internal.h for internal shared functions using

2015-01-02 Thread Chen Gang
Could you provide more detail for it, then I can make patch v3 (or I have no any idea about it). Thanks 发自我的 iPad 在 2014年12月30日,下午8:45,Christoph Hellwig h...@infradead.org 写道: On Mon, Dec 15, 2014 at 12:05:22AM +0800, Chen Gang wrote: For shared inline functions crossed source files

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

2015-01-02 Thread Chen Gang
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). :-) Thanks. On 01/02/2015 05:46 PM, Heiko Carstens wrote: On Thu, Jan 01, 2015 at 10:27:32PM +0800, Chen Gang wrote: For C language, it treats

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

2015-01-01 Thread Chen Gang
definition to match coding styles. Signed-off-by: Chen Gang --- arch/s390/hypfs/hypfs_vm.c|2 +- arch/s390/include/asm/timex.h |9 + 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/arch/s390/hypfs/hypfs_vm.c b/arch/s390/hypfs/hypfs_vm.c index 32040ac..344b622

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

2015-01-01 Thread Chen Gang
definition to match coding styles. Signed-off-by: Chen Gang gang.chen.5...@gmail.com --- arch/s390/hypfs/hypfs_vm.c|2 +- arch/s390/include/asm/timex.h |9 + 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/arch/s390/hypfs/hypfs_vm.c b/arch/s390/hypfs/hypfs_vm.c

Re: [PATCH] arch: uapi: asm: mman.h: Let MADV_FREE have same value for all architectures

2014-12-24 Thread Chen Gang
On 12/06/2014 11:57 PM, Chen Gang wrote: > Hello Maintainers: > [...] > > But for parisc gcc latest upstream compiler, I found several issues, and > I shall try to analyse them and communicate with gcc related members. > Oh, sorry, there is one my configuration issue wh

[PATCH next] net: netfilter: nfnetlink_cthelper: Remove 'const' and '&' to avoid warnings

2014-12-24 Thread Chen Gang
e_t count); ^ Signed-off-by: Chen Gang --- net/netfilter/nfnetlink_cthelper.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/net/netfilter/nfnetlink_cthelper.c b/net/netfilter/nfnetlink_cthelper.c index 9e287cb..a5599fc 100644 --- a/net/netfilter/nfnetlink_cthelper.c

[PATCH next] drivers: net: wireless: ath: wil6210: debugfs: Use 'uint64_t' instead of 'cycles_t' to avoid warnings

2014-12-24 Thread Chen Gang
t32_t divisor); ^ Signed-off-by: Chen Gang --- drivers/net/wireless/ath/wil6210/debugfs.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/wireless/ath/wil6210/debugfs.c b/drivers/net/wireless/ath/wil6210/debugfs.c index 4e6e145..cd991fa 100644 ---

[PATCH next] drivers: net: wireless: ath: wil6210: debugfs: Use 'uint64_t' instead of 'cycles_t' to avoid warnings

2014-12-24 Thread Chen Gang
); ^ Signed-off-by: Chen Gang gang.chen.5...@gmail.com --- drivers/net/wireless/ath/wil6210/debugfs.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/wireless/ath/wil6210/debugfs.c b/drivers/net/wireless/ath/wil6210/debugfs.c index 4e6e145..cd991fa

[PATCH next] net: netfilter: nfnetlink_cthelper: Remove 'const' and '' to avoid warnings

2014-12-24 Thread Chen Gang
); ^ Signed-off-by: Chen Gang gang.chen.5...@gmail.com --- net/netfilter/nfnetlink_cthelper.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/net/netfilter/nfnetlink_cthelper.c b/net/netfilter/nfnetlink_cthelper.c index 9e287cb..a5599fc 100644 --- a/net/netfilter

Re: [PATCH] arch: uapi: asm: mman.h: Let MADV_FREE have same value for all architectures

2014-12-24 Thread Chen Gang
On 12/06/2014 11:57 PM, Chen Gang wrote: Hello Maintainers: [...] But for parisc gcc latest upstream compiler, I found several issues, and I shall try to analyse them and communicate with gcc related members. Oh, sorry, there is one my configuration issue when cross compiling parisc

[PATCH v2] drivers: scsi: megaraid: Add "megaraid_sas_internal.h" for internal shared functions using

2014-12-14 Thread Chen Gang
For shared inline functions crossed source files, better to let it as "static inline" in a header file. For extern functions, better to declare them in header file. Signed-off-by: Chen Gang --- drivers/scsi/megaraid/megaraid_sas_base.c | 76 + drivers/scs

[PATCH v2] drivers: scsi: megaraid: Add megaraid_sas_internal.h for internal shared functions using

2014-12-14 Thread Chen Gang
For shared inline functions crossed source files, better to let it as static inline in a header file. For extern functions, better to declare them in header file. Signed-off-by: Chen Gang gang.chen.5...@gmail.com --- drivers/scsi/megaraid/megaraid_sas_base.c | 76 + drivers

Re: [PATCH] drivers: scsi: megaraid: Add "megaraid_sas_internal.h" for internal shared functions using

2014-12-10 Thread Chen Gang
On 12/10/14 16:47, Sumit Saxena wrote: >> From: Chen Gang [mailto:gang.chen.5...@gmail.com] >> Sent: Sunday, November 30, 2014 8:37 AM >> >> Is it OK? or if possible, please help rebase and send the patch for it >> based on >> the related branches and patches.

Re: [PATCH] drivers: scsi: megaraid: Add megaraid_sas_internal.h for internal shared functions using

2014-12-10 Thread Chen Gang
On 12/10/14 16:47, Sumit Saxena wrote: From: Chen Gang [mailto:gang.chen.5...@gmail.com] Sent: Sunday, November 30, 2014 8:37 AM Is it OK? or if possible, please help rebase and send the patch for it based on the related branches and patches. Sorry for delay in response, patch series sent

Re: [PATCH] arch: uapi: asm: mman.h: Let MADV_FREE have same value for all architectures

2014-12-09 Thread Chen Gang
Hello Andrew: Could you help check this patch, when you have time. And apply it if possible. I guess, this patch is a little urgent, since it changes uapi, if wait a little longer, more user mode programs (e.g. libc) may use MADV_FREE as various values. Thanks. On 12/6/14 06:00, Chen Gang

Re: [PATCH] arch: uapi: asm: mman.h: Let MADV_FREE have same value for all architectures

2014-12-09 Thread Chen Gang
Hello Andrew: Could you help check this patch, when you have time. And apply it if possible. I guess, this patch is a little urgent, since it changes uapi, if wait a little longer, more user mode programs (e.g. libc) may use MADV_FREE as various values. Thanks. On 12/6/14 06:00, Chen Gang

Re: [PATCH] arch: uapi: asm: mman.h: Let MADV_FREE have same value for all architectures

2014-12-07 Thread Chen Gang
On 12/8/14 00:14, John David Anglin wrote: > On 6-Dec-14, at 10:57 AM, Chen Gang wrote: > >> And I also want to consult about parisc assembly instructions, I can not >> find the related reference documents, could you help to have a check (or >> provide some related info

Re: [PATCH] mm: memcontrol: Skip test_mem_cgroup_node_reclaimable() when no MAX_NUMNODES or not more than 1

2014-12-07 Thread Chen Gang
OK, thanks. I shall try to send another patch to upstream kernel within this month. Send from Lenovo A788t. Michal Hocko wrote: >Hi, >this has been fixed last week already >http://marc.info/?l=linux-mm=141751847710864=2 > >On Sat 06-12-14 06:27:57,

Re: [PATCH] mm: memcontrol: Skip test_mem_cgroup_node_reclaimable() when no MAX_NUMNODES or not more than 1

2014-12-07 Thread Chen Gang
OK, thanks. I shall try to send another patch to upstream kernel within this month. Send from Lenovo A788t. Michal Hocko mho...@suse.cz wrote: Hi, this has been fixed last week already http://marc.info/?l=linux-mmm=141751847710864w=2 On Sat 06-12-14 06:27:57, Chen Gang wrote

Re: [PATCH] arch: uapi: asm: mman.h: Let MADV_FREE have same value for all architectures

2014-12-07 Thread Chen Gang
On 12/8/14 00:14, John David Anglin wrote: On 6-Dec-14, at 10:57 AM, Chen Gang wrote: And I also want to consult about parisc assembly instructions, I can not find the related reference documents, could you help to have a check (or provide some related information for it)? related contents

Re: [PATCH] arch: uapi: asm: mman.h: Let MADV_FREE have same value for all architectures

2014-12-06 Thread Chen Gang
' to find the position for saving relate context (registers). Welcome any suggestions, ideas, and completions. Thanks. On 12/06/2014 06:00 AM, Chen Gang wrote: > On 12/05/2014 02:54 PM, Minchan Kim wrote: >> On Fri, Dec 05, 2014 at 06:58:29AM +0800, Chen Gang wrote: >>> For uapi, ne

Re: [PATCH] arch: uapi: asm: mman.h: Let MADV_FREE have same value for all architectures

2014-12-06 Thread Chen Gang
relate context (registers). Welcome any suggestions, ideas, and completions. Thanks. On 12/06/2014 06:00 AM, Chen Gang wrote: On 12/05/2014 02:54 PM, Minchan Kim wrote: On Fri, Dec 05, 2014 at 06:58:29AM +0800, Chen Gang wrote: For uapi, need try to let all macros have same value

[PATCH] mm: memcontrol: Skip test_mem_cgroup_node_reclaimable() when no MAX_NUMNODES or not more than 1

2014-12-05 Thread Chen Gang
sed [-Wunused-function] static bool test_mem_cgroup_node_reclaimable(struct mem_cgroup *memcg, ^ Signed-off-by: Chen Gang --- mm/memcontrol.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mm/memcontrol.c b/mm/memcontrol.c index c6ac50e..d538b08 100644 --- a/mm

Re: [PATCH] arch: uapi: asm: mman.h: Let MADV_FREE have same value for all architectures

2014-12-05 Thread Chen Gang
On 12/05/2014 02:54 PM, Minchan Kim wrote: > On Fri, Dec 05, 2014 at 06:58:29AM +0800, Chen Gang wrote: >> For uapi, need try to let all macros have same value, and MADV_FREE is >> added into main branch recently, so need redefine MADV_FREE for it. >> >> At present,

Re: [PATCH] arch: uapi: asm: mman.h: Let MADV_FREE have same value for all architectures

2014-12-05 Thread Chen Gang
On 12/05/2014 08:14 AM, Darrick J. Wong wrote: > On Fri, Dec 05, 2014 at 06:58:29AM +0800, Chen Gang wrote: >> For uapi, need try to let all macros have same value, and MADV_FREE is >> added into main branch recently, so need redefine MADV_FREE for it. >> >> At present,

Re: [PATCH] arch: uapi: asm: mman.h: Let MADV_FREE have same value for all architectures

2014-12-05 Thread Chen Gang
On 12/05/2014 08:14 AM, Darrick J. Wong wrote: On Fri, Dec 05, 2014 at 06:58:29AM +0800, Chen Gang wrote: For uapi, need try to let all macros have same value, and MADV_FREE is added into main branch recently, so need redefine MADV_FREE for it. At present, '8' can be shared with all

Re: [PATCH] arch: uapi: asm: mman.h: Let MADV_FREE have same value for all architectures

2014-12-05 Thread Chen Gang
On 12/05/2014 02:54 PM, Minchan Kim wrote: On Fri, Dec 05, 2014 at 06:58:29AM +0800, Chen Gang wrote: For uapi, need try to let all macros have same value, and MADV_FREE is added into main branch recently, so need redefine MADV_FREE for it. At present, '8' can be shared with all architectures

[PATCH] mm: memcontrol: Skip test_mem_cgroup_node_reclaimable() when no MAX_NUMNODES or not more than 1

2014-12-05 Thread Chen Gang
] static bool test_mem_cgroup_node_reclaimable(struct mem_cgroup *memcg, ^ Signed-off-by: Chen Gang gang.chen.5...@gmail.com --- mm/memcontrol.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mm/memcontrol.c b/mm/memcontrol.c index c6ac50e..d538b08 100644

[PATCH] arch: uapi: asm: mman.h: Let MADV_FREE have same value for all architectures

2014-12-04 Thread Chen Gang
For uapi, need try to let all macros have same value, and MADV_FREE is added into main branch recently, so need redefine MADV_FREE for it. At present, '8' can be shared with all architectures, so redefine it to '8'. Signed-off-by: Chen Gang --- arch/alpha/include/uapi/asm/mman.h | 2

[PATCH] arch: uapi: asm: mman.h: Let MADV_FREE have same value for all architectures

2014-12-04 Thread Chen Gang
For uapi, need try to let all macros have same value, and MADV_FREE is added into main branch recently, so need redefine MADV_FREE for it. At present, '8' can be shared with all architectures, so redefine it to '8'. Signed-off-by: Chen Gang gang.chen.5...@gmail.com --- arch/alpha/include/uapi

[PATCH] drivers: staging: rtl8192e: Include "asm/unaligned.h" instead of "access_ok.h" in "rtl819x_BAProc.c"

2014-12-02 Thread Chen Gang
ot;asm/unaligned.h" too. And also include "asm/byteorder.h" since it is the first include file". Signed-off-by: Chen Gang --- drivers/staging/rtl8192e/rtl819x_BAProc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192e/rtl819

[PATCH] drivers: staging: rtl8192e: Include asm/unaligned.h instead of access_ok.h in rtl819x_BAProc.c

2014-12-02 Thread Chen Gang
include file. Signed-off-by: Chen Gang gang.chen.5...@gmail.com --- drivers/staging/rtl8192e/rtl819x_BAProc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192e/rtl819x_BAProc.c b/drivers/staging/rtl8192e/rtl819x_BAProc.c index 66ea791..0415e02 100644

Re: [PATCH] arch: uapi: asm: mman.h: Support MADV_FREE for madvise()

2014-12-01 Thread Chen Gang
On 12/2/14 09:55, James Bottomley wrote: > On Mon, 2014-12-01 at 17:01 -0500, Carlos O'Donell wrote: >> On Mon, Dec 1, 2014 at 4:35 PM, Geert Uytterhoeven >> wrote: >>> On Mon, Dec 1, 2014 at 9:52 PM, Max Filippov wrote: >>>> On Mon, Dec 1, 2014

Re: [PATCH] arch: uapi: asm: mman.h: Support MADV_FREE for madvise()

2014-12-01 Thread Chen Gang
On 12/2/14 06:01, Carlos O'Donell wrote: > On Mon, Dec 1, 2014 at 4:35 PM, Geert Uytterhoeven > wrote: >> On Mon, Dec 1, 2014 at 9:52 PM, Max Filippov wrote: >>> On Mon, Dec 1, 2014 at 11:43 PM, Chen Gang wrote: >>>> At present, kernel supports madvise(

[PATCH] arch: uapi: asm: mman.h: Support MADV_FREE for madvise()

2014-12-01 Thread Chen Gang
use in this function) case MADV_FREE: ^ make[1]: *** [mm/madvise.o] Error 1 make: *** [mm] Error 2 Signed-off-by: Chen Gang --- arch/alpha/include/uapi/asm/mman.h | 1 + arch/mips/include/uapi/asm/mman.h | 1 + arch/parisc/include/uapi/asm/mman.h | 1 + arch/xtensa/include/uapi/asm/m

[PATCH] arch: uapi: asm: mman.h: Support MADV_FREE for madvise()

2014-12-01 Thread Chen Gang
use in this function) case MADV_FREE: ^ make[1]: *** [mm/madvise.o] Error 1 make: *** [mm] Error 2 Signed-off-by: Chen Gang --- arch/alpha/include/uapi/asm/mman.h | 1 + arch/mips/include/uapi/asm/mman.h | 1 + arch/parisc/include/uapi/asm/mman.h | 1 + arch/xtensa/include/uapi/asm/m

<    1   2   3   4   5   6   7   8   9   10   >