[PATCH] arch/microblaze/include/asm/uaccess.h: Use pr_devel() instead of pr_debug()

2014-08-05 Thread Chen Gang
, ^ Signed-off-by: Chen Gang gang.chen.5...@gmail.com --- arch/microblaze/include/asm/uaccess.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/microblaze/include/asm/uaccess.h b/arch/microblaze/include/asm/uaccess.h index 0aa0057..59a89a6 100644 --- a/arch/microblaze

Re: [PATCH] drivers/isdn/hisax/Kconfig: Let HISAX_NETJET skip microblaze architecture

2014-08-05 Thread Chen Gang
On 08/05/2014 12:21 PM, Chen Gang wrote: On 08/05/2014 03:44 AM, Geert Uytterhoeven wrote: On Mon, Aug 4, 2014 at 9:23 PM, Jean Delvare jdelv...@suse.de wrote: --- a/drivers/isdn/hisax/Kconfig +++ b/drivers/isdn/hisax/Kconfig @@ -237,7 +237,7 @@ config HISAX_MIC config HISAX_NETJET

Re: [PATCH] drivers/isdn/hisax/Kconfig: Let HISAX_NETJET skip microblaze architecture

2014-08-04 Thread Chen Gang
PU_LITTLE_ENDIAN >> defined on all little-endian architectures, so that driver can depend >> on that if needed? > > Yes, that idea pops up once in a while. > > BTW, these days little endian PPC is also supported by Linux... > >> Alternatively, it might make m

[PATCH] drivers/isdn/hisax/Kconfig: Let HISAX_NETJET skip microblaze architecture

2014-08-04 Thread Chen Gang
'setup_netjet_s': drivers/isdn/hisax/nj_s.c:265:2: error: #error "not running on big endian machines now" #error "not running on big endian machines now" ^ Signed-off-by: Chen Gang --- drivers/isdn/hisax/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 dele

[PATCH] drivers/isdn/hisax/Kconfig: Let HISAX_NETJET skip microblaze architecture

2014-08-04 Thread Chen Gang
'setup_netjet_s': drivers/isdn/hisax/nj_s.c:265:2: error: #error not running on big endian machines now #error not running on big endian machines now ^ Signed-off-by: Chen Gang gang.chen.5...@gmail.com --- drivers/isdn/hisax/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions

Re: [PATCH] drivers/isdn/hisax/Kconfig: Let HISAX_NETJET skip microblaze architecture

2014-08-04 Thread Chen Gang
sounds reasonable to me. Hello Geert: Could you help send the related patch for it? 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] arch: score: Export necessary symbols in related files

2014-08-02 Thread Chen Gang
On 08/02/2014 05:20 PM, Lennox Wu wrote: > Yes, I will issue a PULL request to Linus. Thank you very much! And I shall start allmodconfig for other architecture (microblaze), next week. > Best, > Lennox > > Chen Gang 於 2014/8/2 下午 04:18 寫道: >> On 08/02/2014 02:2

Re: [PATCH] arch: score: Export necessary symbols in related files

2014-08-02 Thread Chen Gang
On 08/02/2014 02:20 PM, Lennox Wu wrote: > It was merged into Score's git tree, and it will show in the next PULL > request. > OK, thanks. And next, will it be integrated into upstream main line? (I guess it will be). Thanks -- Chen Gang Open, share, and attitude like air, water,

Re: [PATCH] arch: score: Export necessary symbols in related files

2014-08-02 Thread Chen Gang
On 08/02/2014 02:20 PM, Lennox Wu wrote: It was merged into Score's git tree, and it will show in the next PULL request. OK, thanks. And next, will it be integrated into upstream main line? (I guess it will be). Thanks -- Chen Gang Open, share, and attitude like air, water, and life which

Re: [PATCH] arch: score: Export necessary symbols in related files

2014-08-02 Thread Chen Gang
On 08/02/2014 05:20 PM, Lennox Wu wrote: Yes, I will issue a PULL request to Linus. Thank you very much! And I shall start allmodconfig for other architecture (microblaze), next week. Best, Lennox Chen Gang 於 2014/8/2 下午 04:18 寫道: On 08/02/2014 02:20 PM, Lennox Wu wrote: It was merged

Re: [PATCH] staging: Change kzalloc to kcalloc

2014-07-24 Thread Chen Gang
On 07/25/2014 11:39 AM, Nick Krause wrote: > On Thu, Jul 24, 2014 at 11:34 PM, Chen Gang wrote: >> >> >> On 07/25/2014 11:30 AM, Nick Krause wrote: >>> On Thu, Jul 24, 2014 at 11:21 PM, Chen Gang >>> wrote: >>>> >>>> >>>

Re: [PATCH] staging: Change kzalloc to kcalloc

2014-07-24 Thread Chen Gang
On 07/25/2014 11:30 AM, Nick Krause wrote: > On Thu, Jul 24, 2014 at 11:21 PM, Chen Gang wrote: >> >> >> On 07/25/2014 11:13 AM, Nick Krause wrote: >>> On Thu, Jul 24, 2014 at 11:09 PM, Chen Gang >>> wrote: >>>> >>>> >>>

Re: [PATCH] staging: Change kzalloc to kcalloc

2014-07-24 Thread Chen Gang
On 07/25/2014 11:13 AM, Nick Krause wrote: > On Thu, Jul 24, 2014 at 11:09 PM, Chen Gang wrote: >> >> >> On 07/25/2014 10:53 AM, Nick Krause wrote: >>> On Thu, Jul 24, 2014 at 10:47 PM, Chen Gang >>> wrote: >>>> >>>> >>>

