* [PATCH] net: calxedaxgmac: fix driver dependencies
@ 2014-09-01 15:39 Bartlomiej Zolnierkiewicz
2014-09-02 2:36 ` Rob Herring
2014-09-02 20:23 ` David Miller
0 siblings, 2 replies; 4+ messages in thread
From: Bartlomiej Zolnierkiewicz @ 2014-09-01 15:39 UTC (permalink / raw)
To: David S. Miller; +Cc: Rob Herring, Kyungmin Park, netdev, linux-kernel
Calxeda 1G/10G XGMAC Ethernet support should be available only on
Calxeda ECX-1000/2000 (Highbank/Midway) platforms.
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Acked-by: Kyungmin Park <kyungmin.park@samsung.com>
Cc: Rob Herring <robh@kernel.org>
---
drivers/net/ethernet/calxeda/Kconfig | 1 +
1 file changed, 1 insertion(+)
Index: b/drivers/net/ethernet/calxeda/Kconfig
===================================================================
--- a/drivers/net/ethernet/calxeda/Kconfig 2014-07-30 14:31:12.159522474 +0200
+++ b/drivers/net/ethernet/calxeda/Kconfig 2014-09-01 17:33:49.232810483 +0200
@@ -1,6 +1,7 @@
config NET_CALXEDA_XGMAC
tristate "Calxeda 1G/10G XGMAC Ethernet driver"
depends on HAS_IOMEM && HAS_DMA
+ depends on ARCH_HIGHBANK || COMPILE_TEST
select CRC32
help
This is the driver for the XGMAC Ethernet IP block found on Calxeda
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] net: calxedaxgmac: fix driver dependencies
2014-09-01 15:39 [PATCH] net: calxedaxgmac: fix driver dependencies Bartlomiej Zolnierkiewicz
@ 2014-09-02 2:36 ` Rob Herring
2014-09-02 9:51 ` Bartlomiej Zolnierkiewicz
2014-09-02 20:23 ` David Miller
1 sibling, 1 reply; 4+ messages in thread
From: Rob Herring @ 2014-09-02 2:36 UTC (permalink / raw)
To: Bartlomiej Zolnierkiewicz
Cc: David S. Miller, Kyungmin Park, netdev,
linux-kernel@vger.kernel.org
On Mon, Sep 1, 2014 at 10:39 AM, Bartlomiej Zolnierkiewicz
<b.zolnierkie@samsung.com> wrote:
> Calxeda 1G/10G XGMAC Ethernet support should be available only on
> Calxeda ECX-1000/2000 (Highbank/Midway) platforms.
>
> Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> Acked-by: Kyungmin Park <kyungmin.park@samsung.com>
> Cc: Rob Herring <robh@kernel.org>
> ---
> drivers/net/ethernet/calxeda/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> Index: b/drivers/net/ethernet/calxeda/Kconfig
> ===================================================================
> --- a/drivers/net/ethernet/calxeda/Kconfig 2014-07-30 14:31:12.159522474 +0200
> +++ b/drivers/net/ethernet/calxeda/Kconfig 2014-09-01 17:33:49.232810483 +0200
> @@ -1,6 +1,7 @@
> config NET_CALXEDA_XGMAC
> tristate "Calxeda 1G/10G XGMAC Ethernet driver"
> depends on HAS_IOMEM && HAS_DMA
> + depends on ARCH_HIGHBANK || COMPILE_TEST
Opinions differ on whether drivers should be restricted in this way.
There is no actual dependency requiring ARCH_HIGHBANK. So I leave it
to David.
Rob
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] net: calxedaxgmac: fix driver dependencies
2014-09-02 2:36 ` Rob Herring
@ 2014-09-02 9:51 ` Bartlomiej Zolnierkiewicz
0 siblings, 0 replies; 4+ messages in thread
From: Bartlomiej Zolnierkiewicz @ 2014-09-02 9:51 UTC (permalink / raw)
To: Rob Herring
Cc: David S. Miller, Kyungmin Park, netdev,
linux-kernel@vger.kernel.org
Hi,
On Monday, September 01, 2014 09:36:39 PM Rob Herring wrote:
> On Mon, Sep 1, 2014 at 10:39 AM, Bartlomiej Zolnierkiewicz
> <b.zolnierkie@samsung.com> wrote:
> > Calxeda 1G/10G XGMAC Ethernet support should be available only on
> > Calxeda ECX-1000/2000 (Highbank/Midway) platforms.
> >
> > Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> > Acked-by: Kyungmin Park <kyungmin.park@samsung.com>
> > Cc: Rob Herring <robh@kernel.org>
> > ---
> > drivers/net/ethernet/calxeda/Kconfig | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > Index: b/drivers/net/ethernet/calxeda/Kconfig
> > ===================================================================
> > --- a/drivers/net/ethernet/calxeda/Kconfig 2014-07-30 14:31:12.159522474 +0200
> > +++ b/drivers/net/ethernet/calxeda/Kconfig 2014-09-01 17:33:49.232810483 +0200
> > @@ -1,6 +1,7 @@
> > config NET_CALXEDA_XGMAC
> > tristate "Calxeda 1G/10G XGMAC Ethernet driver"
> > depends on HAS_IOMEM && HAS_DMA
> > + depends on ARCH_HIGHBANK || COMPILE_TEST
>
> Opinions differ on whether drivers should be restricted in this way.
> There is no actual dependency requiring ARCH_HIGHBANK. So I leave it
> to David.
The hardware is specific to ARCH_HIGHBANK and it is just unavailble on
any other platform (i.e. I have completely no use for this option on
ARM Exynos platform). In such cases we restrict drivers to the specific
hardware platform so they don't pollute configs for other platforms
(i.e. this option is enabled by default in multi_v7_defconfig and once
I go from multi defconfing to single platform one, the option stays
enabled).
Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] net: calxedaxgmac: fix driver dependencies
2014-09-01 15:39 [PATCH] net: calxedaxgmac: fix driver dependencies Bartlomiej Zolnierkiewicz
2014-09-02 2:36 ` Rob Herring
@ 2014-09-02 20:23 ` David Miller
1 sibling, 0 replies; 4+ messages in thread
From: David Miller @ 2014-09-02 20:23 UTC (permalink / raw)
To: b.zolnierkie; +Cc: robh, kyungmin.park, netdev, linux-kernel
From: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Date: Mon, 01 Sep 2014 17:39:04 +0200
> Calxeda 1G/10G XGMAC Ethernet support should be available only on
> Calxeda ECX-1000/2000 (Highbank/Midway) platforms.
>
> Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> Acked-by: Kyungmin Park <kyungmin.park@samsung.com>
Applied.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2014-09-02 20:23 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-09-01 15:39 [PATCH] net: calxedaxgmac: fix driver dependencies Bartlomiej Zolnierkiewicz
2014-09-02 2:36 ` Rob Herring
2014-09-02 9:51 ` Bartlomiej Zolnierkiewicz
2014-09-02 20:23 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).