Hi Heiko,

and another nitpicking comment below:

On Monday 14 September 2009 17:06:46 Heiko Schocher wrote:

<snip>

> diff --git a/cpu/mpc5xxx/cpu.c b/cpu/mpc5xxx/cpu.c
> index f6258c7..a2fc323 100644
> --- a/cpu/mpc5xxx/cpu.c
> +++ b/cpu/mpc5xxx/cpu.c
> @@ -125,6 +125,9 @@ void ft_cpu_setup(void *blob, bd_t *bd)
>       uchar enetaddr[6];
>       char * eth_path = "/" OF_SOC "/ether...@3000";
>  #endif
> +#if defined(CONFIG_OF_IDE_FIXUP)
> +     extern block_dev_desc_t ide_dev_desc[CONFIG_SYS_IDE_MAXDEVICE];
> +#endif
> 
>       do_fixup_by_path_u32(blob, cpu_path, "timebase-frequency", OF_TBCLK, 1);
>       do_fixup_by_path_u32(blob, cpu_path, "bus-frequency", bd->bi_busfreq, 
> 1);
> @@ -137,6 +140,21 @@ void ft_cpu_setup(void *blob, bd_t *bd)
>       do_fixup_by_path(blob, eth_path, "mac-address", enetaddr, 6, 0);
>       do_fixup_by_path(blob, eth_path, "local-mac-address", enetaddr, 6, 0);
>  #endif
> +#if defined(CONFIG_OF_IDE_FIXUP)
> +     if (ide_dev_desc[0].type == DEV_TYPE_UNKNOWN) {
> +             /* NO CF card detected -> delete ata node in DTS */
> +             int nodeoffset = 0;
> +             char nodename[] = "/soc5...@f0000000/a...@3a00";
> +
> +             nodeoffset = fdt_path_offset (blob, nodename);

Please use consistent styles in one file, in this case func() without a space 
before the "(".

Thanks.

Cheers,
Stefan

--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: off...@denx.de
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to