Commit 134d22eb authored by Laurent Pinchart's avatar Laurent Pinchart
Browse files

sh_mobile_hdmi: Remove platform data lcd_dev field 



The field is used to print debug messages only. Remove it.

Signed-off-by: default avatarLaurent Pinchart <laurent.pinchart@ideasonboard.com>
parent 12ee2502

arch/arm/mach-shmobile/board-ap4evb.c

+0 −1
Original line number Diff line number Diff line
@@ -856,7 +856,6 @@ static long ap4evb_clk_optimize(unsigned long target, unsigned long *best_freq, 


static struct sh_mobile_hdmi_info hdmi_info = {

	.lcd_chan = &sh_mobile_lcdc1_info.ch[0],

	.lcd_dev = &lcdc1_device.dev,

	.flags = HDMI_SND_SRC_SPDIF,

	.clk_optimize_parent = ap4evb_clk_optimize,

};

arch/arm/mach-shmobile/board-mackerel.c

+0 −1
Original line number Diff line number Diff line
@@ -480,7 +480,6 @@ static struct platform_device hdmi_lcdc_device = { 


static struct sh_mobile_hdmi_info hdmi_info = {

	.lcd_chan	= &hdmi_lcdc_info.ch[0],

	.lcd_dev	= &hdmi_lcdc_device.dev,

	.flags		= HDMI_SND_SRC_SPDIF,

};

drivers/video/sh_mobile_hdmi.c

+5 −12
Original line number Diff line number Diff line
@@ -1008,11 +1008,9 @@ static void sh_hdmi_display_on(void *arg, struct fb_info *info) 
	 * FB_EVENT_FB_UNBIND notify is also called with info->lock held

	 */

	struct sh_hdmi *hdmi = arg;

	struct sh_mobile_hdmi_info *pdata = hdmi->dev->platform_data;

	struct sh_mobile_lcdc_chan *ch = info->par;



	dev_dbg(hdmi->dev, "%s(%p): state %x\n", __func__,

		pdata->lcd_dev, info->state);

	dev_dbg(hdmi->dev, "%s(%p): state %x\n", __func__, hdmi, info->state);



	/* No need to lock */

	hdmi->info = info;
@@ -1040,9 +1038,8 @@ static void sh_hdmi_display_on(void *arg, struct fb_info *info) 
static void sh_hdmi_display_off(void *arg)

{

	struct sh_hdmi *hdmi = arg;

	struct sh_mobile_hdmi_info *pdata = hdmi->dev->platform_data;



	dev_dbg(hdmi->dev, "%s(%p)\n", __func__, pdata->lcd_dev);

	dev_dbg(hdmi->dev, "%s(%p)\n", __func__, hdmi);

	/* PS mode e->a */

	hdmi_write(hdmi, 0x10, HDMI_SYSTEM_CTRL);

}
@@ -1114,15 +1111,11 @@ static void sh_hdmi_edid_work_fn(struct work_struct *work) 
{

	struct sh_hdmi *hdmi = container_of(work, struct sh_hdmi, edid_work.work);

	struct fb_info *info;

	struct sh_mobile_hdmi_info *pdata = hdmi->dev->platform_data;

	struct sh_mobile_lcdc_chan *ch;

	int ret;



	dev_dbg(hdmi->dev, "%s(%p): begin, hotplug status %d\n", __func__,

		pdata->lcd_dev, hdmi->hp_state);



	if (!pdata->lcd_dev)

		return;

	dev_dbg(hdmi->dev, "%s(%p): begin, hotplug status %d\n", __func__, hdmi,

		hdmi->hp_state);



	mutex_lock(&hdmi->mutex);
@@ -1198,7 +1191,7 @@ static void sh_hdmi_edid_work_fn(struct work_struct *work) 
		hdmi->hp_state = HDMI_HOTPLUG_DISCONNECTED;

	mutex_unlock(&hdmi->mutex);



	dev_dbg(hdmi->dev, "%s(%p): end\n", __func__, pdata->lcd_dev);

	dev_dbg(hdmi->dev, "%s(%p): end\n", __func__, hdmi);

}



static int sh_hdmi_notify(struct notifier_block *nb,

include/video/sh_mobile_hdmi.h

+0 −1
Original line number Diff line number Diff line
@@ -32,7 +32,6 @@ struct clk; 


struct sh_mobile_hdmi_info {

	struct sh_mobile_lcdc_chan_cfg	*lcd_chan;

	struct device			*lcd_dev;

	unsigned int			 flags;

	long (*clk_optimize_parent)(unsigned long target, unsigned long *best_freq,

				    unsigned long *parent_freq);