Skip to content

Commit

Permalink
OMAPDSS: rename omap_dss_device's 'device' field to 'dst'
Browse files Browse the repository at this point in the history
In the old panel device model we had omap_dss_output entities,
representing the encoders in the DSS block. This entity had "device"
field, which pointed to the panel that was using the omap_dss_output.

With the new panel device model, the omap_dss_output is integrated into
omap_dss_device, which now represents a "display entity". Thus the "device"
field, now in omap_dss_device, points to the next entity in the display
entity-chain.

This patch renames the "device" field to "dst", which much better tells
what the field points to.

Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Reviewed-by: Archit Taneja <archit@ti.com>
  • Loading branch information
Tomi Valkeinen committed Aug 30, 2013
1 parent a73fdc6 commit 9560dc1
Show file tree
Hide file tree
Showing 10 changed files with 31 additions and 31 deletions.
10 changes: 5 additions & 5 deletions drivers/video/omap2/displays-new/encoder-tfp410.c
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ static int tfp410_connect(struct omap_dss_device *dssdev,
return r;

dst->src = dssdev;
dssdev->device = dst;
dssdev->dst = dst;

return 0;
}
Expand All @@ -59,12 +59,12 @@ static void tfp410_disconnect(struct omap_dss_device *dssdev,
if (!omapdss_device_is_connected(dssdev))
return;

WARN_ON(dst != dssdev->device);
if (dst != dssdev->device)
WARN_ON(dst != dssdev->dst);
if (dst != dssdev->dst)
return;

dst->src = NULL;
dssdev->device = NULL;
dssdev->dst = NULL;

in->ops.dpi->disconnect(in, &ddata->dssdev);
}
Expand Down Expand Up @@ -244,7 +244,7 @@ static int __exit tfp410_remove(struct platform_device *pdev)

WARN_ON(omapdss_device_is_connected(dssdev));
if (omapdss_device_is_connected(dssdev))
tfp410_disconnect(dssdev, dssdev->device);
tfp410_disconnect(dssdev, dssdev->dst);

omap_dss_put_device(in);

