* gpio: gpio-exynos4 build fails in mainline
@ 2011-05-29 15:30 Randy Dunlap
2011-05-29 15:35 ` Randy Dunlap
0 siblings, 1 reply; 7+ messages in thread
From: Randy Dunlap @ 2011-05-29 15:30 UTC (permalink / raw)
To: lkml; +Cc: Kukjin Kim, Grant Likely
probably should depend on ARM or some ARM sub-arch/mach-type?
drivers/gpio/gpio-exynos4.c:18:11: error: unable to open 'mach/map.h'
on x86_64
---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: gpio: gpio-exynos4 build fails in mainline
2011-05-29 15:30 gpio: gpio-exynos4 build fails in mainline Randy Dunlap
@ 2011-05-29 15:35 ` Randy Dunlap
2011-05-29 15:40 ` Randy Dunlap
0 siblings, 1 reply; 7+ messages in thread
From: Randy Dunlap @ 2011-05-29 15:35 UTC (permalink / raw)
To: lkml; +Cc: Kukjin Kim, Grant Likely
On Sun, 29 May 2011 08:30:32 -0700 Randy Dunlap wrote:
> probably should depend on ARM or some ARM sub-arch/mach-type?
>
>
> drivers/gpio/gpio-exynos4.c:18:11: error: unable to open 'mach/map.h'
>
>
> on x86_64
>
> ---
similarly:
drivers/gpio/gpio-plat-samsung.c:22:11: error: unable to open 'plat/gpio-core.h'
---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: gpio: gpio-exynos4 build fails in mainline
2011-05-29 15:35 ` Randy Dunlap
@ 2011-05-29 15:40 ` Randy Dunlap
2011-05-29 15:44 ` Randy Dunlap
0 siblings, 1 reply; 7+ messages in thread
From: Randy Dunlap @ 2011-05-29 15:40 UTC (permalink / raw)
To: Kyungmin Park, lkml; +Cc: Kukjin Kim, Grant Likely
On Sun, 29 May 2011 08:35:20 -0700 Randy Dunlap wrote:
> On Sun, 29 May 2011 08:30:32 -0700 Randy Dunlap wrote:
>
> > probably should depend on ARM or some ARM sub-arch/mach-type?
> >
> >
> > drivers/gpio/gpio-exynos4.c:18:11: error: unable to open 'mach/map.h'
> >
> >
> > on x86_64
> >
> > ---
>
> similarly:
>
> drivers/gpio/gpio-plat-samsung.c:22:11: error: unable to open 'plat/gpio-core.h'
and
drivers/gpio/gpio-s5pc100.c:21:11: error: unable to open 'mach/map.h'
(this began as x86_64 allmodconfig)
---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: gpio: gpio-exynos4 build fails in mainline
2011-05-29 15:40 ` Randy Dunlap
@ 2011-05-29 15:44 ` Randy Dunlap
2011-05-30 1:24 ` Kyungmin Park
0 siblings, 1 reply; 7+ messages in thread
From: Randy Dunlap @ 2011-05-29 15:44 UTC (permalink / raw)
To: lkml; +Cc: Kyungmin Park, Kukjin Kim, Grant Likely
On Sun, 29 May 2011 08:40:00 -0700 Randy Dunlap wrote:
> On Sun, 29 May 2011 08:35:20 -0700 Randy Dunlap wrote:
>
> > On Sun, 29 May 2011 08:30:32 -0700 Randy Dunlap wrote:
> >
> > > probably should depend on ARM or some ARM sub-arch/mach-type?
> > >
> > >
> > > drivers/gpio/gpio-exynos4.c:18:11: error: unable to open 'mach/map.h'
> > >
> > >
> > > on x86_64
> > >
> > > ---
> >
> > similarly:
> >
> > drivers/gpio/gpio-plat-samsung.c:22:11: error: unable to open 'plat/gpio-core.h'
>
>
> and
>
> drivers/gpio/gpio-s5pc100.c:21:11: error: unable to open 'mach/map.h'
>
>
> (this began as x86_64 allmodconfig)
and
drivers/gpio/gpio-s5pv210.c:17:11: error: unable to open 'plat/gpio-core.h'
---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: gpio: gpio-exynos4 build fails in mainline
2011-05-29 15:44 ` Randy Dunlap
@ 2011-05-30 1:24 ` Kyungmin Park
2011-05-30 2:50 ` Kukjin Kim
2011-05-30 5:54 ` Grant Likely
0 siblings, 2 replies; 7+ messages in thread
From: Kyungmin Park @ 2011-05-30 1:24 UTC (permalink / raw)
To: Randy Dunlap; +Cc: lkml, Grant Likely
On Mon, May 30, 2011 at 12:44 AM, Randy Dunlap <rdunlap@xenotime.net> wrote:
> On Sun, 29 May 2011 08:40:00 -0700 Randy Dunlap wrote:
>
>> On Sun, 29 May 2011 08:35:20 -0700 Randy Dunlap wrote:
>>
>> > On Sun, 29 May 2011 08:30:32 -0700 Randy Dunlap wrote:
>> >
>> > > probably should depend on ARM or some ARM sub-arch/mach-type?
>> > >
>> > >
>> > > drivers/gpio/gpio-exynos4.c:18:11: error: unable to open 'mach/map.h'
>> > >
>> > >
>> > > on x86_64
>> > >
>> > > ---
>> >
>> > similarly:
>> >
>> > drivers/gpio/gpio-plat-samsung.c:22:11: error: unable to open 'plat/gpio-core.h'
>>
>>
>> and
>>
>> drivers/gpio/gpio-s5pc100.c:21:11: error: unable to open 'mach/map.h'
>>
>>
>> (this began as x86_64 allmodconfig)
>
> and
>
> drivers/gpio/gpio-s5pv210.c:17:11: error: unable to open 'plat/gpio-core.h'
Hi,
Right, now it's just moved from mach directories to drivers/gpio
So it requires the proper depends on each ARCH_CPU.
Thank you,
Kyungmin Park
---
diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig
index 5923976..655babf 100644
--- a/drivers/gpio/Kconfig
+++ b/drivers/gpio/Kconfig
@@ -88,24 +88,28 @@ config GPIO_IT8761E
config GPIO_EXYNOS4
bool "Samsung Exynos4 GPIO library support"
+ depends on CPU_EXYNOS4210
default y if CPU_EXYNOS4210
help
Say yes here to support Samsung Exynos4 series SoCs GPIO library
config GPIO_PLAT_SAMSUNG
bool "Samsung SoCs GPIO library support"
+ depends on SAMSUNG_GPIOLIB_4BIT
default y if SAMSUNG_GPIOLIB_4BIT
help
Say yes here to support Samsung SoCs GPIO library
config GPIO_S5PC100
bool "Samsung S5PC100 GPIO library support"
+ depends on CPU_S5PC100
default y if CPU_S5PC100
help
Say yes here to support Samsung S5PC100 SoCs GPIO library
config GPIO_S5PV210
bool "Samsung S5PV210/S5PC110 GPIO library support"
+ depends on CPU_S5PV210
default y if CPU_S5PV210
help
Say yes here to support Samsung S5PV210/S5PC110 SoCs GPIO library
>
>
> ---
> ~Randy
> *** Remember to use Documentation/SubmitChecklist when testing your code ***
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: gpio: gpio-exynos4 build fails in mainline
2011-05-30 1:24 ` Kyungmin Park
@ 2011-05-30 2:50 ` Kukjin Kim
2011-05-30 5:54 ` Grant Likely
1 sibling, 0 replies; 7+ messages in thread
From: Kukjin Kim @ 2011-05-30 2:50 UTC (permalink / raw)
To: Kyungmin Park; +Cc: Randy Dunlap, lkml, Grant Likely
On 05/29/11 18:24, Kyungmin Park wrote:
> On Mon, May 30, 2011 at 12:44 AM, Randy Dunlap<rdunlap@xenotime.net> wrote:
>> On Sun, 29 May 2011 08:40:00 -0700 Randy Dunlap wrote:
(snip)
>>
>> drivers/gpio/gpio-s5pv210.c:17:11: error: unable to open 'plat/gpio-core.h'
>
Hi all,
Grrr, :(
> Hi,
>
> Right, now it's just moved from mach directories to drivers/gpio
> So it requires the proper depends on each ARCH_CPU.
>
Please add me in Cc on this.
> ---
> diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig
> index 5923976..655babf 100644
> --- a/drivers/gpio/Kconfig
> +++ b/drivers/gpio/Kconfig
> @@ -88,24 +88,28 @@ config GPIO_IT8761E
>
> config GPIO_EXYNOS4
> bool "Samsung Exynos4 GPIO library support"
> + depends on CPU_EXYNOS4210
> default y if CPU_EXYNOS4210
> help
> Say yes here to support Samsung Exynos4 series SoCs GPIO library
Basically, GPIO_EXYNOS4 is used only for EXYNOS4(EXYNOS4210) and was
selected as a default on EXYNOS4 before moving into drivers/. I mean, do
we really need to select this manually in kernel configuration(like make
menuconfig)?
And I think, it's more reasonable that depends on ARCH_XXX not CPU_XXX.
Grant, how about following?
+config GPIO_EXYNOS4
+ bool
+ depends on ARCH_EXYNOS4
+ default y if CPU_EXYNOS4210
>
> config GPIO_PLAT_SAMSUNG
> bool "Samsung SoCs GPIO library support"
> + depends on SAMSUNG_GPIOLIB_4BIT
> default y if SAMSUNG_GPIOLIB_4BIT
> help
> Say yes here to support Samsung SoCs GPIO library
>
Same as above.
+config GPIO_PLAT_SAMSUNG
+ bool
+ depends on PLAT_SAMSUNG
+ default y if SAMSUNG_GPIOLIB_4BIT
> config GPIO_S5PC100
> bool "Samsung S5PC100 GPIO library support"
> + depends on CPU_S5PC100
> default y if CPU_S5PC100
> help
> Say yes here to support Samsung S5PC100 SoCs GPIO library
>
Same...
+config GPIO_S5PC100
+ bool
+ depends on ARCH_S5PC100
+ default y if CPU_S5PC100
> config GPIO_S5PV210
> bool "Samsung S5PV210/S5PC110 GPIO library support"
> + depends on CPU_S5PV210
> default y if CPU_S5PV210
> help
> Say yes here to support Samsung S5PV210/S5PC110 SoCs GPIO library
>
Same.
+config GPIO_S5PV210
+ bool
+ depends on ARCH_S5PV210
+ default y if CPU_S5PV210
One more, IMHO, would be better if they could not split like following.
As you know, 'PLAT_SAMSUNG' includes common GPIOlib for other Samsung SoCs.
config GPIO_PLAT_SAMSUNG
...
config GPIO_S5PC100
...
config GPIO_S5PV210
...
config GPIO_EXYNOS4
...
Thanks.
Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: gpio: gpio-exynos4 build fails in mainline
2011-05-30 1:24 ` Kyungmin Park
2011-05-30 2:50 ` Kukjin Kim
@ 2011-05-30 5:54 ` Grant Likely
1 sibling, 0 replies; 7+ messages in thread
From: Grant Likely @ 2011-05-30 5:54 UTC (permalink / raw)
To: Kyungmin Park; +Cc: Randy Dunlap, lkml
On Mon, May 30, 2011 at 10:24:47AM +0900, Kyungmin Park wrote:
> On Mon, May 30, 2011 at 12:44 AM, Randy Dunlap <rdunlap@xenotime.net> wrote:
> > On Sun, 29 May 2011 08:40:00 -0700 Randy Dunlap wrote:
> >
> >> On Sun, 29 May 2011 08:35:20 -0700 Randy Dunlap wrote:
> >>
> >> > On Sun, 29 May 2011 08:30:32 -0700 Randy Dunlap wrote:
> >> >
> >> > > probably should depend on ARM or some ARM sub-arch/mach-type?
> >> > >
> >> > >
> >> > > drivers/gpio/gpio-exynos4.c:18:11: error: unable to open 'mach/map.h'
> >> > >
> >> > >
> >> > > on x86_64
> >> > >
> >> > > ---
> >> >
> >> > similarly:
> >> >
> >> > drivers/gpio/gpio-plat-samsung.c:22:11: error: unable to open 'plat/gpio-core.h'
> >>
> >>
> >> and
> >>
> >> drivers/gpio/gpio-s5pc100.c:21:11: error: unable to open 'mach/map.h'
> >>
> >>
> >> (this began as x86_64 allmodconfig)
> >
> > and
> >
> > drivers/gpio/gpio-s5pv210.c:17:11: error: unable to open 'plat/gpio-core.h'
>
> Hi,
>
> Right, now it's just moved from mach directories to drivers/gpio
> So it requires the proper depends on each ARCH_CPU.
>
> Thank you,
> Kyungmin Park
A variant on this patch merged and pushed out to the following branch. Thanks.
git://git.secretlab.ca/git/linux-2.6 gpio/merge
It should be in linux-next tomorrow.
g.
> ---
> diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig
> index 5923976..655babf 100644
> --- a/drivers/gpio/Kconfig
> +++ b/drivers/gpio/Kconfig
> @@ -88,24 +88,28 @@ config GPIO_IT8761E
>
> config GPIO_EXYNOS4
> bool "Samsung Exynos4 GPIO library support"
> + depends on CPU_EXYNOS4210
> default y if CPU_EXYNOS4210
> help
> Say yes here to support Samsung Exynos4 series SoCs GPIO library
>
> config GPIO_PLAT_SAMSUNG
> bool "Samsung SoCs GPIO library support"
> + depends on SAMSUNG_GPIOLIB_4BIT
> default y if SAMSUNG_GPIOLIB_4BIT
> help
> Say yes here to support Samsung SoCs GPIO library
>
> config GPIO_S5PC100
> bool "Samsung S5PC100 GPIO library support"
> + depends on CPU_S5PC100
> default y if CPU_S5PC100
> help
> Say yes here to support Samsung S5PC100 SoCs GPIO library
>
> config GPIO_S5PV210
> bool "Samsung S5PV210/S5PC110 GPIO library support"
> + depends on CPU_S5PV210
> default y if CPU_S5PV210
> help
> Say yes here to support Samsung S5PV210/S5PC110 SoCs GPIO library
>
> >
> >
> > ---
> > ~Randy
> > *** Remember to use Documentation/SubmitChecklist when testing your code ***
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
> > Please read the FAQ at http://www.tux.org/lkml/
> >
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2011-05-30 5:54 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-05-29 15:30 gpio: gpio-exynos4 build fails in mainline Randy Dunlap
2011-05-29 15:35 ` Randy Dunlap
2011-05-29 15:40 ` Randy Dunlap
2011-05-29 15:44 ` Randy Dunlap
2011-05-30 1:24 ` Kyungmin Park
2011-05-30 2:50 ` Kukjin Kim
2011-05-30 5:54 ` Grant Likely
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.