Re: [PATCH 16/33] OMAPDSS: acx565akm panel: handle gpios in panel driver

On Wednesday 13 February 2013 10:59 PM, Aaro Koskinen wrote:

Hi,

On Wed, Feb 13, 2013 at 07:52:08PM +0530, Archit Taneja wrote:

quoted

+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?

the 'data' member of omap_dss_device has the type 'void *', we need to 
cast it to access the panel_acx565akm_data struct pointer.

Archit