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

[Maxime Ripard]

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.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161114/6a3164b2/attachment.sig>