Re: [PATCH V9 2/5] dma: add Qualcomm Technologies HIDMA management driver

From: Sinan Kaya
Date: Wed Jan 13 2016 - 12:59:33 EST


Hi Mark, Rob, Frank, Grant and other DT experts;

>>>
>>> I'll add support for both. Both having a parent-child relationship while
>>> inside the hypervisor and flat hierarchy while running in the guest
>>> machine for QEMU.
>>
>> I'm happy with that.
>>
>>> I just created a child object of the management driver and channel
>>> driver got probed properly. So, it just a matter of device-tree
>>> documentation at this point.
>>>
>>> It will be like this though.
>>>
>>>>>> hidma {
>>>>>> compatible - "qcom,hidma-1.0";
>>>>>>
>>>>>> /* OPTIONAL management interface registers */
>>>>>> reg = < ... ... >;
>>>>>>
>>>>>> ...
>>>>>>
>>>>>> channel0 {
>>>>>> compatible = "qcom,
>>>>>> reg = < ... ... >;
>>>>>>
>>>>>> ...
>>>>>> };
>>>>>>
>>>>>> ...
>>>>>> };
>>>
>>> This seems to have worked without requiring any work from me.
>>>
>>> I need to associate sysfs next.
>>
>> Ok. I look forward to seeing those patches.
>>
>> Thanks,
>> Mark.
>>
>
> Mark,
> The patch has been posted here. Please review.
>
> https://lkml.org/lkml/2016/1/3/246
>
> Sinan
>
>
>
>

I have been trying to reach to device tree reviewers without any luck for the last two weeks.
It must be the holidays.

I have restructured the code address concerns from Mark Rutland above. The dma-engine maintainer, Vinod Koul;
is looking for an ACK from the DT community before the patches can be incorporated.

Could you please kindly review the series here from DT perspective and ACK it if you are OK?
https://lkml.org/lkml/2016/1/11/517

This document here
https://www.kernel.org/doc/linux/MAINTAINERS

tells me that Rob Herring, Frank Rowand and Grant Likely are the maintainers for device tree.

OPEN FIRMWARE AND FLATTENED DEVICE TREE
M: Rob Herring <robh+dt@xxxxxxxxxx>
M: Frank Rowand <frowand.list@xxxxxxxxx>
M: Grant Likely <grant.likely@xxxxxxxxxx>
L: devicetree@xxxxxxxxxxxxxxx

That's why, I'm reaching out.

--
Sinan Kaya
Qualcomm Technologies, Inc. on behalf of Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project