Re: [PATCH 01/33] clk_ops: change round_rate() to return unsigned long

From: Bryan O'Donoghue
Date: Sun Dec 31 2017 - 08:40:33 EST


On 30/12/17 16:36, Mikko Perttunen wrote:
FWIW, we had this problem some years ago with the Tegra CPU clock - then it was determined that a simpler solution was to have the determine_rate callback support unsigned long rates - so clock drivers that need to return rates higher than 2^31 can instead implement the determine_rate callback. That is what's currently implemented.

Mikko

Granted we could work around it but, having both zero and less than zero indicate error means you can't support larger than LONG_MAX which is I think worth fixing.

---
bod