[U-Boot] [PATCH v4 00/11] mtd/sf: Various fixes

Tom Rini trini at konsulko.com
Sun Dec 9 13:36:33 UTC 2018


On Sun, Dec 09, 2018 at 07:42:07AM +0100, Boris Brezillon wrote:
> +Tom for the question about missing SoBs.
> 
> Hi Jagan,
> 
> On Thu, 6 Dec 2018 00:48:47 +0530
> Jagan Teki <jagan at amarulasolutions.com> wrote:
> 
> > On Sun, Dec 2, 2018 at 3:25 PM Boris Brezillon
> > <boris.brezillon at bootlin.com> wrote:
> > >
> > > Hello,
> > >
> > > This is the 4th version of the mtd / sf fixes patchset. This v4 just
> > > adds a new check in del_mtd_device() (and a debug() when
> > > del_mtd_partitions() fails).
> > >
> > > Regards,
> > >
> > > Boris
> > >
> > > P.S.: travis-ci results =>
> > >       https://travis-ci.org/bbrezillon/u-boot/builds/461943011
> > >
> > > Boris Brezillon (11):
> > >   mtd: Add a function to report when the MTD dev list has been updated
> > >   mtd: Parse mtdparts/mtdids again when the MTD list has been updated
> > >   mtd: Delete partitions attached to the device when a device is deleted
> > >   mtd: sf: Make sure we don't register the same device twice
> > >   mtd: Use get_mtdids() instead of env_get("mtdids") in
> > >     mtd_search_alternate_name()
> > >   mtd: Be more strict on the "mtdparts=" prefix check
> > >   mtd: Make sure the name passed in mtdparts fits in mtd_name[]
> > >   mtd: Make sure we don't parse MTD partitions belonging to another dev
> > >   mtd: Don't stop MTD partition creation when it fails on one device
> > >   mtd: sf: Unregister the MTD device prior to removing the spi_flash obj
> > >   mtd: sf: Make sf_mtd.c more robust  
> > 
> > Applied to u-boot-spi/master
> 
> I see that those patches have been merged in mainline, which is great.
> Just have one question: looks like your SoB is missing while you're
> clearly reported as the committer. Since I found the same problem on
> 2614a208471e ("common: command: tempory buffer should have size of
> command line buf"), I wanted to know if this is a common practice in
> u-boot to not add SoBs when maintainers apply patches to their tree?

So yes (and I should put this into that wiki sections where we do differ
from Linux), if you as a custodian aren't making changes to a patch when
applying it then no, you don't add your own SoB line.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20181209/d2119303/attachment.sig>


More information about the U-Boot mailing list