From: Stefano Babic <sbabic@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 12/12] imx: mx5: Remove legacy iomux support
Date: Sun, 05 May 2013 16:58:40 +0200 [thread overview]
Message-ID: <518673A0.1080705@denx.de> (raw)
In-Reply-To: <201305031518.33328.marex@denx.de>
On 03/05/2013 15:18, 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.
Marek, apart who send patches earlier, I stick with Benoit's approach.
The reason: first patches that provide a general framework (in this
case, the iomux) for the SOC, and then board support already using the
right approach.
Best regards,
Stefano
--
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================
next prev parent reply other threads:[~2013-05-05 14:58 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
2013-05-05 14:58 ` Stefano Babic [this message]
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=518673A0.1080705@denx.de \
--to=sbabic@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