[v3 2/3] pwm: Add Aspeed ast2600 PWM support
billy_tsai at aspeedtech.com
Mon Nov 7 19:47:35 AEDT 2022
On 2022/11/2, 11:56 PM, "Biju Das" <biju.das.jz at bp.renesas.com> wrote:
> > + parent_dev = of_find_device_by_node(np);
> > + priv->clk = devm_clk_get_enabled(&parent_dev->dev, NULL);
> > + if (IS_ERR(priv->clk))
> > + return dev_err_probe(dev, PTR_ERR(priv->clk),
> > + "Couldn't get clock\n");
> What is the use case?
> Is pwm configured by boot loader initially ?
> pwm configured by Linux, not by the bootloader initially?
> Driver needs to handle both cases?
> Just asking, because you are turning on the clock unnecessarily here,
> If you need to address all the use cases. If it is just first one, then
> It is ok.
The driver want to handle all of the use cases. Can you tell me why turning on the clock
is unnecessarily here?
More information about the Linux-aspeed