Re: [PATCH] staging: Change kzalloc to kcalloc

2014-07-24 Thread Chen Gang
On 07/25/2014 10:53 AM, Nick Krause wrote: > On Thu, Jul 24, 2014 at 10:47 PM, Chen Gang wrote: >> >> >> On 07/25/2014 10:20 AM, Nick Krause wrote: >>> On Thu, Jul 24, 2014 at 10:15 PM, Chen Gang >>> wrote: >>>> >>>> Excu

Re: [PATCH] staging: Change kzalloc to kcalloc

2014-07-24 Thread Chen Gang
On 07/25/2014 10:20 AM, Nick Krause wrote: > On Thu, Jul 24, 2014 at 10:15 PM, Chen Gang wrote: >> >> Excuse me, I did not see the patch details, but I guess it is about >> whether welcome a new member to upstream kernel. For me, I have 3 ideas >> about it if I am

Re: [PATCH] staging: Change kzalloc to kcalloc

2014-07-24 Thread Chen Gang
l. >>>> Take the challenge at http://eudyptula-challenge.org. After you've solved >>>> all >>>> tasks we'll accept patches from you. >>>> >>>> -- >>>> Thanks, >>>> //richard >>> That's fine, >>> So Sorry :

Re: [PATCH] staging: Change kzalloc to kcalloc

2014-07-24 Thread Chen Gang
. -- Thanks, //richard That's fine, So Sorry :(. Nick I sent the challenge a email. Seems that there not replying. Patience is a virtue. Thanks, //richard -- Chen Gang Open, share, and attitude like air, water, and life which God blessed -- To unsubscribe from this list: send

Re: [PATCH] staging: Change kzalloc to kcalloc

2014-07-24 Thread Chen Gang
On 07/25/2014 10:20 AM, Nick Krause wrote: On Thu, Jul 24, 2014 at 10:15 PM, Chen Gang gang.chen.5...@gmail.com wrote: Excuse me, I did not see the patch details, but I guess it is about whether welcome a new member to upstream kernel. For me, I have 3 ideas about it if I am a newbie

Re: [PATCH] staging: Change kzalloc to kcalloc

2014-07-24 Thread Chen Gang
On 07/25/2014 10:53 AM, Nick Krause wrote: On Thu, Jul 24, 2014 at 10:47 PM, Chen Gang gang.chen.5...@gmail.com wrote: On 07/25/2014 10:20 AM, Nick Krause wrote: On Thu, Jul 24, 2014 at 10:15 PM, Chen Gang gang.chen.5...@gmail.com wrote: Excuse me, I did not see the patch details, but I

Re: [PATCH] staging: Change kzalloc to kcalloc

2014-07-24 Thread Chen Gang
On 07/25/2014 11:13 AM, Nick Krause wrote: On Thu, Jul 24, 2014 at 11:09 PM, Chen Gang gang.chen.5...@gmail.com wrote: On 07/25/2014 10:53 AM, Nick Krause wrote: On Thu, Jul 24, 2014 at 10:47 PM, Chen Gang gang.chen.5...@gmail.com wrote: On 07/25/2014 10:20 AM, Nick Krause wrote

Re: [PATCH] staging: Change kzalloc to kcalloc

2014-07-24 Thread Chen Gang
On 07/25/2014 11:30 AM, Nick Krause wrote: On Thu, Jul 24, 2014 at 11:21 PM, Chen Gang gang.chen.5...@gmail.com wrote: On 07/25/2014 11:13 AM, Nick Krause wrote: On Thu, Jul 24, 2014 at 11:09 PM, Chen Gang gang.chen.5...@gmail.com wrote: On 07/25/2014 10:53 AM, Nick Krause wrote

Re: [PATCH] staging: Change kzalloc to kcalloc

2014-07-24 Thread Chen Gang
On 07/25/2014 11:39 AM, Nick Krause wrote: On Thu, Jul 24, 2014 at 11:34 PM, Chen Gang gang.chen.5...@gmail.com wrote: On 07/25/2014 11:30 AM, Nick Krause wrote: On Thu, Jul 24, 2014 at 11:21 PM, Chen Gang gang.chen.5...@gmail.com wrote: On 07/25/2014 11:13 AM, Nick Krause wrote

Re: [PATCH] drivers/pwm/Kconfig: Let PWM_CLPS711X depend on HAS_IOMEM

2014-07-23 Thread Chen Gang
Oh, sorry, after a long discussion, we have to keep current status no touch -- let individual modules depends on HAS_IOMEM if they need it. So, this patch is still alive, please help check it, when you have time. Thanks. On 07/14/2014 06:00 PM, Chen Gang wrote: > > 在 2014年7月14日,下午5:33,T

Re: [PATCH] drivers/crypto/Kconfig: Let 'DEV_QCE' depend on both HAS_DMA and HAS_IOMEM

