From: Marek Vasut <marex@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 12/12] imx: mx5: Remove legacy iomux support
Date: Fri, 3 May 2013 15:50:24 +0200 [thread overview]
Message-ID: <201305031550.24291.marex@denx.de> (raw)
In-Reply-To: <2030651606.524331.1367588321529.JavaMail.root@advansee.com>
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@advansee.com>
> > > > > > > >
> > > > > > > > Yes, good move
> > > > > > > >
> > > > > > > > Reviewed-by: Marek Vasut <marex@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
next prev parent reply other threads:[~2013-05-03 13:50 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-02 20:52 [U-Boot] [PATCH 01/12] imx: iomux-mx51: Fix MX51_PAD_EIM_CS2__GPIO2_27 Benoît Thébaudeau
2013-05-02 20:52 ` [U-Boot] [PATCH 02/12] imx: iomux-v3: Add missing definitions to iomux-mx51.h Benoît Thébaudeau
2013-05-02 20:52 ` [U-Boot] [PATCH 03/12] imx: mx51evk: Convert to iomux-v3 Benoît Thébaudeau
2013-05-02 20:52 ` [U-Boot] [PATCH 04/12] imx: mx51_efikamx/sb: " Benoît Thébaudeau
2013-05-03 15:58 ` Matt Sealey
2013-05-03 16:01 ` Benoît Thébaudeau
2013-05-16 18:05 ` Matt Sealey
2013-05-16 18:51 ` Benoît Thébaudeau
2013-05-02 20:52 ` [U-Boot] [PATCH 05/12] imx: vision2: " Benoît Thébaudeau
2013-05-02 20:52 ` [U-Boot] [PATCH 06/12] imx: iomux-v3: Add iomux-mx53.h Benoît Thébaudeau
2013-05-02 20:52 ` [U-Boot] [PATCH 07/12] imx: ima3-mx53: Convert to iomux-v3 Benoît Thébaudeau
2013-05-02 20:52 ` [U-Boot] [PATCH 08/12] imx: mx53ard: " Benoît Thébaudeau
2013-05-02 20:52 ` [U-Boot] [PATCH 09/12] imx: mx53evk: " Benoît Thébaudeau
2013-05-02 20:52 ` [U-Boot] [PATCH 10/12] imx: mx53loco: " Benoît Thébaudeau
2013-05-02 20:52 ` [U-Boot] [PATCH 11/12] imx: mx53smd: " Benoît Thébaudeau
2013-05-02 20:52 ` [U-Boot] [PATCH 12/12] imx: mx5: Remove legacy iomux support Benoît Thébaudeau
2013-05-03 2:38 ` Marek Vasut
2013-05-03 12:09 ` Benoît Thébaudeau
2013-05-03 12:42 ` Marek Vasut
2013-05-03 13:06 ` Benoît Thébaudeau
2013-05-03 13:18 ` Marek Vasut
2013-05-03 13:17 ` Benoît Thébaudeau
2013-05-03 13:39 ` Marek Vasut
2013-05-03 13:38 ` Benoît Thébaudeau
2013-05-03 13:50 ` Marek Vasut [this message]
2013-05-05 14:58 ` Stefano Babic
2013-05-05 14:56 ` Stefano Babic
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201305031550.24291.marex@denx.de \
--to=marex@denx.de \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox