[media] it913x: add chip device ids for binding
authorAntti Palosaari <crope@iki.fi>
Thu, 10 Nov 2016 03:24:50 +0000 (01:24 -0200)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Tue, 31 Jan 2017 12:50:34 +0000 (10:50 -0200)
Driver supports 2 different device versions, AX and BX. Use device
IDs to pass chip version information to driver.

Signed-off-by: Antti Palosaari <crope@iki.fi>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
drivers/media/tuners/it913x.c
drivers/media/tuners/it913x.h
drivers/media/usb/dvb-usb-v2/af9035.c

index 39998066a347708f6ae23b9a517a17d20dac6d4e..27e5bc1c3cb5d8b84f5f7cdab041c3cb2b1f5fff 100644 (file)
@@ -390,6 +390,7 @@ static int it913x_probe(struct platform_device *pdev)
        struct it913x_platform_data *pdata = pdev->dev.platform_data;
        struct dvb_frontend *fe = pdata->fe;
        struct it913x_dev *dev;
+       const struct platform_device_id *id = platform_get_device_id(pdev);
        int ret;
        char *chip_ver_str;
 
@@ -403,7 +404,7 @@ static int it913x_probe(struct platform_device *pdev)
        dev->pdev = pdev;
        dev->regmap = pdata->regmap;
        dev->fe = pdata->fe;
-       dev->chip_ver = pdata->chip_ver;
+       dev->chip_ver = id->driver_data;
        dev->role = pdata->role;
 
        fe->tuner_priv = dev;
@@ -441,6 +442,13 @@ static int it913x_remove(struct platform_device *pdev)
        return 0;
 }
 
+static const struct platform_device_id it913x_id_table[] = {
+       {"it9133ax-tuner", 1},
+       {"it9133bx-tuner", 2},
+       {},
+};
+MODULE_DEVICE_TABLE(platform, it913x_id_table);
+
 static struct platform_driver it913x_driver = {
        .driver = {
                .name   = "it913x",
@@ -448,6 +456,7 @@ static struct platform_driver it913x_driver = {
        },
        .probe          = it913x_probe,
        .remove         = it913x_remove,
+       .id_table       = it913x_id_table,
 };
 
 module_platform_driver(it913x_driver);
index 9fb8e028176d342e8d7a75687418540c1971e3ae..226f657228fbbd5a0f12166c1ffec6745d17d8c8 100644 (file)
  * struct it913x_platform_data - Platform data for the it913x driver
  * @regmap: af9033 demod driver regmap.
  * @dvb_frontend: af9033 demod driver DVB frontend.
- * @chip_ver: Used chip version. 1=IT9133 AX, 2=IT9133 BX.
  * @role: Chip role, single or dual configuration.
  */
 
 struct it913x_platform_data {
        struct regmap *regmap;
        struct dvb_frontend *fe;
-       unsigned int chip_ver:2;
 #define IT913X_ROLE_SINGLE         0
 #define IT913X_ROLE_DUAL_MASTER    1
 #define IT913X_ROLE_DUAL_SLAVE     2
        unsigned int role:2;
 };
 
-/* Backwards compatibility */
-#define it913x_config it913x_platform_data
-
 #endif
index a6ecd52b5917547fdfe9f3d81eab53b4b3596b51..cb8204387d64c40764ebef1e9f345df48999d37c 100644 (file)
@@ -1495,6 +1495,7 @@ static int af9035_tuner_attach(struct dvb_usb_adapter *adap)
        case AF9033_TUNER_IT9135_62:
        {
                struct platform_device *pdev;
+               const char *name;
                struct it913x_platform_data it913x_pdata = {
                        .regmap = state->af9033_config[adap->id].regmap,
                        .fe = adap->fe[0],
@@ -1504,12 +1505,12 @@ static int af9035_tuner_attach(struct dvb_usb_adapter *adap)
                case AF9033_TUNER_IT9135_38:
                case AF9033_TUNER_IT9135_51:
                case AF9033_TUNER_IT9135_52:
-                       it913x_pdata.chip_ver = 1;
+                       name = "it9133ax-tuner";
                        break;
                case AF9033_TUNER_IT9135_60:
                case AF9033_TUNER_IT9135_61:
                case AF9033_TUNER_IT9135_62:
-                       it913x_pdata.chip_ver = 2;
+                       name = "it9133bx-tuner";
                        break;
                default:
                        ret = -ENODEV;
@@ -1526,8 +1527,7 @@ static int af9035_tuner_attach(struct dvb_usb_adapter *adap)
                }
 
                request_module("%s", "it913x");
-               pdev = platform_device_register_data(&d->intf->dev,
-                                                    "it913x",
+               pdev = platform_device_register_data(&d->intf->dev, name,
                                                     PLATFORM_DEVID_AUTO,
                                                     &it913x_pdata,
                                                     sizeof(it913x_pdata));