how to pre-screen patch submissions from a newbie janitor?

Artem Bityutskiy dedekind1 at gmail.com
Sun May 23 12:01:27 EDT 2010


On Wed, 2010-05-19 at 07:40 -0400, Robert P. J. Day wrote:
> On Wed, 19 May 2010, Wolfram Sang wrote:
> 
> > >   however, since he's new to patch creation and submission, i
> > > offered to "vet" his patches first.  i realize that my vetting
> > > carries no weight on the MTD list, but given that i've submitted
> > > lots and lots of patches elsewhere, i can still at least sanity
> > > check what he comes up with before it hits the list.
> > >
> > >   is there a canonical signage for that nowadays?  is it
> > > "Reviewed-By" or something like that?  so that by the time it gets
> > > to the MTD list, he would have done the "Signed-Off-By" and i'd
> > > sign off as a reviewer. is that the way it works?  or is there an
> > > alternative?  thanks.
> >
> > Yes, that sounds like a good case for Reviewed-by. It will surely
> > help these patches knowing there is somebody with experience who
> > already had a look at (and in that case probably build-tested) them.
> >
> > Looking forward to the series,
> 
>   what i'm going to have this trainee(?) do is just simplify the
> config/Kconfig/Makefile combos, which i openly admit is not
> earth-shaking in its complexity.
> 
>   for example. in a number of places elsewhere in the kernel tree,
> i've submitted patches to change things like this in the config menus:
> 
> <M>   OneNAND Device Support  --->
>       LPDDR flash memory drivers  --->
>       UBI - Unsorted block images  --->
> 
>   note how, to select either LPDDR or UBI, you need to *go* to those
> submenus, where you find a top-level selector.  it's simpler to just
> put that selector right on that menu item, as it's done for "OneNAND"
> just above it.  no wasting time.

Yeah, and I assume it is better to be consistent and use the same style
as most of things in the MTD menu have.

>   and another simplification is to cut down the conditional inclusion
> in the makefiles.  for instance, here's tests/Makefile:
> 
> obj-$(CONFIG_MTD_TESTS) += mtd_oobtest.o
> obj-$(CONFIG_MTD_TESTS) += mtd_pagetest.o
> obj-$(CONFIG_MTD_TESTS) += mtd_readtest.o
> ... etc etc ...
> 
>   there's little point having every single line in the Makefile test
> CONFIG_MTD_TESTS when that can all be done with a single test one
> level up that checks whether or not to process the tests/ directory at
> all.

Yeah, sounds good.

-- 
Best Regards,
Artem Bityutskiy (Артём Битюцкий)




More information about the linux-mtd mailing list