Hi Sekhar :) On Wed, Nov 13, 2013 at 11:08 AM, Sekhar Nori <nsek...@ti.com> wrote: > Hi Vaibhav, > > On 11/13/2013 7:38 PM, Vaibhav Bedia wrote: >> On Wed, Nov 13, 2013 at 3:48 AM, Lokesh Vutla <lokeshvu...@ti.com> wrote: >> [...] >>> I checked with hardware folks. There is no register or some way to tell >>> if VTT is present. It is not added in EEPROM also and I have no answer why >>> it >>> is not added in EEPROM..:( >>> It is specific to boards using DDR3. So its good to have it in board files >>> as I did it here >>> instead of adding this check in emif file. >> >> That EEPROM is clearly not getting used the way i think it should be :\ >> I would have made a lot of noise to get details like this added there. > > The EEPROM was designed as a way to differentiate between different TI > EVMs, not as a generic way to differentiate between various possible > board hook-ups. Even if we did define it that way, why would all boards > using AM437x have an onboard EEPROM? > > We could request this information be placed in EEPROM and see if > hardware folks oblige, but I don't see how that's going to be used > beyond TI EVMs. >
I understand the intent of customers to get rid of all the components they can to lower the cost. But if one just thinks about this a bit more, the current solution does a half-hearted attempt to differentiate the boards variants. It doesn't really capture the differences that are there and that is leading to hard coding to a certain extent. >From AM335x boards we should now have a decent idea of what things change across boards that go into production. I don't think it makes sense to throw away all that knowledge and go ahead assuming we will never make a change. The request for change is just to future proof the current code and have the EEPROM actually help us do our jobs. Why? Because life's too short to keep worrying about why a board rev that a you pick up from a neighbor's desk doesn't boot, hooking up the JTAG to trace the DDR setup code, figure out what needs to change in the boot-loader, add in the appropriate check and then get to the task at hand ;) Regards, Vaibhav _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
