Hi, On Wed, Feb 13, 2013 at 07:52:08PM +0530, Archit Taneja wrote: > +static struct panel_acx565akm_data *get_panel_data(struct omap_dss_device *dssdev) > +{ > + return (struct panel_acx565akm_data *) dssdev->data; > +} > + > static int acx_panel_probe(struct omap_dss_device *dssdev) > { > int r; > struct acx565akm_device *md = &acx_dev; > + struct panel_acx565akm_data *panel_data = get_panel_data(dssdev); Why the get_panel_data function is needed, isn't the cast unnecessary? A. -- 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