public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Boris Brezillon <boris.brezillon@bootlin.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v4 00/11] mtd/sf: Various fixes
Date: Sun, 9 Dec 2018 07:42:07 +0100	[thread overview]
Message-ID: <20181209074207.37259968@bbrezillon> (raw)
In-Reply-To: <CAMty3ZA-OmC_m-kadmDPhQJnE1EVr2NBvne5ErS2uTAdvsNocw@mail.gmail.com>

+Tom for the question about missing SoBs.

Hi Jagan,

On Thu, 6 Dec 2018 00:48:47 +0530
Jagan Teki <jagan@amarulasolutions.com> wrote:

> On Sun, Dec 2, 2018 at 3:25 PM Boris Brezillon
> <boris.brezillon@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?

Regards,

Boris

  reply	other threads:[~2018-12-09  6:42 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-12-02  9:54 [U-Boot] [PATCH v4 00/11] mtd/sf: Various fixes Boris Brezillon
2018-12-02  9:54 ` [U-Boot] [PATCH v4 01/11] mtd: Add a function to report when the MTD dev list has been updated Boris Brezillon
2018-12-08 23:40   ` Lukasz Majewski
2018-12-02  9:54 ` [U-Boot] [PATCH v4 02/11] mtd: Parse mtdparts/mtdids again when the MTD " Boris Brezillon
2018-12-02  9:54 ` [U-Boot] [PATCH v4 03/11] mtd: Delete partitions attached to the device when a device is deleted Boris Brezillon
2018-12-02  9:54 ` [U-Boot] [PATCH v4 04/11] mtd: sf: Make sure we don't register the same device twice Boris Brezillon
2018-12-02  9:54 ` [U-Boot] [PATCH v4 05/11] mtd: Use get_mtdids() instead of env_get("mtdids") in mtd_search_alternate_name() Boris Brezillon
2018-12-02  9:54 ` [U-Boot] [PATCH v4 06/11] mtd: Be more strict on the "mtdparts=" prefix check Boris Brezillon
2018-12-02  9:54 ` [U-Boot] [PATCH v4 07/11] mtd: Make sure the name passed in mtdparts fits in mtd_name[] Boris Brezillon
2018-12-02  9:54 ` [U-Boot] [PATCH v4 08/11] mtd: Make sure we don't parse MTD partitions belonging to another dev Boris Brezillon
2018-12-02  9:54 ` [U-Boot] [PATCH v4 09/11] mtd: Don't stop MTD partition creation when it fails on one device Boris Brezillon
2018-12-02  9:54 ` [U-Boot] [PATCH v4 10/11] mtd: sf: Unregister the MTD device prior to removing the spi_flash obj Boris Brezillon
2018-12-02  9:54 ` [U-Boot] [PATCH v4 11/11] mtd: sf: Make sf_mtd.c more robust Boris Brezillon
2018-12-05 11:02 ` [U-Boot] [PATCH v4 00/11] mtd/sf: Various fixes Boris Brezillon
2018-12-05 19:18 ` Jagan Teki
2018-12-09  6:42   ` Boris Brezillon [this message]
2018-12-09 13:36     ` Tom Rini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20181209074207.37259968@bbrezillon \
    --to=boris.brezillon@bootlin.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox