From: Brian Foster <brian.foster@maxim-ic.com>
To: Brian Norris <computersforpeace@gmail.com>
Cc: "linux-mtd@lists.infradead.org" <linux-mtd@lists.infradead.org>,
Artem Bityutskiy <dedekind1@gmail.com>
Subject: Re: [PATCH 2/2] ubinfo: fix `--all' for non-consecutive device numbers
Date: Mon, 30 Jan 2012 09:32:13 +0100 [thread overview]
Message-ID: <201201300932.14012.brian.foster@maxim-ic.com> (raw)
In-Reply-To: <CAN8TOE9hvxBGG40P9RV=-1r9m3wrYGOnBUFvBmZk3nn-8=RAnw@mail.gmail.com>
On Friday 27 January 2012 19:28:59 Brian Norris wrote:
> On Thu, Jan 26, 2012 at 12:53 AM, Brian Foster <brian.foster@maxim-ic.com> wrote:
> > [ ... ] looking at the mtd-utils v1.4.6 ‘mtdinfo.c’,
> > is there a similar issue there? But I have no idea
> > how you could get non-consecutive MTD device numbers?
>
> Yeah, there is a problem there. It's a little more difficult to see,
> but it can happen.
>
> Suppose you have /dev/mtd0 on your system, then you can get an error
> with the following sequence:
>
> # modprobe mtdram
> # modprobe nandsim
> # rmmod mtdram
> # mtdinfo --all
> ...
> mtdinfo: error!: mtd1 does not correspond to any existing MTD device
Ah, Ok! I had a similar thought over the weekend,
and before seeing your e-mail tried to set-up the
situation up on one of our SoC evaluation boards,
using our H/W drivers. But for some reason those
drivers won't unload — I should look into that —
and then saw your e-mails ....
> I'm sending a quick patch set for this.
Yeah, saw it, seemed Ok to me. Thanks.
cheers!
-blf-
--
Brian Foster
Principal MTS, Software | La Ciotat, France
Maxim Integrated Products | Web: http://www.maxim-ic.com/
next prev parent reply other threads:[~2012-01-30 8:33 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-01-26 7:31 [PATCH 1/2] libubi: make `ubi_dev_present()' a library function Brian Norris
2012-01-26 7:31 ` [PATCH 2/2] ubinfo: fix `--all' for non-consecutive device numbers Brian Norris
2012-01-26 8:53 ` Brian Foster
2012-01-27 18:28 ` Brian Norris
2012-01-30 8:32 ` Brian Foster [this message]
2012-01-27 14:51 ` [PATCH 1/2] libubi: make `ubi_dev_present()' a library function Artem Bityutskiy
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=201201300932.14012.brian.foster@maxim-ic.com \
--to=brian.foster@maxim-ic.com \
--cc=computersforpeace@gmail.com \
--cc=dedekind1@gmail.com \
--cc=linux-mtd@lists.infradead.org \
/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