Hi Sam,
Thanks for the clarification, I'll send the divided patches next week.
-- Greetings Konrad
Sam Ravnborg wrote:
On Fri, Jun 05, 2009 at 11:40:04AM +0200, Konrad Eisele wrote:
Hi Sam,
If you want this reviewed then please do as asked and send a set of
individual mails
each containing one logical change - with a description what it contains.
I (and others too) do not go hunting to find patches to review.
Ok, I can do this (next week). This would be 20 mails for 20 files. Is
everybody on the list ok with it?
Yes they are.
>From the list below I can see you have a logical split of the changes.
So you just have to add the corresponding patch too.
If you did this before than document what was changed since then, which
review comments was addressed and especially which was not (and why).
Didnt understand this. There was nothing sent previously.
So please inline patches - with good changelogs.
Ok, I'll send them next.
Try to send one to yourself first and check that you can properly apply the
patch in the mail you receive. This is a good check that your mailer does
not destroy the patch.
I'm not familiar with how patches get applied. Can someone describe
who actually is checking in a patch so that it appears in the final kernel?
Is it you (Sam)?
Is this patch sending only a "review" or do they get checked in afterwards?
If it is only a "review" who is then the one that actually checks them in?
For sparc davem would apply them.
I guees he prefers someone to address all the easy stuff before he starts
looks at them. This is where I and other helps.
Looking forward to see the patches.
Sam
--
To unsubscribe from this list: send the line "unsubscribe sparclinux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
