Re: [PATCH v6 3/4] mfd: aat2870: Convert to use OF bindings

[sashiko-bot@kernel.org]

> Conversion of the AAT2870 driver to use OF bindings requires a few complex
> changes that should be done simultaneously.
> 
> The AAT2870 essentially provides two functions via child devices:
> backlight and regulators. Both functions are fairly self-sufficient and
> may not be populated on the final board. Consequently, the MFD
> registration API was replaced with of_platform_populate(), and each
> sub-device was given its own compatible string.
> 
> Additionally, aat2870-core utilizes an enable GPIO. Obtaining this GPIO
> was converted to use modern gpiod/OF helpers, allowing the redundant
> aat2870_enable() and aat2870_disable() helpers to be removed.
> 
> The aat2870-regulator driver was updated to register each of the four LDOs
> from a dedicated OF node using modern regulator framework.
> 
> The aat2870-backlight driver was changed to populate all required
> properties from a dedicated Device Tree node. Its channel map was updated
> to use u8 instead of int. Furthermore, because the maximum current is now
> parsed as an absolute value rather than an enum entry, the calculation and
> application of the maximum current were updated accordingly.
> 
> All of the changes above allow platform data to be removed entirely.
> 
> Reviewed-by: Daniel Thompson (RISCstar) <danielt@kernel.org> # for backlight
> Signed-off-by: Svyatoslav Ryhel <clamor95@gmail.com>

Sashiko has reviewed this patch and found no issues.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261104123714.234634-1-clamor95@gmail.com?part=3