On 10/12/20 8:23 AM, Peng Fan wrote:
[...]
> +static int nop_phy_probe(struct udevice *dev)
> +{
> + struct nop_phy_priv *priv = dev_get_priv(dev);
> + int ret;
> +
> + if (CONFIG_IS_ENABLED(CLK)) {
> + ret = clk_get_by_name(dev, "main_clk", &priv->nop_clk);So is this main_clk or nop_clk ? Wouldn't it be better if the NOP PHY used the clk_bulk and enabled all clock described in DT ?
