Re: [PATCH 2/4] mfd: rn5t618: Add of compatibles for ADC and power
From: Lee Jones <hidden>
Date: 2021-07-05 07:37:29
Also in:
linux-iio, linux-pm, lkml
On Sat, 03 Jul 2021, Andreas Kemnade wrote:
quoted hunk ↗ jump to hunk
This allows having devicetree nodes for the subdevices. Signed-off-by: Andreas Kemnade <andreas@kemnade.info> --- drivers/mfd/rn5t618.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-)diff --git a/drivers/mfd/rn5t618.c b/drivers/mfd/rn5t618.c index 384acb459427..b916c7471ca3 100644 --- a/drivers/mfd/rn5t618.c +++ b/drivers/mfd/rn5t618.c@@ -24,8 +24,10 @@ static const struct mfd_cell rn5t618_cells[] = { }; static const struct mfd_cell rc5t619_cells[] = { - { .name = "rn5t618-adc" }, - { .name = "rn5t618-power" }, + { .name = "rn5t618-adc", + .of_compatible = "ricoh,rc5t619-adc" }, + { .name = "rn5t618-power", + .of_compatible = "ricoh,rc5t619-power" },
If you're converting entries from single to multi-line, you should place the coding lines one different ones to the brackets. Also, if ordering is unimportant, could you move the multi-line entries to the bottom please?
{ .name = "rn5t618-regulator" },
{ .name = "rc5t619-rtc" },
{ .name = "rn5t618-wdt" },-- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog