From: Robert Schwebel <r.schwebel@pengutronix.de>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] [Patch 0/9] U-boot-V2: Introduce I2C support for SDP3430
Date: Wed, 18 Jun 2008 22:44:07 +0200 [thread overview]
Message-ID: <20080618204407.GR1189@pengutronix.de> (raw)
In-Reply-To: <7A436F7769CA33409C6B44B358BFFF0CD3193101@dlee02.ent.ti.com>
On Wed, Jun 18, 2008 at 03:21:02PM -0500, Menon, Nishanth wrote:
> > - Please don't bring in things which are not part of the current code
> > base. If you add code for subsystems, the subsystems must be there
> > first and *then* their users. Otherwhise we bloat the codebase and we
> > disable the possibility to add *real* support for PCI, USB and S390
> > later :-)
>
> Acked. Yep will redo that :).
Yea, looking forward especially for PCI patches :-)
> > Which part of i2c do we actually *need* in the bootloader? What's
> > *your* use case?
>
> I need i2c_transfer -yes - I can customize the changes just for that.
> I do like i2c_transfers, adapters, clients etc.. my immediate need is
> rather simple -> i2c_transfers.
Ok, so let's start with that.
> > I assume we need i2c host controllers, that's quite clear. We also need
> > an abstraction for i2c devices. The only devices I have seen which
> > really needed a bootloader driver are
> >
> > a) eeproms
> > b) RTCs
> > c) analog/digital in/outputs (i.e. gpio expanders)
> >
> > Other use cases?
>
> USB Transceivers, power management devices.
Yes, but I think what we most of all need is the i2c communication with
the device. We should think about if the abstraction of the devices
*behind* the bus brings us anything; note that you can do lots of things
in scripts in u-boot-v2. Linke writing a shell function for your usb
transceivers which just encapsulates the right predefined i2c_read/write
operations.
> Are we ok with i2ctools movement here?
Hmm? How do you mean that?
rsc
--
Dipl.-Ing. Robert Schwebel | http://www.pengutronix.de
Pengutronix - Linux Solutions for Science and Industry
Handelsregister: Amtsgericht Hildesheim, HRA 2686
Hannoversche Str. 2, 31134 Hildesheim, Germany
Phone: +49-5121-206917-0 | Fax: +49-5121-206917-9
next prev parent reply other threads:[~2008-06-18 20:44 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-06-18 12:33 [U-Boot-Users] [Patch 0/9] U-boot-V2: Introduce I2C support for SDP3430 Menon, Nishanth
2008-06-18 14:55 ` Timur Tabi
2008-06-18 15:15 ` Menon, Nishanth
2008-06-18 15:28 ` Sascha Hauer
2008-06-18 15:40 ` Menon, Nishanth
2008-06-18 15:39 ` Timur Tabi
2008-06-18 15:52 ` Menon, Nishanth
2008-06-18 15:58 ` Sascha Hauer
2008-06-18 16:04 ` Menon, Nishanth
2008-06-18 18:38 ` Robert Schwebel
2008-06-18 18:48 ` Menon, Nishanth
2008-06-18 19:11 ` Robert Schwebel
2008-06-18 20:21 ` Menon, Nishanth
2008-06-18 20:44 ` Robert Schwebel [this message]
2008-06-18 20:52 ` Menon, Nishanth
2008-06-19 15:10 ` Menon, Nishanth
2008-06-19 15:11 ` [U-Boot-Users] [Patch 1/9]U-boot-V2:ID: Add i2c_device_id Menon, Nishanth
2008-06-19 15:11 ` [U-Boot-Users] [Patch 2/9]U-boot-V2:ID: Introduce idr Menon, Nishanth
2008-06-19 15:11 ` [U-Boot-Users] [Patch 3/9]U-boot-V2:I2C: Introduce core Menon, Nishanth
2008-06-19 15:12 ` [U-Boot-Users] [Patch 4/9]U-boot-V2:I2C: Introduce i2c-dev Menon, Nishanth
2008-06-19 15:12 ` [U-Boot-Users] [Patch 5/9]U-boot-V2:cmd: add I2C commands Menon, Nishanth
2008-06-19 20:33 ` Wolfgang Denk
2008-06-20 2:10 ` Menon, Nishanth
2008-06-20 7:28 ` Wolfgang Denk
2008-06-20 10:28 ` Sascha Hauer
2008-06-20 13:35 ` Menon, Nishanth
2008-06-20 15:16 ` Sascha Hauer
2008-06-20 16:28 ` Menon, Nishanth
2008-06-23 7:39 ` Sascha Hauer
2008-06-23 13:29 ` Menon, Nishanth
2008-06-21 2:34 ` [U-Boot-Users] [Patch 4/9 Try 2]U-boot-V2:I2C: Introduce i2c-dev Menon, Nishanth
2008-06-24 8:05 ` Sascha Hauer
2008-06-25 1:48 ` [U-Boot-Users] [Patch 4/9 Try 3]U-boot-V2:I2C: " Menon, Nishanth
2008-06-21 2:34 ` [U-Boot-Users] [Patch 5/9 Try2]U-boot-V2:cmd: add I2C commands Menon, Nishanth
2008-06-21 7:23 ` Wolfgang Denk
2008-06-22 18:56 ` [U-Boot-Users] [Patch 5/9 Try 3]U-boot-V2:cmd: " Menon, Nishanth
2008-06-24 6:18 ` Sascha Hauer
2008-06-25 1:46 ` [U-Boot-Users] [Patch 5/9 Try 4]U-boot-V2:cmd: " Menon, Nishanth
2008-06-19 15:12 ` [U-Boot-Users] [Patch 6/9]U-boot-V2:cmd: I2C commands support Menon, Nishanth
2008-06-19 15:13 ` [U-Boot-Users] [Patch 7/9]U-boot-V2:ARM:OMAP: OMAP classes Menon, Nishanth
2008-06-19 15:13 ` [U-Boot-Users] [Patch 8/9]U-boot-V2:I2C:Busses: OMAP I2C Adapter Menon, Nishanth
2008-06-19 15:13 ` [U-Boot-Users] [Patch 9/9]U-boot-V2:Boards:SDP3430 I2C bus1 Menon, Nishanth
2008-06-18 18:41 ` [U-Boot-Users] [Patch 0/9] U-boot-V2: Introduce I2C support for SDP3430 Robert Schwebel
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=20080618204407.GR1189@pengutronix.de \
--to=r.schwebel@pengutronix.de \
--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 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.