[U-Boot] [PATCH v2 55/55] dm: Update the of-platdata README for the new features

[Simon Glass]

Hi Tom,

On 23 June 2016 at 14:04, Tom Rini <trini at konsulko.com> wrote:
> On Sun, Jun 12, 2016 at 11:33:36PM -0600, Simon Glass wrote:
>
>> Revise the content based on the v2 additions. This is kept as a separate
>> patch to avoid confusing those who have already reviewed the v1 series.
>>
>> Signed-off-by: Simon Glass <sjg at chromium.org>
>> Suggested-by: Tom Rini <trini at konsulko.com>
> [snip]
>> +Converting of-platdata to a useful form
>> +---------------------------------------
>> +
>> +Of course it would be possible use the of-platdata directly in your driver
>> +whenever configuration information is required. However this meands that the
>
> "means"
>
> [snip]
>> +The of-platdata struct contents is copied from the C structure data to the
>
> "is copied" -> "are copied"
>
> And thanks again for doing all of this!

Obviously I still have a test to write, but other than that, what do
you think of this feature?

I put quite a bit of info in the caveats. The benefit is clear but it
is also a bit wonky - e.g. the structure / member naming. I'm really a
little bit nervous about it all. Do you think we can make sure it is
used sparingly?

Regards,
Simon