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: Wed, 10 Jun 2009 00:23:28 +0400 [thread overview]
Message-ID: <20090609202328.GA19866@oksana.dev.rtsoft.ru> (raw)
On Fri, May 01, 2009 at 12:31:54AM +0200, Wolfgang Denk wrote:
> Dear Anton,
>
> In message <20090429215000.GA1092@oksana.dev.rtsoft.ru> you wrote:
> > This patch implements simple hwconfig infrastructure: an
> > interface for software knobs to control a hardware.
>
> Thanks a lot.
>
> > 3. We support hwconfig options with arguments. For example,
> >
> > set hwconfig dr_usb,dr_usb_mode:peripheral,dr_usb_phy_type:ulpi
> >
> > There are three hwconfig options selected:
> > 1. dr_usb - enable Dual-Role USB controller;
> > 2. dr_usb_mode:peripheral - USB in Function mode;
> > 3. dr_usb_phy_type:ulpi - USB should work with ULPI PHYs.
>
> That gives a lot of typing, which in turn results in lots of typing
> errors, which in this case are probably nasty to debug.
>
> Suggestion: instead of
>
> set hwconfig dr_usb,dr_usb_mode:peripheral,dr_usb_phy_type:ulpi
>
> use:
>
> set hwconfig dr_usb:mode=peripheral,phy_type=ulpi
>
> What do you think?
Sorry for the delay. Done.
New patches on the way.
Thanks,
--
Anton Vorontsov
email: cbouatmailru at gmail.com
irc://irc.freenode.net/bd2
next reply other threads:[~2009-06-09 20:23 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-06-09 20:23 Anton Vorontsov [this message]
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
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=20090609202328.GA19866@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.