From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vinod Koul Subject: Re: [PATCH] Improve slave/cyclic DMA engine documentation (was: [PATCH V4 04/14] DMA: PL330: Add DMA_CYCLIC capability) Date: Wed, 27 Jul 2011 08:08:20 +0530 Message-ID: <1311734300.1536.35.camel@vkoul-udesk3> References: <002001cc4ab6$0d7cd590$287680b0$%kim@samsung.com> <20110725103629.GF9653@n2100.arm.linux.org.uk> <1311590884.29703.2.camel@vkoul-mobl4> <20110725105754.GG9653@n2100.arm.linux.org.uk> <1311591717.29703.7.camel@vkoul-mobl4> <20110725113926.GH9653@n2100.arm.linux.org.uk> <1311595303.29703.31.camel@vkoul-mobl4> <20110726075720.GL9653@n2100.arm.linux.org.uk> <1311672933.24316.1.camel@vkoul-mobl4> <20110726172027.GD1655@n2100.arm.linux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-arm-kernel-bounces@lists.infradead.org Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Jassi Brar Cc: linux-samsung-soc@vger.kernel.org, Russell King - ARM Linux , Boojin Kim , vinod.koul@intel.com, Mark Brown , Grant Likely , Kukjin Kim , Dan Williams , linux-arm-kernel@lists.infradead.org List-Id: linux-samsung-soc@vger.kernel.org On Tue, 2011-07-26 at 23:25 +0530, Jassi Brar wrote: > On Tue, Jul 26, 2011 at 10:50 PM, Russell King - ARM Linux > 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. Jassi, Since this was a doc patch, I applied it soon enough, doesn't sound dangerous to me ! If you have any updates you would like, pls feel free to send a patch. > > I just observed it is second time that Vinod applied a patch without any > ack or prior alert. And you can't ignore the fact that you had almost _two_ weeks to ack the patch from Rob, but.... You will get ample time to ack a patch, if you don't you can always point out and I will do the right thing, which in this case was to revert. > > > > >> 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. > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel From mboxrd@z Thu Jan 1 00:00:00 1970 From: vkoul@infradead.org (Vinod Koul) Date: Wed, 27 Jul 2011 08:08:20 +0530 Subject: [PATCH] Improve slave/cyclic DMA engine documentation (was: [PATCH V4 04/14] DMA: PL330: Add DMA_CYCLIC capability) In-Reply-To: References: <002001cc4ab6$0d7cd590$287680b0$%kim@samsung.com> <20110725103629.GF9653@n2100.arm.linux.org.uk> <1311590884.29703.2.camel@vkoul-mobl4> <20110725105754.GG9653@n2100.arm.linux.org.uk> <1311591717.29703.7.camel@vkoul-mobl4> <20110725113926.GH9653@n2100.arm.linux.org.uk> <1311595303.29703.31.camel@vkoul-mobl4> <20110726075720.GL9653@n2100.arm.linux.org.uk> <1311672933.24316.1.camel@vkoul-mobl4> <20110726172027.GD1655@n2100.arm.linux.org.uk> Message-ID: <1311734300.1536.35.camel@vkoul-udesk3> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, 2011-07-26 at 23:25 +0530, Jassi Brar wrote: > On Tue, Jul 26, 2011 at 10:50 PM, Russell King - ARM Linux > 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. Jassi, Since this was a doc patch, I applied it soon enough, doesn't sound dangerous to me ! If you have any updates you would like, pls feel free to send a patch. > > I just observed it is second time that Vinod applied a patch without any > ack or prior alert. And you can't ignore the fact that you had almost _two_ weeks to ack the patch from Rob, but.... You will get ample time to ack a patch, if you don't you can always point out and I will do the right thing, which in this case was to revert. > > > > >> 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. > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel at lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel