From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Thu, 23 Jun 2016 18:55:27 -0400 Subject: [U-Boot] [PATCH v2 55/55] dm: Update the of-platdata README for the new features In-Reply-To: References: <1465796016-18375-1-git-send-email-sjg@chromium.org> <1465796016-18375-56-git-send-email-sjg@chromium.org> <20160623200419.GZ19080@bill-the-cat> Message-ID: <20160623225527.GA19080@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Thu, Jun 23, 2016 at 02:36:55PM -0600, Simon Glass wrote: > Hi Tom, > > On 23 June 2016 at 14:04, Tom Rini 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 > >> Suggested-by: Tom Rini > > [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? Well, I like it. But I'm also not great at spotting problems before we run into them sometimes. > 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? Given the number of places (it feels like) that run in to, or nearly run in to size limits today in SPL with tiny-printf enabled, no, I can't say that I think this will be used sparingly. So is there anything we can do about the structure / member naming to make it less wonky? Or just wait and see how things work out in the end when people start using it more? -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: