> -----Original Message-----
> From: Menon, Nishanth
> Sent: Thursday, May 13, 2010 9:02 PM
> To: G, Manjunath Kondaiah
> Cc: linux-omap@xxxxxxxxxxxxxxx; Agarwal, Preshit; Tony
> Lindgren; Kevin Hilman; Turquette, Mike; V, Hemanth
> Subject: Re: [PATCH v4] OMAP3: Registering sgx device and
> it's platform data
>
> G, Manjunath Kondaiah had written, on 05/13/2010 12:06 AM,
> the following:
> > The SGX powervr_device is registered with it's platform specific
> > data to provide information about setting constraint through
> > omap_pm_set_min_bus_tput.
> >
> > Signed-off-by: Preshit Agarwal <preshit.agarwal@xxxxxx>
> > Signed-off-by: Manjunatha GK <manjugk@xxxxxx>
> > Cc: Tony Lindgren <tony@xxxxxxxxxxx>
> > Cc: Kevin Hilman <khilman@xxxxxxxxxxxxxxxxxxx>
> > Cc: Mike Turquette <mturquette@xxxxxx>
> > Cc: Hemanth V <hemanthv@xxxxxx>
> > ---
> > arch/arm/mach-omap2/devices.c | 21
> +++++++++++++++++++--
> > arch/arm/mach-omap2/include/mach/omap_sgxdef.h | 11 +++++++++++
> > 2 files changed, 30 insertions(+), 2 deletions(-)
> > create mode 100644 arch/arm/mach-omap2/include/mach/omap_sgxdef.h
> >
> > diff --git a/arch/arm/mach-omap2/devices.c
> b/arch/arm/mach-omap2/devices.c
> > index 2271b9b..6349ee5 100644
> > --- a/arch/arm/mach-omap2/devices.c
> > +++ b/arch/arm/mach-omap2/devices.c
> > @@ -26,7 +26,7 @@
> > #include <plat/mux.h>
> > #include <mach/gpio.h>
> > #include <plat/mmc.h>
> > -
> > +#include <mach/omap_sgxdef.h>
> > #include "mux.h"
> >
> > #if defined(CONFIG_VIDEO_OMAP2) ||
> defined(CONFIG_VIDEO_OMAP2_MODULE)
> > @@ -786,6 +786,23 @@ static inline void omap_hdq_init(void)
> > static inline void omap_hdq_init(void) {}
> > #endif
> >
> > +struct sgx_platform_data omap_sgx_data = {
> > + .set_min_bus_tput = NULL,
> > +};
> > +
> > +static struct platform_device powervr_device = {
> > + .name = "pvrsrvkm",
> > + .id = -1,
> > + .dev = {
> > + .platform_data = &omap_sgx_data,
> > + }
> > +};
> > +
> > +static void omap_init_sgx(void)
> > +{
> > + (void) platform_device_register(&powervr_device);
> > +}
> > +
> >
> /*------------------------------------------------------------
> -------------*/
> >
> > static int __init omap2_init_devices(void)
> > @@ -800,7 +817,7 @@ static int __init omap2_init_devices(void)
> > omap_hdq_init();
> > omap_init_sti();
> > omap_init_sha1_md5();
> > -
> > + omap_init_sgx();
> should'nt we depend on has_sgx_feature?
I will change it to: "if (omap3_has_sgx) omap_init_sgx();"
-Manjunath--
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
