[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20110311133016.GF7357@pulham.picochip.com>
Date: Fri, 11 Mar 2011 13:30:16 +0000
From: Jamie Iles <jamie@...ieiles.com>
To: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@...osoft.com>
Cc: Jamie Iles <jamie@...ieiles.com>, netdev@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, nicolas.ferre@...el.com
Subject: Re: [PATCH 4/8] macb: initial support for Cadence GEM
On Fri, Mar 11, 2011 at 02:14:15PM +0100, Jean-Christophe PLAGNIOL-VILLARD wrote:
> On 10:10 Thu 10 Mar , Jamie Iles wrote:
> > The Cadence GEM is based on the MACB Ethernet controller but has a few
> > small changes with regards to register and bitfield placement. This
> > patch adds a new platform driver for gem which sets allows the driver to
> > tell at runtime whether it is targetting a GEM device.
> >
> > Signed-off-by: Jamie Iles <jamie@...ieiles.com>
> could we avoid all this if else everywhere?
I can't really see any other way to do this, but you're right it isn't
particularly nice. Having said that, it is only in the initialization
code so there shouldn't be any real performance impact.
I'm open to ideas though!
Jamie
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists