On Fri, Jan 08, 2016 at 10:14:35AM -0800, Laura Abbott wrote: > On 1/8/16 5:55 AM, Mark Rutland wrote: > >On Fri, Jan 08, 2016 at 12:44:39PM +0000, Mark Brown wrote: > >>On Thu, Jan 07, 2016 at 09:02:14PM -0800, Greg Kroah-Hartman wrote: > >>>On Thu, Jan 07, 2016 at 05:37:44PM +0000, Mark Brown wrote: > >> > >>>>I was just looking at DTs in -next and noticed that there is a patch > >>>>59dfafd03fc (arm64: dts: Add dts files to enable ION on Hi6220 SoC) > >>>>which adds at DT doing something for ION. Are we sure this should be > >>>>going into the main production DT? The bindings haven't been reviewed > >>>>as far as I can tell, the matching driver is only in staging and hasn't > >>>>been posted upstream. > >> > >>>Isn't "staging" upstream enough for this? :) > >> > >>I wouldn't have thought so, DTs are supposed to be an ABI so we want > >>proper review and having had a quick glance this doesn't look like it's > >>a hardware description so it's not clear to me it should be in DT at all. > > > >Indeed. > > > >The driver and the binding before that don't really belong either, > >I would have NAK'd those on devicetree@xxxxxxxxxxxxxxx, though it > >appears I either missed them or they never made it to that list. > > > > From my PoV there should not be a platform-specific ION binding. If we > >need one at all, people should work on the proposed generic binding [1] > >or figure out how to do this with the existing reserved-memory bindings. > > > >Thanks, > >Mark. > > > >[1] https://lkml.org/lkml/2015/10/6/854 > > I posted v2 back in November > (http://article.gmane.org/gmane.linux.drivers.driver-project.devel/80475) > but there wasn't much in the way of review comments. More feedback there > would be appreciated or I can resend. A resend would be appeciated, as that's easier to reply to. I'm still a little fuzzy on why this can't be done with reserved-memory, but that's a discussion better left for the thread with the patches. Thanks, Mark. -- 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