[U-Boot] [PATCH RESEND 6/9] eeprom: Add DS2431 support

[Simon Glass]

Hi Maxime,

On 14 November 2016 at 14:09, Maxime Ripard
<maxime.ripard at free-electrons.com> wrote:
> On Mon, Nov 14, 2016 at 01:44:42PM -0700, Simon Glass wrote:
>> Hi Maxime,
>>
>> On 14 November 2016 at 13:12, Maxime Ripard
>> <maxime.ripard at free-electrons.com> wrote:
>> > On Mon, Nov 14, 2016 at 10:14:57AM -0500, Tom Rini wrote:
>> >> On Mon, Nov 14, 2016 at 02:42:59PM +0100, Maxime Ripard wrote:
>> >> > Hi,
>> >> >
>> >> > On Fri, Nov 11, 2016 at 11:16:39AM -0800, Moritz Fischer wrote:
>> >> > > > +U_BOOT_DRIVER(ds2431) = {
>> >> > > > +       .name           = "ds2431",
>> >> > > > +       .id             = UCLASS_EEPROM,
>> >> > > > +       .ops            = &ds2431_ops,
>> >> > >
>> >> > > Do you want to add a .flags = DM_UC_FLAG_SEQ_ALIAS here?
>> >> >
>> >> > I don't know. I was kind of wondering why U-Boot relies on aliases so
>> >> > much, especially when the Linux DT maintainers are saying that aliases
>> >> > should be avoided entirely, and we'll won't be able to upstream those
>> >> > changes.
>> >>
>> >> Bah.  Do you have a pointer to some discussion about this handy?
>> >
>> > Rob said this multiple times, but here is an example:
>> > http://lkml.iu.edu/hypermail/linux/kernel/1609.1/00653.html
>>
>> Well that's not really an explanation!
>
> Don't shoot the messenger :)
>
> I think I found some other discussion where they discuss it at length:
> https://patchwork.kernel.org/patch/9133903/
>
> The interesting part is when Rob Herring is involved.

Oh dear. That sounds like a change to me. I'm going to leave this one to Tom :-)

Regards,
Simon