On Thu, Feb 09, 2017 at 09:17:58AM -0600, Rob Herring wrote: > Frank, any more comments on this? If not, I plan to apply this series. Well, I find that a little annoying, because DT has the requirement that new bindings are properly documented in Documentation, and it appears that this comes with no documentation what so ever, despite introducing new properties (like the -map-mask-passthru thing). So I'm NAKing it until there's some documentation of how this mechanism is supposed to work. Merely providing an example in a commit log is (IMHO) insufficient. An example doesn't explain how it was created, or how to create an implementation. Remember, we expect people to do exactly that, so we need to give them this information so that they can make use of it. I did try to work out from the code how the -map-mask thing worked, but eventually gave up. -- RMK's Patch system: http://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up according to speedtest.net. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
