mbox

[U-Boot] Pull request: u-boot-sh/rmobile into u-boot-arm/master

Message ID CABMQnV+XmFwo-qSj38XhcevBmLga=DFXbxQ=JLcEjTGjKCt9ew@mail.gmail.com
State Superseded
Delegated to: Albert ARIBAUD
Headers show

Pull-request

git://git.denx.de/u-boot-sh.git rmobile

Message

Nobuhiro Iwamatsu June 10, 2014, 7:58 a.m. UTC 
Dear Albert Aribaud,

Please pull u-boot-sh/rmobile into u-boot-arm/master.

The following changes since commit 0a26e1d6c394aacbf1153977b7348d1dff85db3f:

  arm: fix a double-definition error of _start symbol (2014-06-09
10:36:40 +0200)

are available in the git repository at:

  git://git.denx.de/u-boot-sh.git rmobile

for you to fetch changes up to a59cdc44a9df0887fbb0d7dd5e6229ac0018785b:

  arm: rmobile: lager: Remove NOR-Flash support from boards.cfg
(2014-06-10 16:49:31 +0900)

----------------------------------------------------------------
Nobuhiro Iwamatsu (2):
      arm: rmobile: koelsch: Remove NOR-Flash support from boards.cfg
      arm: rmobile: lager: Remove NOR-Flash support from boards.cfg

 boards.cfg | 2 --
 1 file changed, 2 deletions(-)

Comments

Albert ARIBAUD June 10, 2014, 8:46 a.m. UTC | #1 
Hi Nobuhiro,

On Tue, 10 Jun 2014 16:58:39 +0900, Nobuhiro Iwamatsu
<iwamatsu@nigauri.org> wrote:

> Dear Albert Aribaud,
> 
> Please pull u-boot-sh/rmobile into u-boot-arm/master.
> 
> The following changes since commit 0a26e1d6c394aacbf1153977b7348d1dff85db3f:
> 
>   arm: fix a double-definition error of _start symbol (2014-06-09
> 10:36:40 +0200)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-sh.git rmobile
> 
> for you to fetch changes up to a59cdc44a9df0887fbb0d7dd5e6229ac0018785b:
> 
>   arm: rmobile: lager: Remove NOR-Flash support from boards.cfg
> (2014-06-10 16:49:31 +0900)
> 
> ----------------------------------------------------------------
> Nobuhiro Iwamatsu (2):
>       arm: rmobile: koelsch: Remove NOR-Flash support from boards.cfg
>       arm: rmobile: lager: Remove NOR-Flash support from boards.cfg
> 
>  boards.cfg | 2 --
>  1 file changed, 2 deletions(-)

Isn't the lager commit message slightly wrong, mentioning koelsch
instead of lager at one point? Not that it matters much since we are
finishing removing support here.

Amicalement,
Nobuhiro Iwamatsu June 12, 2014, 12:47 a.m. UTC | #2 
Hi,

Thanks for your review.

2014-06-10 17:46 GMT+09:00 Albert ARIBAUD <albert.u.boot@aribaud.net>:
> Hi Nobuhiro,
>
> On Tue, 10 Jun 2014 16:58:39 +0900, Nobuhiro Iwamatsu
> <iwamatsu@nigauri.org> wrote:
>
>> Dear Albert Aribaud,
>>
>> Please pull u-boot-sh/rmobile into u-boot-arm/master.
>>
>> The following changes since commit 0a26e1d6c394aacbf1153977b7348d1dff85db3f:
>>
>>   arm: fix a double-definition error of _start symbol (2014-06-09
>> 10:36:40 +0200)
>>
>> are available in the git repository at:
>>
>>   git://git.denx.de/u-boot-sh.git rmobile
>>
>> for you to fetch changes up to a59cdc44a9df0887fbb0d7dd5e6229ac0018785b:
>>
>>   arm: rmobile: lager: Remove NOR-Flash support from boards.cfg
>> (2014-06-10 16:49:31 +0900)
>>
>> ----------------------------------------------------------------
>> Nobuhiro Iwamatsu (2):
>>       arm: rmobile: koelsch: Remove NOR-Flash support from boards.cfg
>>       arm: rmobile: lager: Remove NOR-Flash support from boards.cfg
>>
>>  boards.cfg | 2 --
>>  1 file changed, 2 deletions(-)
>
> Isn't the lager commit message slightly wrong, mentioning koelsch
> instead of lager at one point? Not that it matters much since we are
> finishing removing support here.
>

Yes, you are right.
I will update commit message and resend.

> Amicalement,
> --
> Albert.

Best regards,
  Nobuhiro