Patchwork [U-Boot,GIT,PULL] Pull request u-boot-85xx.git

login
register
mail settings
Submitter Kumar Gala
Date Oct. 14, 2011, 7:34 a.m.
Message ID <alpine.LFD.2.00.1110140233570.16644@right.am.freescale.net>
Download mbox
Permalink /patch/119718/
State Accepted
Headers show

Pull-request

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

Comments

Kumar Gala - Oct. 14, 2011, 7:34 a.m.
The following changes since commit d8fffa057c9430fd0c5104ab6ff7db4cdb03db51:

  Merge branch 'master' of git://git.denx.de/u-boot-mips (2011-10-12 22:47:15 +0200)

are available in the git repository at:

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

Haiying Wang (3):
      powerpc/p5020: fixup portal config info
      powerpc/p2041: fixup portal config info
      powerpc/p3041: fixup portal config info

Kuldip Giroh (1):
      powerpc/85xx: Added secure boot option for P2041RDB boards

Kumar Gala (1):
      powerpc/85xx: Drop CONFIG_VIDEO on P1_P2_RDB-PC boards to reduce size

Timur Tabi (2):
      powerpc/p3060: remove all references to RCW bits EC1_EXT, EC2_EXT, and EC3
      phylib: wait for TN2020 to achieve SERDES lane alignment at startup

chenhui zhao (2):
      powerpc/mpc8548cds: Code cleanup and refactoring
      powerpc/mpc8548cds: Add 36-bit support

 arch/powerpc/cpu/mpc85xx/p2041_ids.c    |   20 +++---
 arch/powerpc/cpu/mpc85xx/p3041_ids.c    |   20 +++---
 arch/powerpc/cpu/mpc85xx/p3060_serdes.c |   20 -----
 arch/powerpc/cpu/mpc85xx/p5020_ids.c    |   20 +++---
 arch/powerpc/include/asm/immap_85xx.h   |   10 ---
 board/freescale/mpc8548cds/law.c        |   31 +-------
 board/freescale/mpc8548cds/mpc8548cds.c |    8 +-
 board/freescale/mpc8548cds/tlb.c        |   64 ++++++++--------
 boards.cfg                              |    2 +
 drivers/net/fm/p3060.c                  |   17 ----
 drivers/net/phy/teranetics.c            |   33 ++++++++
 include/configs/MPC8548CDS.h            |  125 +++++++++++++++++++++++--------
 include/configs/p1_p2_rdb_pc.h          |   16 ----
 13 files changed, 196 insertions(+), 190 deletions(-)
Tabi Timur-B04825 - Oct. 14, 2011, 7:15 p.m.
On Fri, Oct 14, 2011 at 2:34 AM, Kumar Gala <galak@kernel.crashing.org> wrote:
> The following changes since commit d8fffa057c9430fd0c5104ab6ff7db4cdb03db51:
>
>  Merge branch 'master' of git://git.denx.de/u-boot-mips (2011-10-12 22:47:15 +0200)
>
> are available in the git repository at:
>
>  git://git.denx.de/u-boot-mpc85xx master

Can you pick up this patch, as well:

http://patchwork.ozlabs.org/patch/109631/

It fixes a real bug.
Kumar Gala - Oct. 14, 2011, 8:27 p.m.
On Oct 14, 2011, at 2:15 PM, Tabi Timur-B04825 wrote:

> On Fri, Oct 14, 2011 at 2:34 AM, Kumar Gala <galak@kernel.crashing.org> wrote:
>> The following changes since commit d8fffa057c9430fd0c5104ab6ff7db4cdb03db51:
>> 
>>  Merge branch 'master' of git://git.denx.de/u-boot-mips (2011-10-12 22:47:15 +0200)
>> 
>> are available in the git repository at:
>> 
>>  git://git.denx.de/u-boot-mpc85xx master
> 
> Can you pick up this patch, as well:
> 
> http://patchwork.ozlabs.org/patch/109631/
> 
> It fixes a real bug.

