Re: [PATCH V6 2/3] dma: add Qualcomm Technologies HIDMA management driver

From: Sinan Kaya
Date: Sun Nov 22 2015 - 21:31:47 EST


>>>
>>> Can it be like
>>>
>>> /sys/â/DEVICExx/
>>> channelYY/
>>> attr1
>>> attr2
>>> â
>>>
>>> ?
>>
>> I'll work on it. I didn't know that you are allowed to create subdirectories
>> in sysfs. I was just creating attributes to keep it simple. But, your
>> suggestion is cleaner.
>>
>>>
>>> I think it will be easier to handle in code and from user. (Similar
>>> way DMAEngine API does for slave DMA devices)
>>
>> Now, the good stuff. Can you clarify your comment? I didn't understand it.
>
> I meant that DMAEngine uses
> /sys/class/dma
> dmaYchannelX/
> attr1
> attr2
> â
>
> layout.
>

I have posted v7 with this change.


--
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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/