Patchwork [U-Boot] Please pull u-boot-ti/master

login
register
mail settings
Submitter Tom Rini
Date March 28, 2012, 9:54 p.m.
Message ID <20120328215441.GC4903@bill-the-cat>
Download mbox
Permalink /patch/149312/
State Accepted
Headers show

Pull-request

git://git.denx.de/u-boot-ti master

Comments

Tom Rini - March 28, 2012, 9:54 p.m.
Hello,

The following changes since commit 4a3ea216414ad762b3718775a65437c13f0accbc:
  Rob Herring (1):
        ARM: highbank: setup env from boot source register

are available in the git repository at:

  git://git.denx.de/u-boot-ti master

Anatolij Gustschin (2):
      board/ti/beagle/beagle.c: Fix build warnings
      arch/arm/cpu/armv7/omap-common/clocks-common.c: Fix build warnings

Tom Rini (3):
      sdrc.c: Fix typo in do_sdrc_init() for SPL
      OMAP: Remove omap1610inn-based boards
      OMAP: Move omap1510inn to Unmaintained / Orphaned

 MAINTAINERS                                    |    8 +-
 arch/arm/cpu/armv7/omap-common/clocks-common.c |    4 +-
 arch/arm/cpu/armv7/omap3/sdrc.c                |    2 +-
 board/ti/beagle/beagle.c                       |    3 +-
 board/ti/omap1610inn/Makefile                  |   45 ---
 board/ti/omap1610inn/config.mk                 |   26 --
 board/ti/omap1610inn/flash.c                   |  495 ------------------------
 board/ti/omap1610inn/lowlevel_init.S           |  452 ---------------------
 board/ti/omap1610inn/omap1610innovator.c       |  309 ---------------
 boards.cfg                                     |   10 +-
 include/configs/omap1610h2.h                   |  197 ----------
 include/configs/omap1610inn.h                  |  202 ----------
 12 files changed, 8 insertions(+), 1745 deletions(-)
 delete mode 100644 board/ti/omap1610inn/Makefile
 delete mode 100644 board/ti/omap1610inn/config.mk
 delete mode 100644 board/ti/omap1610inn/flash.c
 delete mode 100644 board/ti/omap1610inn/lowlevel_init.S
 delete mode 100644 board/ti/omap1610inn/omap1610innovator.c
 delete mode 100644 include/configs/omap1610h2.h
 delete mode 100644 include/configs/omap1610inn.h


While not all of these patches have been around for long, they are
either:
- Remove broken boards (omap1610inn) or make doing so in the future
  easier (omap1510inn)
- Long-standing warning fixes
- Obvious thinko fixes.
Albert ARIBAUD - March 29, 2012, 6:21 a.m.
Hi Tom,

Le 28/03/2012 23:54, Tom Rini a écrit :
> Hello,
>
> The following changes since commit 4a3ea216414ad762b3718775a65437c13f0accbc:
>    Rob Herring (1):
>          ARM: highbank: setup env from boot source register
>
> are available in the git repository at:
>
>    git://git.denx.de/u-boot-ti master
>
> Anatolij Gustschin (2):
>        board/ti/beagle/beagle.c: Fix build warnings
>        arch/arm/cpu/armv7/omap-common/clocks-common.c: Fix build warnings
>
> Tom Rini (3):
>        sdrc.c: Fix typo in do_sdrc_init() for SPL
>        OMAP: Remove omap1610inn-based boards
>        OMAP: Move omap1510inn to Unmaintained / Orphaned
>
>   MAINTAINERS                                    |    8 +-
>   arch/arm/cpu/armv7/omap-common/clocks-common.c |    4 +-
>   arch/arm/cpu/armv7/omap3/sdrc.c                |    2 +-
>   board/ti/beagle/beagle.c                       |    3 +-
>   board/ti/omap1610inn/Makefile                  |   45 ---
>   board/ti/omap1610inn/config.mk                 |   26 --
>   board/ti/omap1610inn/flash.c                   |  495 ------------------------
>   board/ti/omap1610inn/lowlevel_init.S           |  452 ---------------------
>   board/ti/omap1610inn/omap1610innovator.c       |  309 ---------------
>   boards.cfg                                     |   10 +-
>   include/configs/omap1610h2.h                   |  197 ----------
>   include/configs/omap1610inn.h                  |  202 ----------
>   12 files changed, 8 insertions(+), 1745 deletions(-)
>   delete mode 100644 board/ti/omap1610inn/Makefile
>   delete mode 100644 board/ti/omap1610inn/config.mk
>   delete mode 100644 board/ti/omap1610inn/flash.c
>   delete mode 100644 board/ti/omap1610inn/lowlevel_init.S
>   delete mode 100644 board/ti/omap1610inn/omap1610innovator.c
>   delete mode 100644 include/configs/omap1610h2.h
>   delete mode 100644 include/configs/omap1610inn.h
>
>
> While not all of these patches have been around for long, they are
> either:
> - Remove broken boards (omap1610inn) or make doing so in the future
>    easier (omap1510inn)
> - Long-standing warning fixes
> - Obvious thinko fixes.
>

