Thread (4 messages) flat view 4 messages, 2 authors, 2018-01-02

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

From: pure.logic@nexus-software.ie (Bryan O'Donoghue)
Date: 2017-12-31 13:40:17
Also in: dri-devel, linux-amlogic, linux-arm-msm, linux-clk, linux-media, linux-mediatek, linux-mips, linux-omap, linux-renesas-soc, linux-rockchip, linux-rtc, linux-samsung-soc, linux-tegra, lkml

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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help