diff options
author | Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com> | 2010-09-12 19:19:08 +0800 |
---|---|---|
committer | Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com> | 2010-09-17 10:56:24 +0800 |
commit | 427c6085838cb52bf71c316a2c7b5630e94c65f8 (patch) | |
tree | 9901f6970883f785fdbe9d42a8b37c71d182298a /lib | |
parent | 55819d58d4116ff5a3dcc4494a7ace56e5a318b0 (diff) | |
download | barebox-427c6085838cb52bf71c316a2c7b5630e94c65f8.tar.gz barebox-427c6085838cb52bf71c316a2c7b5630e94c65f8.tar.xz |
driver: allow register device with specific id
if you specify id = 0 the next available id will be taken
otherwise fail if already registered
before if you specify one it will have use the next free id anyway
Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
Diffstat (limited to 'lib')
-rw-r--r-- | lib/driver.c | 26 |
1 files changed, 22 insertions, 4 deletions
diff --git a/lib/driver.c b/lib/driver.c index b60074511f..00aca9cf71 100644 --- a/lib/driver.c +++ b/lib/driver.c @@ -56,14 +56,24 @@ struct device_d *get_device_by_name(const char *name) return NULL; } +struct device_d *get_device_by_name_id(const char *name, int id) +{ + struct device_d *dev; + + for_each_device(dev) { + if(!strcmp(dev->name, name) && id == dev->id) + return dev; + } + + return NULL; +} + int get_free_deviceid(const char *name_template) { int i = 0; - char name[MAX_DRIVER_NAME + 3]; while (1) { - sprintf(name, "%s%d", name_template, i); - if (!get_device_by_name(name)) + if (!get_device_by_name_id(name_template, i)) return i; i++; }; @@ -95,7 +105,15 @@ int register_device(struct device_d *new_device) { struct driver_d *drv; - new_device->id = get_free_deviceid(new_device->name); + if (new_device->id == 0) { + new_device->id = get_free_deviceid(new_device->name); + } else { + if (get_device_by_name_id(new_device->name, new_device->id)) { + eprintf("register_device: already registered %s\n", + dev_name(new_device)); + return -EINVAL; + } + } debug ("register_device: %s\n",new_device->name); |