[PATCH 4/6] SPI: S3C64XX: Check before mem-region release

Grant Likely grant.likely at secretlab.ca
Fri Jan 22 02:00:11 EST 2010


On Wed, Jan 20, 2010 at 8:06 PM, jassi brar <jassisinghbrar at gmail.com> wrote:
> On Thu, Jan 21, 2010 at 11:55 AM, Grant Likely
> <grant.likely at secretlab.ca> wrote:
>> On Wed, Jan 20, 2010 at 6:46 PM, jassi brar <jassisinghbrar at gmail.com> wrote:
>>> On Thu, Jan 21, 2010 at 10:35 AM, jassi brar <jassisinghbrar at gmail.com> wrote:
>>>> Hi Grant Likely,
>>>>  You have reviewed this series of patches but didn't ACK/NAK.
>>>> Could you kindly confirm if u accept them or want some changes?
>>>> Thanks.
>>> Nevermind, I just saw ur ACK at my official ID (many maintainers reply
>>> something like
>>> 'pulled now' to the patch thread but thats ok if you ACK personally).
>>> Thanks.
>>
>> I'm using patchwork.kernel.org for collecting spi patches.
>> Unfortunately patchwork doesn't provide any mechanism for emailing
>> "merged" messages when I accept patches.  To compensate, I've started
>> cc'ing patch authors my next- branch update message when I push out an
>> updated tree.
>>
>> You can also always look at patchwork.kernel.org to see if your
>> patches are still in the queue.  I won't forget accidentally drop any
>> patches tracked by patchwork.
> well, it might not just be the submitter, but also others in the thread
> interested in knowing the fate of a particular patch. And having to
> poll some repo for that, isn't very efficient. It's not important to be notified
> _when_ you accept the patch, but only _if_ you accept, which I think
> you would know after seeing the patch and/or some further clarifications
> from the submitter.

Since I review patches on patchwork, I don't have an easy link between
what I'm picking up and my email client.  Only when I need to comment
on something will I actually go and find the thread in my mailer.  So
it is considerably less efficient for me to go and reply to every
patch submission.

Jeremy, would it be possible to have a way of sending out a status
update email from within patchwork?

Thanks,
g.

-- 
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.


More information about the Patchwork mailing list