* Re: + cs553x-gpio-add-amd-cs5535-cs5536-gpio-driver-support-fix-fix.patch added to -mm tree [not found] <200904062355.n36NtLTf018342@imap1.linux-foundation.org> @ 2009-04-10 20:00 ` Randy Dunlap 2009-04-13 1:52 ` Andres Salomon 0 siblings, 1 reply; 6+ messages in thread From: Randy Dunlap @ 2009-04-10 20:00 UTC (permalink / raw) To: akpm Cc: dilinger, david-b, dilinger, jordan, katzj, tiwai, Linux Kernel Mailing List akpm@linux-foundation.org wrote: > The patch titled > cs553x-gpio-add-amd-cs5535-cs5536-gpio-driver-support-fix-fix > has been added to the -mm tree. Its filename is > cs553x-gpio-add-amd-cs5535-cs5536-gpio-driver-support-fix-fix.patch > > Before you just go and hit "reply", please: > a) Consider who else should be cc'ed > b) Prefer to cc a suitable mailing list as well > c) Ideally: find the original patch on the mailing list and do a > reply-to-all to that, adding suitable additional cc's > > *** Remember to use Documentation/SubmitChecklist when testing your code *** > > See http://userweb.kernel.org/~akpm/stuff/added-to-mm.txt to find > out what to do about this > > The current -mm tree may be found at http://userweb.kernel.org/~akpm/mmotm/ > > ------------------------------------------------------ > Subject: cs553x-gpio-add-amd-cs5535-cs5536-gpio-driver-support-fix-fix > From: Andres Salomon <dilinger@queued.net> > > force GPIO_CS553X to be defined when OLPC is selected > > We've begun using the generic GPIO stuff for geodes, so OLPC implicitly > depends upon this. This patch forces selection of it. > > Signed-off-by: Andres Salomon <dilinger@debian.org> > Cc: Takashi Iwai <tiwai@suse.de> > Cc: Jeremy Katz <katzj@redhat.com> > Cc: Jordan Crouse <jordan@cosmicpenguin.net> > Cc: David Brownell <david-b@pacbell.net> > Cc: Randy Dunlap <randy.dunlap@oracle.com> > Signed-off-by: Andrew Morton <akpm@linux-foundation.org> > --- > > arch/x86/Kconfig | 1 + > pci/cs5535audio/Makefile | 0 > 2 files changed, 1 insertion(+) > > diff -puN arch/x86/Kconfig~cs553x-gpio-add-amd-cs5535-cs5536-gpio-driver-support-fix-fix arch/x86/Kconfig > --- a/arch/x86/Kconfig~cs553x-gpio-add-amd-cs5535-cs5536-gpio-driver-support-fix-fix > +++ a/arch/x86/Kconfig > @@ -1972,6 +1972,7 @@ config GEODE_MFGPT_TIMER > > config OLPC > bool "One Laptop Per Child support" > + select GPIO_CS553X > default n > ---help--- > Add support for detecting the unique features of the OLPC In mmotm-2009-0410, this patch causes build errors when GPIOLIB is not enabled since GPIO_CS553X uses gpiolib functions & struct fields: drivers/gpio/cs553x-gpio.c:23: error: field 'chip' has incomplete type drivers/gpio/cs553x-gpio.c:150: error: field name not in record or union initializer drivers/gpio/cs553x-gpio.c:150: error: (near initialization for 'cs553x_gpio_chip.chip') drivers/gpio/cs553x-gpio.c:151: error: field name not in record or union initializer drivers/gpio/cs553x-gpio.c:151: error: (near initialization for 'cs553x_gpio_chip.chip') drivers/gpio/cs553x-gpio.c:153: error: field name not in record or union initializer drivers/gpio/cs553x-gpio.c:153: error: (near initialization for 'cs553x_gpio_chip.chip') drivers/gpio/cs553x-gpio.c:154: error: field name not in record or union initializer drivers/gpio/cs553x-gpio.c:154: error: (near initialization for 'cs553x_gpio_chip.chip') drivers/gpio/cs553x-gpio.c:156: error: field name not in record or union initializer drivers/gpio/cs553x-gpio.c:156: error: (near initialization for 'cs553x_gpio_chip.chip') drivers/gpio/cs553x-gpio.c:157: error: field name not in record or union initializer drivers/gpio/cs553x-gpio.c:157: error: (near initialization for 'cs553x_gpio_chip.chip') drivers/gpio/cs553x-gpio.c:159: error: field name not in record or union initializer drivers/gpio/cs553x-gpio.c:159: error: (near initialization for 'cs553x_gpio_chip.chip') drivers/gpio/cs553x-gpio.c:160: error: field name not in record or union initializer drivers/gpio/cs553x-gpio.c:160: error: (near initialization for 'cs553x_gpio_chip.chip') drivers/gpio/cs553x-gpio.c:197: error: implicit declaration of function 'gpiochip_add' drivers/gpio/cs553x-gpio.c:216: error: implicit declaration of function 'gpiochip_remove' One possible patch is also to select GPIOLIB above in the OLPC config block. -- ~Randy ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: + cs553x-gpio-add-amd-cs5535-cs5536-gpio-driver-support-fix-fix.patch added to -mm tree 2009-04-10 20:00 ` + cs553x-gpio-add-amd-cs5535-cs5536-gpio-driver-support-fix-fix.patch added to -mm tree Randy Dunlap @ 2009-04-13 1:52 ` Andres Salomon 2009-04-13 4:26 ` Randy Dunlap 2009-04-17 18:34 ` + cs553x-gpio-add-amd-cs5535-cs5536-gpio-driver-support-fix-fix.patch added to -mm tree Andrew Morton 0 siblings, 2 replies; 6+ messages in thread From: Andres Salomon @ 2009-04-13 1:52 UTC (permalink / raw) To: Randy Dunlap Cc: akpm, david-b, jordan, katzj, tiwai, Linux Kernel Mailing List On Fri, 10 Apr 2009 13:00:54 -0700 Randy Dunlap <randy.dunlap@oracle.com> wrote: > akpm@linux-foundation.org wrote: > > The patch titled > > cs553x-gpio-add-amd-cs5535-cs5536-gpio-driver-support-fix-fix > > has been added to the -mm tree. Its filename is > > cs553x-gpio-add-amd-cs5535-cs5536-gpio-driver-support-fix-fix.patch > > > > Before you just go and hit "reply", please: > > a) Consider who else should be cc'ed > > b) Prefer to cc a suitable mailing list as well > > c) Ideally: find the original patch on the mailing list and do a > > reply-to-all to that, adding suitable additional cc's > > > > *** Remember to use Documentation/SubmitChecklist when testing your > > code *** > > > > See http://userweb.kernel.org/~akpm/stuff/added-to-mm.txt to find > > out what to do about this > > > > The current -mm tree may be found at > > http://userweb.kernel.org/~akpm/mmotm/ > > > > ------------------------------------------------------ > > Subject: > > cs553x-gpio-add-amd-cs5535-cs5536-gpio-driver-support-fix-fix From: > > Andres Salomon <dilinger@queued.net> > > > > force GPIO_CS553X to be defined when OLPC is selected > > > > We've begun using the generic GPIO stuff for geodes, so OLPC > > implicitly depends upon this. This patch forces selection of it. > > > > Signed-off-by: Andres Salomon <dilinger@debian.org> > > Cc: Takashi Iwai <tiwai@suse.de> > > Cc: Jeremy Katz <katzj@redhat.com> > > Cc: Jordan Crouse <jordan@cosmicpenguin.net> > > Cc: David Brownell <david-b@pacbell.net> > > Cc: Randy Dunlap <randy.dunlap@oracle.com> > > Signed-off-by: Andrew Morton <akpm@linux-foundation.org> > > --- > > > > arch/x86/Kconfig | 1 + > > pci/cs5535audio/Makefile | 0 > > 2 files changed, 1 insertion(+) > > > > diff -puN > > arch/x86/Kconfig~cs553x-gpio-add-amd-cs5535-cs5536-gpio-driver-support-fix-fix > > arch/x86/Kconfig --- > > a/arch/x86/Kconfig~cs553x-gpio-add-amd-cs5535-cs5536-gpio-driver-support-fix-fix > > +++ a/arch/x86/Kconfig @@ -1972,6 +1972,7 @@ config > > GEODE_MFGPT_TIMER config OLPC > > bool "One Laptop Per Child support" > > + select GPIO_CS553X > > default n > > ---help--- > > Add support for detecting the unique features of the OLPC > > > In mmotm-2009-0410, this patch causes build errors when GPIOLIB is > not enabled since GPIO_CS553X uses gpiolib functions & struct fields: > > drivers/gpio/cs553x-gpio.c:23: error: field 'chip' has incomplete type [...] > > > One possible patch is also to select GPIOLIB above in the OLPC config > block. > Does the following help? I think this is what we want (gpio-cs553x uses gpiolib). I'm a bit unclear how the other gpio drivers get away with including linux/gpio.h and using gpio_chip without requiring GPIOLIB. Maybe they implicitly are built on architectures which select CONFIG_ARCH_REQUIRE_GPIOLIB? Force GPIO_CS553X to select GPIOLIB, which it makes use of. Signed-off-by: Andres Salomon <dilinger@debian.org> diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig index 797ff45..17d7364 100644 --- a/drivers/gpio/Kconfig +++ b/drivers/gpio/Kconfig @@ -148,6 +148,7 @@ comment "PCI GPIO expanders:" config GPIO_CS553X tristate "AMD cs5535/cs5536 GPIO support" depends on PCI && !CS5535_GPIO && !MGEODE_LX + select GPIOLIB help The AMD cs5535 and cs5536 southbridges support 28 GPIO pins that can be used for quite a number of things. The cs553x is found on ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: + cs553x-gpio-add-amd-cs5535-cs5536-gpio-driver-support-fix-fix.patch added to -mm tree 2009-04-13 1:52 ` Andres Salomon @ 2009-04-13 4:26 ` Randy Dunlap 2009-04-19 1:01 ` [PATCH] cs553x: fix build dependency problems Randy Dunlap 2009-04-17 18:34 ` + cs553x-gpio-add-amd-cs5535-cs5536-gpio-driver-support-fix-fix.patch added to -mm tree Andrew Morton 1 sibling, 1 reply; 6+ messages in thread From: Randy Dunlap @ 2009-04-13 4:26 UTC (permalink / raw) To: Andres Salomon Cc: akpm, david-b, jordan, katzj, tiwai, Linux Kernel Mailing List Andres Salomon wrote: > On Fri, 10 Apr 2009 13:00:54 -0700 > Randy Dunlap <randy.dunlap@oracle.com> wrote: > >> akpm@linux-foundation.org wrote: >>> The patch titled >>> cs553x-gpio-add-amd-cs5535-cs5536-gpio-driver-support-fix-fix >>> has been added to the -mm tree. Its filename is >>> cs553x-gpio-add-amd-cs5535-cs5536-gpio-driver-support-fix-fix.patch >>> >>> Before you just go and hit "reply", please: >>> a) Consider who else should be cc'ed >>> b) Prefer to cc a suitable mailing list as well >>> c) Ideally: find the original patch on the mailing list and do a >>> reply-to-all to that, adding suitable additional cc's >>> >>> *** Remember to use Documentation/SubmitChecklist when testing your >>> code *** >>> >>> See http://userweb.kernel.org/~akpm/stuff/added-to-mm.txt to find >>> out what to do about this >>> >>> The current -mm tree may be found at >>> http://userweb.kernel.org/~akpm/mmotm/ >>> >>> ------------------------------------------------------ >>> Subject: >>> cs553x-gpio-add-amd-cs5535-cs5536-gpio-driver-support-fix-fix From: >>> Andres Salomon <dilinger@queued.net> >>> >>> force GPIO_CS553X to be defined when OLPC is selected >>> >>> We've begun using the generic GPIO stuff for geodes, so OLPC >>> implicitly depends upon this. This patch forces selection of it. >>> >>> Signed-off-by: Andres Salomon <dilinger@debian.org> >>> Cc: Takashi Iwai <tiwai@suse.de> >>> Cc: Jeremy Katz <katzj@redhat.com> >>> Cc: Jordan Crouse <jordan@cosmicpenguin.net> >>> Cc: David Brownell <david-b@pacbell.net> >>> Cc: Randy Dunlap <randy.dunlap@oracle.com> >>> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> >>> --- >>> >>> arch/x86/Kconfig | 1 + >>> pci/cs5535audio/Makefile | 0 >>> 2 files changed, 1 insertion(+) >>> >>> diff -puN >>> arch/x86/Kconfig~cs553x-gpio-add-amd-cs5535-cs5536-gpio-driver-support-fix-fix >>> arch/x86/Kconfig --- >>> a/arch/x86/Kconfig~cs553x-gpio-add-amd-cs5535-cs5536-gpio-driver-support-fix-fix >>> +++ a/arch/x86/Kconfig @@ -1972,6 +1972,7 @@ config >>> GEODE_MFGPT_TIMER config OLPC >>> bool "One Laptop Per Child support" >>> + select GPIO_CS553X >>> default n >>> ---help--- >>> Add support for detecting the unique features of the OLPC >> >> In mmotm-2009-0410, this patch causes build errors when GPIOLIB is >> not enabled since GPIO_CS553X uses gpiolib functions & struct fields: >> >> drivers/gpio/cs553x-gpio.c:23: error: field 'chip' has incomplete type > [...] >> >> One possible patch is also to select GPIOLIB above in the OLPC config >> block. >> > > Does the following help? I think this is what we want (gpio-cs553x uses > gpiolib). I'm a bit unclear how the other gpio drivers get away with > including linux/gpio.h and using gpio_chip without requiring GPIOLIB. > Maybe they implicitly are built on architectures which select > CONFIG_ARCH_REQUIRE_GPIOLIB? > It helps some (and is needed), but there is another problem. When PCI is not enabled: drivers/gpio/cs553x-gpio.c: In function 'cs553x_gpio_probe': drivers/gpio/cs553x-gpio.c:176: error: implicit declaration of function 'pci_enable_device_io' drivers/gpio/cs553x-gpio.c:182: error: implicit declaration of function 'pci_request_region' drivers/gpio/cs553x-gpio.c:207: error: implicit declaration of function 'pci_release_region' I guess that OLPC needs an additional dependency on PCI ?? > > > Force GPIO_CS553X to select GPIOLIB, which it makes use of. > > Signed-off-by: Andres Salomon <dilinger@debian.org> > > diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig > index 797ff45..17d7364 100644 > --- a/drivers/gpio/Kconfig > +++ b/drivers/gpio/Kconfig > @@ -148,6 +148,7 @@ comment "PCI GPIO expanders:" > config GPIO_CS553X > tristate "AMD cs5535/cs5536 GPIO support" > depends on PCI && !CS5535_GPIO && !MGEODE_LX > + select GPIOLIB > help > The AMD cs5535 and cs5536 southbridges support 28 GPIO pins that > can be used for quite a number of things. The cs553x is found on -- ~Randy ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] cs553x: fix build dependency problems 2009-04-13 4:26 ` Randy Dunlap @ 2009-04-19 1:01 ` Randy Dunlap 2009-04-20 0:00 ` Andres Salomon 0 siblings, 1 reply; 6+ messages in thread From: Randy Dunlap @ 2009-04-19 1:01 UTC (permalink / raw) To: Andres Salomon Cc: akpm, david-b, jordan, katzj, tiwai, Linux Kernel Mailing List Andres, how does this look to you? It fixes the problems that I know about. From: Randy Dunlap <randy.dunlap@oracle.com> Fix multiple build errors due to cs553x-gpio using GPIO APIs and using PCI APIs. Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com> --- arch/x86/Kconfig | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) --- mmotm-2009-0417-1519.orig/arch/x86/Kconfig +++ mmotm-2009-0417-1519/arch/x86/Kconfig @@ -1969,7 +1969,8 @@ config GEODE_MFGPT_TIMER config OLPC bool "One Laptop Per Child support" - select GPIO_CS553X + select GPIO_CS553X if PCI + select GPIOLIB if PCI default n ---help--- Add support for detecting the unique features of the OLPC ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] cs553x: fix build dependency problems 2009-04-19 1:01 ` [PATCH] cs553x: fix build dependency problems Randy Dunlap @ 2009-04-20 0:00 ` Andres Salomon 0 siblings, 0 replies; 6+ messages in thread From: Andres Salomon @ 2009-04-20 0:00 UTC (permalink / raw) To: Randy Dunlap Cc: akpm, david-b, jordan, katzj, tiwai, Linux Kernel Mailing List On Sat, 18 Apr 2009 18:01:18 -0700 Randy Dunlap <randy.dunlap@oracle.com> wrote: > Andres, how does this look to you? > > It fixes the problems that I know about. > > > This may be okay as a separate patch, but doesn't really fix the problem. cs553x-gpio isn't just for OLPC, it can be used by plain Geodes. I've lost track of the state of the patches at this point, but the solution would be to simply have CONFIG_GPIO_CS553X depend upon PCI. > From: Randy Dunlap <randy.dunlap@oracle.com> > > Fix multiple build errors due to cs553x-gpio using GPIO APIs > and using PCI APIs. > > Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com> > --- > arch/x86/Kconfig | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > --- mmotm-2009-0417-1519.orig/arch/x86/Kconfig > +++ mmotm-2009-0417-1519/arch/x86/Kconfig > @@ -1969,7 +1969,8 @@ config GEODE_MFGPT_TIMER > > config OLPC > bool "One Laptop Per Child support" > - select GPIO_CS553X > + select GPIO_CS553X if PCI > + select GPIOLIB if PCI > default n > ---help--- > Add support for detecting the unique features of the OLPC ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: + cs553x-gpio-add-amd-cs5535-cs5536-gpio-driver-support-fix-fix.patch added to -mm tree 2009-04-13 1:52 ` Andres Salomon 2009-04-13 4:26 ` Randy Dunlap @ 2009-04-17 18:34 ` Andrew Morton 1 sibling, 0 replies; 6+ messages in thread From: Andrew Morton @ 2009-04-17 18:34 UTC (permalink / raw) To: Andres Salomon; +Cc: randy.dunlap, david-b, jordan, katzj, tiwai, linux-kernel On Sun, 12 Apr 2009 21:52:03 -0400 Andres Salomon <dilinger@queued.net> wrote: > Force GPIO_CS553X to select GPIOLIB, which it makes use of. > > Signed-off-by: Andres Salomon <dilinger@debian.org> > > diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig > index 797ff45..17d7364 100644 > --- a/drivers/gpio/Kconfig > +++ b/drivers/gpio/Kconfig > @@ -148,6 +148,7 @@ comment "PCI GPIO expanders:" > config GPIO_CS553X > tristate "AMD cs5535/cs5536 GPIO support" > depends on PCI && !CS5535_GPIO && !MGEODE_LX > + select GPIOLIB > help > The AMD cs5535 and cs5536 southbridges support 28 GPIO pins that > can be used for quite a number of things. The cs553x is found on x86_64 allmodconfig: drivers/gpio/Kconfig:28:error: found recursive dependency: GPIOLIB -> GPIO_CS553X -> GPIOLIB ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2009-04-20 0:01 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <200904062355.n36NtLTf018342@imap1.linux-foundation.org>
2009-04-10 20:00 ` + cs553x-gpio-add-amd-cs5535-cs5536-gpio-driver-support-fix-fix.patch added to -mm tree Randy Dunlap
2009-04-13 1:52 ` Andres Salomon
2009-04-13 4:26 ` Randy Dunlap
2009-04-19 1:01 ` [PATCH] cs553x: fix build dependency problems Randy Dunlap
2009-04-20 0:00 ` Andres Salomon
2009-04-17 18:34 ` + cs553x-gpio-add-amd-cs5535-cs5536-gpio-driver-support-fix-fix.patch added to -mm tree Andrew Morton
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox