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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox