Claude review: mfd: lm3533: Remove driver specific regmap wrappers

[Claude Code Review Bot <claude-review@example.com>]

Patch Review

**Bug: Swapped mask/val in lm3533_als_set_int_mode.**

The old `lm3533_update(lm3533, reg, val, mask)` wrapper internally calls `regmap_update_bits(regmap, reg, mask, val)` — the wrapper takes `(val, mask)` while regmap takes `(mask, val)`. Most call sites are converted correctly, but one is not:

```c
-	ret = lm3533_update(als->lm3533, LM3533_REG_ALS_ZONE_INFO, val, mask);
+	ret = regmap_update_bits(als->lm3533->regmap, LM3533_REG_ALS_ZONE_INFO,
+				 val, mask);
```

This should be `mask, val` (not `val, mask`). When `enable=false`, `val=0` and `mask=LM3533_ALS_INT_ENABLE_MASK`. The direct call becomes `regmap_update_bits(regmap, reg, 0, mask)` — a mask of 0 means no bits are touched, so disabling ALS interrupts becomes a no-op. The correct conversion is:

```c
	ret = regmap_update_bits(als->lm3533->regmap, LM3533_REG_ALS_ZONE_INFO,
				 mask, val);
```

All other `lm3533_update` → `regmap_update_bits` conversions in this patch are correct.

The `u8` → `u32` type changes for regmap_read output pointers are correct and necessary since `regmap_read` requires `unsigned int *`.

---
Generated by Claude Code Patch Reviewer