On 8/9/2011 4:10 PM, G, Manjunath Kondaiah wrote:

Add pd_size in the AUXDATA structure so that device drivers which require
platform_data size can pass along with AUXDATA.

It is really needed by device driver? Or is it because omap_device_build is using platform_device_add_data that is doing a copy of the pdata and thus require the size?

Signed-off-by: G, Manjunath Kondaiah<manj...@ti.com>
---
  drivers/of/platform.c       |    2 ++
  include/linux/of_platform.h |    5 +++++
  2 files changed, 7 insertions(+), 0 deletions(-)

diff --git a/drivers/of/platform.c b/drivers/of/platform.c
index ebbbf42..4b27286 100644
--- a/drivers/of/platform.c
+++ b/drivers/of/platform.c
@@ -565,6 +565,7 @@ static int of_platform_bus_create(struct device_node *bus,
        struct platform_device *dev;
        const char *bus_id = NULL;
        void *platform_data = NULL;
+       int pd_size;

Maybe platform_data_size will be a little bit more consistent?

Regards,
Benoit

        int id = -1;
        int rc = 0;

@@ -588,6 +589,7 @@ static int of_platform_bus_create(struct device_node *bus,
                bus_id = auxdata->name;
                id = auxdata->id;
                platform_data = auxdata->platform_data;
+               pd_size = auxdata->pd_size;
        }

        if (of_device_is_compatible(bus, "arm,primecell")) {
diff --git a/include/linux/of_platform.h b/include/linux/of_platform.h
index 252246c..a3be980 100644
--- a/include/linux/of_platform.h
+++ b/include/linux/of_platform.h
@@ -47,6 +47,7 @@ struct of_dev_auxdata {
        char *name;
        int id;
        void *platform_data;
+       int pd_size;
  };

  /* Macro to simplify populating a lookup table */
@@ -58,6 +59,10 @@ struct of_dev_auxdata {
        { .compatible = _compat, .phys_addr = _phys, .name = _name, \
          .id = _id, .platform_data = _pdata }

+#define OF_DEV_AUXDATA_ID_PDSIZE(_compat,_phys,_name,_id,_pdata,_pd_size) \
+       { .compatible = _compat, .phys_addr = _phys, .name = _name, \
+         .id = _id, .platform_data = _pdata, .pd_size = _pd_size }
+
  /**
   * of_platform_driver - Legacy of-aware driver for platform devices.
   *

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to