From: "Robert P. J. Day" <rpjday@crashcourse.ca>
To: Sascha Hauer <s.hauer@pengutronix.de>
Cc: "U-Boot Version 2 (barebox)" <barebox@lists.infradead.org>
Subject: Re: [PATCH] commands: Remove reference to non-existent CONFIG_CMD_I2C.
Date: Mon, 21 Dec 2009 06:33:17 -0500 (EST) [thread overview]
Message-ID: <alpine.LFD.2.00.0912210631160.13694@localhost> (raw)
In-Reply-To: <20091221113052.GG15126@pengutronix.de>
On Mon, 21 Dec 2009, Sascha Hauer wrote:
> On Mon, Dec 21, 2009 at 06:17:19AM -0500, Robert P. J. Day wrote:
> >
> > Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
> >
> > ---
> >
> > the commands directory doesn't even have an i2c.c source file, so
> > it's clearly safe to remove this reference. although i'm assuming
> > that, at some point down the road, there will be an "i2c" command
> > since that would be useful.
>
> Ok, applied.
>
> Is a i2c command really useful? For I2C eeproms you would create an
> epprom driver which creates a file under /dev/ which you can then
> access using the usual mm/mw commands. Maybe an i2c command could be
> useful to register a new device on an I2C bus.
i'm not sure -- maybe it *isn't* all that useful. if at some point
it becomes useful, we can always put this back in. from the original
u-boot, i recall being able to at least run "i2c probe" or something
like that.
in any event, that doesn't exist now so no big deal.
rday
--
========================================================================
Robert P. J. Day Waterloo, Ontario, CANADA
Linux Consulting, Training and Kernel Pedantry.
Web page: http://crashcourse.ca
Twitter: http://twitter.com/rpjday
========================================================================
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
next prev parent reply other threads:[~2009-12-21 11:33 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-12-21 11:17 [PATCH] commands: Remove reference to non-existent CONFIG_CMD_I2C Robert P. J. Day
2009-12-21 11:30 ` Sascha Hauer
2009-12-21 11:33 ` Robert P. J. Day [this message]
2009-12-21 11:40 ` Peter Korsgaard
2009-12-21 11:47 ` Robert P. J. Day
2009-12-21 11:54 ` Peter Korsgaard
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=alpine.LFD.2.00.0912210631160.13694@localhost \
--to=rpjday@crashcourse.ca \
--cc=barebox@lists.infradead.org \
--cc=s.hauer@pengutronix.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.