Need Wolfgang's Ack on that.

- k
Tabi Timur-B04825 - Oct. 14, 2011, 8:29 p.m.
Kumar Gala wrote:
> Need Wolfgang's Ack on that.

Why?  It just fixes a bug in our board code.
Kumar Gala - Oct. 14, 2011, 8:31 p.m.
On Oct 14, 2011, at 3:29 PM, Timur Tabi wrote:

> Kumar Gala wrote:
>> Need Wolfgang's Ack on that.
> 
> Why?  It just fixes a bug in our board code.

But it touches common/cmd_mac.c

- k
Timur Tabi - Oct. 14, 2011, 8:37 p.m.
Kumar Gala wrote:
> But it touches common/cmd_mac.c

I've never understand why that file is in common/, since it applies only to
Freescale SOCs.  There's only one implementation of do_mac, and it's ours.

I could have sworn York posted a patch to move do_mac into sys_eeprom.c.  I
don't see it anywhere, though.
Wolfgang Denk - Oct. 14, 2011, 9:34 p.m.
Dear Kumar Gala,

In message <0C0B0795-E6C4-4D6D-A067-BC22FFA98C75@kernel.crashing.org> you wrote:
> 
> > Can you pick up this patch, as well:
> > 
> > http://patchwork.ozlabs.org/patch/109631/
> > 
> > It fixes a real bug.
> 
> Need Wolfgang's Ack on that.

Sorry for the late review; I see issues with this patch. No ACK as is.

Best regards,

Wolfgang Denk
Wolfgang Denk - Oct. 15, 2011, 7:59 p.m.
Dear Kumar Gala,

In message <alpine.LFD.2.00.1110140233570.16644@right.am.freescale.net> you wrote:
> The following changes since commit d8fffa057c9430fd0c5104ab6ff7db4cdb03db51:
> 
>   Merge branch 'master' of git://git.denx.de/u-boot-mips (2011-10-12 22:47:15 +0200)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-mpc85xx master
> 
> Haiying Wang (3):
>       powerpc/p5020: fixup portal config info
>       powerpc/p2041: fixup portal config info
>       powerpc/p3041: fixup portal config info
> 
> Kuldip Giroh (1):
>       powerpc/85xx: Added secure boot option for P2041RDB boards
> 
> Kumar Gala (1):
>       powerpc/85xx: Drop CONFIG_VIDEO on P1_P2_RDB-PC boards to reduce size
> 
> Timur Tabi (2):
>       powerpc/p3060: remove all references to RCW bits EC1_EXT, EC2_EXT, and EC3
>       phylib: wait for TN2020 to achieve SERDES lane alignment at startup
> 
> chenhui zhao (2):
>       powerpc/mpc8548cds: Code cleanup and refactoring
>       powerpc/mpc8548cds: Add 36-bit support
> 
>  arch/powerpc/cpu/mpc85xx/p2041_ids.c    |   20 +++---
>  arch/powerpc/cpu/mpc85xx/p3041_ids.c    |   20 +++---
>  arch/powerpc/cpu/mpc85xx/p3060_serdes.c |   20 -----
>  arch/powerpc/cpu/mpc85xx/p5020_ids.c    |   20 +++---
>  arch/powerpc/include/asm/immap_85xx.h   |   10 ---
>  board/freescale/mpc8548cds/law.c        |   31 +-------
>  board/freescale/mpc8548cds/mpc8548cds.c |    8 +-
>  board/freescale/mpc8548cds/tlb.c        |   64 ++++++++--------
>  boards.cfg                              |    2 +
>  drivers/net/fm/p3060.c                  |   17 ----
>  drivers/net/phy/teranetics.c            |   33 ++++++++
>  include/configs/MPC8548CDS.h            |  125 +++++++++++++++++++++++--------
>  include/configs/p1_p2_rdb_pc.h          |   16 ----
>  13 files changed, 196 insertions(+), 190 deletions(-)

Applied, thanks.

Best regards,

Wolfgang Denk