diff options
author | Rasmus Villemoes <linux@rasmusvillemoes.dk> | 2022-11-04 09:30:04 +0100 |
---|---|---|
committer | Jakub Kicinski <kuba@kernel.org> | 2022-11-07 17:52:24 -0800 |
commit | ed4314f7729714d788698ade4f9905ee5378ebc0 (patch) | |
tree | 8847ed6b2fea921709ade5603089ff10b27e77ab /Documentation | |
parent | c23fb2c82267638f9d206cb96bb93e1f93ad7828 (diff) | |
download | linux-ed4314f7729714d788698ade4f9905ee5378ebc0.tar.gz |
net: stmmac: dwmac-meson8b: fix meson8b_devm_clk_prepare_enable()
There are two problems with meson8b_devm_clk_prepare_enable(),
introduced in commit a54dc4a49045 ("net: stmmac: dwmac-meson8b:
Make the clock enabling code re-usable"):
- It doesn't pass the clk argument, but instead always the
rgmii_tx_clk of the device.
- It silently ignores the return value of devm_add_action_or_reset().
The former didn't become an actual bug until another user showed up in
the next commit 9308c47640d5 ("net: stmmac: dwmac-meson8b: add support
for the RX delay configuration"). The latter means the callers could
end up with the clock not actually prepared/enabled.
Fixes: a54dc4a49045 ("net: stmmac: dwmac-meson8b: Make the clock enabling code re-usable")
Signed-off-by: Rasmus Villemoes <linux@rasmusvillemoes.dk>
Reviewed-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>
Link: https://lore.kernel.org/r/20221104083004.2212520-1-linux@rasmusvillemoes.dk
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'Documentation')
0 files changed, 0 insertions, 0 deletions