* [PATCH v2 -next] spi: fix spi-sprd-adi build errors when SPI_SPRD_ADI=y and HWSPINLOCK=m @ 2017-09-21 18:02 Randy Dunlap 2017-09-22 9:26 ` Mark Brown 0 siblings, 1 reply; 22+ messages in thread From: Randy Dunlap @ 2017-09-21 18:02 UTC (permalink / raw) To: LKML, linux-spi, Mark Brown From: Randy Dunlap <rdunlap@infradead.org> Fix build errors when CONFIG_HWSPINLOCK=m and SPI_SPRD_ADI=y. That combination is not allowed. drivers/spi/spi-sprd-adi.o: In function `sprd_adi_remove': spi-sprd-adi.c:(.text+0x13): undefined reference to `hwspin_lock_free' drivers/spi/spi-sprd-adi.o: In function `sprd_adi_probe': spi-sprd-adi.c:(.text+0xf5): undefined reference to `of_hwspin_lock_get_id' spi-sprd-adi.c:(.text+0x107): undefined reference to `hwspin_lock_request_specific' spi-sprd-adi.c:(.text+0x22e): undefined reference to `hwspin_lock_free' drivers/spi/spi-sprd-adi.o: In function `sprd_adi_transfer_one': spi-sprd-adi.c:(.text+0x2eb): undefined reference to `__hwspin_lock_timeout' spi-sprd-adi.c:(.text+0x349): undefined reference to `__hwspin_unlock' spi-sprd-adi.c:(.text+0x389): undefined reference to `__hwspin_lock_timeout' spi-sprd-adi.c:(.text+0x3ee): undefined reference to `__hwspin_unlock' v2: allow build with or without HWSPINLOCK, but restrict to =m if HWSPINLOCK=m. Signed-off-by: Randy Dunlap <rdunlap@infradead.org> Cc: Baolin Wang <Baolin.Wang@spreadtrum.com> Cc: Mark Brown <broonie@kernel.org> Cc: linux-spi@vger.kernel.org --- drivers/spi/Kconfig | 1 + 1 file changed, 1 insertion(+) --- linux-next-20170921.orig/drivers/spi/Kconfig +++ linux-next-20170921/drivers/spi/Kconfig @@ -625,6 +625,7 @@ config SPI_SIRF config SPI_SPRD_ADI tristate "Spreadtrum ADI controller" depends on ARCH_SPRD || COMPILE_TEST + depends on HWSPINLOCK || HWSPINLOCK=n help ADI driver based on SPI for Spreadtrum SoCs. ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2 -next] spi: fix spi-sprd-adi build errors when SPI_SPRD_ADI=y and HWSPINLOCK=m @ 2017-09-22 9:26 ` Mark Brown 0 siblings, 0 replies; 22+ messages in thread From: Mark Brown @ 2017-09-22 9:26 UTC (permalink / raw) To: Randy Dunlap; +Cc: LKML, linux-spi [-- Attachment #1: Type: text/plain, Size: 555 bytes --] On Thu, Sep 21, 2017 at 11:02:31AM -0700, Randy Dunlap wrote: > spi-sprd-adi.c:(.text+0x3ee): undefined reference to `__hwspin_unlock' > > v2: allow build with or without HWSPINLOCK, but restrict to =m > if HWSPINLOCK=m. > > Signed-off-by: Randy Dunlap <rdunlap@infradead.org> Please put inter version changelogs after the --- as covered in SubmittingPatches. This still seems like the stubs aren't doing the right thing - if we can use hwspinlocks in a module when they're enabled I'd expect to be able to build the stubs that way too. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2 -next] spi: fix spi-sprd-adi build errors when SPI_SPRD_ADI=y and HWSPINLOCK=m @ 2017-09-22 9:26 ` Mark Brown 0 siblings, 0 replies; 22+ messages in thread From: Mark Brown @ 2017-09-22 9:26 UTC (permalink / raw) To: Randy Dunlap; +Cc: LKML, linux-spi-u79uwXL29TY76Z2rM5mHXA [-- Attachment #1: Type: text/plain, Size: 581 bytes --] On Thu, Sep 21, 2017 at 11:02:31AM -0700, Randy Dunlap wrote: > spi-sprd-adi.c:(.text+0x3ee): undefined reference to `__hwspin_unlock' > > v2: allow build with or without HWSPINLOCK, but restrict to =m > if HWSPINLOCK=m. > > Signed-off-by: Randy Dunlap <rdunlap-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org> Please put inter version changelogs after the --- as covered in SubmittingPatches. This still seems like the stubs aren't doing the right thing - if we can use hwspinlocks in a module when they're enabled I'd expect to be able to build the stubs that way too. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2 -next] spi: fix spi-sprd-adi build errors when SPI_SPRD_ADI=y and HWSPINLOCK=m @ 2017-09-23 1:46 ` Randy Dunlap 0 siblings, 0 replies; 22+ messages in thread From: Randy Dunlap @ 2017-09-23 1:46 UTC (permalink / raw) To: Mark Brown; +Cc: LKML, linux-spi On 09/22/17 02:26, Mark Brown wrote: > On Thu, Sep 21, 2017 at 11:02:31AM -0700, Randy Dunlap wrote: > >> spi-sprd-adi.c:(.text+0x3ee): undefined reference to `__hwspin_unlock' >> >> v2: allow build with or without HWSPINLOCK, but restrict to =m >> if HWSPINLOCK=m. >> >> Signed-off-by: Randy Dunlap <rdunlap@infradead.org> > > Please put inter version changelogs after the --- as covered in > SubmittingPatches. This still seems like the stubs aren't doing the Yep. > right thing - if we can use hwspinlocks in a module when they're enabled > I'd expect to be able to build the stubs that way too. Sorry, I'm not understanding what you are trying to say on that one. The hwspinlock stubs are present if HWSPINLOCK is not enabled (when <linux/hwspinlock.h> is #included). The following kconfig combinations build: HWSPINLOCK SPI_SPRD_ADI n y n m y y y m m m but this combo is not allowed (with the patch) or causes build errors (without the patch): m y -- ~Randy ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2 -next] spi: fix spi-sprd-adi build errors when SPI_SPRD_ADI=y and HWSPINLOCK=m @ 2017-09-23 1:46 ` Randy Dunlap 0 siblings, 0 replies; 22+ messages in thread From: Randy Dunlap @ 2017-09-23 1:46 UTC (permalink / raw) To: Mark Brown; +Cc: LKML, linux-spi-u79uwXL29TY76Z2rM5mHXA On 09/22/17 02:26, Mark Brown wrote: > On Thu, Sep 21, 2017 at 11:02:31AM -0700, Randy Dunlap wrote: > >> spi-sprd-adi.c:(.text+0x3ee): undefined reference to `__hwspin_unlock' >> >> v2: allow build with or without HWSPINLOCK, but restrict to =m >> if HWSPINLOCK=m. >> >> Signed-off-by: Randy Dunlap <rdunlap-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org> > > Please put inter version changelogs after the --- as covered in > SubmittingPatches. This still seems like the stubs aren't doing the Yep. > right thing - if we can use hwspinlocks in a module when they're enabled > I'd expect to be able to build the stubs that way too. Sorry, I'm not understanding what you are trying to say on that one. The hwspinlock stubs are present if HWSPINLOCK is not enabled (when <linux/hwspinlock.h> is #included). The following kconfig combinations build: HWSPINLOCK SPI_SPRD_ADI n y n m y y y m m m but this combo is not allowed (with the patch) or causes build errors (without the patch): m y -- ~Randy -- To unsubscribe from this list: send the line "unsubscribe linux-spi" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2 -next] spi: fix spi-sprd-adi build errors when SPI_SPRD_ADI=y and HWSPINLOCK=m @ 2017-09-25 16:12 ` Mark Brown 0 siblings, 0 replies; 22+ messages in thread From: Mark Brown @ 2017-09-25 16:12 UTC (permalink / raw) To: Randy Dunlap; +Cc: LKML, linux-spi [-- Attachment #1: Type: text/plain, Size: 613 bytes --] On Fri, Sep 22, 2017 at 06:46:46PM -0700, Randy Dunlap wrote: > On 09/22/17 02:26, Mark Brown wrote: > > right thing - if we can use hwspinlocks in a module when they're enabled > > I'd expect to be able to build the stubs that way too. > Sorry, I'm not understanding what you are trying to say on that one. > HWSPINLOCK SPI_SPRD_ADI > but this combo is not allowed (with the patch) or causes build errors > (without the patch): > m y Why is that not just an || COMPILE_TEST dependency then? The dependency you're trying to introduce is weird and confusing, we shouldn't be having to do things like that. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2 -next] spi: fix spi-sprd-adi build errors when SPI_SPRD_ADI=y and HWSPINLOCK=m @ 2017-09-25 16:12 ` Mark Brown 0 siblings, 0 replies; 22+ messages in thread From: Mark Brown @ 2017-09-25 16:12 UTC (permalink / raw) To: Randy Dunlap; +Cc: LKML, linux-spi-u79uwXL29TY76Z2rM5mHXA [-- Attachment #1: Type: text/plain, Size: 613 bytes --] On Fri, Sep 22, 2017 at 06:46:46PM -0700, Randy Dunlap wrote: > On 09/22/17 02:26, Mark Brown wrote: > > right thing - if we can use hwspinlocks in a module when they're enabled > > I'd expect to be able to build the stubs that way too. > Sorry, I'm not understanding what you are trying to say on that one. > HWSPINLOCK SPI_SPRD_ADI > but this combo is not allowed (with the patch) or causes build errors > (without the patch): > m y Why is that not just an || COMPILE_TEST dependency then? The dependency you're trying to introduce is weird and confusing, we shouldn't be having to do things like that. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2 -next] spi: fix spi-sprd-adi build errors when SPI_SPRD_ADI=y and HWSPINLOCK=m 2017-09-25 16:12 ` Mark Brown (?) @ 2017-09-25 17:19 ` Randy Dunlap 2017-09-25 17:36 ` Mark Brown 2017-09-25 19:20 ` Geert Uytterhoeven -1 siblings, 2 replies; 22+ messages in thread From: Randy Dunlap @ 2017-09-25 17:19 UTC (permalink / raw) To: Mark Brown; +Cc: LKML, linux-spi On 09/25/17 09:12, Mark Brown wrote: > On Fri, Sep 22, 2017 at 06:46:46PM -0700, Randy Dunlap wrote: >> On 09/22/17 02:26, Mark Brown wrote: > >>> right thing - if we can use hwspinlocks in a module when they're enabled >>> I'd expect to be able to build the stubs that way too. > >> Sorry, I'm not understanding what you are trying to say on that one. > >> HWSPINLOCK SPI_SPRD_ADI > >> but this combo is not allowed (with the patch) or causes build errors >> (without the patch): >> m y > > Why is that not just an || COMPILE_TEST dependency then? The dependency > you're trying to introduce is weird and confusing, we shouldn't be > having to do things like that. It already uses COMPILE_TEST: config SPI_SPRD_ADI tristate "Spreadtrum ADI controller" depends on ARCH_SPRD || COMPILE_TEST help ADI driver based on SPI for Spreadtrum SoCs. but that allows build errors when SPI_SPRD_ADI=y and HWSPINLOCK=y. As for the dependency that I am adding: + depends on HWSPINLOCK || HWSPINLOCK=n that idiom is used over and over again in Kconfig files to prevent this kind of build problem in loadable modules. -- ~Randy ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2 -next] spi: fix spi-sprd-adi build errors when SPI_SPRD_ADI=y and HWSPINLOCK=m @ 2017-09-25 17:36 ` Mark Brown 0 siblings, 0 replies; 22+ messages in thread From: Mark Brown @ 2017-09-25 17:36 UTC (permalink / raw) To: Randy Dunlap; +Cc: LKML, linux-spi [-- Attachment #1: Type: text/plain, Size: 904 bytes --] On Mon, Sep 25, 2017 at 10:19:56AM -0700, Randy Dunlap wrote: > On 09/25/17 09:12, Mark Brown wrote: > > Why is that not just an || COMPILE_TEST dependency then? The dependency > > you're trying to introduce is weird and confusing, we shouldn't be > > having to do things like that. > It already uses COMPILE_TEST: > config SPI_SPRD_ADI > tristate "Spreadtrum ADI controller" > depends on ARCH_SPRD || COMPILE_TEST > help > ADI driver based on SPI for Spreadtrum SoCs. > but that allows build errors when SPI_SPRD_ADI=y and HWSPINLOCK=y. > As for the dependency that I am adding: > + depends on HWSPINLOCK || HWSPINLOCK=n > that idiom is used over and over again in Kconfig files to prevent this kind of > build problem in loadable modules. And what is that equivalent to in relation to COMPILE_TEST, especially considering the situations where you'd build this without hwspinlock support? [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2 -next] spi: fix spi-sprd-adi build errors when SPI_SPRD_ADI=y and HWSPINLOCK=m @ 2017-09-25 17:36 ` Mark Brown 0 siblings, 0 replies; 22+ messages in thread From: Mark Brown @ 2017-09-25 17:36 UTC (permalink / raw) To: Randy Dunlap; +Cc: LKML, linux-spi-u79uwXL29TY76Z2rM5mHXA [-- Attachment #1: Type: text/plain, Size: 904 bytes --] On Mon, Sep 25, 2017 at 10:19:56AM -0700, Randy Dunlap wrote: > On 09/25/17 09:12, Mark Brown wrote: > > Why is that not just an || COMPILE_TEST dependency then? The dependency > > you're trying to introduce is weird and confusing, we shouldn't be > > having to do things like that. > It already uses COMPILE_TEST: > config SPI_SPRD_ADI > tristate "Spreadtrum ADI controller" > depends on ARCH_SPRD || COMPILE_TEST > help > ADI driver based on SPI for Spreadtrum SoCs. > but that allows build errors when SPI_SPRD_ADI=y and HWSPINLOCK=y. > As for the dependency that I am adding: > + depends on HWSPINLOCK || HWSPINLOCK=n > that idiom is used over and over again in Kconfig files to prevent this kind of > build problem in loadable modules. And what is that equivalent to in relation to COMPILE_TEST, especially considering the situations where you'd build this without hwspinlock support? [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2 -next] spi: fix spi-sprd-adi build errors when SPI_SPRD_ADI=y and HWSPINLOCK=m @ 2017-09-25 19:20 ` Geert Uytterhoeven 0 siblings, 0 replies; 22+ messages in thread From: Geert Uytterhoeven @ 2017-09-25 19:20 UTC (permalink / raw) To: Randy Dunlap; +Cc: Mark Brown, LKML, linux-spi Hi Randy, On Mon, Sep 25, 2017 at 7:19 PM, Randy Dunlap <rdunlap@infradead.org> wrote: > On 09/25/17 09:12, Mark Brown wrote: >> On Fri, Sep 22, 2017 at 06:46:46PM -0700, Randy Dunlap wrote: >>> On 09/22/17 02:26, Mark Brown wrote: >> >>>> right thing - if we can use hwspinlocks in a module when they're enabled >>>> I'd expect to be able to build the stubs that way too. >> >>> Sorry, I'm not understanding what you are trying to say on that one. >> >>> HWSPINLOCK SPI_SPRD_ADI >> >>> but this combo is not allowed (with the patch) or causes build errors >>> (without the patch): >>> m y >> >> Why is that not just an || COMPILE_TEST dependency then? The dependency >> you're trying to introduce is weird and confusing, we shouldn't be >> having to do things like that. > > It already uses COMPILE_TEST: > > config SPI_SPRD_ADI > tristate "Spreadtrum ADI controller" > depends on ARCH_SPRD || COMPILE_TEST > help > ADI driver based on SPI for Spreadtrum SoCs. > > but that allows build errors when SPI_SPRD_ADI=y and HWSPINLOCK=y. You mean "when SPI_SPRD_ADI=y and HWSPINLOCK=m"? > As for the dependency that I am adding: > + depends on HWSPINLOCK || HWSPINLOCK=n > > that idiom is used over and over again in Kconfig files to prevent this kind of > build problem in loadable modules. Indeed, it's confusing, but used all over the place. The issue is builtin drivers that depend on a modular API. The clean way is to separate API and implementation, so the API can be builtin, and the implementation can be modular. Hence the API should provide stubs that call into function pointers, to be registered by the module providing the implementation. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2 -next] spi: fix spi-sprd-adi build errors when SPI_SPRD_ADI=y and HWSPINLOCK=m @ 2017-09-25 19:20 ` Geert Uytterhoeven 0 siblings, 0 replies; 22+ messages in thread From: Geert Uytterhoeven @ 2017-09-25 19:20 UTC (permalink / raw) To: Randy Dunlap; +Cc: Mark Brown, LKML, linux-spi Hi Randy, On Mon, Sep 25, 2017 at 7:19 PM, Randy Dunlap <rdunlap-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org> wrote: > On 09/25/17 09:12, Mark Brown wrote: >> On Fri, Sep 22, 2017 at 06:46:46PM -0700, Randy Dunlap wrote: >>> On 09/22/17 02:26, Mark Brown wrote: >> >>>> right thing - if we can use hwspinlocks in a module when they're enabled >>>> I'd expect to be able to build the stubs that way too. >> >>> Sorry, I'm not understanding what you are trying to say on that one. >> >>> HWSPINLOCK SPI_SPRD_ADI >> >>> but this combo is not allowed (with the patch) or causes build errors >>> (without the patch): >>> m y >> >> Why is that not just an || COMPILE_TEST dependency then? The dependency >> you're trying to introduce is weird and confusing, we shouldn't be >> having to do things like that. > > It already uses COMPILE_TEST: > > config SPI_SPRD_ADI > tristate "Spreadtrum ADI controller" > depends on ARCH_SPRD || COMPILE_TEST > help > ADI driver based on SPI for Spreadtrum SoCs. > > but that allows build errors when SPI_SPRD_ADI=y and HWSPINLOCK=y. You mean "when SPI_SPRD_ADI=y and HWSPINLOCK=m"? > As for the dependency that I am adding: > + depends on HWSPINLOCK || HWSPINLOCK=n > > that idiom is used over and over again in Kconfig files to prevent this kind of > build problem in loadable modules. Indeed, it's confusing, but used all over the place. The issue is builtin drivers that depend on a modular API. The clean way is to separate API and implementation, so the API can be builtin, and the implementation can be modular. Hence the API should provide stubs that call into function pointers, to be registered by the module providing the implementation. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- To unsubscribe from this list: send the line "unsubscribe linux-spi" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2 -next] spi: fix spi-sprd-adi build errors when SPI_SPRD_ADI=y and HWSPINLOCK=m 2017-09-25 19:20 ` Geert Uytterhoeven (?) @ 2017-09-25 19:22 ` Mark Brown 2017-09-25 19:31 ` Randy Dunlap -1 siblings, 1 reply; 22+ messages in thread From: Mark Brown @ 2017-09-25 19:22 UTC (permalink / raw) To: Geert Uytterhoeven; +Cc: Randy Dunlap, LKML, linux-spi [-- Attachment #1: Type: text/plain, Size: 519 bytes --] On Mon, Sep 25, 2017 at 09:20:31PM +0200, Geert Uytterhoeven wrote: > The issue is builtin drivers that depend on a modular API. The clean way > is to separate API and implementation, so the API can be builtin, and the > implementation can be modular. > Hence the API should provide stubs that call into function pointers, to be > registered by the module providing the implementation. In this case the problem is even more basic in that the driver does actually depend on having hwspinlocks for any production use. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2 -next] spi: fix spi-sprd-adi build errors when SPI_SPRD_ADI=y and HWSPINLOCK=m 2017-09-25 19:22 ` Mark Brown @ 2017-09-25 19:31 ` Randy Dunlap 2017-09-25 19:44 ` Mark Brown 0 siblings, 1 reply; 22+ messages in thread From: Randy Dunlap @ 2017-09-25 19:31 UTC (permalink / raw) To: Mark Brown, Geert Uytterhoeven; +Cc: LKML, linux-spi On 09/25/17 12:22, Mark Brown wrote: > On Mon, Sep 25, 2017 at 09:20:31PM +0200, Geert Uytterhoeven wrote: > >> The issue is builtin drivers that depend on a modular API. The clean way >> is to separate API and implementation, so the API can be builtin, and the >> implementation can be modular. >> Hence the API should provide stubs that call into function pointers, to be >> registered by the module providing the implementation. > > In this case the problem is even more basic in that the driver does > actually depend on having hwspinlocks for any production use. > so just add: depends on HWSPINLOCK Is that satisfactory to you? -- ~Randy ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2 -next] spi: fix spi-sprd-adi build errors when SPI_SPRD_ADI=y and HWSPINLOCK=m @ 2017-09-25 19:44 ` Mark Brown 0 siblings, 0 replies; 22+ messages in thread From: Mark Brown @ 2017-09-25 19:44 UTC (permalink / raw) To: Randy Dunlap; +Cc: Geert Uytterhoeven, LKML, linux-spi [-- Attachment #1: Type: text/plain, Size: 383 bytes --] On Mon, Sep 25, 2017 at 12:31:03PM -0700, Randy Dunlap wrote: > On 09/25/17 12:22, Mark Brown wrote: > > In this case the problem is even more basic in that the driver does > > actually depend on having hwspinlocks for any production use. > so just add: > depends on HWSPINLOCK > Is that satisfactory to you? That would then get in the way of build test coverage. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2 -next] spi: fix spi-sprd-adi build errors when SPI_SPRD_ADI=y and HWSPINLOCK=m @ 2017-09-25 19:44 ` Mark Brown 0 siblings, 0 replies; 22+ messages in thread From: Mark Brown @ 2017-09-25 19:44 UTC (permalink / raw) To: Randy Dunlap; +Cc: Geert Uytterhoeven, LKML, linux-spi [-- Attachment #1: Type: text/plain, Size: 383 bytes --] On Mon, Sep 25, 2017 at 12:31:03PM -0700, Randy Dunlap wrote: > On 09/25/17 12:22, Mark Brown wrote: > > In this case the problem is even more basic in that the driver does > > actually depend on having hwspinlocks for any production use. > so just add: > depends on HWSPINLOCK > Is that satisfactory to you? That would then get in the way of build test coverage. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2 -next] spi: fix spi-sprd-adi build errors when SPI_SPRD_ADI=y and HWSPINLOCK=m @ 2017-09-25 19:53 ` Randy Dunlap 0 siblings, 0 replies; 22+ messages in thread From: Randy Dunlap @ 2017-09-25 19:53 UTC (permalink / raw) To: Mark Brown; +Cc: Geert Uytterhoeven, LKML, linux-spi On 09/25/17 12:44, Mark Brown wrote: > On Mon, Sep 25, 2017 at 12:31:03PM -0700, Randy Dunlap wrote: >> On 09/25/17 12:22, Mark Brown wrote: > >>> In this case the problem is even more basic in that the driver does >>> actually depend on having hwspinlocks for any production use. > >> so just add: >> depends on HWSPINLOCK > >> Is that satisfactory to you? > > That would then get in the way of build test coverage. I don't agree, but whatever. I give up. -- ~Randy ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2 -next] spi: fix spi-sprd-adi build errors when SPI_SPRD_ADI=y and HWSPINLOCK=m @ 2017-09-25 19:53 ` Randy Dunlap 0 siblings, 0 replies; 22+ messages in thread From: Randy Dunlap @ 2017-09-25 19:53 UTC (permalink / raw) To: Mark Brown; +Cc: Geert Uytterhoeven, LKML, linux-spi On 09/25/17 12:44, Mark Brown wrote: > On Mon, Sep 25, 2017 at 12:31:03PM -0700, Randy Dunlap wrote: >> On 09/25/17 12:22, Mark Brown wrote: > >>> In this case the problem is even more basic in that the driver does >>> actually depend on having hwspinlocks for any production use. > >> so just add: >> depends on HWSPINLOCK > >> Is that satisfactory to you? > > That would then get in the way of build test coverage. I don't agree, but whatever. I give up. -- ~Randy -- To unsubscribe from this list: send the line "unsubscribe linux-spi" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2 -next] spi: fix spi-sprd-adi build errors when SPI_SPRD_ADI=y and HWSPINLOCK=m @ 2017-09-25 21:11 ` Mark Brown 0 siblings, 0 replies; 22+ messages in thread From: Mark Brown @ 2017-09-25 21:11 UTC (permalink / raw) To: Randy Dunlap; +Cc: Geert Uytterhoeven, LKML, linux-spi [-- Attachment #1: Type: text/plain, Size: 486 bytes --] On Mon, Sep 25, 2017 at 12:53:20PM -0700, Randy Dunlap wrote: > On 09/25/17 12:44, Mark Brown wrote: > > That would then get in the way of build test coverage. > I don't agree, but whatever. I give up. OK. Just as a general thing it's really important that dependency changes be thought through and express what the constraints are in ways that reflect what's actually going on and what's useful for users, it seems to be a frequent problem and it does get in the way. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2 -next] spi: fix spi-sprd-adi build errors when SPI_SPRD_ADI=y and HWSPINLOCK=m @ 2017-09-25 21:11 ` Mark Brown 0 siblings, 0 replies; 22+ messages in thread From: Mark Brown @ 2017-09-25 21:11 UTC (permalink / raw) To: Randy Dunlap; +Cc: Geert Uytterhoeven, LKML, linux-spi [-- Attachment #1: Type: text/plain, Size: 486 bytes --] On Mon, Sep 25, 2017 at 12:53:20PM -0700, Randy Dunlap wrote: > On 09/25/17 12:44, Mark Brown wrote: > > That would then get in the way of build test coverage. > I don't agree, but whatever. I give up. OK. Just as a general thing it's really important that dependency changes be thought through and express what the constraints are in ways that reflect what's actually going on and what's useful for users, it seems to be a frequent problem and it does get in the way. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2 -next] spi: fix spi-sprd-adi build errors when SPI_SPRD_ADI=y and HWSPINLOCK=m @ 2017-09-25 22:32 ` Randy Dunlap 0 siblings, 0 replies; 22+ messages in thread From: Randy Dunlap @ 2017-09-25 22:32 UTC (permalink / raw) To: LKML; +Cc: Mark Brown, Geert Uytterhoeven, linux-spi, Baolin Wang On 09/25/17 14:11, Mark Brown wrote: > On Mon, Sep 25, 2017 at 12:53:20PM -0700, Randy Dunlap wrote: >> On 09/25/17 12:44, Mark Brown wrote: > >>> That would then get in the way of build test coverage. > >> I don't agree, but whatever. I give up. > > OK. Just as a general thing it's really important that dependency > changes be thought through and express what the constraints are in ways > that reflect what's actually going on and what's useful for users, it > seems to be a frequent problem and it does get in the way. Yes, it's a frequent problem that drivers do not build due to some kconfig dependency issue(s). I would prefer to see the driver maintainer(s) fix build errors themselves, but they are not always aware of the idiosyncrasies of kconfig. I think that people often submit drivers (or driver changes) after building it in one (working) configuration only, without regard for the other possible combinations. -- ~Randy ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2 -next] spi: fix spi-sprd-adi build errors when SPI_SPRD_ADI=y and HWSPINLOCK=m @ 2017-09-25 22:32 ` Randy Dunlap 0 siblings, 0 replies; 22+ messages in thread From: Randy Dunlap @ 2017-09-25 22:32 UTC (permalink / raw) To: LKML; +Cc: Mark Brown, Geert Uytterhoeven, linux-spi, Baolin Wang On 09/25/17 14:11, Mark Brown wrote: > On Mon, Sep 25, 2017 at 12:53:20PM -0700, Randy Dunlap wrote: >> On 09/25/17 12:44, Mark Brown wrote: > >>> That would then get in the way of build test coverage. > >> I don't agree, but whatever. I give up. > > OK. Just as a general thing it's really important that dependency > changes be thought through and express what the constraints are in ways > that reflect what's actually going on and what's useful for users, it > seems to be a frequent problem and it does get in the way. Yes, it's a frequent problem that drivers do not build due to some kconfig dependency issue(s). I would prefer to see the driver maintainer(s) fix build errors themselves, but they are not always aware of the idiosyncrasies of kconfig. I think that people often submit drivers (or driver changes) after building it in one (working) configuration only, without regard for the other possible combinations. -- ~Randy -- To unsubscribe from this list: send the line "unsubscribe linux-spi" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 22+ messages in thread
end of thread, other threads:[~2017-09-25 22:33 UTC | newest] Thread overview: 22+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2017-09-21 18:02 [PATCH v2 -next] spi: fix spi-sprd-adi build errors when SPI_SPRD_ADI=y and HWSPINLOCK=m Randy Dunlap 2017-09-22 9:26 ` Mark Brown 2017-09-22 9:26 ` Mark Brown 2017-09-23 1:46 ` Randy Dunlap 2017-09-23 1:46 ` Randy Dunlap 2017-09-25 16:12 ` Mark Brown 2017-09-25 16:12 ` Mark Brown 2017-09-25 17:19 ` Randy Dunlap 2017-09-25 17:36 ` Mark Brown 2017-09-25 17:36 ` Mark Brown 2017-09-25 19:20 ` Geert Uytterhoeven 2017-09-25 19:20 ` Geert Uytterhoeven 2017-09-25 19:22 ` Mark Brown 2017-09-25 19:31 ` Randy Dunlap 2017-09-25 19:44 ` Mark Brown 2017-09-25 19:44 ` Mark Brown 2017-09-25 19:53 ` Randy Dunlap 2017-09-25 19:53 ` Randy Dunlap 2017-09-25 21:11 ` Mark Brown 2017-09-25 21:11 ` Mark Brown 2017-09-25 22:32 ` Randy Dunlap 2017-09-25 22:32 ` Randy Dunlap
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.