On Mon, Oct 30, 2023 at 08:41:48PM +0100, Artur Weber wrote:

+	for (i = 0; i < pmctrl_count; i++) {
+		ret = regmap_write(regmap, pmctrl_addr + i, mode_mask);
+		if (ret)
+			return ret;
+	}

Why not a bulk write?  What happens when the new values are partially
written, both if there's an error and just transiently?