public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Timur Tabi <timur@freescale.com>
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 10:39:30 -0500	[thread overview]
Message-ID: <48592C32.60900@freescale.com> (raw)
In-Reply-To: <7A436F7769CA33409C6B44B358BFFF0CD3192D75@dlee02.ent.ti.com>

Menon, Nishanth wrote:

> Please see U-boot v2 here: 

Sorry, I didn't notice that your patch is for U-Boot V2.

> It does not have i2c support. And IMHO, using kernel like i2c addition in U-Boot v2 is appropriate. Why this is not a bloat U-Boot v2?

If it isn't, then it means that U-Boot V2 will be pretty bloated in general.

> A) Now, if you just want i2c_transfer without any device interface, you don't need to compile in the CHARDEV.
> B) Since we use a different linking strategy in U-Boot-v2 which uses gcc link stage to decide which functions be retained and which not, it is pretty neat to see all unused functionality disappear.
> C) kernel has additional bus and class types which are useful when we think in terms of varied features, in u-boot v2, we don't have a bus and class, just a plain device+driver interface like the kernel + ioctls to allow "apps" to invoke functionality within the driver for the device. Very linux like, call it a micro-linux ;). So kernel i2c architecture fits u_boot v2 better that u-boot v1's architecture.
> D) U-boot v2 has insmod + rmmod -> though this is still to be fixed, once it is fully functioning, you can loadb just the module feature you need. In fact, you could even have a filesystem with just modules....
> E) Ofcourse, we work in a single threaded, no interrupt mode in u-boot v2+ don't care for any power management -> more savings there

Sounds to me like U-Boot V2 is just a repacked Linux kernel.  We could probably
save ourselves a lot of trouble if we just took the DDR and CPU register
initialization code in U-Boot and moved it to the kernel.

-- 
Timur Tabi
Linux kernel developer at Freescale

  parent reply	other threads:[~2008-06-18 15:39 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 [this message]
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
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=48592C32.60900@freescale.com \
    --to=timur@freescale.com \
    --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