[U-Boot-Users] [Patch 0/9] U-boot-V2: Introduce I2C support for SDP3430

[Menon, Nishanth]

Robert,
> -----Original Message-----
> From: Robert Schwebel [mailto:r.schwebel at pengutronix.de]
> Sent: Wednesday, June 18, 2008 2:12 PM
> To: Menon, Nishanth
> Cc: Robert Schwebel; Sascha Hauer; Timur Tabi; Laurent Desnogues; Kamat, Nishant;
> philip.balister at gmail.com; u-boot-users at lists.sourceforge.net
> Subject: Re: [U-Boot-Users] [Patch 0/9] U-boot-V2: Introduce I2C support for SDP3430
>
> I've commented your first patch in the series; in short:
>
> - It shows nicely that it is easy to copy things over from Linux -
>   that's a good sign that we didn't do too many things the wrong way. But
>   note, it is *not* the main intention of u-boot-v2 to be a Linux clone.
>   It is a bootloader, and it tries to be one with a neat design,
>   combining the best of POSIX, Linux and U-Boot-v1. Copying *ideas* is
>   good, but let's do it in a way that it still is u-boot.
>
> - Please don't bring in things which are not part of the current code
>   base. If you add code for subsystems, the subsystems must be there
>   first and *then* their users. Otherwhise we bloat the codebase and we
>   disable the possibility to add *real* support for PCI, USB and S390
>   later :-)
Acked. Yep will redo that :).

> Which part of i2c do we actually *need* in the bootloader? What's *your*
> use case?
I need i2c_transfer -yes - I can customize the changes just for that. I do like i2c_transfers, adapters, clients etc.. my immediate need is rather simple -> i2c_transfers.

>
> I assume we need i2c host controllers, that's quite clear. We also need
> an abstraction for i2c devices. The only devices I have seen which
> really needed a bootloader driver are
>
> a) eeproms
> b) RTCs
> c) analog/digital in/outputs (i.e. gpio expanders)
>
> Other use cases?
USB Transceivers, power management devices.

Do we need SMBUS support? I don't. probably algos would be a good idea for other SOCs.

Are we ok with i2ctools movement here?

Regards,
Nishanth Menon