Re: [PATCH v3] ARM: dts: twl4030: Add iio properties for bci subnode

Re: [PATCH v3] ARM: dts: twl4030: Add iio properties for bci subnode

[Belisko Marek]

Tony sorry I forgot to add you to the recipients for this patchset.
Can you please queue this patch. Many thanks.

@Sebastian: thanks for noticing

On Sat, Sep 26, 2015 at 7:29 PM, Sebastian Reichel <sre-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> wrote:
> Hi Marek,
>
> On Fri, Sep 25, 2015 at 10:20:47PM +0200, Marek Belisko wrote:
>> Added new iio properties which are required for twl4030-charger driver and
>> allow to use twl4030-madc indirectly.
>>
>> Signed-off-by: Marek Belisko <marek-xXXSsgcRVICgSpxsJD1C4w@public.gmane.org>
>> ---
>>  arch/arm/boot/dts/twl4030.dtsi | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/twl4030.dtsi b/arch/arm/boot/dts/twl4030.dtsi
>> index 36ae916..482b7aa 100644
>> --- a/arch/arm/boot/dts/twl4030.dtsi
>> +++ b/arch/arm/boot/dts/twl4030.dtsi
>> @@ -22,6 +22,8 @@
>>       charger: bci {
>>               compatible = "ti,twl4030-bci";
>>               interrupts = <9>, <2>;
>> +             io-channels = <&twl4030_madc 11>;
>> +             io-channel-name = "vac";
>>               bci3v1-supply = <&vusb3v1>;
>>       };
>
> Acked-By: Sebastian Reichel <sre-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
>
> This patch is missing correct To/Cc:
>
> $ grep -A5 "OMAP DEVICE TREE SUPPORT" MAINTAINERS
> OMAP DEVICE TREE SUPPORT
> M:      Benoît Cousson <bcousson-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
> M:      Tony Lindgren <tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
> L:      linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> L:      devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> S:      Maintained
>
> -- Sebastian

BR,

marek

-- 
as simple and primitive as possible
-------------------------------------------------
Marek Belisko - OPEN-NANDRA
Freelance Developer

Ruska Nova Ves 219 | Presov, 08005 Slovak Republic
Tel: +421 915 052 184
skype: marekwhite
twitter: #opennandra
web: http://open-nandra.com
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH v3] ARM: dts: twl4030: Add iio properties for bci subnode

[Tony Lindgren]

* Belisko Marek <marek.belisko-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> [150926 13:02]:
> Tony sorry I forgot to add you to the recipients for this patchset.
> Can you please queue this patch. Many thanks.

OK applying into omap-for-v4.4/dt thanks.

Tony
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH v3] ARM: dts: twl4030: Add iio properties for bci subnode

[Tony Lindgren]

* Tony Lindgren <tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org> [151012 14:43]:
> * Belisko Marek <marek.belisko-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> [150926 13:02]:
> > Tony sorry I forgot to add you to the recipients for this patchset.
> > Can you please queue this patch. Many thanks.
> 
> OK applying into omap-for-v4.4/dt thanks.

Actually dropping this one, it makes build fail as we don't
have twl4030_madc yet.

Maybe please send a separate set of dts patches for me.

Regards,

Tony
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH v3] ARM: dts: twl4030: Add iio properties for bci subnode

[Sebastian Reichel]

[-- Attachment #1: Type: text/plain, Size: 846 bytes --]

Hi,

On Mon, Oct 12, 2015 at 03:20:10PM -0700, Tony Lindgren wrote:
> * Tony Lindgren <tony@atomide.com> [151012 14:43]:
> > * Belisko Marek <marek.belisko@gmail.com> [150926 13:02]:
> > > Tony sorry I forgot to add you to the recipients for this patchset.
> > > Can you please queue this patch. Many thanks.
> > 
> > OK applying into omap-for-v4.4/dt thanks.
> 
> Actually dropping this one, it makes build fail as we don't
> have twl4030_madc yet.
> 
> Maybe please send a separate set of dts patches for me.

mh that's strange, twl4030-madc is there since a long time. But
checking the patch, it seems Marek got the phandle wrong:

$ grep -B1 "ti,twl4030-madc" arch/arm/boot/dts/twl4030.dtsi
	twl_madc: madc {
		compatible = "ti,twl4030-madc";

Once that is fixed, the patch should have no dependencies.

-- Sebastian

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

Re: [PATCH v3] ARM: dts: twl4030: Add iio properties for bci subnode

[Belisko Marek]

Hi,

On Tue, Oct 13, 2015 at 1:53 AM, Sebastian Reichel <sre@kernel.org> wrote:
> Hi,
>
> On Mon, Oct 12, 2015 at 03:20:10PM -0700, Tony Lindgren wrote:
>> * Tony Lindgren <tony@atomide.com> [151012 14:43]:
>> > * Belisko Marek <marek.belisko@gmail.com> [150926 13:02]:
>> > > Tony sorry I forgot to add you to the recipients for this patchset.
>> > > Can you please queue this patch. Many thanks.
>> >
>> > OK applying into omap-for-v4.4/dt thanks.
>>
>> Actually dropping this one, it makes build fail as we don't
>> have twl4030_madc yet.
>>
>> Maybe please send a separate set of dts patches for me.
>
> mh that's strange, twl4030-madc is there since a long time. But
> checking the patch, it seems Marek got the phandle wrong:
OK, thanks for letting me know. I'll send update soon. Thanks.
>
> $ grep -B1 "ti,twl4030-madc" arch/arm/boot/dts/twl4030.dtsi
>         twl_madc: madc {
>                 compatible = "ti,twl4030-madc";
>
> Once that is fixed, the patch should have no dependencies.
>
> -- Sebastian

BR,

marek

-- 
as simple and primitive as possible
-------------------------------------------------
Marek Belisko - OPEN-NANDRA
Freelance Developer

Ruska Nova Ves 219 | Presov, 08005 Slovak Republic
Tel: +421 915 052 184
skype: marekwhite
twitter: #opennandra
web: http://open-nandra.com