On Tue, 22 Jan 2013, Anton Vorontsov wrote: > On Tue, Jan 22, 2013 at 09:27:17AM +0000, Lee Jones wrote: > > On Mon, 21 Jan 2013, Anton Vorontsov wrote: > > > > > On Mon, Jan 21, 2013 at 12:03:59PM +0000, Lee Jones wrote: > > > > When compile testing the new AB8500 Battery Management changes > > > > due for inclusion into upstream, there were a few minor niggles > > > > which required repairing, or adapting for use against the > > > > Mainline kernel. This patch is a collection of them all. > > > > > > No. This is the third time I'm saying it: the last time I checked, this is > > > not how we do development in the mainline. > > > > Okay, all of these changes have now been fixed-up into the patches > > which caused the issues. I have also added Stable to the patch you > > requested it on. > > > > How would you like to proceed? Do you want a pull-request, or for me > > to send the patches to the list again? > > Either way will work. > > Thanks! > > Anton > > p.s. I took another look at the patches, and noticed one issue that is > constantly there: you place your signed-off-by line on top of everyones', > but that's wrong. Sign offs are going downwards, not upwards. This is > quite important since the lines encode how the code "travels" between > companies and submitters. Currently, one might assume that the origin of > the code is always Linaro, which is not quite true. (And 'From:' doesn't > always mean the author, fwiw.)
Thanks for the info. I'll move them all and re-submit. -- Lee Jones Linaro ST-Ericsson Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
