media: cedrus: Set the platform driver data earlier
commit708938f849upstream. The cedrus_hw_resume() crashes with NULL deference on driver probe if runtime PM is disabled because it uses platform data that hasn't been set up yet. Fix this by setting the platform data earlier during probe. Cc: stable@vger.kernel.org Fixes:50e761516f(media: platform: Add Cedrus VPU decoder driver) Signed-off-by: Dmitry Osipenko <dmitry.osipenko@collabora.com> Signed-off-by: Nicolas Dufresne <nicolas.dufresne@collabora.com> Reviewed-by: Samuel Holland <samuel@sholland.org> Acked-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com> Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl> Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
dbdd3b1448
commit
bda8120e5b
@@ -399,6 +399,8 @@ static int cedrus_probe(struct platform_device *pdev)
|
||||
if (!dev)
|
||||
return -ENOMEM;
|
||||
|
||||
platform_set_drvdata(pdev, dev);
|
||||
|
||||
dev->vfd = cedrus_video_device;
|
||||
dev->dev = &pdev->dev;
|
||||
dev->pdev = pdev;
|
||||
@@ -469,8 +471,6 @@ static int cedrus_probe(struct platform_device *pdev)
|
||||
goto err_m2m_mc;
|
||||
}
|
||||
|
||||
platform_set_drvdata(pdev, dev);
|
||||
|
||||
return 0;
|
||||
|
||||
err_m2m_mc:
|
||||
|
||||
Reference in New Issue
Block a user