diff options
| author | Hans Verkuil <hverkuil-cisco@xs4all.nl> | 2023-01-26 13:33:10 +0100 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@kernel.org> | 2023-02-08 08:31:49 +0100 |
| commit | 5a1a39a8ac30c7f116d427f7271927d976e4723e (patch) | |
| tree | dc301ab109f7e05834bcb9a851810e942aae686c /drivers/media | |
| parent | media: dvb-frontends: cxd2880: return 0 instead of 'ret'. (diff) | |
| download | linux-5a1a39a8ac30c7f116d427f7271927d976e4723e.tar.xz linux-5a1a39a8ac30c7f116d427f7271927d976e4723e.zip | |
media: marvell: change return to goto for proper unwind
The probe function used 'goto out' everywhere, except in one
place where it returned an error. That too should be a 'goto out'.
This fixes a smatch warning:
mmp-driver.c:257 mmpcam_probe() warn: missing unwind goto?
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
Diffstat (limited to 'drivers/media')
| -rw-r--r-- | drivers/media/platform/marvell/mmp-driver.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/platform/marvell/mmp-driver.c b/drivers/media/platform/marvell/mmp-driver.c index df16899ab1cb..ef22bf8f276c 100644 --- a/drivers/media/platform/marvell/mmp-driver.c +++ b/drivers/media/platform/marvell/mmp-driver.c @@ -254,7 +254,7 @@ static int mmpcam_probe(struct platform_device *pdev) */ ret = mccic_register(mcam); if (ret) - return ret; + goto out; /* * Add OF clock provider. |