From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Jones Subject: Re: [PATCH v2 3/4] mfd: axp20x: Add a cell for the usb power_supply part of the axp20x PMICs Date: Wed, 24 Jun 2015 12:23:33 +0100 Message-ID: <20150624112333.GH15013@x1> References: <1434211748-25572-1-git-send-email-hdegoede@redhat.com> <1434211748-25572-4-git-send-email-hdegoede@redhat.com> Reply-To: lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Return-path: Content-Disposition: inline In-Reply-To: <1434211748-25572-4-git-send-email-hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> List-Post: , List-Help: , List-Archive: , List-Unsubscribe: , To: Hans de Goede Cc: Sebastian Reichel , Dmitry Eremin-Solenikov , David Woodhouse , Maxime Ripard , Bruno =?iso-8859-1?Q?Pr=E9mont?= , linux-pm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree , linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org List-Id: devicetree@vger.kernel.org > Add a cell for the usb power_supply part of the axp20x PMICs. Why are you duplicating the subject line? > Note that this cell is only for the usb power_supply part and not the > ac-power / battery-charger / rtc-backup-bat-charger bits. >=20 > Depending on the board each of those must be enabled / disabled separatel= y > in devicetree as most boards do not use all 4. So in dt each one needs it= s > own child-node of the axp20x node. Another reason for using separate chil= d > nodes for each is so that other devicetree nodes can have a power-supply > property with a phandle referencing a node representing a single > power-supply. >=20 > The decision to use a separate devicetree node for each is reflected on > the kernel side by each getting its own mfd-cell / platform_device and > platform-driver. You don't really need to say any of this, as this is the 'norm'. What you didn't mention however, is that you're taking the opportunity to fix some formatting issues and that there are no functional changes in these lines. > Cc: Bruno Pr=C3=A9mont > Signed-off-by: Hans de Goede > --- > Changes in v2: > -Use DEFINE_RES_IRQ_NAMED > -Change indentation of axp20x_cells initializers to avoid line wrapping > --- > drivers/mfd/axp20x.c | 20 ++++++++++++++++---- > 1 file changed, 16 insertions(+), 4 deletions(-) Patch looks okay however: Acked-by: Lee Jones > diff --git a/drivers/mfd/axp20x.c b/drivers/mfd/axp20x.c > index f9a3c2d..ca4a604 100644 > --- a/drivers/mfd/axp20x.c > +++ b/drivers/mfd/axp20x.c > @@ -113,6 +113,13 @@ static struct resource axp20x_pek_resources[] =3D { > }, > }; > =20 > +static struct resource axp20x_usb_power_supply_resources[] =3D { > + DEFINE_RES_IRQ_NAMED(AXP20X_IRQ_VBUS_PLUGIN, "VBUS_PLUGIN"), > + DEFINE_RES_IRQ_NAMED(AXP20X_IRQ_VBUS_REMOVAL, "VBUS_REMOVAL"), > + DEFINE_RES_IRQ_NAMED(AXP20X_IRQ_VBUS_VALID, "VBUS_VALID"), > + DEFINE_RES_IRQ_NAMED(AXP20X_IRQ_VBUS_NOT_VALID, "VBUS_NOT_VALID"), > +}; > + > static struct resource axp22x_pek_resources[] =3D { > { > .name =3D "PEK_DBR", > @@ -363,11 +370,16 @@ static const struct regmap_irq_chip axp288_regmap_i= rq_chip =3D { > =20 > static struct mfd_cell axp20x_cells[] =3D { > { > - .name =3D "axp20x-pek", > - .num_resources =3D ARRAY_SIZE(axp20x_pek_resources), > - .resources =3D axp20x_pek_resources, > + .name =3D "axp20x-pek", > + .num_resources =3D ARRAY_SIZE(axp20x_pek_resources), > + .resources =3D axp20x_pek_resources, > }, { > - .name =3D "axp20x-regulator", > + .name =3D "axp20x-regulator", > + }, { > + .name =3D "axp20x-usb-power-supply", > + .of_compatible =3D "x-powers,axp202-usb-power-supply", > + .num_resources =3D ARRAY_SIZE(axp20x_usb_power_supply_resources), > + .resources =3D axp20x_usb_power_supply_resources, > }, > }; > =20 --=20 Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org =E2=94=82 Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog --=20 You received this message because you are subscribed to the Google Groups "= linux-sunxi" group. To unsubscribe from this group and stop receiving emails from it, send an e= mail to linux-sunxi+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org For more options, visit https://groups.google.com/d/optout.