2014-07-23 Thread Chen Gang
Oh, sorry, this patch can still be alive, after a long discussion, we need keep current status no touch -- still let individual modules to depends on HAS_IOMEM and/or HAS_DMA, if they need them. So please help check the patch, when you have time. Thanks. On 07/14/2014 08:12 PM, Chen Gang wrote

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-23 Thread Chen Gang
On 07/23/2014 07:30 PM, Dan Carpenter wrote: > On Wed, Jul 23, 2014 at 07:09:22PM +0800, Chen Gang wrote: >> >> >> On 07/17/2014 05:19 PM, Chen Gang wrote: >>> >>> >>> On 07/17/2014 05:16 PM, Dan Carpenter wrote: >>>&g

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-23 Thread Chen Gang
On 07/17/2014 05:19 PM, Chen Gang wrote: > > > On 07/17/2014 05:16 PM, Dan Carpenter wrote: >> On Thu, Jul 17, 2014 at 04:59:09PM +0800, Chen Gang wrote: >>>>> + return (__force void __iomem *)ERR_PTR(-ENXIO); >>>> >>>> T

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-23 Thread Chen Gang
On 07/17/2014 05:19 PM, Chen Gang wrote: On 07/17/2014 05:16 PM, Dan Carpenter wrote: On Thu, Jul 17, 2014 at 04:59:09PM +0800, Chen Gang wrote: + return (__force void __iomem *)ERR_PTR(-ENXIO); There's apparently an IOMEM_ERR_PTR() for this nowadays... IOMEM_ERR_PTR() is defined

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-23 Thread Chen Gang
On 07/23/2014 07:30 PM, Dan Carpenter wrote: On Wed, Jul 23, 2014 at 07:09:22PM +0800, Chen Gang wrote: On 07/17/2014 05:19 PM, Chen Gang wrote: On 07/17/2014 05:16 PM, Dan Carpenter wrote: On Thu, Jul 17, 2014 at 04:59:09PM +0800, Chen Gang wrote: + return (__force void __iomem

Re: [PATCH] drivers/crypto/Kconfig: Let 'DEV_QCE' depend on both HAS_DMA and HAS_IOMEM

2014-07-23 Thread Chen Gang
Oh, sorry, this patch can still be alive, after a long discussion, we need keep current status no touch -- still let individual modules to depends on HAS_IOMEM and/or HAS_DMA, if they need them. So please help check the patch, when you have time. Thanks. On 07/14/2014 08:12 PM, Chen Gang wrote

Re: [PATCH] drivers/pwm/Kconfig: Let PWM_CLPS711X depend on HAS_IOMEM

2014-07-23 Thread Chen Gang
Oh, sorry, after a long discussion, we have to keep current status no touch -- let individual modules depends on HAS_IOMEM if they need it. So, this patch is still alive, please help check it, when you have time. Thanks. On 07/14/2014 06:00 PM, Chen Gang wrote: 在 2014年7月14日,下午5:33,Thierry

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-22 Thread Chen Gang
On 07/22/2014 06:32 PM, Arnd Bergmann wrote: > On Sunday 20 July 2014 17:45:40 Chen Gang wrote: >>> >>> Next, I shall: >>> >>> - Remove HAS_IOMEM and NO_IOMEM from kernel, firstly. >>> >>> - Try to make dummy IOMEM functions for sc

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-22 Thread Chen Gang
On 07/22/2014 06:32 PM, Arnd Bergmann wrote: On Sunday 20 July 2014 17:45:40 Chen Gang wrote: Next, I shall: - Remove HAS_IOMEM and NO_IOMEM from kernel, firstly. - Try to make dummy IOMEM functions for score architecture. - Continue discussing with UML for it. Oh, sorry, I

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-20 Thread Chen Gang
On 07/20/2014 05:51 PM, Chen Gang wrote: > On 07/20/2014 05:45 PM, Richard Weinberger wrote: >> >> >> Am 20.07.2014 10:38, schrieb Chen Gang: >>> On 07/19/2014 02:02 AM, Chen Gang wrote: >>>>> 2014-07-18 18:51 GMT+08:00 Richard Weinberger :

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-20 Thread Chen Gang
On 07/20/2014 05:45 PM, Richard Weinberger wrote: > > > Am 20.07.2014 10:38, schrieb Chen Gang: >> On 07/19/2014 02:02 AM, Chen Gang wrote: >>>> 2014-07-18 18:51 GMT+08:00 Richard Weinberger : >>>>> Am 18.07.2014 12:44, schrieb Chen Gang: >>>&

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-20 Thread Chen Gang
On 07/20/2014 04:38 PM, Chen Gang wrote: > On 07/19/2014 02:02 AM, Chen Gang wrote: >>> 2014-07-18 18:51 GMT+08:00 Richard Weinberger : >>>> Am 18.07.2014 12:44, schrieb Chen Gang: >>>>> On 07/18/2014 03:35 PM, Richard Weinberger wrote: >&g

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-20 Thread Chen Gang
On 07/19/2014 02:02 AM, Chen Gang wrote: >> 2014-07-18 18:51 GMT+08:00 Richard Weinberger : >>> Am 18.07.2014 12:44, schrieb Chen Gang: >>>> On 07/18/2014 03:35 PM, Richard Weinberger wrote: >>>>> Am 18.07.2014 02:36, schrieb Chen Gang: >>>&g

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-20 Thread Chen Gang
On 07/19/2014 02:02 AM, Chen Gang wrote: 2014-07-18 18:51 GMT+08:00 Richard Weinberger rich...@nod.at: Am 18.07.2014 12:44, schrieb Chen Gang: On 07/18/2014 03:35 PM, Richard Weinberger wrote: Am 18.07.2014 02:36, schrieb Chen Gang: On 07/18/2014 02:09 AM, Richard Weinberger wrote: Am

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-20 Thread Chen Gang
On 07/20/2014 04:38 PM, Chen Gang wrote: On 07/19/2014 02:02 AM, Chen Gang wrote: 2014-07-18 18:51 GMT+08:00 Richard Weinberger rich...@nod.at: Am 18.07.2014 12:44, schrieb Chen Gang: On 07/18/2014 03:35 PM, Richard Weinberger wrote: Am 18.07.2014 02:36, schrieb Chen Gang: On 07/18/2014 02

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-20 Thread Chen Gang
On 07/20/2014 05:45 PM, Richard Weinberger wrote: Am 20.07.2014 10:38, schrieb Chen Gang: On 07/19/2014 02:02 AM, Chen Gang wrote: 2014-07-18 18:51 GMT+08:00 Richard Weinberger rich...@nod.at: Am 18.07.2014 12:44, schrieb Chen Gang: On 07/18/2014 03:35 PM, Richard Weinberger wrote: Am

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-20 Thread Chen Gang
On 07/20/2014 05:51 PM, Chen Gang wrote: On 07/20/2014 05:45 PM, Richard Weinberger wrote: Am 20.07.2014 10:38, schrieb Chen Gang: On 07/19/2014 02:02 AM, Chen Gang wrote: 2014-07-18 18:51 GMT+08:00 Richard Weinberger rich...@nod.at: Am 18.07.2014 12:44, schrieb Chen Gang: On 07/18/2014 03

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-18 Thread Chen Gang
-07-18 18:51 GMT+08:00 Richard Weinberger : >> Am 18.07.2014 12:44, schrieb Chen Gang: >>> On 07/18/2014 03:35 PM, Richard Weinberger wrote: >>>> Am 18.07.2014 02:36, schrieb Chen Gang: >>>>> >>>>> On 07/18/2014 02:09 AM, Richard Weinberger wro

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-18 Thread Chen Gang
On 07/18/2014 03:35 PM, Richard Weinberger wrote: > Am 18.07.2014 02:36, schrieb Chen Gang: >> >> On 07/18/2014 02:09 AM, Richard Weinberger wrote: >>> Am 17.07.2014 12:48, schrieb Arnd Bergmann: >>>> AFAICT, NO_IOMEM only has a real purpose on UML these days

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-18 Thread Chen Gang
+08:00 Richard Weinberger rich...@nod.at: Am 18.07.2014 12:44, schrieb Chen Gang: On 07/18/2014 03:35 PM, Richard Weinberger wrote: Am 18.07.2014 02:36, schrieb Chen Gang: On 07/18/2014 02:09 AM, Richard Weinberger wrote: Am 17.07.2014 12:48, schrieb Arnd Bergmann: AFAICT, NO_IOMEM only has

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-18 Thread Chen Gang
On 07/18/2014 03:35 PM, Richard Weinberger wrote: Am 18.07.2014 02:36, schrieb Chen Gang: On 07/18/2014 02:09 AM, Richard Weinberger wrote: Am 17.07.2014 12:48, schrieb Arnd Bergmann: AFAICT, NO_IOMEM only has a real purpose on UML these days. Could we take a shortcut here and make

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-17 Thread Chen Gang
UML appear in generic include directory. And then remove all HAS_IOMEM and NO_IOMEM from kernel. If score sticks to NO_IOMEM, and can not remove score from kernel only because of this reason. I also suggest implement dummy functions within score itself. Thanks. -- Chen Gang Open, share, and attit

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-17 Thread Chen Gang
On 07/18/2014 05:05 AM, Arnd Bergmann wrote: > On Thursday 17 July 2014 16:41:14 Chris Metcalf wrote: >> On 7/17/2014 7:28 AM, Chen Gang wrote: >>> On 07/17/2014 06:48 PM, Arnd Bergmann wrote: >>>> AFAICT, NO_IOMEM only has a real purpose on UML these days. Coul

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-17 Thread Chen Gang
On 07/17/2014 06:38 PM, Arnd Bergmann wrote: > On Thursday 17 July 2014 17:29:31 Chen Gang wrote: >>> >>> COMPILE_TEST is a great tool in general, but it has its limits. >>> In particular, the case for !CONFIG_IOMEM is completely obscure >>> and we won't

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-17 Thread Chen Gang
On 07/17/2014 06:58 PM, Richard Weinberger wrote: > Am 17.07.2014 12:28, schrieb Arnd Bergmann: >> On Thursday 17 July 2014 11:26:57 Richard Weinberger wrote: >>> Am 17.07.2014 11:20, schrieb Arnd Bergmann: >>>> On Thursday 17 July

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-17 Thread Chen Gang
On 07/17/2014 06:48 PM, Arnd Bergmann wrote: > On Thursday 17 July 2014 12:40:25 Lars-Peter Clausen wrote: >> On 07/17/2014 11:20 AM, Arnd Bergmann wrote: >>> On Thursday 17 July 2014 09:27:58 Chen Gang wrote: >>>> gfp_t gfp_

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-17 Thread Chen Gang
ils members. At present, the related score maintainers are still active in upstream kernel, so we also need the related maintainers' ideas and suggestions. Thanks -- Chen Gang Open, share, and attitude like air, water, and life which God blessed -- To unsubscribe from this list: send the line &qu

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-17 Thread Chen Gang
On 07/17/2014 05:20 PM, Arnd Bergmann wrote: > On Thursday 17 July 2014 09:27:58 Chen Gang wrote: >> gfp_t gfp_mask, unsigned int order); >> extern void devm_free_pages(struct device *dev, unsigned long addr); >> >> +#ifde

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-17 Thread Chen Gang
On 07/17/2014 05:16 PM, Dan Carpenter wrote: > On Thu, Jul 17, 2014 at 04:59:09PM +0800, Chen Gang wrote: >>>> + return (__force void __iomem *)ERR_PTR(-ENXIO); >>> >>> There's apparently an IOMEM_ERR_PTR() for this nowadays... >>> >> &g

