On Tue, Sep 6, 2016 at 8:18 PM, Maxime Ripard <maxime.ripard@xxxxxxxxxxxxxxxxxx> wrote: > Some dividers might have a maximum value that is lower than the width of > the register. > > Add a field to _ccu_div to handle those case properly. If the field is set > to 0, the code will assume that the maximum value is the maximum one that > can be used with the field register width. This is a bit confusing. What is the maximum referring to? The raw value in the register? Or the actual divider? Personally I'd go with the maximum valid value of the register. You could get rid of the special power-of-2 handling code you added. Either way I think the message and the field name should be more explicit. > > Otherwise, we'll use whatever value has been set. > > Signed-off-by: Maxime Ripard <maxime.ripard@xxxxxxxxxxxxxxxxxx> [...] The code looks correct. Regards ChenYu -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html