Applied to u-boot-arm/master, thanks.

Amicalement,
Tom Rini - March 29, 2012, 6:01 p.m.
On Wed, Mar 28, 2012 at 2:54 PM, Tom Rini <trini@ti.com> wrote:
> Hello,
>
> The following changes since commit 4a3ea216414ad762b3718775a65437c13f0accbc:
>  Rob Herring (1):
>        ARM: highbank: setup env from boot source register
>
> are available in the git repository at:
>
>  git://git.denx.de/u-boot-ti master
>
> Anatolij Gustschin (2):
>      board/ti/beagle/beagle.c: Fix build warnings
>      arch/arm/cpu/armv7/omap-common/clocks-common.c: Fix build warnings
>
> Tom Rini (3):
>      sdrc.c: Fix typo in do_sdrc_init() for SPL
>      OMAP: Remove omap1610inn-based boards
>      OMAP: Move omap1510inn to Unmaintained / Orphaned

Well this is embarrassing.  I'd have sworn I tested -s omap but I must
not have as omap1510inn doesn't build after that change as it doesn't
build as there's no such thing as arch/arm/cpu/arm925t/omap/.  Please
drop that change out, sorry for the noise.
Albert ARIBAUD - March 29, 2012, 11:51 p.m.
Hi Tom,

Le 29/03/2012 20:01, Tom Rini a écrit :
> On Wed, Mar 28, 2012 at 2:54 PM, Tom Rini<trini@ti.com>  wrote:
>> Hello,
>>
>> The following changes since commit 4a3ea216414ad762b3718775a65437c13f0accbc:
>>   Rob Herring (1):
>>         ARM: highbank: setup env from boot source register
>>
>> are available in the git repository at:
>>
>>   git://git.denx.de/u-boot-ti master
>>
>> Anatolij Gustschin (2):
>>       board/ti/beagle/beagle.c: Fix build warnings
>>       arch/arm/cpu/armv7/omap-common/clocks-common.c: Fix build warnings
>>
>> Tom Rini (3):
>>       sdrc.c: Fix typo in do_sdrc_init() for SPL
>>       OMAP: Remove omap1610inn-based boards
>>       OMAP: Move omap1510inn to Unmaintained / Orphaned
>
> Well this is embarrassing.  I'd have sworn I tested -s omap but I must
> not have as omap1510inn doesn't build after that change as it doesn't
> build as there's no such thing as arch/arm/cpu/arm925t/omap/.  Please
> drop that change out, sorry for the noise.

Hmm... I would not expect omap1510inn to build after it has been 
orphaned, so I am unsure what you mean exactly here. Also, can you 
precisely list the change(s) that I should drop?

Amicalement,
Tom Rini - March 30, 2012, 12:58 a.m.
On Thu, Mar 29, 2012 at 4:51 PM, Albert ARIBAUD
<albert.u.boot@aribaud.net> wrote:
> Hi Tom,
>
> Le 29/03/2012 20:01, Tom Rini a écrit :
>>
>> On Wed, Mar 28, 2012 at 2:54 PM, Tom Rini<trini@ti.com>  wrote:
>>>
>>> Hello,
>>>
>>> The following changes since commit
>>> 4a3ea216414ad762b3718775a65437c13f0accbc:
>>>  Rob Herring (1):
>>>        ARM: highbank: setup env from boot source register
>>>
>>> are available in the git repository at:
>>>
>>>  git://git.denx.de/u-boot-ti master
>>>
>>> Anatolij Gustschin (2):
>>>      board/ti/beagle/beagle.c: Fix build warnings
>>>      arch/arm/cpu/armv7/omap-common/clocks-common.c: Fix build warnings
>>>
>>> Tom Rini (3):
>>>      sdrc.c: Fix typo in do_sdrc_init() for SPL
>>>      OMAP: Remove omap1610inn-based boards
>>>      OMAP: Move omap1510inn to Unmaintained / Orphaned
>>
>>
>> Well this is embarrassing.  I'd have sworn I tested -s omap but I must
>> not have as omap1510inn doesn't build after that change as it doesn't
>> build as there's no such thing as arch/arm/cpu/arm925t/omap/.  Please
>> drop that change out, sorry for the noise.
>
>
> Hmm... I would not expect omap1510inn to build after it has been orphaned,
> so I am unsure what you mean exactly here. Also, can you precisely list the
> change(s) that I should drop?

So, omap1510inn has no active maintainer, it's listed as the same as
omap1610inn which was not building.  omap1510inn was however building
until 'OMAP: Move omap1510inn to Unmaintained / Orphaned' in which I
also added it to the omap SoC family, forgetting that it also forced
trying to build a non-existent directory, breaking it.
Albert ARIBAUD - March 30, 2012, 5:35 a.m.
Hi Tom,

