From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marex@denx.de>
Date: Tue, 15 Oct 2019 01:28:16 +0200
Subject: [U-Boot] [RESEND PATCH 2/5] usb: host: dwc2: add support for clk
In-Reply-To: <20191014080025.11245-3-patrick.delaunay@st.com>
References: <20191014080025.11245-1-patrick.delaunay@st.com>
 <20191014080025.11245-3-patrick.delaunay@st.com>
Message-ID: <77804378-5ea1-4b37-cb54-8fce77351d9b@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On 10/14/19 10:00 AM, Patrick Delaunay wrote:
> Add support for clock with driver model.
> 

Same question as with the PHY -- is there now a mandatory dependency on
the DM CLK ?

[...]

> @@ -1403,6 +1429,7 @@ static int dwc2_usb_remove(struct udevice *dev)
>  	dwc2_uninit_common(priv->regs);
>  
>  	reset_release_bulk(&priv->resets);
> +	clk_release_bulk(&priv->clks);

Shouldn't there be some clk_...disable() here ?