> -----Original Message----- > From: Wolfgang Denk [mailto:[email protected]] > Sent: Tuesday, November 24, 2009 1:21 AM > To: Hiremath, Vaibhav > Cc: [email protected] > Subject: Re: [U-Boot] [PATCH 5/5] AM3517: Add support for EMIF4 > > Dear [email protected], > > In message <[email protected]> you > wrote: > > From: Vaibhav Hiremath <[email protected]> > > > > Changes:- > > > > - Created 2 seperate config option > > * CONFIG_SDRC (omap34xx family) > > * CONFIG_EMIF4 (am3517 family) > > - Changed all omap3 dependent file for CONFIG_SDRC > > > > Tested On:- > > > > - AM3517EVM board > > - OMAP3EVM board > > - BEAGLE board > > > > I would need help here from people to validate this change for > SDP/ZOOMx > > and other OMAP3 version of boards. > > This commit adds a lot of #ifdef's - maybe it would make more sense > to split code into different files? > [Hiremath, Vaibhav] Initially I thought of this, but since currently AM3517 is the only device in OMAP3 series which uses EMIF and all others use SDRC. So I thought lets start with this approach and get peoples opinion on this, and then we can split the files.
Thanks, Vaibhav > Best regards, > > Wolfgang Denk > > -- > DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev > Zundel > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, > Germany > Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: > [email protected] > You are an excellent tactician, Captain. You let your second in > com- > mand attack while you sit and watch for weakness. > -- Khan Noonian Singh, "Space Seed", stardate 3141.9 _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
