On Fri, Mar 24, 2023 at 5:50 PM Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx> wrote: > The semantic of mcp_set_mask() was surprising to me when I first read > that driver. So it was unexpected that in the call > > mcp_set_mask(mcp, MCP_OLAT, mask, value); > > value was a bool. Make the function a thinner wrapper around > regmap_update_bits() and rename it to also have a similar name. > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx> Both patches applied, thanks! Yours, Linus Walleij