diff options
author | Russell King <rmk+kernel@arm.linux.org.uk> | 2015-08-14 06:22:50 -0400 |
---|---|---|
committer | Russell King <rmk+kernel@arm.linux.org.uk> | 2015-09-29 14:26:05 -0400 |
commit | a3584f60f4898c9479931cdca1dc19f758af45fb (patch) | |
tree | 0a3e7413a28089052d2dbf28e4ce65f569eac444 /drivers/gpu/drm/i2c | |
parent | eed64b5963f1496be62ba41f536346b2186727a0 (diff) |
drm/i2c: tda998x: kill struct tda998x_priv2
Kill the redundant tda998x_priv2 structure now that its only member is
the struct tda998x_priv.
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
Diffstat (limited to 'drivers/gpu/drm/i2c')
-rw-r--r-- | drivers/gpu/drm/i2c/tda998x_drv.c | 80 |
1 files changed, 38 insertions, 42 deletions
diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c b/drivers/gpu/drm/i2c/tda998x_drv.c index a2a463cec244..8bca9155ee18 100644 --- a/drivers/gpu/drm/i2c/tda998x_drv.c +++ b/drivers/gpu/drm/i2c/tda998x_drv.c | |||
@@ -1330,21 +1330,17 @@ fail: | |||
1330 | return -ENXIO; | 1330 | return -ENXIO; |
1331 | } | 1331 | } |
1332 | 1332 | ||
1333 | struct tda998x_priv2 { | 1333 | #define conn_to_tda998x_priv(x) \ |
1334 | struct tda998x_priv base; | 1334 | container_of(x, struct tda998x_priv, connector); |
1335 | }; | ||
1336 | |||
1337 | #define conn_to_tda998x_priv2(x) \ | ||
1338 | container_of(x, struct tda998x_priv2, base.connector); | ||
1339 | 1335 | ||
1340 | #define enc_to_tda998x_priv2(x) \ | 1336 | #define enc_to_tda998x_priv(x) \ |
1341 | container_of(x, struct tda998x_priv2, base.encoder); | 1337 | container_of(x, struct tda998x_priv, encoder); |
1342 | 1338 | ||
1343 | static void tda998x_encoder2_dpms(struct drm_encoder *encoder, int mode) | 1339 | static void tda998x_encoder2_dpms(struct drm_encoder *encoder, int mode) |
1344 | { | 1340 | { |
1345 | struct tda998x_priv2 *priv = enc_to_tda998x_priv2(encoder); | 1341 | struct tda998x_priv *priv = enc_to_tda998x_priv(encoder); |
1346 | 1342 | ||
1347 | tda998x_encoder_dpms(&priv->base, mode); | 1343 | tda998x_encoder_dpms(priv, mode); |
1348 | } | 1344 | } |
1349 | 1345 | ||
1350 | static void tda998x_encoder_prepare(struct drm_encoder *encoder) | 1346 | static void tda998x_encoder_prepare(struct drm_encoder *encoder) |
@@ -1361,9 +1357,9 @@ static void tda998x_encoder2_mode_set(struct drm_encoder *encoder, | |||
1361 | struct drm_display_mode *mode, | 1357 | struct drm_display_mode *mode, |
1362 | struct drm_display_mode *adjusted_mode) | 1358 | struct drm_display_mode *adjusted_mode) |
1363 | { | 1359 | { |
1364 | struct tda998x_priv2 *priv = enc_to_tda998x_priv2(encoder); | 1360 | struct tda998x_priv *priv = enc_to_tda998x_priv(encoder); |
1365 | 1361 | ||
1366 | tda998x_encoder_mode_set(&priv->base, mode, adjusted_mode); | 1362 | tda998x_encoder_mode_set(priv, mode, adjusted_mode); |
1367 | } | 1363 | } |
1368 | 1364 | ||
1369 | static const struct drm_encoder_helper_funcs tda998x_encoder_helper_funcs = { | 1365 | static const struct drm_encoder_helper_funcs tda998x_encoder_helper_funcs = { |
@@ -1378,9 +1374,9 @@ static const struct drm_encoder_helper_funcs tda998x_encoder_helper_funcs = { | |||
1378 | 1374 | ||
1379 | static void tda998x_encoder_destroy(struct drm_encoder *encoder) | 1375 | static void tda998x_encoder_destroy(struct drm_encoder *encoder) |
1380 | { | 1376 | { |
1381 | struct tda998x_priv2 *priv = enc_to_tda998x_priv2(encoder); | 1377 | struct tda998x_priv *priv = enc_to_tda998x_priv(encoder); |
1382 | 1378 | ||
1383 | tda998x_destroy(&priv->base); | 1379 | tda998x_destroy(priv); |
1384 | drm_encoder_cleanup(encoder); | 1380 | drm_encoder_cleanup(encoder); |
1385 | } | 1381 | } |
1386 | 1382 | ||
@@ -1390,25 +1386,25 @@ static const struct drm_encoder_funcs tda998x_encoder_funcs = { | |||
1390 | 1386 | ||
1391 | static int tda998x_connector_get_modes(struct drm_connector *connector) | 1387 | static int tda998x_connector_get_modes(struct drm_connector *connector) |
1392 | { | 1388 | { |
1393 | struct tda998x_priv2 *priv = conn_to_tda998x_priv2(connector); | 1389 | struct tda998x_priv *priv = conn_to_tda998x_priv(connector); |
1394 | 1390 | ||
1395 | return tda998x_encoder_get_modes(&priv->base, connector); | 1391 | return tda998x_encoder_get_modes(priv, connector); |
1396 | } | 1392 | } |
1397 | 1393 | ||
1398 | static int tda998x_connector_mode_valid(struct drm_connector *connector, | 1394 | static int tda998x_connector_mode_valid(struct drm_connector *connector, |
1399 | struct drm_display_mode *mode) | 1395 | struct drm_display_mode *mode) |
1400 | { | 1396 | { |
1401 | struct tda998x_priv2 *priv = conn_to_tda998x_priv2(connector); | 1397 | struct tda998x_priv *priv = conn_to_tda998x_priv(connector); |
1402 | 1398 | ||
1403 | return tda998x_encoder_mode_valid(&priv->base, mode); | 1399 | return tda998x_encoder_mode_valid(priv, mode); |
1404 | } | 1400 | } |
1405 | 1401 | ||
1406 | static struct drm_encoder * | 1402 | static struct drm_encoder * |
1407 | tda998x_connector_best_encoder(struct drm_connector *connector) | 1403 | tda998x_connector_best_encoder(struct drm_connector *connector) |
1408 | { | 1404 | { |
1409 | struct tda998x_priv2 *priv = conn_to_tda998x_priv2(connector); | 1405 | struct tda998x_priv *priv = conn_to_tda998x_priv(connector); |
1410 | 1406 | ||
1411 | return &priv->base.encoder; | 1407 | return &priv->encoder; |
1412 | } | 1408 | } |
1413 | 1409 | ||
1414 | static | 1410 | static |
@@ -1421,9 +1417,9 @@ const struct drm_connector_helper_funcs tda998x_connector_helper_funcs = { | |||
1421 | static enum drm_connector_status | 1417 | static enum drm_connector_status |
1422 | tda998x_connector_detect(struct drm_connector *connector, bool force) | 1418 | tda998x_connector_detect(struct drm_connector *connector, bool force) |
1423 | { | 1419 | { |
1424 | struct tda998x_priv2 *priv = conn_to_tda998x_priv2(connector); | 1420 | struct tda998x_priv *priv = conn_to_tda998x_priv(connector); |
1425 | 1421 | ||
1426 | return tda998x_encoder_detect(&priv->base); | 1422 | return tda998x_encoder_detect(priv); |
1427 | } | 1423 | } |
1428 | 1424 | ||
1429 | static void tda998x_connector_destroy(struct drm_connector *connector) | 1425 | static void tda998x_connector_destroy(struct drm_connector *connector) |
@@ -1444,7 +1440,7 @@ static int tda998x_bind(struct device *dev, struct device *master, void *data) | |||
1444 | struct tda998x_encoder_params *params = dev->platform_data; | 1440 | struct tda998x_encoder_params *params = dev->platform_data; |
1445 | struct i2c_client *client = to_i2c_client(dev); | 1441 | struct i2c_client *client = to_i2c_client(dev); |
1446 | struct drm_device *drm = data; | 1442 | struct drm_device *drm = data; |
1447 | struct tda998x_priv2 *priv; | 1443 | struct tda998x_priv *priv; |
1448 | u32 crtcs = 0; | 1444 | u32 crtcs = 0; |
1449 | int ret; | 1445 | int ret; |
1450 | 1446 | ||
@@ -1463,58 +1459,58 @@ static int tda998x_bind(struct device *dev, struct device *master, void *data) | |||
1463 | crtcs = 1 << 0; | 1459 | crtcs = 1 << 0; |
1464 | } | 1460 | } |
1465 | 1461 | ||
1466 | priv->base.connector.interlace_allowed = 1; | 1462 | priv->connector.interlace_allowed = 1; |
1467 | priv->base.encoder.possible_crtcs = crtcs; | 1463 | priv->encoder.possible_crtcs = crtcs; |
1468 | 1464 | ||
1469 | ret = tda998x_create(client, &priv->base); | 1465 | ret = tda998x_create(client, priv); |
1470 | if (ret) | 1466 | if (ret) |
1471 | return ret; | 1467 | return ret; |
1472 | 1468 | ||
1473 | if (!dev->of_node && params) | 1469 | if (!dev->of_node && params) |
1474 | tda998x_encoder_set_config(&priv->base, params); | 1470 | tda998x_encoder_set_config(priv, params); |
1475 | 1471 | ||
1476 | tda998x_encoder_set_polling(&priv->base, &priv->base.connector); | 1472 | tda998x_encoder_set_polling(priv, &priv->connector); |
1477 | 1473 | ||
1478 | drm_encoder_helper_add(&priv->base.encoder, &tda998x_encoder_helper_funcs); | 1474 | drm_encoder_helper_add(&priv->encoder, &tda998x_encoder_helper_funcs); |
1479 | ret = drm_encoder_init(drm, &priv->base.encoder, &tda998x_encoder_funcs, | 1475 | ret = drm_encoder_init(drm, &priv->encoder, &tda998x_encoder_funcs, |
1480 | DRM_MODE_ENCODER_TMDS); | 1476 | DRM_MODE_ENCODER_TMDS); |
1481 | if (ret) | 1477 | if (ret) |
1482 | goto err_encoder; | 1478 | goto err_encoder; |
1483 | 1479 | ||
1484 | drm_connector_helper_add(&priv->base.connector, | 1480 | drm_connector_helper_add(&priv->connector, |
1485 | &tda998x_connector_helper_funcs); | 1481 | &tda998x_connector_helper_funcs); |
1486 | ret = drm_connector_init(drm, &priv->base.connector, | 1482 | ret = drm_connector_init(drm, &priv->connector, |
1487 | &tda998x_connector_funcs, | 1483 | &tda998x_connector_funcs, |
1488 | DRM_MODE_CONNECTOR_HDMIA); | 1484 | DRM_MODE_CONNECTOR_HDMIA); |
1489 | if (ret) | 1485 | if (ret) |
1490 | goto err_connector; | 1486 | goto err_connector; |
1491 | 1487 | ||
1492 | ret = drm_connector_register(&priv->base.connector); | 1488 | ret = drm_connector_register(&priv->connector); |
1493 | if (ret) | 1489 | if (ret) |
1494 | goto err_sysfs; | 1490 | goto err_sysfs; |
1495 | 1491 | ||
1496 | priv->base.connector.encoder = &priv->base.encoder; | 1492 | priv->connector.encoder = &priv->encoder; |
1497 | drm_mode_connector_attach_encoder(&priv->base.connector, &priv->base.encoder); | 1493 | drm_mode_connector_attach_encoder(&priv->connector, &priv->encoder); |
1498 | 1494 | ||
1499 | return 0; | 1495 | return 0; |
1500 | 1496 | ||
1501 | err_sysfs: | 1497 | err_sysfs: |
1502 | drm_connector_cleanup(&priv->base.connector); | 1498 | drm_connector_cleanup(&priv->connector); |
1503 | err_connector: | 1499 | err_connector: |
1504 | drm_encoder_cleanup(&priv->base.encoder); | 1500 | drm_encoder_cleanup(&priv->encoder); |
1505 | err_encoder: | 1501 | err_encoder: |
1506 | tda998x_destroy(&priv->base); | 1502 | tda998x_destroy(priv); |
1507 | return ret; | 1503 | return ret; |
1508 | } | 1504 | } |
1509 | 1505 | ||
1510 | static void tda998x_unbind(struct device *dev, struct device *master, | 1506 | static void tda998x_unbind(struct device *dev, struct device *master, |
1511 | void *data) | 1507 | void *data) |
1512 | { | 1508 | { |
1513 | struct tda998x_priv2 *priv = dev_get_drvdata(dev); | 1509 | struct tda998x_priv *priv = dev_get_drvdata(dev); |
1514 | 1510 | ||
1515 | drm_connector_cleanup(&priv->base.connector); | 1511 | drm_connector_cleanup(&priv->connector); |
1516 | drm_encoder_cleanup(&priv->base.encoder); | 1512 | drm_encoder_cleanup(&priv->encoder); |
1517 | tda998x_destroy(&priv->base); | 1513 | tda998x_destroy(priv); |
1518 | } | 1514 | } |
1519 | 1515 | ||
1520 | static const struct component_ops tda998x_ops = { | 1516 | static const struct component_ops tda998x_ops = { |