[PATCH] Improve slave/cyclic DMA engine documentation (was: [PATCH V4 04/14] DMA: PL330: Add DMA_CYCLIC capability)

Jassi Brar jassisinghbrar at gmail.com
Tue Jul 26 13:55:13 EDT 2011


On Tue, Jul 26, 2011 at 10:50 PM, Russell King - ARM Linux
<linux at arm.linux.org.uk> wrote:
> On Tue, Jul 26, 2011 at 08:07:44PM +0530, Jassi Brar wrote:
>> Dear Vinod,
>>
>>  Since it came from the RMK, most probably it'll be the best.
>>
>> But applying patches upon personal timeout seems very dangerous.
>
> Ehh what?  Is there any contention over this documentation patch?
I haven't yet read it... even after reading I would object only if I found
your patch disturbing enough to disrupt my bowel movements. Which
I don't think would be the case.

I just observed it is second time that Vinod applied a patch without any
ack or prior alert.

>
>> People not responding doesn't mean only either people agree completely
>> or they don't care. Some might be interested but too busy with current tasks
>> that they need time to check... please make some policy for such cases.
>>
>> It already happened with the patch from Rob, which you probably have to
>> revert.
>>
>> IMHO, if nobody replied, maybe you could first ack the patch and wait
>> for, say a week, before applying?
>> That way people will know they have to hurry if they care otherwise
>> the patch is going upstream as such.
>
> A week is far too long.  That's how patches get lost and missed.
>
He may decide to wait shorter, but imho a week after the first ack
isn't that long.



More information about the linux-arm-kernel mailing list