[PATCH 3/4] fec: add support for Freescale i.MX25 PDK (3DS)
Baruch Siach
baruch at tkos.co.il
Wed Dec 16 01:34:06 EST 2009
Hi Greg,
On Wed, Dec 16, 2009 at 10:13:56AM +1000, Greg Ungerer wrote:
> Baruch Siach wrote:
> >On Tue, Dec 15, 2009 at 09:52:24PM +1000, Greg Ungerer wrote:
> >>On 12/15/2009 06:31 PM, Baruch Siach wrote:
> >>>+#ifndef CONFIG_M5272
> >>I would suggest making this conditional on FEC_MIIGSK_ENR.
> >>Although the CONFIG_M5272 is the only case here currently,
> >>that may change over the years. And using this here may not
> >>be obvious to the causual code reader, since the register
> >>offset definitions don't explicitly key on CONFIG_M5272.
> >
> >OK, I'll change this conditional.
> >
> >Can I take this as an Ack from you?
>
> With that conditional check changed, sure:
>
> Acked-by: Greg Ungerer <gerg at uclinux.org>
Thanks. The updated patch below.
baruch
More information about the linux-arm-kernel
mailing list