All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anton Vorontsov <avorontsov@ru.mvista.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3 0/8] hwconfig and some its users
Date: Thu, 11 Jun 2009 21:01:23 +0400	[thread overview]
Message-ID: <20090611170123.GA7533@oksana.dev.rtsoft.ru> (raw)
In-Reply-To: <20090611105533.7aeec1ee.kim.phillips@freescale.com>

On Thu, Jun 11, 2009 at 10:55:33AM -0500, Kim Phillips wrote:
[...]
> if you don't want to type, things like this are possible but they
> have to depend on the order given:
> 
> dr_usb.mode = host; .phy_type = ulpi; esdhc;
> 
> however when automating/scripting concatenation of them, it's useful
> to not have to depend on their order:
> 
> dr_usb.mode = host; esdhc; dr_usb.phy_type = ulpi;
> 
> ...so as you can see I've come up with the dot ('.') in order to
> eliminate the less familiar (and therefore more misleading) colon
> (':').
> 
> What do you think?

"dr_usb.mode = host; .phy_type = ulpi; esdhc;" <- that isn't obvious
that .phy_type is a continuation of dr_usb. :-/

It's kind of more special format then the original...

> Kim
> 
> p.s., your representation above is the best, but now it's starting to
> look like the frontend to a C compiler:
> 
> "dr_usb { mode = host; phy_type = ulpi; }; esdhc;"

I like it too:
http://lists.denx.de/pipermail/u-boot/2009-May/051875.html

Wolfgang replied that we should keep things simple, and if we
want something complicated we should use device-tree:
http://lists.denx.de/pipermail/u-boot/2009-May/051886.html

Since hwconfig is designed to be *simple* hw configuration
interface, I tend to agree with Wolfgang.

Technically we don't need hwconfig, it is only needed to make
configuration easier for reference boards that have quite a
lot of hw options. Hwconfig can be substituted by

# mw 0x... 0x... # manually modify some bcsrs
# mw 0x... 0x... # manually modify pinmux
# fdt addr ... # now manually fixup the device tree
# fdt set ... ...
# fdt mknode ... ...


In a final product you barely need the hwconfig stuff, because
product boards only define things they use (and that's why we
must keep things working without CONFIG_HWCONFIG).

Thanks,

-- 
Anton Vorontsov
email: cbouatmailru at gmail.com
irc://irc.freenode.net/bd2

  parent reply	other threads:[~2009-06-11 17:01 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-09 20:23 [U-Boot] [PATCH v3 0/8] hwconfig and some its users Anton Vorontsov
2009-06-09 20:25 ` [U-Boot] [PATCH 1/8] Add simple hwconfig infrastructure Anton Vorontsov
2009-07-16 20:26   ` Wolfgang Denk
2009-06-09 20:25 ` [U-Boot] [PATCH 2/8] fsl_esdhc: Add device tree fixups Anton Vorontsov
2009-07-16 20:27   ` Wolfgang Denk
2009-06-09 20:25 ` [U-Boot] [PATCH 3/8] mpc83xx: MPC837XERDB: Add support for FSL eSDHC Anton Vorontsov
2009-07-16 20:27   ` Wolfgang Denk
2009-06-09 20:25 ` [U-Boot] [PATCH 4/8] mpc83xx: MPC837XEMDS: Fixup eSDHC nodes in device tree Anton Vorontsov
2009-07-16 20:27   ` Wolfgang Denk
2009-06-09 20:25 ` [U-Boot] [PATCH 5/8] fdt_support, usb: Move fdt_fixup_dr_usb routine to drivers/usb/otg Anton Vorontsov
2009-07-16 20:44   ` Wolfgang Denk
2009-07-16 20:45   ` Wolfgang Denk
2009-06-09 20:25 ` [U-Boot] [PATCH 6/8] fsl_dr_usb: Fixup disabled USB controllers nodes in device tree Anton Vorontsov
2009-07-16 20:46   ` Wolfgang Denk
2009-06-09 20:25 ` [U-Boot] [PATCH 7/8] mpc83xx: MPC8315ERDB: Use hwconfig for board type selection Anton Vorontsov
2009-07-16 20:47   ` Wolfgang Denk
2009-06-09 20:25 ` [U-Boot] [PATCH 8/8] mpc83xx: MPC837xEMDS: Use hwconfig instead of pci_external_arbiter variable Anton Vorontsov
2009-07-16 20:47   ` Wolfgang Denk
2009-06-11 14:09 ` [U-Boot] [PATCH v3 0/8] hwconfig and some its users Kim Phillips
2009-06-11 14:39   ` Anton Vorontsov
2009-06-11 15:55     ` Kim Phillips
2009-06-11 16:03       ` Wolfgang Denk
2009-06-11 16:31         ` Kim Phillips
2009-06-11 17:01       ` Anton Vorontsov [this message]
2009-06-11 21:53         ` Kim Phillips
2009-07-07 12:19           ` Anton Vorontsov
2009-07-07 22:38             ` Kim Phillips
2009-07-16 20:52               ` Wolfgang Denk
2009-07-16 21:06                 ` Anton Vorontsov
2009-07-21 14:18                   ` Kumar Gala

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=20090611170123.GA7533@oksana.dev.rtsoft.ru \
    --to=avorontsov@ru.mvista.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 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.