| Message ID | alpine.DEB.2.00.1207261349350.14468@utopia.booyaka.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Hi Paul, On 07/26/2012 09:55 PM, Paul Walmsley wrote: > > Hello Michael, > > thanks for the patch. I've queued it for 3.7 after writing a changelog. > Two minor comments for future patches: we don't accept patches with a > non-null changelog - usually just something brief works fine. Also Tony > wants all OMAP patches to be cc'ed to the linux-arm-kernel mailing list. > > regards, > > - Paul Thanks for the comments, I'll adhere to them in the future. I assume you meant you _only_ accept patches with a non-null changelog. I referred to http://omappedia.org/wiki/Releasing_to_Linux_kernel_using_patches_and_emails before my submission, and didn't see these two points. A few Q's: Can I add these two points to that Wiki page? Or is there a more authoritative checklist somewhere? Why is it queued for 3.7? I thought 3.6 was in the works? Where is it queued? thanks, Michael MATRIX VISION GmbH, Talstrasse 16, DE-71570 Oppenweiler Registergericht: Amtsgericht Stuttgart, HRB 271090 Geschaeftsfuehrer: Gerhard Thullner, Werner Armingeon, Uwe Furtner, Erhard Meier
Hello Michael, On Fri, 27 Jul 2012, Michael Jones wrote: > Thanks for the comments, I'll adhere to them in the future. I assume you > meant you _only_ accept patches with a non-null changelog. Indeed, that's right, I misstated it. > I referred to > http://omappedia.org/wiki/Releasing_to_Linux_kernel_using_patches_and_emails > before my submission, and didn't see these two points. A few Q's: > > Can I add these two points to that Wiki page? Sounds reasonable, but I don't know who maintains that page. > Or is there a more authoritative checklist somewhere? There is; it's Documentation/SubmittingPatches which can be viewed here: http://git.kernel.org/?p=linux/kernel/git/torvalds/linux.git;a=blob;f=Documentation/SubmittingPatches > Why is it queued for 3.7? It is a low-priority fix (but valuable nonetheless, of course). > I thought 3.6 was in the works? Most of the significant new patches for 3.6 were tested and queued several weeks ago. Now we are queuing bugfix and regression patches for 3.6-rc and new features and low-priority fixes for 3.7. > Where is it queued? It is currently queued in a git branch on my local drive. - Paul
Hi Paul, Thanks for the explanations. On 07/27/2012 09:14 AM, Paul Walmsley wrote: > Hello Michael, > [snip] > There is; it's Documentation/SubmittingPatches which can be viewed here: > > http://git.kernel.org/?p=linux/kernel/git/torvalds/linux.git;a=blob;f=Documentation/SubmittingPatches I looked at that, too, but it wasn't clear to me that even such a trivial patch should have an "explanation body", as the description there assumes a functional change is being introduced. Now I know! [snip] > > - Paul > -Michael MATRIX VISION GmbH, Talstrasse 16, DE-71570 Oppenweiler Registergericht: Amtsgericht Stuttgart, HRB 271090 Geschaeftsfuehrer: Gerhard Thullner, Werner Armingeon, Uwe Furtner, Erhard Meier
diff --git a/arch/arm/mach-omap2/clock3xxx_data.c b/arch/arm/mach-omap2/clock3xxx_data.c index 1efdec2..025b5d9 100644 --- a/arch/arm/mach-omap2/clock3xxx_data.c +++ b/arch/arm/mach-omap2/clock3xxx_data.c @@ -3223,7 +3223,6 @@ static struct clk dummy_apb_pclk = { * clkdev */ -/* XXX At some point we should rename this file to clock3xxx_data.c */ static struct omap_clk omap3xxx_clks[] = { CLK(NULL, "apb_pclk", &dummy_apb_pclk, CK_3XXX), CLK(NULL, "omap_32k_fck", &omap_32k_fck, CK_3XXX),