Re: [v13 2/2] pwm: Add Aspeed ast2600 PWM support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Philipp,

On 2021/11/30, 5:52 PM, "Philipp Zabel" <p.zabel@xxxxxxxxxxxxxx> wrote:

    On Mon, 2021-11-29 at 14:43 +0800, Billy Tsai wrote:
    [...]
    >   > +	ret = clk_prepare_enable(priv->clk);
    >   > +	if (ret)
    >   > +		return dev_err_probe(dev, ret, "Couldn't enable clock\n");
    >   > +
    >   > +	ret = reset_control_deassert(priv->reset);
    >   > +	if (ret) {
    >   > +		dev_err_probe(dev, ret, "Couldn't deassert reset control\n");
    >   > +		goto err_disable_clk;
    >   > +	}

    >   Is there any reason to keep the clocks running and the controller out of
    >   reset while the PWM outputs are disabled?

Can you tell me about your concerns with this process?
In my opinion, they are used to provide the clock and de-assert the reset of the PWM engine. If we didn't release
them in probe stage the CPU can't and shouldn't read the register of the PWM engine when call the get_state.
Assume that we want to adjust them dynamically, the driver needs to add more conditions to check and keep the status
of each PWM channel, which is not a good deal for the server platform.

Thanks

Best Regards,
Billy Tsai






[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux