* [PATCH] ARM: mxs: Select FEC driver @ 2011-11-29 18:50 Fabio Estevam 2011-11-29 19:23 ` Marek Vasut 2011-11-29 19:33 ` Uwe Kleine-König 0 siblings, 2 replies; 6+ messages in thread From: Fabio Estevam @ 2011-11-29 18:50 UTC (permalink / raw) To: linux-arm-kernel Let the FEC driver be built by default. Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com> --- arch/arm/configs/mxs_defconfig | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/arch/arm/configs/mxs_defconfig b/arch/arm/configs/mxs_defconfig index 6ee781b..f99bcfc 100644 --- a/arch/arm/configs/mxs_defconfig +++ b/arch/arm/configs/mxs_defconfig @@ -59,6 +59,8 @@ CONFIG_DEVTMPFS=y # CONFIG_BLK_DEV is not set CONFIG_NETDEVICES=y CONFIG_NET_ETHERNET=y +CONFIG_NET_VENDOR_FREESCALE=y +CONFIG_FEC=y CONFIG_ENC28J60=y # CONFIG_NETDEV_1000 is not set # CONFIG_NETDEV_10000 is not set -- 1.7.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH] ARM: mxs: Select FEC driver 2011-11-29 18:50 [PATCH] ARM: mxs: Select FEC driver Fabio Estevam @ 2011-11-29 19:23 ` Marek Vasut 2011-11-29 19:33 ` Uwe Kleine-König 1 sibling, 0 replies; 6+ messages in thread From: Marek Vasut @ 2011-11-29 19:23 UTC (permalink / raw) To: linux-arm-kernel > Let the FEC driver be built by default. > > Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com> > --- > arch/arm/configs/mxs_defconfig | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/arch/arm/configs/mxs_defconfig > b/arch/arm/configs/mxs_defconfig index 6ee781b..f99bcfc 100644 > --- a/arch/arm/configs/mxs_defconfig > +++ b/arch/arm/configs/mxs_defconfig > @@ -59,6 +59,8 @@ CONFIG_DEVTMPFS=y > # CONFIG_BLK_DEV is not set > CONFIG_NETDEVICES=y > CONFIG_NET_ETHERNET=y > +CONFIG_NET_VENDOR_FREESCALE=y > +CONFIG_FEC=y > CONFIG_ENC28J60=y > # CONFIG_NETDEV_1000 is not set > # CONFIG_NETDEV_10000 is not set Acked-by: Marek Vasut <marek.vasut@gmail.com> ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] ARM: mxs: Select FEC driver 2011-11-29 18:50 [PATCH] ARM: mxs: Select FEC driver Fabio Estevam 2011-11-29 19:23 ` Marek Vasut @ 2011-11-29 19:33 ` Uwe Kleine-König 2011-11-30 8:22 ` Shawn Guo 1 sibling, 1 reply; 6+ messages in thread From: Uwe Kleine-König @ 2011-11-29 19:33 UTC (permalink / raw) To: linux-arm-kernel On Tue, Nov 29, 2011 at 04:50:07PM -0200, Fabio Estevam wrote: > Let the FEC driver be built by default. > > Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com> > --- > arch/arm/configs/mxs_defconfig | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/arch/arm/configs/mxs_defconfig b/arch/arm/configs/mxs_defconfig > index 6ee781b..f99bcfc 100644 > --- a/arch/arm/configs/mxs_defconfig > +++ b/arch/arm/configs/mxs_defconfig > @@ -59,6 +59,8 @@ CONFIG_DEVTMPFS=y > # CONFIG_BLK_DEV is not set > CONFIG_NETDEVICES=y > CONFIG_NET_ETHERNET=y > +CONFIG_NET_VENDOR_FREESCALE=y > +CONFIG_FEC=y I'd prefer to add default y or maybe default y if SOME_SENSIBLE_OPTION to NET_VENDOR_FREESCALE and FEC instead. We already had that, but Shawn removed it in 230dec6 (net/fec: add imx6q enet support) (by mistake if I remember an earlier thread here correctly). Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-K?nig | Industrial Linux Solutions | http://www.pengutronix.de/ | ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] ARM: mxs: Select FEC driver 2011-11-29 19:33 ` Uwe Kleine-König @ 2011-11-30 8:22 ` Shawn Guo 2011-11-30 9:06 ` Uwe Kleine-König 0 siblings, 1 reply; 6+ messages in thread From: Shawn Guo @ 2011-11-30 8:22 UTC (permalink / raw) To: linux-arm-kernel On Tue, Nov 29, 2011 at 08:33:04PM +0100, Uwe Kleine-K?nig wrote: > On Tue, Nov 29, 2011 at 04:50:07PM -0200, Fabio Estevam wrote: > > Let the FEC driver be built by default. > > > > Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com> > > --- > > arch/arm/configs/mxs_defconfig | 2 ++ > > 1 files changed, 2 insertions(+), 0 deletions(-) > > > > diff --git a/arch/arm/configs/mxs_defconfig b/arch/arm/configs/mxs_defconfig > > index 6ee781b..f99bcfc 100644 > > --- a/arch/arm/configs/mxs_defconfig > > +++ b/arch/arm/configs/mxs_defconfig > > @@ -59,6 +59,8 @@ CONFIG_DEVTMPFS=y > > # CONFIG_BLK_DEV is not set > > CONFIG_NETDEVICES=y > > CONFIG_NET_ETHERNET=y > > +CONFIG_NET_VENDOR_FREESCALE=y > > +CONFIG_FEC=y > I'd prefer to add > > default y > > or maybe > > default y if SOME_SENSIBLE_OPTION > > to NET_VENDOR_FREESCALE and FEC instead. We already had that, but Shawn > removed it in > > 230dec6 (net/fec: add imx6q enet support) > > (by mistake if I remember an earlier thread here correctly). > Here is what the commit changed. diff --git a/drivers/net/ethernet/freescale/Kconfig b/drivers/net/ethernet/freescale/Kconfig index 4dbe41f..1cf6716 100644 --- a/drivers/net/ethernet/freescale/Kconfig +++ b/drivers/net/ethernet/freescale/Kconfig @@ -7,7 +7,7 @@ config NET_VENDOR_FREESCALE default y depends on FSL_SOC || QUICC_ENGINE || CPM1 || CPM2 || PPC_MPC512x || \ M523x || M527x || M5272 || M528x || M520x || M532x || \ - IMX_HAVE_PLATFORM_FEC || MXS_HAVE_PLATFORM_FEC || \ + ARCH_MXC || ARCH_MXS || \ (PPC_MPC52xx && PPC_BESTCOMM) ---help--- If you have a network (Ethernet) card belonging to this class, say Y @@ -16,16 +16,15 @@ config NET_VENDOR_FREESCALE Note that the answer to this question doesn't directly affect the kernel: saying N will just cause the configurator to skip all - the questions about IBM devices. If you say Y, you will be asked for - your specific card in the following questions. + the questions about Freescale devices. If you say Y, you will be + asked for your specific card in the following questions. if NET_VENDOR_FREESCALE config FEC bool "FEC ethernet controller (of ColdFire and some i.MX CPUs)" depends on (M523x || M527x || M5272 || M528x || M520x || M532x || \ - IMX_HAVE_PLATFORM_FEC || MXS_HAVE_PLATFORM_FEC) - default IMX_HAVE_PLATFORM_FEC || MXS_HAVE_PLATFORM_FEC if ARM + ARCH_MXC || ARCH_MXS) select PHYLIB ---help--- Say Y here if you want to use the built-in 10/100 Fast ethernet Since for device tree support, we have neither IMX_HAVE_PLATFORM_FEC nor MXS_HAVE_PLATFORM_FEC set. It's all about change 'IMX_HAVE_PLATFORM_FEC || MXS_HAVE_PLATFORM_FEC' to 'ARCH_MXC || ARCH_MXS'. I do not see a mistake there. -- Regards, Shawn ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH] ARM: mxs: Select FEC driver 2011-11-30 8:22 ` Shawn Guo @ 2011-11-30 9:06 ` Uwe Kleine-König 2011-11-30 13:35 ` Shawn Guo 0 siblings, 1 reply; 6+ messages in thread From: Uwe Kleine-König @ 2011-11-30 9:06 UTC (permalink / raw) To: linux-arm-kernel Hello Shawn, On Wed, Nov 30, 2011 at 04:22:41PM +0800, Shawn Guo wrote: > On Tue, Nov 29, 2011 at 08:33:04PM +0100, Uwe Kleine-K?nig wrote: > > On Tue, Nov 29, 2011 at 04:50:07PM -0200, Fabio Estevam wrote: > > > Let the FEC driver be built by default. > > > > > > Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com> > > > --- > > > arch/arm/configs/mxs_defconfig | 2 ++ > > > 1 files changed, 2 insertions(+), 0 deletions(-) > > > > > > diff --git a/arch/arm/configs/mxs_defconfig b/arch/arm/configs/mxs_defconfig > > > index 6ee781b..f99bcfc 100644 > > > --- a/arch/arm/configs/mxs_defconfig > > > +++ b/arch/arm/configs/mxs_defconfig > > > @@ -59,6 +59,8 @@ CONFIG_DEVTMPFS=y > > > # CONFIG_BLK_DEV is not set > > > CONFIG_NETDEVICES=y > > > CONFIG_NET_ETHERNET=y > > > +CONFIG_NET_VENDOR_FREESCALE=y > > > +CONFIG_FEC=y > > I'd prefer to add > > > > default y > > > > or maybe > > > > default y if SOME_SENSIBLE_OPTION > > > > to NET_VENDOR_FREESCALE and FEC instead. We already had that, but Shawn > > removed it in > > > > 230dec6 (net/fec: add imx6q enet support) > > > > (by mistake if I remember an earlier thread here correctly). > > > Here is what the commit changed. > > diff --git a/drivers/net/ethernet/freescale/Kconfig b/drivers/net/ethernet/freescale/Kconfig > index 4dbe41f..1cf6716 100644 > --- a/drivers/net/ethernet/freescale/Kconfig > +++ b/drivers/net/ethernet/freescale/Kconfig > @@ -7,7 +7,7 @@ config NET_VENDOR_FREESCALE > default y > depends on FSL_SOC || QUICC_ENGINE || CPM1 || CPM2 || PPC_MPC512x || \ > M523x || M527x || M5272 || M528x || M520x || M532x || \ > - IMX_HAVE_PLATFORM_FEC || MXS_HAVE_PLATFORM_FEC || \ > + ARCH_MXC || ARCH_MXS || \ > (PPC_MPC52xx && PPC_BESTCOMM) > ---help--- > If you have a network (Ethernet) card belonging to this class, say Y > @@ -16,16 +16,15 @@ config NET_VENDOR_FREESCALE > > Note that the answer to this question doesn't directly affect the > kernel: saying N will just cause the configurator to skip all > - the questions about IBM devices. If you say Y, you will be asked for > - your specific card in the following questions. > + the questions about Freescale devices. If you say Y, you will be > + asked for your specific card in the following questions. > > if NET_VENDOR_FREESCALE > > config FEC > bool "FEC ethernet controller (of ColdFire and some i.MX CPUs)" > depends on (M523x || M527x || M5272 || M528x || M520x || M532x || \ > - IMX_HAVE_PLATFORM_FEC || MXS_HAVE_PLATFORM_FEC) > - default IMX_HAVE_PLATFORM_FEC || MXS_HAVE_PLATFORM_FEC if ARM > + ARCH_MXC || ARCH_MXS) > select PHYLIB > ---help--- > Say Y here if you want to use the built-in 10/100 Fast ethernet > > Since for device tree support, we have neither IMX_HAVE_PLATFORM_FEC > nor MXS_HAVE_PLATFORM_FEC set. It's all about change > 'IMX_HAVE_PLATFORM_FEC || MXS_HAVE_PLATFORM_FEC' to 'ARCH_MXC || > ARCH_MXS'. I do not see a mistake there. You did s/IMX_HAVE_PLATFORM_FEC/ARCH_MXC/ and s/MXS_HAVE_PLATFORM_FEC/ARCH_MXS/ in the dependencies of FEC (which is OK and intended). And you removed default IMX_HAVE_PLATFORM_FEC || MXS_HAVE_PLATFORM_FEC if ARM without a substitute. I suggest to add default ARCH_MXC || ARCH_MXS if ARM (with "if ARM" only being there to document where the ARCH_MX? symbols come from). NET_VENDOR_FREESCALE already has a default y, so adding that to the defconfigs shouldn't be necessary. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-K?nig | Industrial Linux Solutions | http://www.pengutronix.de/ | ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] ARM: mxs: Select FEC driver 2011-11-30 9:06 ` Uwe Kleine-König @ 2011-11-30 13:35 ` Shawn Guo 0 siblings, 0 replies; 6+ messages in thread From: Shawn Guo @ 2011-11-30 13:35 UTC (permalink / raw) To: linux-arm-kernel On Wed, Nov 30, 2011 at 10:06:41AM +0100, Uwe Kleine-K?nig wrote: > Hello Shawn, > > On Wed, Nov 30, 2011 at 04:22:41PM +0800, Shawn Guo wrote: > > On Tue, Nov 29, 2011 at 08:33:04PM +0100, Uwe Kleine-K?nig wrote: > > > On Tue, Nov 29, 2011 at 04:50:07PM -0200, Fabio Estevam wrote: > > > > Let the FEC driver be built by default. > > > > > > > > Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com> > > > > --- > > > > arch/arm/configs/mxs_defconfig | 2 ++ > > > > 1 files changed, 2 insertions(+), 0 deletions(-) > > > > > > > > diff --git a/arch/arm/configs/mxs_defconfig b/arch/arm/configs/mxs_defconfig > > > > index 6ee781b..f99bcfc 100644 > > > > --- a/arch/arm/configs/mxs_defconfig > > > > +++ b/arch/arm/configs/mxs_defconfig > > > > @@ -59,6 +59,8 @@ CONFIG_DEVTMPFS=y > > > > # CONFIG_BLK_DEV is not set > > > > CONFIG_NETDEVICES=y > > > > CONFIG_NET_ETHERNET=y > > > > +CONFIG_NET_VENDOR_FREESCALE=y > > > > +CONFIG_FEC=y > > > I'd prefer to add > > > > > > default y > > > > > > or maybe > > > > > > default y if SOME_SENSIBLE_OPTION > > > > > > to NET_VENDOR_FREESCALE and FEC instead. We already had that, but Shawn > > > removed it in > > > > > > 230dec6 (net/fec: add imx6q enet support) > > > > > > (by mistake if I remember an earlier thread here correctly). > > > > > Here is what the commit changed. > > > > diff --git a/drivers/net/ethernet/freescale/Kconfig b/drivers/net/ethernet/freescale/Kconfig > > index 4dbe41f..1cf6716 100644 > > --- a/drivers/net/ethernet/freescale/Kconfig > > +++ b/drivers/net/ethernet/freescale/Kconfig > > @@ -7,7 +7,7 @@ config NET_VENDOR_FREESCALE > > default y > > depends on FSL_SOC || QUICC_ENGINE || CPM1 || CPM2 || PPC_MPC512x || \ > > M523x || M527x || M5272 || M528x || M520x || M532x || \ > > - IMX_HAVE_PLATFORM_FEC || MXS_HAVE_PLATFORM_FEC || \ > > + ARCH_MXC || ARCH_MXS || \ > > (PPC_MPC52xx && PPC_BESTCOMM) > > ---help--- > > If you have a network (Ethernet) card belonging to this class, say Y > > @@ -16,16 +16,15 @@ config NET_VENDOR_FREESCALE > > > > Note that the answer to this question doesn't directly affect the > > kernel: saying N will just cause the configurator to skip all > > - the questions about IBM devices. If you say Y, you will be asked for > > - your specific card in the following questions. > > + the questions about Freescale devices. If you say Y, you will be > > + asked for your specific card in the following questions. > > > > if NET_VENDOR_FREESCALE > > > > config FEC > > bool "FEC ethernet controller (of ColdFire and some i.MX CPUs)" > > depends on (M523x || M527x || M5272 || M528x || M520x || M532x || \ > > - IMX_HAVE_PLATFORM_FEC || MXS_HAVE_PLATFORM_FEC) > > - default IMX_HAVE_PLATFORM_FEC || MXS_HAVE_PLATFORM_FEC if ARM > > + ARCH_MXC || ARCH_MXS) > > select PHYLIB > > ---help--- > > Say Y here if you want to use the built-in 10/100 Fast ethernet > > > > Since for device tree support, we have neither IMX_HAVE_PLATFORM_FEC > > nor MXS_HAVE_PLATFORM_FEC set. It's all about change > > 'IMX_HAVE_PLATFORM_FEC || MXS_HAVE_PLATFORM_FEC' to 'ARCH_MXC || > > ARCH_MXS'. I do not see a mistake there. > You did s/IMX_HAVE_PLATFORM_FEC/ARCH_MXC/ and > s/MXS_HAVE_PLATFORM_FEC/ARCH_MXS/ in the dependencies of FEC (which is > OK and intended). And you removed > > default IMX_HAVE_PLATFORM_FEC || MXS_HAVE_PLATFORM_FEC if ARM > > without a substitute. I suggest to add > > default ARCH_MXC || ARCH_MXS if ARM > > (with "if ARM" only being there to document where the ARCH_MX? symbols > come from). NET_VENDOR_FREESCALE already has a default y, so adding that > to the defconfigs shouldn't be necessary. > Yes, you are all right. That was a mistake :) -- Regards, Shawn ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2011-11-30 13:35 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2011-11-29 18:50 [PATCH] ARM: mxs: Select FEC driver Fabio Estevam 2011-11-29 19:23 ` Marek Vasut 2011-11-29 19:33 ` Uwe Kleine-König 2011-11-30 8:22 ` Shawn Guo 2011-11-30 9:06 ` Uwe Kleine-König 2011-11-30 13:35 ` Shawn Guo
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).