[U-Boot] [PATCH 12/12] imx: mx5: Remove legacy iomux support

Marek Vasut marex at denx.de
Fri May 3 15:50:24 CEST 2013


Dear Benoît Thébaudeau,

> Dear Marek Vasut,
> 
> On Friday, May 3, 2013 3:39:39 PM, Marek Vasut wrote:
> > Dear Benoît Thébaudeau,
> > 
> > > Dear Marek Vasut,
> > > 
> > > On Friday, May 3, 2013 3:18:33 PM, Marek Vasut wrote:
> > > > Dear Benoît Thébaudeau,
> > > > 
> > > > > Dear Marek Vasut,
> > > > > 
> > > > > On Friday, May 3, 2013 2:42:04 PM, Marek Vasut wrote:
> > > > > > Dear Benoît Thébaudeau,
> > > > > > 
> > > > > > > Dear Marek Vasut,
> > > > > > > 
> > > > > > > On Friday, May 3, 2013 4:38:43 AM, Marek Vasut wrote:
> > > > > > > > Dear Benoît Thébaudeau,
> > > > > > > > 
> > > > > > > > > Legacy iomux support is no longer needed now that all
> > > > > > > > > boards have been converted to iomux-v3.
> > > > > > > > > 
> > > > > > > > > Signed-off-by: Benoît Thébaudeau
> > > > > > > > > <benoit.thebaudeau at advansee.com>
> > > > > > > > 
> > > > > > > > Yes, good move
> > > > > > > > 
> > > > > > > > Reviewed-by: Marek Vasut <marex at denx.de>
> > > > > > > 
> > > > > > > How do you plan to handle the conversion to iomux-v3 of the
> > > > > > > not-yet-applied
> > > > > > > 
> > > > > > > m53evk:
> > > > > > >  - You rebase your series on mine?
> > > > > > >  - I rebase my series on yours?
> > > > > > 
> > > > > > Did you not send this stuff after m53evk?
> > > > > 
> > > > > Correct.
> > > > > 
> > > > > > So let's let Stefano merge it all
> > > > > > in
> > > > > > sequence
> > > > > 
> > > > > We can do that, but this will add another conversion patch that
> > > > > would not be needed the other way around, for something not yet
> > > > > applied.
> > > > > 
> > > > > > (poor Stefano though, patches seem to be piling up).
> > > > > 
> > > > > True.
> > > > > 
> > > > > > >  - I convert m53evk, then I send you the file so that you can
> > > > > > >  refactor your
> > > > > > >  
> > > > > > >    series and trivially rebase it on mine?
> > > > > > 
> > > > > > I can convert it, but being lazy as hell, of course the idea of
> > > > > > you doing all the work seems very tempting ;-)
> > > > > 
> > > > > After having done all other boards, one more should be quick. ;)
> > > > > 
> > > > > So I will:
> > > > >  - Extract m53evk.c from your series.
> > > > >  - Convert it to iomux-v3.
> > > > >  - Send it to you.
> > > > > 
> > > > > Then you will:
> > > > >  - Update and resend your series with this file.
> > > > > 
> > > > > Then Stefano will:
> > > > >  - Apply my mx25/35/5x series.
> > > > >  - Apply your converted m53evk series.
> > > > 
> > > > Hm, I will still argument with the fact that yours was sent later and
> > > > for me, this will mean another round of testing. If we swap them, I
> > > > will have a good bisect point right before your series too.
> > > 
> > > OK, then I will just send a standalone patch converting m53evk to
> > > iomux-v3, and I will tell Stefano where to insert it in my mx5x iomux
> > > series, or he can also choose to merge it to your series as a fixup
> > > patch, depending on the order he prefers to apply things.
> > 
> > You can also just send a V2 of this single patch, no ?
> 
> But there is no V1!?
> 
> Or I could just send a V3 concatenating all my iomux series (there are
> dependencies among those series) and rebasing them on top of yours in order
> to make things easier and clearer for Stefano.

Bah, I didn't know that. Ok, if you're up to fixing up M53EVK, send a patch that 
does it and let Stefano (or me?) amend it to the M53EVK patch.

Best regards,
Marek Vasut


More information about the U-Boot mailing list