quoted
quoted
quoted
quoted
"Mark" == Mark Brown [off-list ref] writes:
Mark> On Thu, Oct 06, 2011 at 05:41:34PM +0200, Peter Korsgaard wrote:
>> + trng->clk = clk_get(&pdev->dev, NULL);
>> + if (IS_ERR(trng->clk))
>> + return PTR_ERR(trng->clk);
>> +
>> + ret = clk_enable(trng->clk);
>> + if (ret)
>> + goto err_enable;
Mark> Is it worth enabling the hardware enabled only when the device is open?
Potentially, though I doubt it uses significant power. I kept it like
this for simplicity, to match the other drivers and because hw_random
doesn't have an open/close callback (init is called right away in
hwrng_register or on current change).
--
Bye, Peter Korsgaard