[PATCH] Bluetooth: hci_ll: Use the devm_clk_get_optional() helper

From: Christophe JAILLET
Date: Sat May 20 2023 - 11:17:53 EST


Use devm_clk_get_optional() instead of hand writing it.
This is slightly less verbose and improves the semantic.

This also simplifies ll_open() because clk_prepare_enable() already handles
NULL clk.

Signed-off-by: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx>
---
drivers/bluetooth/hci_ll.c | 7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/drivers/bluetooth/hci_ll.c b/drivers/bluetooth/hci_ll.c
index 4a0b5c3160c2..de463d8bf265 100644
--- a/drivers/bluetooth/hci_ll.c
+++ b/drivers/bluetooth/hci_ll.c
@@ -129,8 +129,7 @@ static int ll_open(struct hci_uart *hu)
if (hu->serdev) {
struct ll_device *lldev = serdev_device_get_drvdata(hu->serdev);

- if (!IS_ERR(lldev->ext_clk))
- clk_prepare_enable(lldev->ext_clk);
+ clk_prepare_enable(lldev->ext_clk);
}

return 0;
@@ -703,8 +702,8 @@ static int hci_ti_probe(struct serdev_device *serdev)
if (IS_ERR(lldev->enable_gpio))
return PTR_ERR(lldev->enable_gpio);

- lldev->ext_clk = devm_clk_get(&serdev->dev, "ext_clock");
- if (IS_ERR(lldev->ext_clk) && PTR_ERR(lldev->ext_clk) != -ENOENT)
+ lldev->ext_clk = devm_clk_get_optional(&serdev->dev, "ext_clock");
+ if (IS_ERR(lldev->ext_clk))
return PTR_ERR(lldev->ext_clk);

of_property_read_u32(serdev->dev.of_node, "max-speed", &max_speed);
--
2.34.1