Hi Juuso,

On 2026-08-12T09:48:39, Juuso Rinta <[email protected]> wrote:

> The hardware still performs the actual clamping; max_timeout_ms only
> lets the uclass report the effective value in the start-path print.

Since the uclass now knows both the requested value and the
driver-reported maximum, would it be cleaner to have the uclass clamp
timeout_ms before calling ops->start(), rather than reporting a value
the driver may or may not honour? The sandbox driver has had to grow
its own clamp in patch 4 purely so the print path is testable. If the
uclass clamped, real drivers would never see an out-of-range value and
the sandbox change in patch 4 would not be needed. What do you think?

Regards,
Simon

Reply via email to