Re: [PATCH v3] arch/score/include/uapi/asm/ptrace.h: Remove the macros which not be used currently

2014-07-17 Thread Chen Gang
t;> >> >>> Best, >>> Lennox >>> >>> 2014-07-17 9:49 GMT+08:00 David Rientjes : >>>> >>>> On Thu, 26 Jun 2014, David Rientjes wrote: >>>> >>>>> On Thu, 26 Jun 2014, Chen Gang wrote: >>>>> &g

Re: [PATCH] arch: score: Export necessary symbols in related files

2014-07-17 Thread Chen Gang
On 07/17/2014 10:22 AM, Lennox Wu wrote: > Acked-by: Lennox Wu > OK, thank you for your work. > 2014-07-09 16:48 GMT+08:00 Chen Gang : >> 'csum_partial_copy_from_user' and 'flush_dcache_page' are also needed by >> outside modules, so need export them in the related fil

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-17 Thread Chen Gang
On 07/17/2014 04:37 PM, Thierry Reding wrote: > On Thu, Jul 17, 2014 at 09:27:58AM +0800, Chen Gang wrote: > [...] >> diff --git a/include/linux/device.h b/include/linux/device.h >> index c2421e0..a7500c3 100644 >> --- a/include/linux/device.h >> +++ b/include/linux

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-17 Thread Chen Gang
On 07/17/2014 04:37 PM, Thierry Reding wrote: On Thu, Jul 17, 2014 at 09:27:58AM +0800, Chen Gang wrote: [...] diff --git a/include/linux/device.h b/include/linux/device.h index c2421e0..a7500c3 100644 --- a/include/linux/device.h +++ b/include/linux/device.h @@ -630,7 +630,16 @@ extern

