From: Sascha Hauer <s.hauer@pengutronix.de>
To: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
Cc: barebox@lists.infradead.org
Subject: Re: [PATCH 1/4] driver: allow register device with specific id
Date: Fri, 17 Sep 2010 15:54:14 +0200 [thread overview]
Message-ID: <20100917135414.GG1473@pengutronix.de> (raw)
In-Reply-To: <20100917135037.GB20438@game.jcrosoft.org>
On Fri, Sep 17, 2010 at 03:50:37PM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote:
> On 15:45 Fri 17 Sep , Sascha Hauer wrote:
> > Hi J,
> >
> > On Sun, Sep 12, 2010 at 01:27:37PM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > > if you specify id = 0 the next available id will be taken
> > > otherwise fail if already registered
> >
> > I think we need a better solution here. Being able to specify an id
> > is a good idea in general, but since 0 is a valid id we should be able
> > to specify exactly this id aswell.
> > An idea is to use -1 for a dynamic id, but of course this needs changes
> > to all existing devices.
> I prefer to do it a second step
> in this case
I prefer doing it first, because it makes sure the problem is getting
solved.
Sascha
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
next prev parent reply other threads:[~2010-09-17 13:54 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-09-12 11:27 [PATCH 1/4] driver: allow register device with specific id Jean-Christophe PLAGNIOL-VILLARD
2010-09-12 11:27 ` [PATCH 2/4] device: fix dev_name Jean-Christophe PLAGNIOL-VILLARD
2010-09-12 11:27 ` [PATCH 3/4] amba-pl011: probe fail if we can get the clock Jean-Christophe PLAGNIOL-VILLARD
2010-09-12 11:38 ` Jean-Christophe PLAGNIOL-VILLARD
2010-09-17 13:52 ` Sascha Hauer
2010-09-17 13:51 ` Sascha Hauer
2010-09-17 13:54 ` Jean-Christophe PLAGNIOL-VILLARD
2010-09-12 11:27 ` [PATCH 4/4] stm8815: fix the uart device clock match Jean-Christophe PLAGNIOL-VILLARD
2010-09-17 13:45 ` [PATCH 1/4] driver: allow register device with specific id Sascha Hauer
2010-09-17 13:50 ` Jean-Christophe PLAGNIOL-VILLARD
2010-09-17 13:54 ` Sascha Hauer [this message]
2010-09-17 13:55 ` Jean-Christophe PLAGNIOL-VILLARD
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=20100917135414.GG1473@pengutronix.de \
--to=s.hauer@pengutronix.de \
--cc=barebox@lists.infradead.org \
--cc=plagnioj@jcrosoft.com \
/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.