]> git.proxmox.com Git - mirror_ubuntu-jammy-kernel.git/commitdiff
media: vidtv: adapter->mdev was set too late
authorHans Verkuil <hverkuil-cisco@xs4all.nl>
Mon, 1 Feb 2021 08:37:40 +0000 (09:37 +0100)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Fri, 5 Feb 2021 22:52:58 +0000 (23:52 +0100)
The media device has to be initialized and assigned to adapter->mdev
before the dvb devices are created, since that will trigger the
automatic creation of the topology.

Rework this code to achieve this.

Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
drivers/media/test-drivers/vidtv/vidtv_bridge.c

index 6ab17a83bced7ab0c1e08cf29e14a9d4db3097e0..75617709c8ce2dbb23069890735b6801bb0d9efb 100644 (file)
@@ -416,6 +416,7 @@ static int vidtv_bridge_dvb_init(struct vidtv_dvb *dvb)
        ret = vidtv_bridge_register_adap(dvb);
        if (ret < 0)
                goto fail_adapter;
+       dvb_register_media_controller(&dvb->adapter, &dvb->mdev);
 
        for (i = 0; i < NUM_FE; ++i) {
                ret = vidtv_bridge_probe_demod(dvb, i);
@@ -495,6 +496,15 @@ static int vidtv_bridge_probe(struct platform_device *pdev)
 
        dvb->pdev = pdev;
 
+#ifdef CONFIG_MEDIA_CONTROLLER_DVB
+       dvb->mdev.dev = &pdev->dev;
+
+       strscpy(dvb->mdev.model, "vidtv", sizeof(dvb->mdev.model));
+       strscpy(dvb->mdev.bus_info, "platform:vidtv", sizeof(dvb->mdev.bus_info));
+
+       media_device_init(&dvb->mdev);
+#endif
+
        ret = vidtv_bridge_dvb_init(dvb);
        if (ret < 0)
                goto err_dvb;
@@ -504,20 +514,12 @@ static int vidtv_bridge_probe(struct platform_device *pdev)
        platform_set_drvdata(pdev, dvb);
 
 #ifdef CONFIG_MEDIA_CONTROLLER_DVB
-       dvb->mdev.dev = &pdev->dev;
-
-       strscpy(dvb->mdev.model, "vidtv", sizeof(dvb->mdev.model));
-       strscpy(dvb->mdev.bus_info, "platform:vidtv", sizeof(dvb->mdev.bus_info));
-
-       media_device_init(&dvb->mdev);
        ret = media_device_register(&dvb->mdev);
        if (ret) {
                dev_err(dvb->mdev.dev,
                        "media device register failed (err=%d)\n", ret);
                goto err_media_device_register;
        }
-
-       dvb_register_media_controller(&dvb->adapter, &dvb->mdev);
 #endif /* CONFIG_MEDIA_CONTROLLER_DVB */
 
        dev_info(&pdev->dev, "Successfully initialized vidtv!\n");