Re: [PATCH] arch: score: Export necessary symbols in related files

2014-07-17 Thread Chen Gang
On 07/17/2014 10:22 AM, Lennox Wu wrote: Acked-by: Lennox Wulennox...@gmail.com OK, thank you for your work. 2014-07-09 16:48 GMT+08:00 Chen Gang gang.chen.5...@gmail.com: 'csum_partial_copy_from_user' and 'flush_dcache_page' are also needed by outside modules, so need export them

Re: [PATCH v3] arch/score/include/uapi/asm/ptrace.h: Remove the macros which not be used currently

2014-07-17 Thread Chen Gang
, 26 Jun 2014, Chen Gang wrote: 'COUNTER' and other same kind macros are too common to use, and easy to get conflict with other modules. At present, they are not used, so it is OK to simply remove them. And the related warning (allmodconfig with score): CC [M] drivers/md/raid1.o

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-17 Thread Chen Gang
On 07/17/2014 05:16 PM, Dan Carpenter wrote: On Thu, Jul 17, 2014 at 04:59:09PM +0800, Chen Gang wrote: + return (__force void __iomem *)ERR_PTR(-ENXIO); There's apparently an IOMEM_ERR_PTR() for this nowadays... IOMEM_ERR_PTR() is defined within lib/devres.c, not in ./include. But may

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-17 Thread Chen Gang
On 07/17/2014 05:20 PM, Arnd Bergmann wrote: On Thursday 17 July 2014 09:27:58 Chen Gang wrote: gfp_t gfp_mask, unsigned int order); extern void devm_free_pages(struct device *dev, unsigned long addr); +#ifdef CONFIG_HAS_IOMEM void __iomem

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-17 Thread Chen Gang
shall try to analyse (it is about link symbols), and maybe need communicate with gcc/binutils members. At present, the related score maintainers are still active in upstream kernel, so we also need the related maintainers' ideas and suggestions. Thanks -- Chen Gang Open, share, and attitude like

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-17 Thread Chen Gang
On 07/17/2014 06:48 PM, Arnd Bergmann wrote: On Thursday 17 July 2014 12:40:25 Lars-Peter Clausen wrote: On 07/17/2014 11:20 AM, Arnd Bergmann wrote: On Thursday 17 July 2014 09:27:58 Chen Gang wrote: gfp_t gfp_mask, unsigned int order); extern

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-17 Thread Chen Gang
On 07/17/2014 06:58 PM, Richard Weinberger wrote: Am 17.07.2014 12:28, schrieb Arnd Bergmann: On Thursday 17 July 2014 11:26:57 Richard Weinberger wrote: Am 17.07.2014 11:20, schrieb Arnd Bergmann: On Thursday 17 July 2014 09:27:58 Chen Gang wrote

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-17 Thread Chen Gang
On 07/17/2014 06:38 PM, Arnd Bergmann wrote: On Thursday 17 July 2014 17:29:31 Chen Gang wrote: COMPILE_TEST is a great tool in general, but it has its limits. In particular, the case for !CONFIG_IOMEM is completely obscure and we won't find any bugs by allowing more drivers to be built

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-17 Thread Chen Gang
On 07/18/2014 05:05 AM, Arnd Bergmann wrote: On Thursday 17 July 2014 16:41:14 Chris Metcalf wrote: On 7/17/2014 7:28 AM, Chen Gang wrote: On 07/17/2014 06:48 PM, Arnd Bergmann wrote: AFAICT, NO_IOMEM only has a real purpose on UML these days. Could we take a shortcut here and make

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-17 Thread Chen Gang
all HAS_IOMEM and NO_IOMEM from kernel. If score sticks to NO_IOMEM, and can not remove score from kernel only because of this reason. I also suggest implement dummy functions within score itself. Thanks. -- Chen Gang Open, share, and attitude like air, water, and life which God blessed

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-16 Thread Chen Gang
On 07/15/2014 10:38 PM, Chen Gang wrote: > On 07/15/2014 09:11 AM, Chen Gang wrote: >> >> >> On 07/15/2014 08:53 AM, Guenter Roeck wrote: >>> On 07/14/2014 05:34 PM, Chen Gang wrote: >>>> On 07/14/2014 05:22 PM, Chen Gang wrote: >>>

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-16 Thread Chen Gang
On 07/15/2014 10:38 PM, Chen Gang wrote: On 07/15/2014 09:11 AM, Chen Gang wrote: On 07/15/2014 08:53 AM, Guenter Roeck wrote: On 07/14/2014 05:34 PM, Chen Gang wrote: On 07/14/2014 05:22 PM, Chen Gang wrote: 在 2014年7月14日,下午4:57,Richard Weinberger rich...@nod.at 写道: Am 14.07.2014 10:48

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-15 Thread Chen Gang
On 07/15/2014 09:11 AM, Chen Gang wrote: > > > On 07/15/2014 08:53 AM, Guenter Roeck wrote: >> On 07/14/2014 05:34 PM, Chen Gang wrote: >>> On 07/14/2014 05:22 PM, Chen Gang wrote: >>>> >>>> 在 2014年7月14日,下午4:57,Richard Weinberger 写道: >>&g

