[U-Boot] [PATCH v4 05/15] dm: Rename struct device_id to udevice_id

Jon Loeliger loeliger at gmail.com
Fri Jun 6 22:26:45 CEST 2014


Didn't this hit upstream master already?
54c5d08a09e631f88738db54c75395c6457c2157

Or is this subsequent cleanup for straglers in this branch?

Thanks,
jdl

On Fri, Jun 6, 2014 at 2:13 PM, Simon Glass <sjg at chromium.org> wrote:
> It is best to avoid having any occurence of 'struct device' in driver
> model, so rename to achieve this.
>
> - Add new patch to rename struct device_id to udevice_id
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
> Changes in v4: None
> Changes in v3: None
> Changes in v2: None
>
>  doc/driver-model/README.txt | 2 +-
>  drivers/core/lists.c        | 2 +-
>  drivers/demo/demo-shape.c   | 2 +-
>  drivers/demo/demo-simple.c  | 2 +-
>  drivers/gpio/sandbox.c      | 2 +-
>  include/dm/device.h         | 6 +++---
>  test/dm/test-fdt.c          | 2 +-
>  7 files changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/doc/driver-model/README.txt b/doc/driver-model/README.txt
> index dcecb9a..4af89b0 100644
> --- a/doc/driver-model/README.txt
> +++ b/doc/driver-model/README.txt
> @@ -315,7 +315,7 @@ is little or no 'driver model' code to write.
>  - Moved some data from code into data structure - e.g. store a pointer to
>  the driver operations structure in the driver, rather than passing it
>  to the driver bind function.
> -- Rename some structures to make them more similar to Linux (struct device
> +- Rename some structures to make them more similar to Linux (struct udevice
>  instead of struct instance, struct platdata, etc.)
>  - Change the name 'core' to 'uclass', meaning U-Boot class. It seems that
>  this concept relates to a class of drivers (or a subsystem). We shouldn't
> diff --git a/drivers/core/lists.c b/drivers/core/lists.c
> index 205b140..9f2917f 100644
> --- a/drivers/core/lists.c
> +++ b/drivers/core/lists.c
> @@ -94,7 +94,7 @@ int lists_bind_drivers(struct udevice *parent)
>   * tree error
>   */
>  static int driver_check_compatible(const void *blob, int offset,
> -                                  const struct device_id *of_match)
> +                                  const struct udevice_id *of_match)
>  {
>         int ret;
>
> diff --git a/drivers/demo/demo-shape.c b/drivers/demo/demo-shape.c
> index a68cc10..3fa9c59 100644
> --- a/drivers/demo/demo-shape.c
> +++ b/drivers/demo/demo-shape.c
> @@ -111,7 +111,7 @@ static int shape_ofdata_to_platdata(struct udevice *dev)
>         return 0;
>  }
>
> -static const struct device_id demo_shape_id[] = {
> +static const struct udevice_id demo_shape_id[] = {
>         { "demo-shape", 0 },
>         { },
>  };
> diff --git a/drivers/demo/demo-simple.c b/drivers/demo/demo-simple.c
> index 11def86..2bcb7df 100644
> --- a/drivers/demo/demo-simple.c
> +++ b/drivers/demo/demo-simple.c
> @@ -32,7 +32,7 @@ static int demo_shape_ofdata_to_platdata(struct udevice *dev)
>         return demo_parse_dt(dev);
>  }
>
> -static const struct device_id demo_shape_id[] = {
> +static const struct udevice_id demo_shape_id[] = {
>         { "demo-simple", 0 },
>         { },
>  };
> diff --git a/drivers/gpio/sandbox.c b/drivers/gpio/sandbox.c
> index 09cebe2..75ada5d 100644
> --- a/drivers/gpio/sandbox.c
> +++ b/drivers/gpio/sandbox.c
> @@ -239,7 +239,7 @@ static int gpio_sandbox_probe(struct udevice *dev)
>         return 0;
>  }
>
> -static const struct device_id sandbox_gpio_ids[] = {
> +static const struct udevice_id sandbox_gpio_ids[] = {
>         { .compatible = "sandbox,gpio" },
>         { }
>  };
> diff --git a/include/dm/device.h b/include/dm/device.h
> index ec04982..19f2039 100644
> --- a/include/dm/device.h
> +++ b/include/dm/device.h
> @@ -75,11 +75,11 @@ struct udevice {
>  #define device_active(dev)     ((dev)->flags & DM_FLAG_ACTIVATED)
>
>  /**
> - * struct device_id - Lists the compatible strings supported by a driver
> + * struct udevice_id - Lists the compatible strings supported by a driver
>   * @compatible: Compatible string
>   * @data: Data for this compatible string
>   */
> -struct device_id {
> +struct udevice_id {
>         const char *compatible;
>         ulong data;
>  };
> @@ -121,7 +121,7 @@ struct device_id {
>  struct driver {
>         char *name;
>         enum uclass_id id;
> -       const struct device_id *of_match;
> +       const struct udevice_id *of_match;
>         int (*bind)(struct udevice *dev);
>         int (*probe)(struct udevice *dev);
>         int (*remove)(struct udevice *dev);
> diff --git a/test/dm/test-fdt.c b/test/dm/test-fdt.c
> index 6eccf11..98e3936 100644
> --- a/test/dm/test-fdt.c
> +++ b/test/dm/test-fdt.c
> @@ -53,7 +53,7 @@ static int testfdt_drv_probe(struct udevice *dev)
>         return 0;
>  }
>
> -static const struct device_id testfdt_ids[] = {
> +static const struct udevice_id testfdt_ids[] = {
>         {
>                 .compatible = "denx,u-boot-fdt-test",
>                 .data = DM_TEST_TYPE_FIRST },
> --
> 2.0.0.526.g5318336
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot


More information about the U-Boot mailing list