Expand Down
10 changes: 5 additions & 5 deletions drivers/video/omap2/displays-new/encoder-tpd12s015.c
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ static int tpd_connect(struct omap_dss_device *dssdev,
return r;

dst->src = dssdev;
dssdev->device = dst;
dssdev->dst = dst;

INIT_COMPLETION(ddata->hpd_completion);

Expand Down Expand Up @@ -95,15 +95,15 @@ static void tpd_disconnect(struct omap_dss_device *dssdev,
struct panel_drv_data *ddata = to_panel_data(dssdev);
struct omap_dss_device *in = ddata->in;

WARN_ON(dst != dssdev->device);
WARN_ON(dst != dssdev->dst);

if (dst != dssdev->device)
if (dst != dssdev->dst)
return;

gpio_set_value_cansleep(ddata->ct_cp_hpd_gpio, 0);

dst->src = NULL;
dssdev->device = NULL;
dssdev->dst = NULL;

in->ops.hdmi->disconnect(in, &ddata->dssdev);
}
Expand Down Expand Up @@ -372,7 +372,7 @@ static int __exit tpd_remove(struct platform_device *pdev)

WARN_ON(omapdss_device_is_connected(dssdev));
if (omapdss_device_is_connected(dssdev))
tpd_disconnect(dssdev, dssdev->device);
tpd_disconnect(dssdev, dssdev->dst);

omap_dss_put_device(in);

Expand Down
4 changes: 2 additions & 2 deletions drivers/video/omap2/dss/apply.c
Original file line number Diff line number Diff line change
Expand Up @@ -428,8 +428,8 @@ static struct omap_dss_device *dss_mgr_get_device(struct omap_overlay_manager *m
if (dssdev == NULL)
return NULL;

while (dssdev->device)
dssdev = dssdev->device;
while (dssdev->dst)
dssdev = dssdev->dst;

if (dssdev->driver)
return dssdev;
Expand Down
4 changes: 2 additions & 2 deletions drivers/video/omap2/dss/dpi.c
Original file line number Diff line number Diff line change
Expand Up @@ -643,9 +643,9 @@ static int dpi_connect(struct omap_dss_device *dssdev,
static void dpi_disconnect(struct omap_dss_device *dssdev,
struct omap_dss_device *dst)
{
WARN_ON(dst != dssdev->device);
WARN_ON(dst != dssdev->dst);

if (dst != dssdev->device)
if (dst != dssdev->dst)
return;

omapdss_output_unset_device(dssdev);
Expand Down
4 changes: 2 additions & 2 deletions drivers/video/omap2/dss/dsi.c
Original file line number Diff line number Diff line change
Expand Up @@ -5264,9 +5264,9 @@ static int dsi_connect(struct omap_dss_device *dssdev,
static void dsi_disconnect(struct omap_dss_device *dssdev,
struct omap_dss_device *dst)
{
WARN_ON(dst != dssdev->device);
WARN_ON(dst != dssdev->dst);

if (dst != dssdev->device)
if (dst != dssdev->dst)
return;

omapdss_output_unset_device(dssdev);
Expand Down
4 changes: 2 additions & 2 deletions drivers/video/omap2/dss/hdmi.c
Original file line number Diff line number Diff line change
Expand Up @@ -900,9 +900,9 @@ static int hdmi_connect(struct omap_dss_device *dssdev,
static void hdmi_disconnect(struct omap_dss_device *dssdev,
struct omap_dss_device *dst)
{
WARN_ON(dst != dssdev->device);
WARN_ON(dst != dssdev->dst);

if (dst != dssdev->device)
if (dst != dssdev->dst)
return;

omapdss_output_unset_device(dssdev);
Expand Down
16 changes: 8 additions & 8 deletions drivers/video/omap2/dss/output.c
Original file line number Diff line number Diff line change
Expand Up @@ -34,9 +34,9 @@ int omapdss_output_set_device(struct omap_dss_device *out,

mutex_lock(&output_lock);

if (out->device) {
if (out->dst) {
DSSERR("output already has device %s connected to it\n",
out->device->name);
out->dst->name);
r = -EINVAL;
goto err;
}
Expand All @@ -47,7 +47,7 @@ int omapdss_output_set_device(struct omap_dss_device *out,
goto err;
}

out->device = dssdev;
out->dst = dssdev;
dssdev->src = out;

mutex_unlock(&output_lock);
Expand All @@ -66,21 +66,21 @@ int omapdss_output_unset_device(struct omap_dss_device *out)

mutex_lock(&output_lock);

if (!out->device) {
if (!out->dst) {
DSSERR("output doesn't have a device connected to it\n");
r = -EINVAL;
goto err;
}

if (out->device->state != OMAP_DSS_DISPLAY_DISABLED) {
if (out->dst->state != OMAP_DSS_DISPLAY_DISABLED) {
DSSERR("device %s is not disabled, cannot unset device\n",
out->device->name);
out->dst->name);
r = -EINVAL;
goto err;
}

out->device->src = NULL;
out->device = NULL;
out->dst->src = NULL;
out->dst = NULL;

mutex_unlock(&output_lock);

Expand Down
4 changes: 2 additions & 2 deletions drivers/video/omap2/dss/sdi.c
Original file line number Diff line number Diff line change
Expand Up @@ -306,9 +306,9 @@ static int sdi_connect(struct omap_dss_device *dssdev,
static void sdi_disconnect(struct omap_dss_device *dssdev,
struct omap_dss_device *dst)
{
WARN_ON(dst != dssdev->device);
WARN_ON(dst != dssdev->dst);

if (dst != dssdev->device)
if (dst != dssdev->dst)
return;

omapdss_output_unset_device(dssdev);
Expand Down
4 changes: 2 additions & 2 deletions drivers/video/omap2/dss/venc.c
Original file line number Diff line number Diff line change
Expand Up @@ -753,9 +753,9 @@ static int venc_connect(struct omap_dss_device *dssdev,
static void venc_disconnect(struct omap_dss_device *dssdev,
struct omap_dss_device *dst)
{
WARN_ON(dst != dssdev->device);
WARN_ON(dst != dssdev->dst);

if (dst != dssdev->device)
if (dst != dssdev->dst)
return;

omapdss_output_unset_device(dssdev);
Expand Down
2 changes: 1 addition & 1 deletion include/video/omapdss.h
Original file line number Diff line number Diff line change
Expand Up @@ -793,7 +793,7 @@ struct omap_dss_device {
/* dynamic fields */
struct omap_overlay_manager *manager;

struct omap_dss_device *device;
struct omap_dss_device *dst;
};

struct omap_dss_hdmi_data
Expand Down

0 comments on commit 9560dc1

Please sign in to comment.