Minki/linux
1
0
Fork 1
mirror of https://github.com/torvalds/linux.git synced 2024-12-25 20:32:22 +00:00
Code Issues Packages Projects Releases Wiki Activity

staging: most: dim2: use if statements instead of ?: expressions

Browse Source 
For better maintainability, replace conditional expressions with if
statements in the drivers' probe routine.

Signed-off-by: Nikita Yushchenko <nikita.yoush@cogentembedded.com>
Link: https://lore.kernel.org/r/20210927160649.29209-1-nikita.yoush@cogentembedded.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Nikita Yushchenko 2021-09-27 19:06:49 +03:00 committed by Greg Kroah-Hartman
parent 9b27a62d11
commit bd46a1f12c
1 changed files with 12 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
drivers/staging/most/dim2/dim2.c
View File

@ -732,7 +732,8 @@ static int dim2_probe(struct platform_device *pdev)
struct dim2_hdm *dev;
struct resource *res;
int ret, i;
u8 dev_fcnt, hal_ret;
u8 hal_ret;
u8 dev_fcnt = fcnt;
int irq;
enum { MLB_INT_IDX, AHB0_INT_IDX };
@ -765,16 +766,16 @@ static int dim2_probe(struct platform_device *pdev)
of_id = of_match_node(dim2_of_match, pdev->dev.of_node);
pdata = of_id->data;
ret = pdata && pdata->enable ? pdata->enable(pdev) : 0;
if (ret)
return ret;
dev->disable_platform = pdata ? pdata->disable : NULL;
if (pdata && pdata->fcnt)
dev_fcnt = pdata->fcnt;
else
dev_fcnt = fcnt;
if (pdata) {
if (pdata->enable) {
ret = pdata->enable(pdev);
if (ret)
return ret;
}
dev->disable_platform = pdata->disable;
if (pdata->fcnt)
dev_fcnt = pdata->fcnt;
}
dev_info(&pdev->dev, "sync: num of frames per sub-buffer: %u\n",
dev_fcnt);