Hello,
On Mon, Mar 03, 2025 at 01:52:49PM +0200, Abel Vesa wrote:
The PWM allow configuring the PWM resolution from 8 bits PWM values up to 15 bits values, for the Hi-Res PWMs, and then either 6-bit or 9-bit for the normal PWMs. The current implementation loops through all possible resolutions (PWM sizes), for the PWM subtype, on top of the already existing process of determining the prediv, exponent and refclk.
The first and second issues are related to capping the computed PWM value.
I just took a very quick look. I'd say squash the first and second patch into a single one. Splitting a change that fixes the same issue in the two branches of an if condition has no benefit.
Other than that this patch set would also benefit from what I wrote in the review of the other patch you send: Please mention a request where the result becomes wrong. This considerably simplifies understanding your changes.
Thanks Uwe