[U-Boot] [PATCH v2 3/3] arm, am335x: add support for 3 siemens boards

[Tom Rini]

On Tue, Jul 30, 2013 at 09:13:26AM +0200, Heiko Schocher wrote:
> Hello Lokesh,
> 
> Am 30.07.2013 07:39, schrieb Lokesh Vutla:
> >Hi Heiko,
> >On Tuesday 30 July 2013 10:39 AM, Heiko Schocher wrote:
> >>add support for the am335x based boards from siemens:
> >>
> [...]
> >>- [U-Boot] net, phy, cpsw: fix gigabit register access
> >>   http://patchwork.ozlabs.org/patch/261087/
> >I have just posted a V2 of AM33xx cleanup series, Can you rebase on top of that.
> >There was a bug in my series.  Multiplier for DDR DPLL was hard coded for all the boards.
> >IN V2, the common file expects  a function which returns ddr_dpll_parms.
> >So can you add the following code in your board files:
> >
> >#define OSC	(V_OSCK/1000000)
> >const struct dpll_params dpll_ddr = {
> >		<DXR2_PLL_FREQ>, OSC-1, 1, -1, -1, -1, -1};
> >const struct dpll_params *get_dpll_ddr_params(void)
> >{
> >		return&dpll_ddr;
> >}
> >
> >The M value we pass in config_ddr is not used(should be cleaned up)
> >Please let me know if I am not clear.
> >Sorry for the noise...
> 
> No problem, thanks for the info... Hmm.. just rebased against
> top of u-boot-ti and your patches, but ethernet is not longer
> working :-( I get:
> 
> U-Boot# ping 192.168.1.1
> link up on port 0, speed 1000, full duplex
> data abort
> 
>     MAYBE you should read doc/README.arm-unaligned-accesses
> 
> pc : [<9ff6ed7c>]          lr : [<9ff6f810>]
> sp : 9ef40d50  ip : 0019010c     fp : 00000000
> r10: 33221100  r9 : 0000002e     r8 : 9ef40f38
> r7 : 9ffed56c  r6 : 9ef45db0     r5 : 00000004  r4 : 9ef45d50
> r3 : 000005ee  r2 : 9ffee2c0     r1 : 9ef45db0  r0 : 9ef45d50
> Flags: nzCv  IRQs off  FIQs on  Mode SVC_32
> Resetting CPU ...
> 
> resetting ...
> 
> U-Boot SPL 2013.07-00098-g2b19a9e (Jul 30 2013 - 09:00:03)
> 
> Digging in this ...
> 
> Hmm... @tom:
> 
> My patch:
> [U-Boot] net, phy, cpsw: fix gigabit register access
> http://patchwork.ozlabs.org/patch/261087/
> 
> has the following diff:
> drivers/net/cpsw.c    | 2 +-
>  drivers/net/phy/phy.c | 6 +++++-
>  2 Dateien ge??ndert, 6 Zeilen hinzugef??gt(+), 2 Zeilen entfernt(-)
> 
> Looking in u-boot-ti tree it has:
> http://git.denx.de/?p=u-boot/u-boot-ti.git;a=commitdiff;h=7614ef78db2edcfb174da6a21a6756f5de08cc8e
> 
> it changes also boards.cfg !? There went something wrong while applying
> this patch ... Could you have a look at this issue?

Whoops, I did an ammend when I shoulda done a rebase --continue when
dealing with
http://git.denx.de/?p=u-boot/u-boot-ti.git;a=commit;h=317fab2df2b06d44d1f91bd44d421f3ef1013ac9

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20130731/1edad663/attachment.pgp>