On 04/04/2011 03:42 PM, Wolfgang Denk wrote:
> Dear Kumar Gala,
>
> In message<16cf6cad-477c-4e04-93e0-1dadcf05c...@kernel.crashing.org>  you 
> wrote:

[snip]

> Timur: does this fix any real problem?
>
>>> powerpc/85xx: introduce 'fdt verify' command
>>
>> I'm wondering if its better to just set the dtb props instead of verifying.
>
> Unfortunately Timur's posting broke the mail thread because he failed
> to add keep reference headers in his patch postings.  I'm not happy
> with this implementation either.
>
> In any case this needs the ACK from the responsible custodian (FDT =>
> gvb).

I don't see any problem with the patch other than a minor typo  On the 
last line, s/patch/match./

>> +/*
>> + * Verify the addresses for all of the PCI controllers
>> + *
>> + * PCI is complicated because there is no correlation between the numbering
>> + * of the controllers by U-Boot and the numbering the device tree.  So we 
>> need
>> + * to search all of the aliases until we find a patch
>> + */

It appears to provide a valuable service.

Acked-by: Gerald Van Baren <vanba...@cideas.com>

> Best regards,
>
> Wolfgang Denk
>

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to