Hi Steven,

On 24/05/12 07:35, Steven King wrote:
Add support for the Freescale Coldfire M5441x; as these parts have an enet-mac,
add a quirk to distinguish them from the other Coldfire parts so we can use
the existing enet-mac support.

Signed-off-by: Steven King<sfk...@fdwdc.com>
---
  drivers/net/ethernet/freescale/Kconfig |    5 ++---
  drivers/net/ethernet/freescale/fec.c   |    6 +++++-
  drivers/net/ethernet/freescale/fec.h   |    3 ++-
  3 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/drivers/net/ethernet/freescale/Kconfig 
b/drivers/net/ethernet/freescale/Kconfig
index 3574e14..f9aa244 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 || \
-                  ARCH_MXC || ARCH_MXS || (PPC_MPC52xx&&  PPC_BESTCOMM)
+                  M5441x || ARCH_MXC || ARCH_MXS || (PPC_MPC52xx&&  
PPC_BESTCOMM)
        ---help---
          If you have a network (Ethernet) card belonging to this class, say Y
          and read the Ethernet-HOWTO, available from
@@ -22,8 +22,7 @@ if NET_VENDOR_FREESCALE

  config FEC
        tristate "FEC ethernet controller (of ColdFire and some i.MX CPUs)"
-       depends on (M523x || M527x || M5272 || M528x || M520x || M532x || \
-                  ARCH_MXC || SOC_IMX28)
+       depends on (M523x || M527x || M5272 || M528x || M520x || M532x || 
M5441x || ARCH_MXC || SOC_IMX28)
        default ARCH_MXC || SOC_IMX28 if ARM
        select PHYLIB
        ---help---
diff --git a/drivers/net/ethernet/freescale/fec.c 
b/drivers/net/ethernet/freescale/fec.c
index a12b3f5..4cb1c90 100644
--- a/drivers/net/ethernet/freescale/fec.c
+++ b/drivers/net/ethernet/freescale/fec.c
@@ -93,6 +93,9 @@ static struct platform_device_id fec_devtype[] = {
                .name = "imx6q-fec",
                .driver_data = FEC_QUIRK_ENET_MAC | FEC_QUIRK_HAS_GBIT,
        }, {
+               .name = "enet-fec",
+               .driver_data = FEC_QUIRK_ENET_MAC,
+       }, {
                /* sentinel */
        }
  };
@@ -186,7 +189,8 @@ MODULE_PARM_DESC(macaddr, "FEC Ethernet MAC address");
   * account when setting it.
   */
  #if defined(CONFIG_M523x) || defined(CONFIG_M527x) || defined(CONFIG_M528x) 
|| \
-    defined(CONFIG_M520x) || defined(CONFIG_M532x) || defined(CONFIG_ARM)
+    defined(CONFIG_M520x) || defined(CONFIG_M532x) || defined(CONFIG_ARM) || \
+    defined(CONFIG_M5441x)

I would probably group all the ColdFire defines together. But that is
a pretty small detail really.

Otherwise I am fine with this:

Acked-by: Greg Ungerer <g...@uclinux.org>

Regards
Greg



  #define       OPT_FRAME_SIZE  (PKT_MAXBUF_SIZE<<  16)
  #else
  #define       OPT_FRAME_SIZE  0
diff --git a/drivers/net/ethernet/freescale/fec.h 
b/drivers/net/ethernet/freescale/fec.h
index 8408c62..298cfb7 100644
--- a/drivers/net/ethernet/freescale/fec.h
+++ b/drivers/net/ethernet/freescale/fec.h
@@ -15,7 +15,8 @@

  #if defined(CONFIG_M523x) || defined(CONFIG_M527x) || defined(CONFIG_M528x) 
|| \
      defined(CONFIG_M520x) || defined(CONFIG_M532x) || \
-    defined(CONFIG_ARCH_MXC) || defined(CONFIG_SOC_IMX28)
+    defined(CONFIG_ARCH_MXC) || defined(CONFIG_SOC_IMX28) || \
+    defined(CONFIG_M5441x)
  /*
   *    Just figures, Motorola would have to change the offsets for
   *    registers in the same peripheral device on different models





--
------------------------------------------------------------------------
Greg Ungerer  --  Principal Engineer        EMAIL:     g...@snapgear.com
SnapGear Group, McAfee                      PHONE:       +61 7 3435 2888
8 Gardner Close                             FAX:         +61 7 3217 5323
Milton, QLD, 4064, Australia                WEB: http://www.SnapGear.com
_______________________________________________
uClinux-dev mailing list
uClinux-dev@uclinux.org
http://mailman.uclinux.org/mailman/listinfo/uclinux-dev
This message was resent by uclinux-dev@uclinux.org
To unsubscribe see:
http://mailman.uclinux.org/mailman/options/uclinux-dev

Reply via email to