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

[Heiko Schocher]

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?

Thanks.

bye,
Heiko
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany