Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1621078 > unrolled thread
| Started by | Tobias Klauser <tklauser@distanz.ch> |
|---|---|
| First post | 2017-04-11 11:30 +0200 |
| Last post | 2017-04-12 09:00 +0200 |
| Articles | 5 — 2 participants |
Back to article view | Back to linux.kernel
[PATCH] fpga: allow to compile-test Altera FPGA bridge drivers Tobias Klauser <tklauser@distanz.ch> - 2017-04-11 11:30 +0200
Re: [PATCH] fpga: allow to compile-test Altera FPGA bridge drivers Alan Tull <atull@kernel.org> - 2017-04-11 16:30 +0200
Re: [PATCH] fpga: allow to compile-test Altera FPGA bridge drivers Tobias Klauser <tklauser@distanz.ch> - 2017-04-11 16:40 +0200
Re: [PATCH] fpga: allow to compile-test Altera FPGA bridge drivers Alan Tull <atull@kernel.org> - 2017-04-11 17:30 +0200
Re: [PATCH] fpga: allow to compile-test Altera FPGA bridge drivers Tobias Klauser <tklauser@distanz.ch> - 2017-04-12 09:00 +0200
| From | Tobias Klauser <tklauser@distanz.ch> |
|---|---|
| Date | 2017-04-11 11:30 +0200 |
| Subject | [PATCH] fpga: allow to compile-test Altera FPGA bridge drivers |
| Message-ID | <tv0hA-543-23@gated-at.bofh.it> |
Add COMPILE_TEST to the Kconfig entries for the Altera SoCFPGA FPGA Bridge and the Altera FPGA Freeze Bridge drivers to allow test-compiling them on other architectures. Signed-off-by: Tobias Klauser <tklauser@distanz.ch> --- drivers/fpga/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig index 161ba9dccede..38ab05d261bf 100644 --- a/drivers/fpga/Kconfig +++ b/drivers/fpga/Kconfig @@ -69,14 +69,14 @@ config FPGA_BRIDGE config SOCFPGA_FPGA_BRIDGE tristate "Altera SoCFPGA FPGA Bridges" - depends on ARCH_SOCFPGA && FPGA_BRIDGE + depends on (ARCH_SOCFPGA || COMPILE_TEST) && FPGA_BRIDGE help Say Y to enable drivers for FPGA bridges for Altera SOCFPGA devices. config ALTERA_FREEZE_BRIDGE tristate "Altera FPGA Freeze Bridge" - depends on ARCH_SOCFPGA && FPGA_BRIDGE + depends on (ARCH_SOCFPGA || COMPILE_TEST) && FPGA_BRIDGE help Say Y to enable drivers for Altera FPGA Freeze bridges. A freeze bridge is a bridge that exists in the FPGA fabric to -- 2.12.2
[toc] | [next] | [standalone]
| From | Alan Tull <atull@kernel.org> |
|---|---|
| Date | 2017-04-11 16:30 +0200 |
| Message-ID | <tv4XV-8aE-37@gated-at.bofh.it> |
| In reply to | #1621078 |
On Tue, Apr 11, 2017 at 4:22 AM, Tobias Klauser <tklauser@distanz.ch> wrote: Hi Tobias, Thanks for your fixes that you have been sending. I disagree with this particular one (see below), but I appreciate your efforts. > Add COMPILE_TEST to the Kconfig entries for the Altera SoCFPGA FPGA > Bridge and the Altera FPGA Freeze Bridge drivers to allow test-compiling > them on other architectures. > > Signed-off-by: Tobias Klauser <tklauser@distanz.ch> > --- > drivers/fpga/Kconfig | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig > index 161ba9dccede..38ab05d261bf 100644 > --- a/drivers/fpga/Kconfig > +++ b/drivers/fpga/Kconfig > @@ -69,14 +69,14 @@ config FPGA_BRIDGE > > config SOCFPGA_FPGA_BRIDGE > tristate "Altera SoCFPGA FPGA Bridges" > - depends on ARCH_SOCFPGA && FPGA_BRIDGE > + depends on (ARCH_SOCFPGA || COMPILE_TEST) && FPGA_BRIDGE It doesn't make sense to compile test these on other arch's. The bridges enabled by this only appear on Cyclone 5 SoCFPGA parts. > help > Say Y to enable drivers for FPGA bridges for Altera SOCFPGA > devices. > > config ALTERA_FREEZE_BRIDGE > tristate "Altera FPGA Freeze Bridge" > - depends on ARCH_SOCFPGA && FPGA_BRIDGE > + depends on (ARCH_SOCFPGA || COMPILE_TEST) && FPGA_BRIDGE These bridges can be used on Altera PCIe FPGA's. So this shouldn't have been dependent on ARCH_SOCFPGA in the first place. Alan > help > Say Y to enable drivers for Altera FPGA Freeze bridges. A > freeze bridge is a bridge that exists in the FPGA fabric to > -- > 2.12.2 > >
[toc] | [prev] | [next] | [standalone]
| From | Tobias Klauser <tklauser@distanz.ch> |
|---|---|
| Date | 2017-04-11 16:40 +0200 |
| Message-ID | <tv57B-8eV-63@gated-at.bofh.it> |
| In reply to | #1621355 |
Hi Alan, On 2017-04-11 at 16:27:03 +0200, Alan Tull <atull@kernel.org> wrote: > On Tue, Apr 11, 2017 at 4:22 AM, Tobias Klauser <tklauser@distanz.ch> wrote: > > Hi Tobias, > > Thanks for your fixes that you have been sending. I disagree with > this particular one (see below), but I appreciate your efforts. Thanks for your feedback on the patch. > > Add COMPILE_TEST to the Kconfig entries for the Altera SoCFPGA FPGA > > Bridge and the Altera FPGA Freeze Bridge drivers to allow test-compiling > > them on other architectures. > > > > Signed-off-by: Tobias Klauser <tklauser@distanz.ch> > > --- > > drivers/fpga/Kconfig | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig > > index 161ba9dccede..38ab05d261bf 100644 > > --- a/drivers/fpga/Kconfig > > +++ b/drivers/fpga/Kconfig > > @@ -69,14 +69,14 @@ config FPGA_BRIDGE > > > > config SOCFPGA_FPGA_BRIDGE > > tristate "Altera SoCFPGA FPGA Bridges" > > - depends on ARCH_SOCFPGA && FPGA_BRIDGE > > + depends on (ARCH_SOCFPGA || COMPILE_TEST) && FPGA_BRIDGE > > It doesn't make sense to compile test these on other arch's. > The bridges enabled by this only appear on Cyclone 5 SoCFPGA parts. There are many drivers in the tree where this is the case (i.e. the driver is compiled on an arch where the corresponding hardware is not available). Still it could be benefitial IMO in order to catch compiler errors and warnings, e.g. due to API/header changes earlier on. I occasionally found this quite handy for janitorial/cleanup patches where I didn't have the corresponding cross-compiler/hardware. In any case, I don't feel strongly about this. So feel free to drop at your discretion. > > help > > Say Y to enable drivers for FPGA bridges for Altera SOCFPGA > > devices. > > > > config ALTERA_FREEZE_BRIDGE > > tristate "Altera FPGA Freeze Bridge" > > - depends on ARCH_SOCFPGA && FPGA_BRIDGE > > + depends on (ARCH_SOCFPGA || COMPILE_TEST) && FPGA_BRIDGE > > These bridges can be used on Altera PCIe FPGA's. So this shouldn't > have been dependent on ARCH_SOCFPGA in the first place. Ok. So the dependency on ARCH_SOCFPGA should be dropped altogether? I can send an updated patch if you want. > Alan > > > help > > Say Y to enable drivers for Altera FPGA Freeze bridges. A > > freeze bridge is a bridge that exists in the FPGA fabric to > > -- > > 2.12.2 > > > > >
[toc] | [prev] | [next] | [standalone]
| From | Alan Tull <atull@kernel.org> |
|---|---|
| Date | 2017-04-11 17:30 +0200 |
| Message-ID | <tv5TY-km-21@gated-at.bofh.it> |
| In reply to | #1621373 |
On Tue, Apr 11, 2017 at 9:35 AM, Tobias Klauser <tklauser@distanz.ch> wrote: > Hi Alan, > > On 2017-04-11 at 16:27:03 +0200, Alan Tull <atull@kernel.org> wrote: >> On Tue, Apr 11, 2017 at 4:22 AM, Tobias Klauser <tklauser@distanz.ch> wrote: >> >> Hi Tobias, >> >> Thanks for your fixes that you have been sending. I disagree with >> this particular one (see below), but I appreciate your efforts. > > Thanks for your feedback on the patch. > >> > Add COMPILE_TEST to the Kconfig entries for the Altera SoCFPGA FPGA >> > Bridge and the Altera FPGA Freeze Bridge drivers to allow test-compiling >> > them on other architectures. >> > >> > Signed-off-by: Tobias Klauser <tklauser@distanz.ch> >> > --- >> > drivers/fpga/Kconfig | 4 ++-- >> > 1 file changed, 2 insertions(+), 2 deletions(-) >> > >> > diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig >> > index 161ba9dccede..38ab05d261bf 100644 >> > --- a/drivers/fpga/Kconfig >> > +++ b/drivers/fpga/Kconfig >> > @@ -69,14 +69,14 @@ config FPGA_BRIDGE >> > >> > config SOCFPGA_FPGA_BRIDGE >> > tristate "Altera SoCFPGA FPGA Bridges" >> > - depends on ARCH_SOCFPGA && FPGA_BRIDGE >> > + depends on (ARCH_SOCFPGA || COMPILE_TEST) && FPGA_BRIDGE >> >> It doesn't make sense to compile test these on other arch's. >> The bridges enabled by this only appear on Cyclone 5 SoCFPGA parts. > > There are many drivers in the tree where this is the case (i.e. the > driver is compiled on an arch where the corresponding hardware is not > available). Still it could be benefitial IMO in order to catch compiler > errors and warnings, e.g. due to API/header changes earlier on. I > occasionally found this quite handy for janitorial/cleanup patches where > I didn't have the corresponding cross-compiler/hardware. That sounds good and useful. > > In any case, I don't feel strongly about this. So feel free to drop at > your discretion. > >> > help >> > Say Y to enable drivers for FPGA bridges for Altera SOCFPGA >> > devices. >> > >> > config ALTERA_FREEZE_BRIDGE >> > tristate "Altera FPGA Freeze Bridge" >> > - depends on ARCH_SOCFPGA && FPGA_BRIDGE >> > + depends on (ARCH_SOCFPGA || COMPILE_TEST) && FPGA_BRIDGE >> >> These bridges can be used on Altera PCIe FPGA's. So this shouldn't >> have been dependent on ARCH_SOCFPGA in the first place. > > Ok. So the dependency on ARCH_SOCFPGA should be dropped altogether? I > can send an updated patch if you want. Yes please. Thanks! ALan > >> Alan >> >> > help >> > Say Y to enable drivers for Altera FPGA Freeze bridges. A >> > freeze bridge is a bridge that exists in the FPGA fabric to >> > -- >> > 2.12.2 >> > >> > >>
[toc] | [prev] | [next] | [standalone]
| From | Tobias Klauser <tklauser@distanz.ch> |
|---|---|
| Date | 2017-04-12 09:00 +0200 |
| Message-ID | <tvkpX-1dW-5@gated-at.bofh.it> |
| In reply to | #1621410 |
On 2017-04-11 at 17:21:19 +0200, Alan Tull <atull@kernel.org> wrote: > On Tue, Apr 11, 2017 at 9:35 AM, Tobias Klauser <tklauser@distanz.ch> wrote: > > Hi Alan, > > > > On 2017-04-11 at 16:27:03 +0200, Alan Tull <atull@kernel.org> wrote: > >> On Tue, Apr 11, 2017 at 4:22 AM, Tobias Klauser <tklauser@distanz.ch> wrote: > >> > >> Hi Tobias, > >> > >> Thanks for your fixes that you have been sending. I disagree with > >> this particular one (see below), but I appreciate your efforts. > > > > Thanks for your feedback on the patch. > > > >> > Add COMPILE_TEST to the Kconfig entries for the Altera SoCFPGA FPGA > >> > Bridge and the Altera FPGA Freeze Bridge drivers to allow test-compiling > >> > them on other architectures. > >> > > >> > Signed-off-by: Tobias Klauser <tklauser@distanz.ch> > >> > --- > >> > drivers/fpga/Kconfig | 4 ++-- > >> > 1 file changed, 2 insertions(+), 2 deletions(-) > >> > > >> > diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig > >> > index 161ba9dccede..38ab05d261bf 100644 > >> > --- a/drivers/fpga/Kconfig > >> > +++ b/drivers/fpga/Kconfig > >> > @@ -69,14 +69,14 @@ config FPGA_BRIDGE > >> > > >> > config SOCFPGA_FPGA_BRIDGE > >> > tristate "Altera SoCFPGA FPGA Bridges" > >> > - depends on ARCH_SOCFPGA && FPGA_BRIDGE > >> > + depends on (ARCH_SOCFPGA || COMPILE_TEST) && FPGA_BRIDGE > >> > >> It doesn't make sense to compile test these on other arch's. > >> The bridges enabled by this only appear on Cyclone 5 SoCFPGA parts. > > > > There are many drivers in the tree where this is the case (i.e. the > > driver is compiled on an arch where the corresponding hardware is not > > available). Still it could be benefitial IMO in order to catch compiler > > errors and warnings, e.g. due to API/header changes earlier on. I > > occasionally found this quite handy for janitorial/cleanup patches where > > I didn't have the corresponding cross-compiler/hardware. > > That sounds good and useful. > > > > > In any case, I don't feel strongly about this. So feel free to drop at > > your discretion. > > > >> > help > >> > Say Y to enable drivers for FPGA bridges for Altera SOCFPGA > >> > devices. > >> > > >> > config ALTERA_FREEZE_BRIDGE > >> > tristate "Altera FPGA Freeze Bridge" > >> > - depends on ARCH_SOCFPGA && FPGA_BRIDGE > >> > + depends on (ARCH_SOCFPGA || COMPILE_TEST) && FPGA_BRIDGE > >> > >> These bridges can be used on Altera PCIe FPGA's. So this shouldn't > >> have been dependent on ARCH_SOCFPGA in the first place. > > > > Ok. So the dependency on ARCH_SOCFPGA should be dropped altogether? I > > can send an updated patch if you want. > > Yes please. Thanks, I'll send a v2.
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web