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  wrote:
> Hi,
>
> On Mon, Oct 12, 2015 at 03:20:10PM -0700, Tony Lindgren wrote:
>> * Tony Lindgren  [151012 14:43]:
>> > * Belisko Marek  [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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

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  wrote:
> Hi,
>
> On Mon, Oct 12, 2015 at 03:20:10PM -0700, Tony Lindgren wrote:
>> * Tony Lindgren  [151012 14:43]:
>> > * Belisko Marek  [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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

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

[Sebastian Reichel]

Hi,

On Mon, Oct 12, 2015 at 03:20:10PM -0700, Tony Lindgren wrote:
> * Tony Lindgren  [151012 14:43]:
> > * Belisko Marek  [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


signature.asc
Description: PGP signature

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

[Tony Lindgren]

* Tony Lindgren  [151012 14:43]:
> * Belisko Marek  [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 linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

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

[Tony Lindgren]

* Belisko Marek  [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 linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

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

[Tony Lindgren]

* Tony Lindgren  [151012 14:43]:
> * Belisko Marek  [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 linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

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

[Sebastian Reichel]

Hi,

On Mon, Oct 12, 2015 at 03:20:10PM -0700, Tony Lindgren wrote:
> * Tony Lindgren  [151012 14:43]:
> > * Belisko Marek  [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


signature.asc
Description: PGP signature

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

[Tony Lindgren]

* Belisko Marek  [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 linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

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  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 
>> ---
>>  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 = <_madc 11>;
>> + io-channel-name = "vac";
>>   bci3v1-supply = <>;
>>   };
>
> Acked-By: Sebastian Reichel 
>
> This patch is missing correct To/Cc:
>
> $ grep -A5 "OMAP DEVICE TREE SUPPORT" MAINTAINERS
> OMAP DEVICE TREE SUPPORT
> M:  Benoît Cousson 
> M:  Tony Lindgren 
> L:  linux-o...@vger.kernel.org
> L:  devicet...@vger.kernel.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 linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

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

[Sebastian Reichel]

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 
> ---
>  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 = <_madc 11>;
> + io-channel-name = "vac";
>   bci3v1-supply = <>;
>   };

Acked-By: Sebastian Reichel 

This patch is missing correct To/Cc:

$ grep -A5 "OMAP DEVICE TREE SUPPORT" MAINTAINERS
OMAP DEVICE TREE SUPPORT
M:  Benoît Cousson 
M:  Tony Lindgren 
L:  linux-o...@vger.kernel.org
L:  devicet...@vger.kernel.org
S:  Maintained

-- Sebastian


signature.asc
Description: Digital signature

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

[Sebastian Reichel]

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 
> ---
>  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 = <_madc 11>;
> + io-channel-name = "vac";
>   bci3v1-supply = <>;
>   };

Acked-By: Sebastian Reichel 

This patch is missing correct To/Cc:

$ grep -A5 "OMAP DEVICE TREE SUPPORT" MAINTAINERS
OMAP DEVICE TREE SUPPORT
M:  Benoît Cousson 
M:  Tony Lindgren 
L:  linux-o...@vger.kernel.org
L:  devicet...@vger.kernel.org
S:  Maintained

-- Sebastian


signature.asc
Description: Digital signature

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  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 
>> ---
>>  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 = <_madc 11>;
>> + io-channel-name = "vac";
>>   bci3v1-supply = <>;
>>   };
>
> Acked-By: Sebastian Reichel 
>
> This patch is missing correct To/Cc:
>
> $ grep -A5 "OMAP DEVICE TREE SUPPORT" MAINTAINERS
> OMAP DEVICE TREE SUPPORT
> M:  Benoît Cousson 
> M:  Tony Lindgren 
> L:  linux-o...@vger.kernel.org
> L:  devicet...@vger.kernel.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 linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/