> -----Original Message----- > From: Tom Rini [mailto:[email protected]] > Sent: Thursday, November 10, 2011 8:11 PM > To: Kumar Nath, Chandan > Cc: [email protected]; Chemparathy, Cyril > Subject: Re: [U-Boot] [PATCH 1/2] TI: netdev: add driver for cpsw > ethernet device > > On Wed, Nov 9, 2011 at 10:47 PM, Kumar Nath, Chandan > <[email protected]> wrote: > >> -----Original Message----- > >> From: Tom Rini [mailto:[email protected]] > >> Sent: Friday, October 21, 2011 9:09 PM > >> To: Kumar Nath, Chandan > >> Cc: [email protected]; Chemparathy, Cyril > >> Subject: Re: [U-Boot] [PATCH 1/2] TI: netdev: add driver for cpsw > >> ethernet device > >> > >> On Fri, Oct 21, 2011 at 12:02 AM, Chandan Nath <[email protected]> > >> wrote: > >> > From: Cyril Chemparathy <[email protected]> > >> > > >> > CPSW is an on-chip ethernet switch that is found on various SoCs > from > >> Texas > >> > Instruments. This patch adds a simple driver (based on the Linux > >> driver) for > >> > this hardware module. > >> > >> Have you made the driver safe with dcache enabled? > > > > No, driver is not made safe with dcache enabled. Can you please tell > me > > why I need to enable dcache. > > Because we really want to enable the dcache for performance reasons. > Given that to enable dcache on am335x we just need: > #ifndef CONFIG_SYS_DCACHE_OFF > void enable_caches(void) > { > /* Enable D-cache. I-cache is already enabled in start.S */ > dcache_enable(); > } > #endif > > We really want to not add drivers that aren't cache safe, especially > when it's not a lot of work to do so. >
I tried enabling dcache by adding above code. But when I executed dhcp command, it was not able to find any IP address and keeps trying infinitely. > -- > Tom _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
