public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Miquel Raynal <miquel.raynal@bootlin.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH V3] mtd: uboot: Fix hanging during mtd list command
Date: Wed, 10 Oct 2018 15:14:07 +0200	[thread overview]
Message-ID: <20181010151407.71b67c31@xps13> (raw)
In-Reply-To: <CAMty3ZDGQ5xgDmc8n_LsuzbzHGsStAY1yDm=cXY79X-5zLcG_g@mail.gmail.com>

Hi Jagan,

Jagan Teki <jagan@amarulasolutions.com> wrote on Wed, 10 Oct 2018
18:41:39 +0530:

> On Wed, Oct 10, 2018 at 6:39 PM Miquel Raynal <miquel.raynal@bootlin.com> wrote:
> >
> > Hi Jagan,
> >
> > Jagan Teki <jagan@amarulasolutions.com> wrote on Wed, 10 Oct 2018
> > 11:32:02 +0530:
> >  
> > > On Tue, Oct 9, 2018 at 12:43 AM Adam Ford <aford173@gmail.com> wrote:  
> > > >
> > > > Some boards (like omap3_logic) hang when trying to access
> > > > address 0. This happens when executing the new 'mtd list' command.
> > > > This patch enhances the checks for conditions that would
> > > > preclude mtd_probe_devices() from operating.
> > > >
> > > > Fixes: 5db66b3aee6f ("cmd: mtd: add 'mtd' command")
> > > > Suggested-by: Boris Brezillon <boris.brezillon@bootlin.com>
> > > > Signed-off-by: Adam Ford <aford173@gmail.com>
> > > > Reviewed-by: Boris Brezillon <boris.brezillon@bootlin.com>
> > > > ---  
> > >
> > > Applied to u-boot-spi/master  
> >
> > Thanks for taking the patch but I wonder: shouldn't we apply this patch
> > sooner, ie. in the next -rc? Actually, it has barely nothing to do with
> > the SPI subsystem anyway.  
> 
> Yes, will send the PR soon.


Ok, no problem then. Thanks for carrying it.

Thanks,
Miquèl

      reply	other threads:[~2018-10-10 13:14 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-10-08 19:13 [U-Boot] [PATCH V3] mtd: uboot: Fix hanging during mtd list command Adam Ford
2018-10-08 20:02 ` Miquel Raynal
2018-10-10  6:02 ` Jagan Teki
2018-10-10 13:09   ` Miquel Raynal
2018-10-10 13:11     ` Jagan Teki
2018-10-10 13:14       ` Miquel Raynal [this message]

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=20181010151407.71b67c31@xps13 \
    --to=miquel.raynal@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