Re: [PATCH] fs/ext4/super.c: Let 'counters' switched by SMP

2014-07-15 Thread Chen Gang
On 07/15/2014 05:31 PM, Theodore Ts'o wrote: > On Tue, Jul 15, 2014 at 09:05:29AM +0800, Chen Gang wrote: >> When SMP is disabled, 'struct percpu_counter' has no 'counters' member, >> so need skip it when in !SMP. >> >> The related git commit: >> >> "f

Re: [PATCH] fs/ext4/super.c: Let 'counters' switched by SMP

2014-07-15 Thread Chen Gang
On 07/15/2014 05:31 PM, Theodore Ts'o wrote: On Tue, Jul 15, 2014 at 09:05:29AM +0800, Chen Gang wrote: When SMP is disabled, 'struct percpu_counter' has no 'counters' member, so need skip it when in !SMP. The related git commit: feab139 ext4: rearrange initialization to fix EXT4FS_DEBUG

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-15 Thread Chen Gang
On 07/15/2014 09:11 AM, Chen Gang wrote: On 07/15/2014 08:53 AM, Guenter Roeck wrote: On 07/14/2014 05:34 PM, Chen Gang wrote: On 07/14/2014 05:22 PM, Chen Gang wrote: 在 2014年7月14日,下午4:57,Richard Weinberger rich...@nod.at 写道: Am 14.07.2014 10:48, schrieb Lars-Peter Clausen: On 07/14

Re: [PATCH v2] drivers: staging: lustre: Use 'force_die' instead of 'die' to avoid compiling issue

2014-07-14 Thread Chen Gang
在 2014年7月15日,上午9:40,Greg Kroah-Hartman 写道: > On Mon, Jul 14, 2014 at 08:04:15PM +0800, Chen Gang wrote: >> >> For drivers/staging/lustre/lustre/include/lustre_sec.h:391: >> >> - staging tree: use '\t ' between 'die' and '('. >> >> -

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-14 Thread Chen Gang
On 07/15/2014 08:53 AM, Guenter Roeck wrote: > On 07/14/2014 05:34 PM, Chen Gang wrote: >> On 07/14/2014 05:22 PM, Chen Gang wrote: >>> >>> 在 2014年7月14日,下午4:57,Richard Weinberger 写道: >>> >>>> Am 14.07.2014 10:48, schrieb Lars-Peter Clausen: >&g

[PATCH] fs/ext4/super.c: Let 'counters' switched by SMP

2014-07-14 Thread Chen Gang
member named 'counters' if (EXT4_SB(sb)->s_freeinodes_counter.counters) ^ Signed-off-by: Chen Gang --- fs/ext4/super.c | 4 1 file changed, 4 insertions(+) diff --git a/fs/ext4/super.c b/fs/ext4/super.c index 30767cd..3cd4fe1 100644 --- a/fs/ext4/

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-14 Thread Chen Gang
On 07/14/2014 05:22 PM, Chen Gang wrote: > > 在 2014年7月14日,下午4:57,Richard Weinberger 写道: > >> Am 14.07.2014 10:48, schrieb Lars-Peter Clausen: >>> On 07/14/2014 10:31 AM, Richard Weinberger wrote: >>>> Am 13.07.2014 22:17, schrieb Greg Kroah-Hartman: >

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-14 Thread Chen Gang
On 07/14/2014 05:22 PM, Chen Gang wrote: > > 在 2014年7月14日,下午4:57,Richard Weinberger 写道: > >> Am 14.07.2014 10:48, schrieb Lars-Peter Clausen: >>> On 07/14/2014 10:31 AM, Richard Weinberger wrote: >>>> Am 13.07.2014 22:17, schrieb Greg Kroah-Hartman: >

Re: [PATCH] drivers/crypto/Kconfig: Let 'DEV_QCE' depend on both HAS_DMA and HAS_IOMEM

