Hi Kevin,
On Mon, Oct 17, 2011 at 1:41 PM, Ohad Ben-Cohen <ohad@xxxxxxxxxx> wrote:
> Expose omap_device_{alloc, delete, register} so we can use them outside
> of omap_device.c.
Can you please take this one ?
Please tell me if you want a refreshed version against 3.2-rc1.
Thanks!
Ohad.
>
> This approach allows users, which need to manipulate an archdata member
> of a device before it is registered, to do so. This is also useful
> for users who have their devices created very early so they can be used
> at ->reserve() time to reserve CMA memory.
>
> The immediate use case for this is to set the private iommu archdata
> member, which binds a device to its associated iommu controller.
> This way, generic code will be able to attach omap devices to their
> iommus, without calling any omap-specific API.
>
> With this in hand, we can further clean the existing mainline OMAP iommu
> driver and its mainline users, and focus on generic IOMMU approaches
> for future users (rpmsg/remoteproc and the upcoming generic DMA API).
>
> This patch is still considered an interim solution until DT fully materializes
> for omap; at that point, this functionality will be removed as DT will
> take care of creating the devices and configuring them correctly.
>
> Tested on OMAP4 with a generic rpmsg/remoteproc that doesn't use any
> omap-specific IOMMU API anymore.
>
> Signed-off-by: Ohad Ben-Cohen <ohad@xxxxxxxxxx>
> ---
> arch/arm/plat-omap/include/plat/omap_device.h | 7 +++++++
> arch/arm/plat-omap/omap_device.c | 13 +++----------
> 2 files changed, 10 insertions(+), 10 deletions(-)
>
> diff --git a/arch/arm/plat-omap/include/plat/omap_device.h b/arch/arm/plat-omap/include/plat/omap_device.h
> index 12c5b0c..28d7a38 100644
> --- a/arch/arm/plat-omap/include/plat/omap_device.h
> +++ b/arch/arm/plat-omap/include/plat/omap_device.h
> @@ -100,6 +100,13 @@ struct platform_device *omap_device_build_ss(const char *pdev_name, int pdev_id,
> struct omap_device_pm_latency *pm_lats,
> int pm_lats_cnt, int is_early_device);
>
> +struct omap_device *omap_device_alloc(struct platform_device *pdev,
> + struct omap_hwmod **ohs, int oh_cnt,
> + struct omap_device_pm_latency *pm_lats,
> + int pm_lats_cnt);
> +void omap_device_delete(struct omap_device *od);
> +int omap_device_register(struct platform_device *pdev);
> +
> void __iomem *omap_device_get_rt_va(struct omap_device *od);
> struct device *omap_device_get_by_hwmod_name(const char *oh_name);
>
> diff --git a/arch/arm/plat-omap/omap_device.c b/arch/arm/plat-omap/omap_device.c
> index cd90bed..913179c 100644
> --- a/arch/arm/plat-omap/omap_device.c
> +++ b/arch/arm/plat-omap/omap_device.c
> @@ -96,14 +96,7 @@
> #define USE_WAKEUP_LAT 0
> #define IGNORE_WAKEUP_LAT 1
>
> -static int omap_device_register(struct platform_device *pdev);
> static int omap_early_device_register(struct platform_device *pdev);
> -static struct omap_device *omap_device_alloc(struct platform_device *pdev,
> - struct omap_hwmod **ohs, int oh_cnt,
> - struct omap_device_pm_latency *pm_lats,
> - int pm_lats_cnt);
> -static void omap_device_delete(struct omap_device *od);
> -
>
> static struct omap_device_pm_latency omap_default_latency[] = {
> {
> @@ -508,7 +501,7 @@ static int omap_device_fill_resources(struct omap_device *od,
> *
> * Returns an struct omap_device pointer or ERR_PTR() on error;
> */
> -static struct omap_device *omap_device_alloc(struct platform_device *pdev,
> +struct omap_device *omap_device_alloc(struct platform_device *pdev,
> struct omap_hwmod **ohs, int oh_cnt,
> struct omap_device_pm_latency *pm_lats,
> int pm_lats_cnt)
> @@ -590,7 +583,7 @@ oda_exit1:
> return ERR_PTR(ret);
> }
>
> -static void omap_device_delete(struct omap_device *od)
> +void omap_device_delete(struct omap_device *od)
> {
> if (!od)
> return;
> @@ -816,7 +809,7 @@ static struct dev_pm_domain omap_device_pm_domain = {
> * platform_device_register() on the underlying platform_device.
> * Returns the return value of platform_device_register().
> */
> -static int omap_device_register(struct platform_device *pdev)
> +int omap_device_register(struct platform_device *pdev)
> {
> pr_debug("omap_device: %s: registering\n", pdev->name);
>
> --
> 1.7.4.1
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
