From: Tom <Tom.Rix@windriver.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH ARM 3/3 v3] change s3c24x0 register struct members to lower case
Date: Sat, 13 Feb 2010 11:02:39 -0600 [thread overview]
Message-ID: <4B76DB2F.1070505@windriver.com> (raw)
In-Reply-To: <1f3430fb1002130806h5f89a77ci5e45c0f0589cefe2@mail.gmail.com>
Minkyu Kang wrote:
> Dear Tom,
>
> On 12 February 2010 22:07, Tom <Tom.Rix@windriver.com> wrote:
>> Minkyu Kang wrote:
>>> Dear Kevin,
>>>
>>> On 12 February 2010 05:29, kevin.morfitt at fearnside-systems.co.uk
>>> <kevin.morfitt@fearnside-systems.co.uk> wrote:
>>>> On 11/02/2010 19:04, Tom wrote:
>>>>> Kevin Morfitt wrote:
>>>>>> Changes the names of the s3c24x0 register struct members from
>>>>>> upper-case
>>>>>> to lower-case.
>>>>>>
>>>>>> Signed-off-by: Kevin Morfitt<kevin.morfitt@fearnside-systems.co.uk>
>>>>>> ---
>>>>>>
>>>>>> v2 changes - re-number to be 3/3
>>>>>> v3 changes - re-based to modified patch 2/3
>>>>>>
>>>>>> checkpatch.pl reports no errors. MAKEALL ARM9 reports no new warnings
>>>>>> or errors.
>>>>>>
>>>>> I had problem applying v2 patch 1 to u-boot-arm and u-boot-samsung
>>>>> Can you check if this patchset is correctly rebased.
>>>> It's based against Monday's u-boot-master which hasn't changed since
>>>> then. v1 was against u-boot-samsung but Wolfgang asked me to do it
>>>> against u-boot-master:
>>>>
>>>> "Please adhere to posting guidelines. Patches posted ere MUST be based
>>>> either on the "master" or on the "next" trees."
>>>>
>>>> http://lists.denx.de/pipermail/u-boot/2009-December/065420.html
>>> If so, who can apply this patchset?
>>> I think need to rebase for u-boot-arm or u-boot-samsung.
>>>
>> Kevin is correct.
>>
>> This is a problem with the arm and samsung branch diverging from master.
>> The testing of this patch will stall a bit until we are sync-ed up again.
>> I will make sure this happens in the next day or so.
>>
ARM has been rebased to master.
I checked this patchset yesterday and there was still a problem with applying
them. Ultimately I think this set will need to be rebased. Waiting until the
ARM changes have been pulled into master is fine.
>> Minkyu,
>> There are 3 commits on samsung/master.
>> Are these ready for a pull request?
>> Please send me one if they are.
>>
>> Tom
>>
>>
>>
>
> I'll send it next week.
>
No rush, this is fine.
Tom
> Thanks
> Minkyu Kang
prev parent reply other threads:[~2010-02-13 17:02 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-02-11 17:51 [U-Boot] [PATCH ARM 3/3 v3] change s3c24x0 register struct members to lower case Kevin Morfitt
2010-02-11 19:04 ` Tom
2010-02-11 20:29 ` kevin.morfitt at fearnside-systems.co.uk
2010-02-12 5:19 ` Minkyu Kang
2010-02-12 13:07 ` Tom
2010-02-13 16:06 ` Minkyu Kang
2010-02-13 17:02 ` Tom [this message]
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=4B76DB2F.1070505@windriver.com \
--to=tom.rix@windriver.com \
--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