Commit 35deca3d authored by Tomi Valkeinen's avatar Tomi Valkeinen
Browse files

OMAPDSS: interface drivers register their panel devices 



Currently the higher level omapdss platform driver gets the list of
displays in its platform data, and uses that list to create the
omap_dss_device for each display.

With DT, the logical way to do the above is to list the displays under
each individual output, i.e. we'd have "dpi" node, under which we would
have the display that uses DPI. In other words, each output driver
handles the displays that use that particular output.

To make the current code ready for DT, this patch modifies the output
drivers so that each of them creates the display devices which use that
output. However, instead of changing the platform data to suit this
method, each output driver is passed the full list of displays, and the
drivers pick the displays that are meant for them. This allows us to
keep the old platform data, and thus we avoid the need to change the
board files.

Signed-off-by: default avatarTomi Valkeinen <tomi.valkeinen@ti.com>
parent c018c673

arch/arm/mach-omap2/display.c

+6 −5
Original line number Diff line number Diff line
@@ -327,7 +327,7 @@ int __init omap_display_init(struct omap_dss_board_info *board_data) 
	dss_pdev = create_dss_pdev(curr_dss_hwmod[0].dev_name,

			curr_dss_hwmod[0].id,

			curr_dss_hwmod[0].oh_name,

			NULL, 0,

			board_data, sizeof(*board_data),

			NULL);



	if (IS_ERR(dss_pdev)) {
@@ -341,7 +341,7 @@ int __init omap_display_init(struct omap_dss_board_info *board_data) 
		pdev = create_dss_pdev(curr_dss_hwmod[i].dev_name,

				curr_dss_hwmod[i].id,

				curr_dss_hwmod[i].oh_name,

				NULL, 0,

				board_data, sizeof(*board_data),

				dss_pdev);



		if (IS_ERR(pdev)) {
@@ -354,15 +354,16 @@ int __init omap_display_init(struct omap_dss_board_info *board_data) 


	/* Create devices for DPI and SDI */



	pdev = create_simple_dss_pdev("omapdss_dpi", -1, NULL, 0, dss_pdev);

	pdev = create_simple_dss_pdev("omapdss_dpi", -1,

			board_data, sizeof(*board_data), dss_pdev);

	if (IS_ERR(pdev)) {

		pr_err("Could not build platform_device for omapdss_dpi\n");

		return PTR_ERR(pdev);

	}



	if (cpu_is_omap34xx()) {

		pdev = create_simple_dss_pdev("omapdss_sdi", -1, NULL, 0,

				dss_pdev);

		pdev = create_simple_dss_pdev("omapdss_sdi", -1,

				board_data, sizeof(*board_data), dss_pdev);

		if (IS_ERR(pdev)) {

			pr_err("Could not build platform_device for omapdss_sdi\n");

			return PTR_ERR(pdev);

drivers/video/omap2/dss/core.c

+17 −33
Original line number Diff line number Diff line
@@ -56,9 +56,6 @@ bool dss_debug; 
module_param_named(debug, dss_debug, bool, 0644);

#endif



static int omap_dss_register_device(struct omap_dss_device *);

static void omap_dss_unregister_device(struct omap_dss_device *);



/* REGULATORS */



struct regulator *dss_get_vdds_dsi(void)
@@ -209,7 +206,6 @@ static int __init omap_dss_probe(struct platform_device *pdev) 
{

	struct omap_dss_board_info *pdata = pdev->dev.platform_data;

	int r;

	int i;



	core.pdev = pdev;
@@ -229,25 +225,8 @@ static int __init omap_dss_probe(struct platform_device *pdev) 
	else if (pdata->default_device)

		core.default_display_name = pdata->default_device->name;



	for (i = 0; i < pdata->num_devices; ++i) {

		struct omap_dss_device *dssdev = pdata->devices[i];



		r = omap_dss_register_device(dssdev);

		if (r) {

			DSSERR("device %d %s register failed %d\n", i,

				dssdev->name ?: "unnamed", r);



			while (--i >= 0)

				omap_dss_unregister_device(pdata->devices[i]);



			goto err_register;

		}

	}



	return 0;



err_register:

	dss_uninitialize_debugfs();

err_debugfs:



	return r;
@@ -255,17 +234,11 @@ err_debugfs: 


static int omap_dss_remove(struct platform_device *pdev)

{

	struct omap_dss_board_info *pdata = pdev->dev.platform_data;

	int i;



	dss_uninitialize_debugfs();



	dss_uninit_overlays(pdev);

	dss_uninit_overlay_managers(pdev);



	for (i = 0; i < pdata->num_devices; ++i)

		omap_dss_unregister_device(pdata->devices[i]);



	return 0;

}
@@ -467,25 +440,36 @@ static void omap_dss_dev_release(struct device *dev) 
	reset_device(dev, 0);

}



static int omap_dss_register_device(struct omap_dss_device *dssdev)

int omap_dss_register_device(struct omap_dss_device *dssdev,

		struct device *parent, int disp_num)

{

	static int dev_num;



	WARN_ON(!dssdev->driver_name);



	reset_device(&dssdev->dev, 1);

	dssdev->dev.bus = &dss_bus_type;

	dssdev->dev.parent = &dss_bus;

	dssdev->dev.parent = parent;

	dssdev->dev.release = omap_dss_dev_release;

	dev_set_name(&dssdev->dev, "display%d", dev_num++);

	dev_set_name(&dssdev->dev, "display%d", disp_num);

	return device_register(&dssdev->dev);

}



static void omap_dss_unregister_device(struct omap_dss_device *dssdev)

void omap_dss_unregister_device(struct omap_dss_device *dssdev)

{

	device_unregister(&dssdev->dev);

}



static int dss_unregister_dss_dev(struct device *dev, void *data)

{

	struct omap_dss_device *dssdev = to_dss_device(dev);

	omap_dss_unregister_device(dssdev);

	return 0;

}



void omap_dss_unregister_child_devices(struct device *parent)

{

	device_for_each_child(parent, NULL, dss_unregister_dss_dev);

}



/* BUS */

static int __init omap_dss_bus_register(void)

{

drivers/video/omap2/dss/dpi.c

+17 −0
Original line number Diff line number Diff line
@@ -367,11 +367,28 @@ int dpi_init_display(struct omap_dss_device *dssdev) 


static int __init omap_dpi_probe(struct platform_device *pdev)

{

	struct omap_dss_board_info *pdata = pdev->dev.platform_data;

	int i, r;



	for (i = 0; i < pdata->num_devices; ++i) {

		struct omap_dss_device *dssdev = pdata->devices[i];



		if (dssdev->type != OMAP_DISPLAY_TYPE_DPI)

			continue;



		r = omap_dss_register_device(dssdev, &pdev->dev, i);

		if (r)

			DSSERR("device %s register failed: %d\n",

					dssdev->name, r);

	}



	return 0;

}



static int __exit omap_dpi_remove(struct platform_device *pdev)

{

	omap_dss_unregister_child_devices(&pdev->dev);



	return 0;

}

drivers/video/omap2/dss/dsi.c

+18 −0
Original line number Diff line number Diff line
@@ -4614,6 +4614,7 @@ static int __init omap_dsihw_probe(struct platform_device *dsidev) 
	int r, i, dsi_module = dsi_get_dsidev_id(dsidev);

	struct resource *dsi_mem;

	struct dsi_data *dsi;

	struct omap_dss_board_info *pdata = dsidev->dev.platform_data;



	dsi = devm_kzalloc(&dsidev->dev, sizeof(*dsi), GFP_KERNEL);

	if (!dsi)
@@ -4700,6 +4701,21 @@ static int __init omap_dsihw_probe(struct platform_device *dsidev) 
	else

		dsi->num_lanes_supported = 3;



	for (i = 0; i < pdata->num_devices; ++i) {

		struct omap_dss_device *dssdev = pdata->devices[i];



		if (dssdev->type != OMAP_DISPLAY_TYPE_DSI)

			continue;



		if (dssdev->phy.dsi.module != dsi_module)

			continue;



		r = omap_dss_register_device(dssdev, &dsidev->dev, i);

		if (r)

			DSSERR("device %s register failed: %d\n",

				dssdev->name, r);

	}



	dsi_runtime_put(dsidev);



	if (dsi_module == 0)
@@ -4727,6 +4743,8 @@ static int __exit omap_dsihw_remove(struct platform_device *dsidev) 


	WARN_ON(dsi->scp_clk_refcount > 0);



	omap_dss_unregister_child_devices(&dsidev->dev);



	pm_runtime_disable(&dsidev->dev);



	dsi_put_clocks(dsidev);

drivers/video/omap2/dss/dss.h

+5 −0
Original line number Diff line number Diff line
@@ -165,6 +165,11 @@ void dss_dsi_disable_pads(int dsi_id, unsigned lane_mask); 
int dss_set_min_bus_tput(struct device *dev, unsigned long tput);

int dss_debugfs_create_file(const char *name, void (*write)(struct seq_file *));



int omap_dss_register_device(struct omap_dss_device *dssdev,

		struct device *parent, int disp_num);

void omap_dss_unregister_device(struct omap_dss_device *dssdev);

void omap_dss_unregister_child_devices(struct device *parent);



/* apply */

void dss_apply_init(void);

int dss_mgr_wait_for_go(struct omap_overlay_manager *mgr);

CRA Git | Maintained and supported by SUSTech CRA and CCSE