[U-Boot] [PATCH v2 2/3] Armada100: Enable Ethernet support for GplugD

[Marek Vasut <marek.vasut@gmail.com>]

On Thursday, August 25, 2011 02:09:46 PM Ajay Bhargav wrote:
> ----- "Marek Vasut" <marek.vasut@gmail.com> wrote:
> > On Thursday, August 25, 2011 07:15:17 AM Ajay Bhargav wrote:
> > > ----- "Marek Vasut" <marek.vasut@gmail.com> wrote:
> > > [...]
> > > 
> > > > > +/* Disable DCACHE */
> > > > > +#define CONFIG_SYS_DCACHE_OFF
> > > > 
> > > > ARMADA100 is still ARMv5? Then maybe you can try Hong Xu's patches
> > 
> > for
> > 
> > > > dcache on
> > > > armv5. You'll have to modify the ethernet driver to be aware of
> > 
> > the
> > 
> > > > dcache then
> > > > though.
> > > > 
> > > > As dcache support is on the way, it might be worth it.
> > > 
> > > Yes ARMADA100 is ARMv5. I had problems with dcache enabled, driver
> > 
> > get
> > 
> > > stuck. Can you please link me to Hong Xu's patches? Is there a
> > 
> > problem if
> > 
> > > i keep it disabled?
> > 
> > http://lists.denx.de/pipermail/u-boot/2011-August/098405.html Look
> > around this.
> > You'd better search for V2 of the patches or V3 in U-Boot ML archive.
> > 
> > There's no problem if you keep it disabled, but eventually, you'll
> > have to
> > enable it ;-)
> > 
> > Cheers
> 
> Hi Marek,
> 
> I searched mail list after I emailed you.. There has been a lot of
> discussion going on Hong Xu's patches. I think it would be better if I
> work on enabling dcache once the patch is mainlined.

You can always give it a whirl to test the patch. You can also find pieces of 
your driver that will need redesign because of the cache-line alignment and 
avoid such trouble in advance. But this is just an offer, it's not a firm 
requirement for the patch to be mainlined!

Cheers
> 
> Thanks,
> Ajay Bhargav