ION DTS changes for HiKey in -next
laura at labbott.name
Mon Jan 11 07:18:57 PST 2016
On 1/8/16 9:05 PM, Greg Kroah-Hartman wrote:
> 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.
>>> The driver and the binding before that don't really belong either,
>>> I would have NAK'd those on devicetree at vger.kernel.org, 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 
>>> or figure out how to do this with the existing reserved-memory bindings.
>>>  https://lkml.org/lkml/2015/10/6/854
>> I posted v2 back in November
>> but there wasn't much in the way of review comments. More feedback there
>> would be appreciated or I can resend.
> I still have those in my queue, but was waiting for someone to chime in
> on them. I guess if no one objected, that means I should accept them?
Before accepting them. I'd rather have some acknowledgement from the DT
maintainers that these bindings are moving in the right direction or
at least a better idea than the per platform bindings.
> greg k-h
More information about the linux-arm-kernel