2014-07-14 Thread Chen Gang
Hello all: This patch is obsoleted according to the other discussion thread about IOMEM and COMPILE_TEST. I shall fix it, and maybe not need touch drivers/crypto again, so maybe not need send patch v2 for it. Thanks. On 07/13/2014 11:01 AM, Chen Gang wrote: > 'DEV_QCE' needs both HAS_

Re: [PATCH v2] drivers: staging: lustre: Use 'force_die' instead of 'die' to avoid compiling issue

2014-07-14 Thread Chen Gang
make[2]: *** [drivers/staging/lustre] Error 2 > make[1]: *** [drivers/staging] Error 2 > make: *** [drivers] Error 2 > > > Signed-off-by: Chen Gang > --- > drivers/staging/lustre/lustre/include/lustre_sec.h | 2 +- > drivers/staging/lustre/lustre/ptlrpc/sec.c

[PATCH v2] drivers: staging: lustre: Use 'force_die' instead of 'die' to avoid compiling issue

2014-07-14 Thread Chen Gang
* [drivers/staging/lustre/lustre/ptlrpc/sec.o] Error 1 make[4]: *** [drivers/staging/lustre/lustre/ptlrpc] Error 2 make[3]: *** [drivers/staging/lustre/lustre] Error 2 make[2]: *** [drivers/staging/lustre] Error 2 make[1]: *** [drivers/staging] Error 2 make: *** [drivers] Error 2 Signed

Re: [PATCH] drivers/pwm/Kconfig: Let PWM_CLPS711X depend on HAS_IOMEM

2014-07-14 Thread Chen Gang
在 2014年7月14日,下午5:33,Thierry Reding 写道: > On Mon, Jul 14, 2014 at 05:14:05PM +0800, Chen Gang wrote: >> >> 在 2014年7月14日,下午4:18,Thierry Reding 写道: >> >>> On Sun, Jul 13, 2014 at 07:49:27PM +0800, Chen Gang wrote: >>>> PWM_CLPS711X need

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-14 Thread Chen Gang
not really any >> different from making the driver selectable via COMPILE_TEST for any other >> platform. To hit the issue you'd have to instantiate a device driver >> instance for a device that >> physically does not exist. This will always result in a failure. > > Okay, you ha

Re: [PATCH] drivers/pwm/Kconfig: Let PWM_CLPS711X depend on HAS_IOMEM

2014-07-14 Thread Chen Gang
在 2014年7月14日,下午4:18,Thierry Reding 写道: > On Sun, Jul 13, 2014 at 07:49:27PM +0800, Chen Gang wrote: >> PWM_CLPS711X needs HAS_IOMEM, so let it depend on HAS_IOMEM. >> >> The related error (with allmodconfig under score): >> >>MODPOST 1365 modules

Re: [PATCH] drivers/pwm/Kconfig: Let PWM_CLPS711X depend on HAS_IOMEM

2014-07-14 Thread Chen Gang
在 2014年7月14日,下午4:18,Thierry Reding thierry.red...@gmail.com 写道: On Sun, Jul 13, 2014 at 07:49:27PM +0800, Chen Gang wrote: PWM_CLPS711X needs HAS_IOMEM, so let it depend on HAS_IOMEM. The related error (with allmodconfig under score): MODPOST 1365 modules ERROR

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-14 Thread Chen Gang
for a device that physically does not exist. This will always result in a failure. Okay, you have convinced me. :) OK, thank all of you, and I shall send the related patch for it. I will try to finish it within this week. Thanks. — Chen Gang Open, share, and attitude like air, water, and life

Re: [PATCH] drivers/pwm/Kconfig: Let PWM_CLPS711X depend on HAS_IOMEM

2014-07-14 Thread Chen Gang
在 2014年7月14日,下午5:33,Thierry Reding thierry.red...@gmail.com 写道: On Mon, Jul 14, 2014 at 05:14:05PM +0800, Chen Gang wrote: 在 2014年7月14日,下午4:18,Thierry Reding thierry.red...@gmail.com 写道: On Sun, Jul 13, 2014 at 07:49:27PM +0800, Chen Gang wrote: PWM_CLPS711X needs HAS_IOMEM, so let

[PATCH v2] drivers: staging: lustre: Use 'force_die' instead of 'die' to avoid compiling issue

2014-07-14 Thread Chen Gang
/lustre/ptlrpc/sec.o] Error 1 make[4]: *** [drivers/staging/lustre/lustre/ptlrpc] Error 2 make[3]: *** [drivers/staging/lustre/lustre] Error 2 make[2]: *** [drivers/staging/lustre] Error 2 make[1]: *** [drivers/staging] Error 2 make: *** [drivers] Error 2 Signed-off-by: Chen Gang gang.chen.5

Re: [PATCH v2] drivers: staging: lustre: Use 'force_die' instead of 'die' to avoid compiling issue

2014-07-14 Thread Chen Gang
Signed-off-by: Chen Gang gang.chen.5...@gmail.com --- drivers/staging/lustre/lustre/include/lustre_sec.h | 2 +- drivers/staging/lustre/lustre/ptlrpc/sec.c | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre_sec.h

Re: [PATCH] drivers/crypto/Kconfig: Let 'DEV_QCE' depend on both HAS_DMA and HAS_IOMEM

