From mboxrd@z Thu Jan 1 00:00:00 1970 From: plagnioj@jcrosoft.com (Jean-Christophe PLAGNIOL-VILLARD) Date: Tue, 5 Apr 2011 13:57:51 +0200 Subject: [PATCHv2 0/9] macb: add support for Cadence GEM In-Reply-To: <20110405114754.GG4797@pulham.picochip.com> References: <20110321111838.GC2610@pulham.picochip.com> <20110322161811.GF6466@game.jcrosoft.org> <20110322163917.GD10058@pulham.picochip.com> <20110322175523.GE10058@pulham.picochip.com> <20110324162517.GI6466@game.jcrosoft.org> <20110331094041.GD3371@pulham.picochip.com> <20110405102842.GC22192@game.jcrosoft.org> <20110405104910.GE4797@pulham.picochip.com> <20110405112102.GB19268@game.jcrosoft.org> <20110405114754.GG4797@pulham.picochip.com> Message-ID: <20110405115751.GA24997@game.jcrosoft.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 12:47 Tue 05 Apr , Jamie Iles wrote: > On Tue, Apr 05, 2011 at 01:21:02PM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote: > > On 11:49 Tue 05 Apr , Jamie Iles wrote: > > > On Tue, Apr 05, 2011 at 12:28:42PM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote: > > > > work fine on 9263ek except the IP version detection. > > > > > > > > the at91 macb ip version is supposed to be at 0x0601010C but it's not. > > > > At least on 9263 it's 0x0001010C. So we can not detect the arch at runtime > > > > but we can detect that it's a macb. > > > > > > > > So could keep the ifdef for 2 archs but use the ip version on arm > > > > > > OK, well I think my patches are already doing that so should be OK as > > > they are. > > > > > > Russell, are you able to take these through your tree (I think they > > > count as consolidation work) or should I ask Stephen for a tree in > > > linux-next for a while first? > > no please do not us the is_gem but the same way as I did in the ip detection > > keep the version register and then check it. > > > > as this ip can be used on other arch we do not want to see thousands > > of is_xxx > > But GEM isn't an architecture/machine type, it's a new Cadence Ethernet > controller that follows on from MACB, not some arch specific tweaks so > we really only have two options - MACB or GEM. I agree but you can have this ip in other soc so we need to try to make it as generic as possible and socs can have specific tweaks So use the version register and mask it is better in my mind and more flexible It's a shame that we can not detect the diff between avr32 and at91 via ip version. I hope that ip variation could be detected via version register. Next time. > > Still, if it's important to you then I'll make the change. If you don't ming please Best Regards, J. From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean-Christophe PLAGNIOL-VILLARD Subject: Re: [PATCHv2 0/9] macb: add support for Cadence GEM Date: Tue, 5 Apr 2011 13:57:51 +0200 Message-ID: <20110405115751.GA24997@game.jcrosoft.org> References: <20110321111838.GC2610@pulham.picochip.com> <20110322161811.GF6466@game.jcrosoft.org> <20110322163917.GD10058@pulham.picochip.com> <20110322175523.GE10058@pulham.picochip.com> <20110324162517.GI6466@game.jcrosoft.org> <20110331094041.GD3371@pulham.picochip.com> <20110405102842.GC22192@game.jcrosoft.org> <20110405104910.GE4797@pulham.picochip.com> <20110405112102.GB19268@game.jcrosoft.org> <20110405114754.GG4797@pulham.picochip.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Russell King - ARM Linux , netdev@vger.kernel.org, Nicolas Ferre , David Miller , Peter Korsgaard , Andrew Victor , linux-arm-kernel@lists.infradead.org To: Jamie Iles Return-path: Received: from 63.mail-out.ovh.net ([91.121.185.56]:42246 "HELO 63.mail-out.ovh.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1752886Ab1DEMFc (ORCPT ); Tue, 5 Apr 2011 08:05:32 -0400 Content-Disposition: inline In-Reply-To: <20110405114754.GG4797@pulham.picochip.com> Sender: netdev-owner@vger.kernel.org List-ID: On 12:47 Tue 05 Apr , Jamie Iles wrote: > On Tue, Apr 05, 2011 at 01:21:02PM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote: > > On 11:49 Tue 05 Apr , Jamie Iles wrote: > > > On Tue, Apr 05, 2011 at 12:28:42PM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote: > > > > work fine on 9263ek except the IP version detection. > > > > > > > > the at91 macb ip version is supposed to be at 0x0601010C but it's not. > > > > At least on 9263 it's 0x0001010C. So we can not detect the arch at runtime > > > > but we can detect that it's a macb. > > > > > > > > So could keep the ifdef for 2 archs but use the ip version on arm > > > > > > OK, well I think my patches are already doing that so should be OK as > > > they are. > > > > > > Russell, are you able to take these through your tree (I think they > > > count as consolidation work) or should I ask Stephen for a tree in > > > linux-next for a while first? > > no please do not us the is_gem but the same way as I did in the ip detection > > keep the version register and then check it. > > > > as this ip can be used on other arch we do not want to see thousands > > of is_xxx > > But GEM isn't an architecture/machine type, it's a new Cadence Ethernet > controller that follows on from MACB, not some arch specific tweaks so > we really only have two options - MACB or GEM. I agree but you can have this ip in other soc so we need to try to make it as generic as possible and socs can have specific tweaks So use the version register and mask it is better in my mind and more flexible It's a shame that we can not detect the diff between avr32 and at91 via ip version. I hope that ip variation could be detected via version register. Next time. > > Still, if it's important to you then I'll make the change. If you don't ming please Best Regards, J.