On 12/8/22 17:53, Xavier Drudis Ferran wrote:
[...]
+++ b/drivers/phy/rockchip/phy-rockchip-inno-usb2.c
@@ -7,7 +7,7 @@
*/
#include <common.h>
-#include <clk.h>
+#include <clk-uclass.h>
#include <dm.h>
#include <asm/global_data.h>
#include <dm/device_compat.h>
@@ -168,6 +168,9 @@ static struct phy_ops rockchip_usb2phy_ops = {
.of_xlate = rockchip_usb2phy_of_xlate,
};
+static struct clk_ops rockchip_usb2phy_clk_ops = {
+};
Is this empty structure needed ? Why ?
Either it shouldn't be here, or it should implement some callbacks, like
clock enable/disable ?
Otherwise looks much better, thanks !