diff options
| author | Yong Wu <yong.wu@mediatek.com> | 2022-01-13 19:10:55 +0800 |
|---|---|---|
| committer | Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com> | 2022-01-25 17:51:51 +0100 |
| commit | a6945f4566d4f77a4054720f6649ff921fe1ae64 (patch) | |
| tree | 08cc889821b7fed0dfc57a65c3ee3bd5b1bb175f /drivers/memory/mtk-smi.c | |
| parent | 6d86f23c35fe7b479ceef4d3f1eef925996945fd (diff) | |
| download | lwn-a6945f4566d4f77a4054720f6649ff921fe1ae64.tar.gz lwn-a6945f4566d4f77a4054720f6649ff921fe1ae64.zip | |
memory: mtk-smi: handle positive return value for clk_bulk_prepare_enable
Function clk_bulk_prepare_enable() returns 0 for success or a negative
number for error, although the common style for the callers is to check
always for any non-zero return value (just like its implementation in
clk.h does). Adjust the code to such coding style.
Signed-off-by: Yong Wu <yong.wu@mediatek.com>
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Link: https://lore.kernel.org/r/20220113111057.29918-6-yong.wu@mediatek.com
[krzysztof: rewrite commit msg]
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
Diffstat (limited to 'drivers/memory/mtk-smi.c')
| -rw-r--r-- | drivers/memory/mtk-smi.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/memory/mtk-smi.c b/drivers/memory/mtk-smi.c index e201e5976f34..5ebd7176f133 100644 --- a/drivers/memory/mtk-smi.c +++ b/drivers/memory/mtk-smi.c @@ -480,7 +480,7 @@ static int __maybe_unused mtk_smi_larb_resume(struct device *dev) int ret; ret = clk_bulk_prepare_enable(larb->smi.clk_num, larb->smi.clks); - if (ret < 0) + if (ret) return ret; /* Configure the basic setting for this larb */ |