All of lore.kernel.org
 help / color / mirror / Atom feed
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
=====================================================================

  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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.