Le 30/03/2012 02:58, Tom Rini a écrit :
> On Thu, Mar 29, 2012 at 4:51 PM, Albert ARIBAUD
> <albert.u.boot@aribaud.net>  wrote:
>> Hi Tom,
>>
>> Le 29/03/2012 20:01, Tom Rini a écrit :
>>>
>>> On Wed, Mar 28, 2012 at 2:54 PM, Tom Rini<trini@ti.com>    wrote:
>>>>
>>>> Hello,
>>>>
>>>> The following changes since commit
>>>> 4a3ea216414ad762b3718775a65437c13f0accbc:
>>>>   Rob Herring (1):
>>>>         ARM: highbank: setup env from boot source register
>>>>
>>>> are available in the git repository at:
>>>>
>>>>   git://git.denx.de/u-boot-ti master
>>>>
>>>> Anatolij Gustschin (2):
>>>>       board/ti/beagle/beagle.c: Fix build warnings
>>>>       arch/arm/cpu/armv7/omap-common/clocks-common.c: Fix build warnings
>>>>
>>>> Tom Rini (3):
>>>>       sdrc.c: Fix typo in do_sdrc_init() for SPL
>>>>       OMAP: Remove omap1610inn-based boards
>>>>       OMAP: Move omap1510inn to Unmaintained / Orphaned
>>>
>>>
>>> Well this is embarrassing.  I'd have sworn I tested -s omap but I must
>>> not have as omap1510inn doesn't build after that change as it doesn't
>>> build as there's no such thing as arch/arm/cpu/arm925t/omap/.  Please
>>> drop that change out, sorry for the noise.
>>
>>
>> Hmm... I would not expect omap1510inn to build after it has been orphaned,
>> so I am unsure what you mean exactly here. Also, can you precisely list the
>> change(s) that I should drop?
>
> So, omap1510inn has no active maintainer, it's listed as the same as
> omap1610inn which was not building.  omap1510inn was however building
> until 'OMAP: Move omap1510inn to Unmaintained / Orphaned' in which I
> also added it to the omap SoC family, forgetting that it also forced
> trying to build a non-existent directory, breaking it.

Ok, throwing out 'OMAP: Move omap1510inn to Unmaintained / Orphaned' 
from u-boot-arm/master.

Amicalement,
Tom Rini - March 30, 2012, 2:16 p.m.
On Thu, Mar 29, 2012 at 10:35 PM, Albert ARIBAUD
<albert.u.boot@aribaud.net> wrote:
> Hi Tom,
>
> Le 30/03/2012 02:58, Tom Rini a écrit :
>
>> On Thu, Mar 29, 2012 at 4:51 PM, Albert ARIBAUD
>> <albert.u.boot@aribaud.net>  wrote:
>>>
>>> Hi Tom,
>>>
>>> Le 29/03/2012 20:01, Tom Rini a écrit :
>>>>
>>>>
>>>> On Wed, Mar 28, 2012 at 2:54 PM, Tom Rini<trini@ti.com>    wrote:
>>>>>
>>>>>
>>>>> Hello,
>>>>>
>>>>> The following changes since commit
>>>>> 4a3ea216414ad762b3718775a65437c13f0accbc:
>>>>>  Rob Herring (1):
>>>>>        ARM: highbank: setup env from boot source register
>>>>>
>>>>> are available in the git repository at:
>>>>>
>>>>>  git://git.denx.de/u-boot-ti master
>>>>>
>>>>> Anatolij Gustschin (2):
>>>>>      board/ti/beagle/beagle.c: Fix build warnings
>>>>>      arch/arm/cpu/armv7/omap-common/clocks-common.c: Fix build warnings
>>>>>
>>>>> Tom Rini (3):
>>>>>      sdrc.c: Fix typo in do_sdrc_init() for SPL
>>>>>      OMAP: Remove omap1610inn-based boards
>>>>>      OMAP: Move omap1510inn to Unmaintained / Orphaned
>>>>
>>>>
>>>>
>>>> Well this is embarrassing.  I'd have sworn I tested -s omap but I must
>>>> not have as omap1510inn doesn't build after that change as it doesn't
>>>> build as there's no such thing as arch/arm/cpu/arm925t/omap/.  Please
>>>> drop that change out, sorry for the noise.
>>>
>>>
>>>
>>> Hmm... I would not expect omap1510inn to build after it has been
>>> orphaned,
>>> so I am unsure what you mean exactly here. Also, can you precisely list
>>> the
>>> change(s) that I should drop?
>>
>>
>> So, omap1510inn has no active maintainer, it's listed as the same as
>> omap1610inn which was not building.  omap1510inn was however building
>> until 'OMAP: Move omap1510inn to Unmaintained / Orphaned' in which I
>> also added it to the omap SoC family, forgetting that it also forced
>> trying to build a non-existent directory, breaking it.
>
>
> Ok, throwing out 'OMAP: Move omap1510inn to Unmaintained / Orphaned' from
> u-boot-arm/master.

Thanks, sorry again for the noise.