2014-07-14 Thread Chen Gang
Hello all: This patch is obsoleted according to the other discussion thread about IOMEM and COMPILE_TEST. I shall fix it, and maybe not need touch drivers/crypto again, so maybe not need send patch v2 for it. Thanks. On 07/13/2014 11:01 AM, Chen Gang wrote: 'DEV_QCE' needs both HAS_DMA

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-14 Thread Chen Gang
On 07/14/2014 05:22 PM, Chen Gang wrote: 在 2014年7月14日,下午4:57,Richard Weinberger rich...@nod.at 写道: Am 14.07.2014 10:48, schrieb Lars-Peter Clausen: On 07/14/2014 10:31 AM, Richard Weinberger wrote: Am 13.07.2014 22:17, schrieb Greg Kroah-Hartman: On Sun, Jul 13, 2014 at 09:33:38PM +0200

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-14 Thread Chen Gang
On 07/14/2014 05:22 PM, Chen Gang wrote: 在 2014年7月14日,下午4:57,Richard Weinberger rich...@nod.at 写道: Am 14.07.2014 10:48, schrieb Lars-Peter Clausen: On 07/14/2014 10:31 AM, Richard Weinberger wrote: Am 13.07.2014 22:17, schrieb Greg Kroah-Hartman: On Sun, Jul 13, 2014 at 09:33:38PM +0200

[PATCH] fs/ext4/super.c: Let 'counters' switched by SMP

2014-07-14 Thread Chen Gang
'counters' if (EXT4_SB(sb)-s_freeinodes_counter.counters) ^ Signed-off-by: Chen Gang gang.chen.5...@gmail.com --- fs/ext4/super.c | 4 1 file changed, 4 insertions(+) diff --git a/fs/ext4/super.c b/fs/ext4/super.c index 30767cd..3cd4fe1 100644 --- a/fs

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-14 Thread Chen Gang
On 07/15/2014 08:53 AM, Guenter Roeck wrote: On 07/14/2014 05:34 PM, Chen Gang wrote: On 07/14/2014 05:22 PM, Chen Gang wrote: 在 2014年7月14日,下午4:57,Richard Weinberger rich...@nod.at 写道: Am 14.07.2014 10:48, schrieb Lars-Peter Clausen: On 07/14/2014 10:31 AM, Richard Weinberger wrote: Am

Re: [PATCH v2] drivers: staging: lustre: Use 'force_die' instead of 'die' to avoid compiling issue

2014-07-14 Thread Chen Gang
在 2014年7月15日,上午9:40,Greg Kroah-Hartman gre...@linuxfoundation.org 写道: On Mon, Jul 14, 2014 at 08:04:15PM +0800, Chen Gang wrote: For drivers/staging/lustre/lustre/include/lustre_sec.h:391: - staging tree: use '\t ' between 'die' and '('. - linux-next tree: use ' ' between 'die

Re: [PATCH] drivers: staging: lustre: Use 'force_die' instead of 'die' to avoid compiling issue

2014-07-13 Thread Chen Gang
On 07/14/2014 06:50 AM, Greg Kroah-Hartman wrote: > On Mon, Jul 14, 2014 at 06:38:24AM +0800, Chen Gang wrote: >> On 07/14/2014 06:31 AM, Chen Gang wrote: >>> On 07/14/2014 05:41 AM, Chen Gang wrote: >>>> On 07/14/2014 03:05 AM, Greg Kroah-Hartman wrote: >>&g

Re: [PATCH] drivers: staging: lustre: Use 'force_die' instead of 'die' to avoid compiling issue

2014-07-13 Thread Chen Gang
On 07/14/2014 06:31 AM, Chen Gang wrote: > On 07/14/2014 05:41 AM, Chen Gang wrote: >> On 07/14/2014 03:05 AM, Greg Kroah-Hartman wrote: >>> On Sun, Jul 13, 2014 at 10:50:55PM +0800, Chen Gang wrote: >>>> Some of architectures have already defined 'die' as macro, s

Re: [PATCH] drivers: staging: lustre: Use 'force_die' instead of 'die' to avoid compiling issue

2014-07-13 Thread Chen Gang
On 07/14/2014 05:41 AM, Chen Gang wrote: > On 07/14/2014 03:05 AM, Greg Kroah-Hartman wrote: >> On Sun, Jul 13, 2014 at 10:50:55PM +0800, Chen Gang wrote: >>> Some of architectures have already defined 'die' as macro, so can not use >>> this common name as d

Re: [PATCH] drivers: staging: lustre: Use 'force_die' instead of 'die' to avoid compiling issue

2014-07-13 Thread Chen Gang
On 07/14/2014 03:05 AM, Greg Kroah-Hartman wrote: > On Sun, Jul 13, 2014 at 10:50:55PM +0800, Chen Gang wrote: >> Some of architectures have already defined 'die' as macro, so can not use >> this common name as declaration in other modules, or will cause compiling >> issue

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-13 Thread Chen Gang
that was built with COMPILE_TEST on > your real hardware since it contains so many drivers that are completely > useless on your hardware. The idea of COMPILE_TEST is to have as much compile > test exposure as possible to the code that is enabled by COMPILE_TEST. > Stubbing out ioremap() and

[PATCH] drivers: staging: lustre: Use 'force_die' instead of 'die' to avoid compiling issue

2014-07-13 Thread Chen Gang
ers/staging/lustre/lustre/ptlrpc/sec.o] Error 1 make[4]: *** [drivers/staging/lustre/lustre/ptlrpc] Error 2 make[3]: *** [drivers/staging/lustre/lustre] Error 2 make[2]: *** [drivers/staging/lustre] Error 2 make[1]: *** [drivers/staging] Error 2 make: *** [drivers] Error 2 Signed

<    4   5   6   7   8   9   10   11   12   13   >