From: "Ian S. Nelson" <ian.nelson@echostar.com>
To: David Woodhouse <dwmw2@infradead.org>,
"mtd@infradead.org" <mtd@infradead.org>
Subject: Re: Proc stuff
Date: Thu, 05 Apr 2001 10:08:14 -0600 [thread overview]
Message-ID: <3ACC986E.DE2F4C61@echostar.com> (raw)
In-Reply-To: 27773.986474836@redhat.com
David Woodhouse wrote:
> ian.nelson@echostar.com said:
> > I need some extra bits added to the proc/mtd. For the product I'm
> > building we want to have things from the flash registers like the
> > Device ID and "AMD flash"
>
> Those aren't particularly generic bits of information - why do you think
> they should be present at that level? That kind of detail is reported in the
> boot messages, not in /proc/mtd. Does your eth0 device provide a way to find
> out which chipset it is?
Actually, it does now. I thought about parsing the log or dmesg but we're
turning logging off in the product because it's unneeded disk I/O and disk I/O
is critical. I was looking at the IDE directory under proc and there is an
entry for your southbridge/IDE controller and some other specific things so
that I can dig around in /proc/ide and know exactly what drives and
IDE controllers I've got on the system.
> Perhaps we could make the CFI drivers put slightly more informative
> information in the 'name' field, but that doesn't work nicely if you have
> partitions which override the original name.
We could, I was just offering to try and do a complete job if someone else
wants or needs anything in there. During our manufacturing test we need to
specifically identify each and every part of our hardware and so I've been
stuffing that into /proc. I will do it that way for our particular kernel and
code but if I can write something that will be helpful to others I want to do
that.
Maybe the solution is to create a /proc/mtd directory and then have a few
different files in there which can either be generic or specific. There are
other places in the kernel that do this so it's un unprecedented. /proc is
nice and easy to read so you can build scripts and simple things to read it.
An ioctl could also work.
Ian
To unsubscribe, send "unsubscribe mtd" to majordomo@infradead.org
next prev parent reply other threads:[~2001-04-05 16:09 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-04-04 22:48 Proc stuff Ian S. Nelson
2001-04-05 12:47 ` David Woodhouse
2001-04-05 14:41 ` Johan Adolfsson
2001-04-05 14:50 ` David Woodhouse
2001-04-05 16:08 ` Ian S. Nelson [this message]
2001-04-05 16:33 ` David Woodhouse
2001-04-05 20:09 ` Eric W. Biederman
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=3ACC986E.DE2F4C61@echostar.com \
--to=ian.nelson@echostar.com \
--cc=dwmw2@infradead.org \
--cc=mtd@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