> -----Original Message----- > From: Menon, Nishanth > Sent: Thursday, May 13, 2010 9:02 PM > To: G, Manjunath Kondaiah > Cc: [email protected]; 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 <[email protected]> > > Signed-off-by: Manjunatha GK <[email protected]> > > Cc: Tony Lindgren <[email protected]> > > Cc: Kevin Hilman <[email protected]> > > Cc: Mike Turquette <[email protected]> > > Cc: Hemanth V <[email protected]> > > --- > > 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 [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
