From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marek.vasut@gmail.com>
Date: Wed, 14 Sep 2011 14:11:21 +0200
Subject: [U-Boot] [PATCH 03/15] FEC: Add support for iMX28 quirks
In-Reply-To: <4E705511.6080409@denx.de>
References: <1315800409-19876-1-git-send-email-marek.vasut@gmail.com>
	<1315800409-19876-4-git-send-email-marek.vasut@gmail.com>
	<4E705511.6080409@denx.de>
Message-ID: <201109141411.21236.marek.vasut@gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Wednesday, September 14, 2011 09:17:37 AM Stefano Babic wrote:
> On 09/12/2011 06:06 AM, Marek Vasut wrote:
> > Signed-off-by: Marek Vasut <marek.vasut@gmail.com>
> > Cc: Ben Warren <biggerbadderben@gmail.com>
> > Cc: Stefano Babic <sbabic@denx.de>
> > Cc: Wolfgang Denk <wd@denx.de>
> > Cc: Detlev Zundel <dzu@denx.de>
> > ---
> 
> Hi Marek,
> 
> only a small question:
> >  drivers/net/fec_mxc.c |   44
> >  ++++++++++++++++++++++++++++++++++++++++++-- 1 files changed, 42
> >  insertions(+), 2 deletions(-)
> > 
> > diff --git a/drivers/net/fec_mxc.c b/drivers/net/fec_mxc.c
> > index cfe2176..f50433dd 100644
> > --- a/drivers/net/fec_mxc.c
> > +++ b/drivers/net/fec_mxc.c
> > @@ -42,6 +42,14 @@ DECLARE_GLOBAL_DATA_PTR;
> > 
> >  #define	CONFIG_FEC_XCV_TYPE	MII100
> >  #endif
> > 
> > +/*
> > + * The i.MX28 operates with packets in big endian. We need to swap them
> > before + * sending and after receiving.
> > + */
> > +#ifdef	CONFIG_MX28
> > +#define	CONFIG_FEC_MXC_SWAP_PACKET
> > +#endif
> > +
> 
> Do we really need two defines ? If the MX28 has the oddity to swap
> packets and this must be always done for this SOC, we could protect the
> code only with CONFIG_MX28. I think in the code it is already clear that
> in case of MX28 the packets are swapped.

Well if any other CPU with this problem pops up on the market, we should be 
ready.

> 
> Best regards,
> Stefano Babic