All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/3] LPC4337 CIAA-NXP DTS changes
@ 2015-11-23 12:37 ` Ariel D'Alessandro
  0 siblings, 0 replies; 32+ messages in thread
From: Ariel D'Alessandro @ 2015-11-23 12:37 UTC (permalink / raw)
  To: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	devicetree-u79uwXL29TY76Z2rM5mHXA
  Cc: Jmanabian-Re5JQEeQqe8AvxtiuMwx3w, robh+dt-DgEjT+Ai2ygdnm+yROfE0A,
	pawel.moll-5wv7dgnIgG8, mark.rutland-5wv7dgnIgG8,
	ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
	galak-sgV2jX0FEOL9JmXXK+q4OQ, linux-lFZ/pmaqli7XmaaqVzeoHQ,
	Ariel D'Alessandro

Hi,

This patch set includes some DTS changes for LPC4337 CIAA-NXP board.
It is based on tag next-20151120 of the linux-next repository. It has
been successfully tested on a LPC4337 CIAA-NXP Board.

Ariel D'Alessandro (3):
  ARM: dts: lpc4337-ciaa: enable EEPROM memory
  ARM: dts: lpc4337-ciaa: enable SCT-PWM
  ARM: dts: lpc4337-ciaa: add i2c0 and devices

 arch/arm/boot/dts/lpc4337-ciaa.dts | 38 ++++++++++++++++++++++++++++++++++++++
 1 file changed, 38 insertions(+)

-- 
2.6.2

--
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

^ permalink raw reply	[flat|nested] 32+ messages in thread
* [PATCH 0/3] LPC4337 CIAA-NXP DTS changes
@ 2015-11-23 12:37 ` Ariel D'Alessandro
  0 siblings, 0 replies; 32+ messages in thread
From: Ariel D'Alessandro @ 2015-11-23 12:37 UTC (permalink / raw)
  To: linux-arm-kernel

Hi,

This patch set includes some DTS changes for LPC4337 CIAA-NXP board.
It is based on tag next-20151120 of the linux-next repository. It has
been successfully tested on a LPC4337 CIAA-NXP Board.

Ariel D'Alessandro (3):
  ARM: dts: lpc4337-ciaa: enable EEPROM memory
  ARM: dts: lpc4337-ciaa: enable SCT-PWM
  ARM: dts: lpc4337-ciaa: add i2c0 and devices

 arch/arm/boot/dts/lpc4337-ciaa.dts | 38 ++++++++++++++++++++++++++++++++++++++
 1 file changed, 38 insertions(+)

-- 
2.6.2

^ permalink raw reply	[flat|nested] 32+ messages in thread
* [PATCH 1/3] ARM: dts: lpc4337-ciaa: enable EEPROM memory
  2015-11-23 12:37 ` Ariel D'Alessandro
@ 2015-11-23 12:37     ` Ariel D'Alessandro
  -1 siblings, 0 replies; 32+ messages in thread
From: Ariel D'Alessandro @ 2015-11-23 12:37 UTC (permalink / raw)
  To: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	devicetree-u79uwXL29TY76Z2rM5mHXA
  Cc: Jmanabian-Re5JQEeQqe8AvxtiuMwx3w, robh+dt-DgEjT+Ai2ygdnm+yROfE0A,
	pawel.moll-5wv7dgnIgG8, mark.rutland-5wv7dgnIgG8,
	ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
	galak-sgV2jX0FEOL9JmXXK+q4OQ, linux-lFZ/pmaqli7XmaaqVzeoHQ,
	Ariel D'Alessandro

The CIAA-NXP board has a NXP LPC4337 SoC that includes a 16 KiB
EEPROM memory.

Signed-off-by: Ariel D'Alessandro <ariel-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org>
---
 arch/arm/boot/dts/lpc4337-ciaa.dts | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/arm/boot/dts/lpc4337-ciaa.dts b/arch/arm/boot/dts/lpc4337-ciaa.dts
index 5f500c1..03bdf77 100644
--- a/arch/arm/boot/dts/lpc4337-ciaa.dts
+++ b/arch/arm/boot/dts/lpc4337-ciaa.dts
@@ -155,6 +155,10 @@
 	};
 };
 
+&eeprom {
+	status = "okay";
+};
+
 &enet_tx_clk {
 	clock-frequency = <50000000>;
 };
-- 
2.6.2

--
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

^ permalink raw reply related	[flat|nested] 32+ messages in thread
* [PATCH 1/3] ARM: dts: lpc4337-ciaa: enable EEPROM memory
@ 2015-11-23 12:37     ` Ariel D'Alessandro
  0 siblings, 0 replies; 32+ messages in thread
From: Ariel D'Alessandro @ 2015-11-23 12:37 UTC (permalink / raw)
  To: linux-arm-kernel

The CIAA-NXP board has a NXP LPC4337 SoC that includes a 16 KiB
EEPROM memory.

Signed-off-by: Ariel D'Alessandro <ariel@vanguardiasur.com.ar>
---
 arch/arm/boot/dts/lpc4337-ciaa.dts | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/arm/boot/dts/lpc4337-ciaa.dts b/arch/arm/boot/dts/lpc4337-ciaa.dts
index 5f500c1..03bdf77 100644
--- a/arch/arm/boot/dts/lpc4337-ciaa.dts
+++ b/arch/arm/boot/dts/lpc4337-ciaa.dts
@@ -155,6 +155,10 @@
 	};
 };
 
+&eeprom {
+	status = "okay";
+};
+
 &enet_tx_clk {
 	clock-frequency = <50000000>;
 };
-- 
2.6.2

^ permalink raw reply related	[flat|nested] 32+ messages in thread
* [PATCH 2/3] ARM: dts: lpc4337-ciaa: enable SCT-PWM
  2015-11-23 12:37 ` Ariel D'Alessandro
@ 2015-11-23 12:37     ` Ariel D'Alessandro
  -1 siblings, 0 replies; 32+ messages in thread
From: Ariel D'Alessandro @ 2015-11-23 12:37 UTC (permalink / raw)
  To: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	devicetree-u79uwXL29TY76Z2rM5mHXA
  Cc: Jmanabian-Re5JQEeQqe8AvxtiuMwx3w, robh+dt-DgEjT+Ai2ygdnm+yROfE0A,
	pawel.moll-5wv7dgnIgG8, mark.rutland-5wv7dgnIgG8,
	ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
	galak-sgV2jX0FEOL9JmXXK+q4OQ, linux-lFZ/pmaqli7XmaaqVzeoHQ,
	Ariel D'Alessandro

Enable the PWM based on the State Configurable Timer (SCT) included in
the LPC4337 SoC of the CIAA-NXP board.

Signed-off-by: Ariel D'Alessandro <ariel-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org>
---
 arch/arm/boot/dts/lpc4337-ciaa.dts | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/arm/boot/dts/lpc4337-ciaa.dts b/arch/arm/boot/dts/lpc4337-ciaa.dts
index 03bdf77..1a23589 100644
--- a/arch/arm/boot/dts/lpc4337-ciaa.dts
+++ b/arch/arm/boot/dts/lpc4337-ciaa.dts
@@ -170,6 +170,10 @@
 	pinctrl-0 = <&enet_rmii_pins>;
 };
 
+&sct_pwm {
+	status = "okay";
+};
+
 &ssp1 {
 	status = "okay";
 	pinctrl-names = "default";
-- 
2.6.2

--
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

^ permalink raw reply related	[flat|nested] 32+ messages in thread
* [PATCH 2/3] ARM: dts: lpc4337-ciaa: enable SCT-PWM
@ 2015-11-23 12:37     ` Ariel D'Alessandro
  0 siblings, 0 replies; 32+ messages in thread
From: Ariel D'Alessandro @ 2015-11-23 12:37 UTC (permalink / raw)
  To: linux-arm-kernel

Enable the PWM based on the State Configurable Timer (SCT) included in
the LPC4337 SoC of the CIAA-NXP board.

Signed-off-by: Ariel D'Alessandro <ariel@vanguardiasur.com.ar>
---
 arch/arm/boot/dts/lpc4337-ciaa.dts | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/arm/boot/dts/lpc4337-ciaa.dts b/arch/arm/boot/dts/lpc4337-ciaa.dts
index 03bdf77..1a23589 100644
--- a/arch/arm/boot/dts/lpc4337-ciaa.dts
+++ b/arch/arm/boot/dts/lpc4337-ciaa.dts
@@ -170,6 +170,10 @@
 	pinctrl-0 = <&enet_rmii_pins>;
 };
 
+&sct_pwm {
+	status = "okay";
+};
+
 &ssp1 {
 	status = "okay";
 	pinctrl-names = "default";
-- 
2.6.2

^ permalink raw reply related	[flat|nested] 32+ messages in thread
* [PATCH 3/3] ARM: dts: lpc4337-ciaa: add i2c0 and devices
  2015-11-23 12:37 ` Ariel D'Alessandro
@ 2015-11-23 12:37     ` Ariel D'Alessandro
  -1 siblings, 0 replies; 32+ messages in thread
From: Ariel D'Alessandro @ 2015-11-23 12:37 UTC (permalink / raw)
  To: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	devicetree-u79uwXL29TY76Z2rM5mHXA
  Cc: Jmanabian-Re5JQEeQqe8AvxtiuMwx3w, robh+dt-DgEjT+Ai2ygdnm+yROfE0A,
	pawel.moll-5wv7dgnIgG8, mark.rutland-5wv7dgnIgG8,
	ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
	galak-sgV2jX0FEOL9JmXXK+q4OQ, linux-lFZ/pmaqli7XmaaqVzeoHQ,
	Ariel D'Alessandro

Add I2C0 and two I2C EEPROM devices on the CIAA-NXP board:

* 24AA1025 EEPROM, 1Mbit: it is accessed as two 512Kbit EEPROMs.
* 24AA025E48 EEPROM, 2kbit.

Signed-off-by: Ariel D'Alessandro <ariel-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org>
---
 arch/arm/boot/dts/lpc4337-ciaa.dts | 30 ++++++++++++++++++++++++++++++
 1 file changed, 30 insertions(+)

diff --git a/arch/arm/boot/dts/lpc4337-ciaa.dts b/arch/arm/boot/dts/lpc4337-ciaa.dts
index 1a23589..d81047a 100644
--- a/arch/arm/boot/dts/lpc4337-ciaa.dts
+++ b/arch/arm/boot/dts/lpc4337-ciaa.dts
@@ -99,6 +99,14 @@
 		};
 	};
 
+	i2c0_pins: i2c0-pins {
+		i2c0_pins_cfg {
+			pins = "i2c0_scl", "i2c0_sda";
+			function = "i2c0";
+			input-enable;
+		};
+	};
+
 	ssp_pins: ssp-pins {
 		ssp1_cs {
 			pins = "p6_7";
@@ -163,6 +171,28 @@
 	clock-frequency = <50000000>;
 };
 
+&i2c0 {
+	status = "okay";
+	pinctrl-names = "default";
+	pinctrl-0 = <&i2c0_pins>;
+	clock-frequency = <400000>;
+
+	eeprom@50 {
+		compatible = "microchip,24c512";
+		reg = <0x50>;
+	};
+
+	eeprom@51 {
+		compatible = "microchip,24c02";
+		reg = <0x51>;
+	};
+
+	eeprom@54 {
+		compatible = "microchip,24c512";
+		reg = <0x54>;
+	};
+};
+
 &mac {
 	status = "okay";
 	phy-mode = "rmii";
-- 
2.6.2

--
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

^ permalink raw reply related	[flat|nested] 32+ messages in thread
* [PATCH 3/3] ARM: dts: lpc4337-ciaa: add i2c0 and devices
@ 2015-11-23 12:37     ` Ariel D'Alessandro
  0 siblings, 0 replies; 32+ messages in thread
From: Ariel D'Alessandro @ 2015-11-23 12:37 UTC (permalink / raw)
  To: linux-arm-kernel

Add I2C0 and two I2C EEPROM devices on the CIAA-NXP board:

* 24AA1025 EEPROM, 1Mbit: it is accessed as two 512Kbit EEPROMs.
* 24AA025E48 EEPROM, 2kbit.

Signed-off-by: Ariel D'Alessandro <ariel@vanguardiasur.com.ar>
---
 arch/arm/boot/dts/lpc4337-ciaa.dts | 30 ++++++++++++++++++++++++++++++
 1 file changed, 30 insertions(+)

diff --git a/arch/arm/boot/dts/lpc4337-ciaa.dts b/arch/arm/boot/dts/lpc4337-ciaa.dts
index 1a23589..d81047a 100644
--- a/arch/arm/boot/dts/lpc4337-ciaa.dts
+++ b/arch/arm/boot/dts/lpc4337-ciaa.dts
@@ -99,6 +99,14 @@
 		};
 	};
 
+	i2c0_pins: i2c0-pins {
+		i2c0_pins_cfg {
+			pins = "i2c0_scl", "i2c0_sda";
+			function = "i2c0";
+			input-enable;
+		};
+	};
+
 	ssp_pins: ssp-pins {
 		ssp1_cs {
 			pins = "p6_7";
@@ -163,6 +171,28 @@
 	clock-frequency = <50000000>;
 };
 
+&i2c0 {
+	status = "okay";
+	pinctrl-names = "default";
+	pinctrl-0 = <&i2c0_pins>;
+	clock-frequency = <400000>;
+
+	eeprom at 50 {
+		compatible = "microchip,24c512";
+		reg = <0x50>;
+	};
+
+	eeprom at 51 {
+		compatible = "microchip,24c02";
+		reg = <0x51>;
+	};
+
+	eeprom at 54 {
+		compatible = "microchip,24c512";
+		reg = <0x54>;
+	};
+};
+
 &mac {
 	status = "okay";
 	phy-mode = "rmii";
-- 
2.6.2

^ permalink raw reply related	[flat|nested] 32+ messages in thread
* Re: [PATCH 0/3] LPC4337 CIAA-NXP DTS changes
  2015-11-23 12:37 ` Ariel D'Alessandro
@ 2015-11-23 12:43     ` Ariel D'Alessandro
  -1 siblings, 0 replies; 32+ messages in thread
From: Ariel D'Alessandro @ 2015-11-23 12:43 UTC (permalink / raw)
  To: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	devicetree-u79uwXL29TY76Z2rM5mHXA
  Cc: manabian-Re5JQEeQqe8AvxtiuMwx3w, robh+dt-DgEjT+Ai2ygdnm+yROfE0A,
	pawel.moll-5wv7dgnIgG8, mark.rutland-5wv7dgnIgG8,
	ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
	galak-sgV2jX0FEOL9JmXXK+q4OQ, linux-lFZ/pmaqli7XmaaqVzeoHQ

(+Cc Joachim)

El 23/11/15 a las 09:37, Ariel D'Alessandro escribió:
> Hi,
> 
> This patch set includes some DTS changes for LPC4337 CIAA-NXP board.
> It is based on tag next-20151120 of the linux-next repository. It has
> been successfully tested on a LPC4337 CIAA-NXP Board.
> 
> Ariel D'Alessandro (3):
>   ARM: dts: lpc4337-ciaa: enable EEPROM memory
>   ARM: dts: lpc4337-ciaa: enable SCT-PWM
>   ARM: dts: lpc4337-ciaa: add i2c0 and devices
> 
>  arch/arm/boot/dts/lpc4337-ciaa.dts | 38 ++++++++++++++++++++++++++++++++++++++
>  1 file changed, 38 insertions(+)
> 

-- 
Ariel D'Alessandro, VanguardiaSur
www.vanguardiasur.com.ar
--
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

^ permalink raw reply	[flat|nested] 32+ messages in thread
* [PATCH 0/3] LPC4337 CIAA-NXP DTS changes
@ 2015-11-23 12:43     ` Ariel D'Alessandro
  0 siblings, 0 replies; 32+ messages in thread
From: Ariel D'Alessandro @ 2015-11-23 12:43 UTC (permalink / raw)
  To: linux-arm-kernel

(+Cc Joachim)

El 23/11/15 a las 09:37, Ariel D'Alessandro escribi?:
> Hi,
> 
> This patch set includes some DTS changes for LPC4337 CIAA-NXP board.
> It is based on tag next-20151120 of the linux-next repository. It has
> been successfully tested on a LPC4337 CIAA-NXP Board.
> 
> Ariel D'Alessandro (3):
>   ARM: dts: lpc4337-ciaa: enable EEPROM memory
>   ARM: dts: lpc4337-ciaa: enable SCT-PWM
>   ARM: dts: lpc4337-ciaa: add i2c0 and devices
> 
>  arch/arm/boot/dts/lpc4337-ciaa.dts | 38 ++++++++++++++++++++++++++++++++++++++
>  1 file changed, 38 insertions(+)
> 

-- 
Ariel D'Alessandro, VanguardiaSur
www.vanguardiasur.com.ar

^ permalink raw reply	[flat|nested] 32+ messages in thread
* Re: [PATCH 0/3] LPC4337 CIAA-NXP DTS changes
  2015-11-23 12:43     ` Ariel D'Alessandro
@ 2015-11-23 18:17         ` Joachim Eastwood
  -1 siblings, 0 replies; 32+ messages in thread
From: Joachim Eastwood @ 2015-11-23 18:17 UTC (permalink / raw)
  To: Ariel D'Alessandro
  Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	devicetree-u79uwXL29TY76Z2rM5mHXA, Rob Herring, Pawel Moll,
	Mark Rutland, Ian Campbell, Kumar Gala, Russell King - ARM Linux

Hi Ariel,

On 23 November 2015 at 13:43, Ariel D'Alessandro
<ariel-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org> wrote:
> (+Cc Joachim)
>
> El 23/11/15 a las 09:37, Ariel D'Alessandro escribió:
>> Hi,
>>
>> This patch set includes some DTS changes for LPC4337 CIAA-NXP board.
>> It is based on tag next-20151120 of the linux-next repository. It has
>> been successfully tested on a LPC4337 CIAA-NXP Board.
>>
>> Ariel D'Alessandro (3):
>>   ARM: dts: lpc4337-ciaa: enable EEPROM memory
>>   ARM: dts: lpc4337-ciaa: enable SCT-PWM
>>   ARM: dts: lpc4337-ciaa: add i2c0 and devices

If you want me to pick up the patches please put me in the To field so
they end up in my inbox.


regards,
Joachim Eastwood
--
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

^ permalink raw reply	[flat|nested] 32+ messages in thread
* [PATCH 0/3] LPC4337 CIAA-NXP DTS changes
@ 2015-11-23 18:17         ` Joachim Eastwood
  0 siblings, 0 replies; 32+ messages in thread
From: Joachim Eastwood @ 2015-11-23 18:17 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Ariel,

On 23 November 2015 at 13:43, Ariel D'Alessandro
<ariel@vanguardiasur.com.ar> wrote:
> (+Cc Joachim)
>
> El 23/11/15 a las 09:37, Ariel D'Alessandro escribi?:
>> Hi,
>>
>> This patch set includes some DTS changes for LPC4337 CIAA-NXP board.
>> It is based on tag next-20151120 of the linux-next repository. It has
>> been successfully tested on a LPC4337 CIAA-NXP Board.
>>
>> Ariel D'Alessandro (3):
>>   ARM: dts: lpc4337-ciaa: enable EEPROM memory
>>   ARM: dts: lpc4337-ciaa: enable SCT-PWM
>>   ARM: dts: lpc4337-ciaa: add i2c0 and devices

If you want me to pick up the patches please put me in the To field so
they end up in my inbox.


regards,
Joachim Eastwood

^ permalink raw reply	[flat|nested] 32+ messages in thread
* Re: [PATCH 0/3] LPC4337 CIAA-NXP DTS changes
  2015-11-23 18:17         ` Joachim Eastwood
@ 2015-11-25 12:22             ` Ariel D'Alessandro
  -1 siblings, 0 replies; 32+ messages in thread
From: Ariel D'Alessandro @ 2015-11-25 12:22 UTC (permalink / raw)
  To: Joachim Eastwood
  Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	devicetree-u79uwXL29TY76Z2rM5mHXA, Rob Herring, Pawel Moll,
	Mark Rutland, Ian Campbell, Kumar Gala, Russell King - ARM Linux

Joachim,

El 23/11/15 a las 15:17, Joachim Eastwood escribió:
> Hi Ariel,
> 
> On 23 November 2015 at 13:43, Ariel D'Alessandro
> <ariel-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org> wrote:
>> (+Cc Joachim)
>>
>> El 23/11/15 a las 09:37, Ariel D'Alessandro escribió:
>>> Hi,
>>>
>>> This patch set includes some DTS changes for LPC4337 CIAA-NXP board.
>>> It is based on tag next-20151120 of the linux-next repository. It has
>>> been successfully tested on a LPC4337 CIAA-NXP Board.
>>>
>>> Ariel D'Alessandro (3):
>>>   ARM: dts: lpc4337-ciaa: enable EEPROM memory
>>>   ARM: dts: lpc4337-ciaa: enable SCT-PWM
>>>   ARM: dts: lpc4337-ciaa: add i2c0 and devices
> 
> If you want me to pick up the patches please put me in the To field so
> they end up in my inbox.

OK, I'm resending with you in the To field.

-- 
Ariel D'Alessandro, VanguardiaSur
www.vanguardiasur.com.ar
--
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

^ permalink raw reply	[flat|nested] 32+ messages in thread
* [PATCH 0/3] LPC4337 CIAA-NXP DTS changes
@ 2015-11-25 12:22             ` Ariel D'Alessandro
  0 siblings, 0 replies; 32+ messages in thread
From: Ariel D'Alessandro @ 2015-11-25 12:22 UTC (permalink / raw)
  To: linux-arm-kernel

Joachim,

El 23/11/15 a las 15:17, Joachim Eastwood escribi?:
> Hi Ariel,
> 
> On 23 November 2015 at 13:43, Ariel D'Alessandro
> <ariel@vanguardiasur.com.ar> wrote:
>> (+Cc Joachim)
>>
>> El 23/11/15 a las 09:37, Ariel D'Alessandro escribi?:
>>> Hi,
>>>
>>> This patch set includes some DTS changes for LPC4337 CIAA-NXP board.
>>> It is based on tag next-20151120 of the linux-next repository. It has
>>> been successfully tested on a LPC4337 CIAA-NXP Board.
>>>
>>> Ariel D'Alessandro (3):
>>>   ARM: dts: lpc4337-ciaa: enable EEPROM memory
>>>   ARM: dts: lpc4337-ciaa: enable SCT-PWM
>>>   ARM: dts: lpc4337-ciaa: add i2c0 and devices
> 
> If you want me to pick up the patches please put me in the To field so
> they end up in my inbox.

OK, I'm resending with you in the To field.

-- 
Ariel D'Alessandro, VanguardiaSur
www.vanguardiasur.com.ar

^ permalink raw reply	[flat|nested] 32+ messages in thread
* Re: [PATCH 0/3] LPC4337 CIAA-NXP DTS changes
  2015-11-23 12:37 ` Ariel D'Alessandro
@ 2015-11-25 12:27     ` Ariel D'Alessandro
  -1 siblings, 0 replies; 32+ messages in thread
From: Ariel D'Alessandro @ 2015-11-25 12:27 UTC (permalink / raw)
  To: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	manabian-Re5JQEeQqe8AvxtiuMwx3w
  Cc: robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
	mark.rutland-5wv7dgnIgG8, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
	galak-sgV2jX0FEOL9JmXXK+q4OQ, linux-lFZ/pmaqli7XmaaqVzeoHQ

(+To: Joachim)

El 23/11/15 a las 09:37, Ariel D'Alessandro escribió:
> Hi,
> 
> This patch set includes some DTS changes for LPC4337 CIAA-NXP board.
> It is based on tag next-20151120 of the linux-next repository. It has
> been successfully tested on a LPC4337 CIAA-NXP Board.
> 
> Ariel D'Alessandro (3):
>   ARM: dts: lpc4337-ciaa: enable EEPROM memory
>   ARM: dts: lpc4337-ciaa: enable SCT-PWM
>   ARM: dts: lpc4337-ciaa: add i2c0 and devices
> 
>  arch/arm/boot/dts/lpc4337-ciaa.dts | 38 ++++++++++++++++++++++++++++++++++++++
>  1 file changed, 38 insertions(+)
> 

-- 
Ariel D'Alessandro, VanguardiaSur
www.vanguardiasur.com.ar
--
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

^ permalink raw reply	[flat|nested] 32+ messages in thread
* [PATCH 0/3] LPC4337 CIAA-NXP DTS changes
@ 2015-11-25 12:27     ` Ariel D'Alessandro
  0 siblings, 0 replies; 32+ messages in thread
From: Ariel D'Alessandro @ 2015-11-25 12:27 UTC (permalink / raw)
  To: linux-arm-kernel

(+To: Joachim)

El 23/11/15 a las 09:37, Ariel D'Alessandro escribi?:
> Hi,
> 
> This patch set includes some DTS changes for LPC4337 CIAA-NXP board.
> It is based on tag next-20151120 of the linux-next repository. It has
> been successfully tested on a LPC4337 CIAA-NXP Board.
> 
> Ariel D'Alessandro (3):
>   ARM: dts: lpc4337-ciaa: enable EEPROM memory
>   ARM: dts: lpc4337-ciaa: enable SCT-PWM
>   ARM: dts: lpc4337-ciaa: add i2c0 and devices
> 
>  arch/arm/boot/dts/lpc4337-ciaa.dts | 38 ++++++++++++++++++++++++++++++++++++++
>  1 file changed, 38 insertions(+)
> 

-- 
Ariel D'Alessandro, VanguardiaSur
www.vanguardiasur.com.ar

^ permalink raw reply	[flat|nested] 32+ messages in thread
* Re: [PATCH 1/3] ARM: dts: lpc4337-ciaa: enable EEPROM memory
  2015-11-23 12:37     ` Ariel D'Alessandro
@ 2015-11-25 12:28         ` Ariel D'Alessandro
  -1 siblings, 0 replies; 32+ messages in thread
From: Ariel D'Alessandro @ 2015-11-25 12:28 UTC (permalink / raw)
  To: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	manabian-Re5JQEeQqe8AvxtiuMwx3w
  Cc: robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
	mark.rutland-5wv7dgnIgG8, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
	galak-sgV2jX0FEOL9JmXXK+q4OQ, linux-lFZ/pmaqli7XmaaqVzeoHQ

(+To: Joachim)

El 23/11/15 a las 09:37, Ariel D'Alessandro escribió:
> The CIAA-NXP board has a NXP LPC4337 SoC that includes a 16 KiB
> EEPROM memory.
> 
> Signed-off-by: Ariel D'Alessandro <ariel-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org>
> ---
>  arch/arm/boot/dts/lpc4337-ciaa.dts | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/lpc4337-ciaa.dts b/arch/arm/boot/dts/lpc4337-ciaa.dts
> index 5f500c1..03bdf77 100644
> --- a/arch/arm/boot/dts/lpc4337-ciaa.dts
> +++ b/arch/arm/boot/dts/lpc4337-ciaa.dts
> @@ -155,6 +155,10 @@
>  	};
>  };
>  
> +&eeprom {
> +	status = "okay";
> +};
> +
>  &enet_tx_clk {
>  	clock-frequency = <50000000>;
>  };
> 

-- 
Ariel D'Alessandro, VanguardiaSur
www.vanguardiasur.com.ar
--
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

^ permalink raw reply	[flat|nested] 32+ messages in thread
* [PATCH 1/3] ARM: dts: lpc4337-ciaa: enable EEPROM memory
@ 2015-11-25 12:28         ` Ariel D'Alessandro
  0 siblings, 0 replies; 32+ messages in thread
From: Ariel D'Alessandro @ 2015-11-25 12:28 UTC (permalink / raw)
  To: linux-arm-kernel

(+To: Joachim)

El 23/11/15 a las 09:37, Ariel D'Alessandro escribi?:
> The CIAA-NXP board has a NXP LPC4337 SoC that includes a 16 KiB
> EEPROM memory.
> 
> Signed-off-by: Ariel D'Alessandro <ariel@vanguardiasur.com.ar>
> ---
>  arch/arm/boot/dts/lpc4337-ciaa.dts | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/lpc4337-ciaa.dts b/arch/arm/boot/dts/lpc4337-ciaa.dts
> index 5f500c1..03bdf77 100644
> --- a/arch/arm/boot/dts/lpc4337-ciaa.dts
> +++ b/arch/arm/boot/dts/lpc4337-ciaa.dts
> @@ -155,6 +155,10 @@
>  	};
>  };
>  
> +&eeprom {
> +	status = "okay";
> +};
> +
>  &enet_tx_clk {
>  	clock-frequency = <50000000>;
>  };
> 

-- 
Ariel D'Alessandro, VanguardiaSur
www.vanguardiasur.com.ar

^ permalink raw reply	[flat|nested] 32+ messages in thread
* Re: [PATCH 2/3] ARM: dts: lpc4337-ciaa: enable SCT-PWM
  2015-11-23 12:37     ` Ariel D'Alessandro
@ 2015-11-25 12:28         ` Ariel D'Alessandro
  -1 siblings, 0 replies; 32+ messages in thread
From: Ariel D'Alessandro @ 2015-11-25 12:28 UTC (permalink / raw)
  To: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	manabian-Re5JQEeQqe8AvxtiuMwx3w
  Cc: robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
	mark.rutland-5wv7dgnIgG8, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
	galak-sgV2jX0FEOL9JmXXK+q4OQ, linux-lFZ/pmaqli7XmaaqVzeoHQ

(+To: Joachim)

El 23/11/15 a las 09:37, Ariel D'Alessandro escribió:
> Enable the PWM based on the State Configurable Timer (SCT) included in
> the LPC4337 SoC of the CIAA-NXP board.
> 
> Signed-off-by: Ariel D'Alessandro <ariel-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org>
> ---
>  arch/arm/boot/dts/lpc4337-ciaa.dts | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/lpc4337-ciaa.dts b/arch/arm/boot/dts/lpc4337-ciaa.dts
> index 03bdf77..1a23589 100644
> --- a/arch/arm/boot/dts/lpc4337-ciaa.dts
> +++ b/arch/arm/boot/dts/lpc4337-ciaa.dts
> @@ -170,6 +170,10 @@
>  	pinctrl-0 = <&enet_rmii_pins>;
>  };
>  
> +&sct_pwm {
> +	status = "okay";
> +};
> +
>  &ssp1 {
>  	status = "okay";
>  	pinctrl-names = "default";
> 

-- 
Ariel D'Alessandro, VanguardiaSur
www.vanguardiasur.com.ar
--
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

^ permalink raw reply	[flat|nested] 32+ messages in thread
* [PATCH 2/3] ARM: dts: lpc4337-ciaa: enable SCT-PWM
@ 2015-11-25 12:28         ` Ariel D'Alessandro
  0 siblings, 0 replies; 32+ messages in thread
From: Ariel D'Alessandro @ 2015-11-25 12:28 UTC (permalink / raw)
  To: linux-arm-kernel

(+To: Joachim)

El 23/11/15 a las 09:37, Ariel D'Alessandro escribi?:
> Enable the PWM based on the State Configurable Timer (SCT) included in
> the LPC4337 SoC of the CIAA-NXP board.
> 
> Signed-off-by: Ariel D'Alessandro <ariel@vanguardiasur.com.ar>
> ---
>  arch/arm/boot/dts/lpc4337-ciaa.dts | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/lpc4337-ciaa.dts b/arch/arm/boot/dts/lpc4337-ciaa.dts
> index 03bdf77..1a23589 100644
> --- a/arch/arm/boot/dts/lpc4337-ciaa.dts
> +++ b/arch/arm/boot/dts/lpc4337-ciaa.dts
> @@ -170,6 +170,10 @@
>  	pinctrl-0 = <&enet_rmii_pins>;
>  };
>  
> +&sct_pwm {
> +	status = "okay";
> +};
> +
>  &ssp1 {
>  	status = "okay";
>  	pinctrl-names = "default";
> 

-- 
Ariel D'Alessandro, VanguardiaSur
www.vanguardiasur.com.ar

^ permalink raw reply	[flat|nested] 32+ messages in thread
* Re: [PATCH 3/3] ARM: dts: lpc4337-ciaa: add i2c0 and devices
  2015-11-23 12:37     ` Ariel D'Alessandro
@ 2015-11-25 12:28         ` Ariel D'Alessandro
  -1 siblings, 0 replies; 32+ messages in thread
From: Ariel D'Alessandro @ 2015-11-25 12:28 UTC (permalink / raw)
  To: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	manabian-Re5JQEeQqe8AvxtiuMwx3w
  Cc: robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
	mark.rutland-5wv7dgnIgG8, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
	galak-sgV2jX0FEOL9JmXXK+q4OQ, linux-lFZ/pmaqli7XmaaqVzeoHQ

(+To: Joachim)

El 23/11/15 a las 09:37, Ariel D'Alessandro escribió:
> Add I2C0 and two I2C EEPROM devices on the CIAA-NXP board:
> 
> * 24AA1025 EEPROM, 1Mbit: it is accessed as two 512Kbit EEPROMs.
> * 24AA025E48 EEPROM, 2kbit.
> 
> Signed-off-by: Ariel D'Alessandro <ariel-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org>
> ---
>  arch/arm/boot/dts/lpc4337-ciaa.dts | 30 ++++++++++++++++++++++++++++++
>  1 file changed, 30 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/lpc4337-ciaa.dts b/arch/arm/boot/dts/lpc4337-ciaa.dts
> index 1a23589..d81047a 100644
> --- a/arch/arm/boot/dts/lpc4337-ciaa.dts
> +++ b/arch/arm/boot/dts/lpc4337-ciaa.dts
> @@ -99,6 +99,14 @@
>  		};
>  	};
>  
> +	i2c0_pins: i2c0-pins {
> +		i2c0_pins_cfg {
> +			pins = "i2c0_scl", "i2c0_sda";
> +			function = "i2c0";
> +			input-enable;
> +		};
> +	};
> +
>  	ssp_pins: ssp-pins {
>  		ssp1_cs {
>  			pins = "p6_7";
> @@ -163,6 +171,28 @@
>  	clock-frequency = <50000000>;
>  };
>  
> +&i2c0 {
> +	status = "okay";
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&i2c0_pins>;
> +	clock-frequency = <400000>;
> +
> +	eeprom@50 {
> +		compatible = "microchip,24c512";
> +		reg = <0x50>;
> +	};
> +
> +	eeprom@51 {
> +		compatible = "microchip,24c02";
> +		reg = <0x51>;
> +	};
> +
> +	eeprom@54 {
> +		compatible = "microchip,24c512";
> +		reg = <0x54>;
> +	};
> +};
> +
>  &mac {
>  	status = "okay";
>  	phy-mode = "rmii";
> 

-- 
Ariel D'Alessandro, VanguardiaSur
www.vanguardiasur.com.ar
--
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

^ permalink raw reply	[flat|nested] 32+ messages in thread
* [PATCH 3/3] ARM: dts: lpc4337-ciaa: add i2c0 and devices
@ 2015-11-25 12:28         ` Ariel D'Alessandro
  0 siblings, 0 replies; 32+ messages in thread
From: Ariel D'Alessandro @ 2015-11-25 12:28 UTC (permalink / raw)
  To: linux-arm-kernel

(+To: Joachim)

El 23/11/15 a las 09:37, Ariel D'Alessandro escribi?:
> Add I2C0 and two I2C EEPROM devices on the CIAA-NXP board:
> 
> * 24AA1025 EEPROM, 1Mbit: it is accessed as two 512Kbit EEPROMs.
> * 24AA025E48 EEPROM, 2kbit.
> 
> Signed-off-by: Ariel D'Alessandro <ariel@vanguardiasur.com.ar>
> ---
>  arch/arm/boot/dts/lpc4337-ciaa.dts | 30 ++++++++++++++++++++++++++++++
>  1 file changed, 30 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/lpc4337-ciaa.dts b/arch/arm/boot/dts/lpc4337-ciaa.dts
> index 1a23589..d81047a 100644
> --- a/arch/arm/boot/dts/lpc4337-ciaa.dts
> +++ b/arch/arm/boot/dts/lpc4337-ciaa.dts
> @@ -99,6 +99,14 @@
>  		};
>  	};
>  
> +	i2c0_pins: i2c0-pins {
> +		i2c0_pins_cfg {
> +			pins = "i2c0_scl", "i2c0_sda";
> +			function = "i2c0";
> +			input-enable;
> +		};
> +	};
> +
>  	ssp_pins: ssp-pins {
>  		ssp1_cs {
>  			pins = "p6_7";
> @@ -163,6 +171,28 @@
>  	clock-frequency = <50000000>;
>  };
>  
> +&i2c0 {
> +	status = "okay";
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&i2c0_pins>;
> +	clock-frequency = <400000>;
> +
> +	eeprom at 50 {
> +		compatible = "microchip,24c512";
> +		reg = <0x50>;
> +	};
> +
> +	eeprom at 51 {
> +		compatible = "microchip,24c02";
> +		reg = <0x51>;
> +	};
> +
> +	eeprom at 54 {
> +		compatible = "microchip,24c512";
> +		reg = <0x54>;
> +	};
> +};
> +
>  &mac {
>  	status = "okay";
>  	phy-mode = "rmii";
> 

-- 
Ariel D'Alessandro, VanguardiaSur
www.vanguardiasur.com.ar

^ permalink raw reply	[flat|nested] 32+ messages in thread
* Re: [PATCH 2/3] ARM: dts: lpc4337-ciaa: enable SCT-PWM
  2015-11-25 12:28         ` Ariel D'Alessandro
@ 2015-12-14 22:48             ` Joachim Eastwood
  -1 siblings, 0 replies; 32+ messages in thread
From: Joachim Eastwood @ 2015-12-14 22:48 UTC (permalink / raw)
  To: Ariel D'Alessandro
  Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	devicetree-u79uwXL29TY76Z2rM5mHXA, Rob Herring, Pawel Moll,
	Mark Rutland, Ian Campbell, Kumar Gala, Russell King - ARM Linux

Hi Ariel,

Sorry for the late reply. I have just started to put together my DT
for 4.5 branch now.

On 25 November 2015 at 13:28, Ariel D'Alessandro
<ariel-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org> wrote:
> (+To: Joachim)
>
> El 23/11/15 a las 09:37, Ariel D'Alessandro escribió:
>> Enable the PWM based on the State Configurable Timer (SCT) included in
>> the LPC4337 SoC of the CIAA-NXP board.
>>
>> Signed-off-by: Ariel D'Alessandro <ariel-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org>
>> ---
>>  arch/arm/boot/dts/lpc4337-ciaa.dts | 4 ++++
>>  1 file changed, 4 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/lpc4337-ciaa.dts b/arch/arm/boot/dts/lpc4337-ciaa.dts
>> index 03bdf77..1a23589 100644
>> --- a/arch/arm/boot/dts/lpc4337-ciaa.dts
>> +++ b/arch/arm/boot/dts/lpc4337-ciaa.dts
>> @@ -170,6 +170,10 @@
>>       pinctrl-0 = <&enet_rmii_pins>;
>>  };
>>
>> +&sct_pwm {
>> +     status = "okay";
>> +};
>> +

Isn't this a bit useless without any pinmux setup?


regards,
Joachim Eastwood
--
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

^ permalink raw reply	[flat|nested] 32+ messages in thread
* [PATCH 2/3] ARM: dts: lpc4337-ciaa: enable SCT-PWM
@ 2015-12-14 22:48             ` Joachim Eastwood
  0 siblings, 0 replies; 32+ messages in thread
From: Joachim Eastwood @ 2015-12-14 22:48 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Ariel,

Sorry for the late reply. I have just started to put together my DT
for 4.5 branch now.

On 25 November 2015 at 13:28, Ariel D'Alessandro
<ariel@vanguardiasur.com.ar> wrote:
> (+To: Joachim)
>
> El 23/11/15 a las 09:37, Ariel D'Alessandro escribi?:
>> Enable the PWM based on the State Configurable Timer (SCT) included in
>> the LPC4337 SoC of the CIAA-NXP board.
>>
>> Signed-off-by: Ariel D'Alessandro <ariel@vanguardiasur.com.ar>
>> ---
>>  arch/arm/boot/dts/lpc4337-ciaa.dts | 4 ++++
>>  1 file changed, 4 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/lpc4337-ciaa.dts b/arch/arm/boot/dts/lpc4337-ciaa.dts
>> index 03bdf77..1a23589 100644
>> --- a/arch/arm/boot/dts/lpc4337-ciaa.dts
>> +++ b/arch/arm/boot/dts/lpc4337-ciaa.dts
>> @@ -170,6 +170,10 @@
>>       pinctrl-0 = <&enet_rmii_pins>;
>>  };
>>
>> +&sct_pwm {
>> +     status = "okay";
>> +};
>> +

Isn't this a bit useless without any pinmux setup?


regards,
Joachim Eastwood

^ permalink raw reply	[flat|nested] 32+ messages in thread
* Re: [PATCH 1/3] ARM: dts: lpc4337-ciaa: enable EEPROM memory
  2015-11-25 12:28         ` Ariel D'Alessandro
@ 2015-12-14 22:52             ` Joachim Eastwood
  -1 siblings, 0 replies; 32+ messages in thread
From: Joachim Eastwood @ 2015-12-14 22:52 UTC (permalink / raw)
  To: Ariel D'Alessandro
  Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	devicetree-u79uwXL29TY76Z2rM5mHXA, Rob Herring, Pawel Moll,
	Mark Rutland, Ian Campbell, Kumar Gala, Russell King - ARM Linux

Hi Ariel,

On 25 November 2015 at 13:28, Ariel D'Alessandro
<ariel-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org> wrote:
> (+To: Joachim)
>
> El 23/11/15 a las 09:37, Ariel D'Alessandro escribió:
>> The CIAA-NXP board has a NXP LPC4337 SoC that includes a 16 KiB
>> EEPROM memory.
>>
>> Signed-off-by: Ariel D'Alessandro <ariel-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org>
>> ---
>>  arch/arm/boot/dts/lpc4337-ciaa.dts | 4 ++++
>>  1 file changed, 4 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/lpc4337-ciaa.dts b/arch/arm/boot/dts/lpc4337-ciaa.dts
>> index 5f500c1..03bdf77 100644
>> --- a/arch/arm/boot/dts/lpc4337-ciaa.dts
>> +++ b/arch/arm/boot/dts/lpc4337-ciaa.dts
>> @@ -155,6 +155,10 @@
>>       };
>>  };
>>
>> +&eeprom {
>> +     status = "okay";
>> +};
>> +

I already had a patch that enabled the eeprom for all boards that
included the lpc4357.dtsi from when I tested your driver. So opted to
use that patch instead of this one. This is really a SoC device
feature that doesn't require any board resources so I think it is
better to have this entry in lpc4357.dtsi.


regards,
Joachim Eastwood
--
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

^ permalink raw reply	[flat|nested] 32+ messages in thread
* [PATCH 1/3] ARM: dts: lpc4337-ciaa: enable EEPROM memory
@ 2015-12-14 22:52             ` Joachim Eastwood
  0 siblings, 0 replies; 32+ messages in thread
From: Joachim Eastwood @ 2015-12-14 22:52 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Ariel,

On 25 November 2015 at 13:28, Ariel D'Alessandro
<ariel@vanguardiasur.com.ar> wrote:
> (+To: Joachim)
>
> El 23/11/15 a las 09:37, Ariel D'Alessandro escribi?:
>> The CIAA-NXP board has a NXP LPC4337 SoC that includes a 16 KiB
>> EEPROM memory.
>>
>> Signed-off-by: Ariel D'Alessandro <ariel@vanguardiasur.com.ar>
>> ---
>>  arch/arm/boot/dts/lpc4337-ciaa.dts | 4 ++++
>>  1 file changed, 4 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/lpc4337-ciaa.dts b/arch/arm/boot/dts/lpc4337-ciaa.dts
>> index 5f500c1..03bdf77 100644
>> --- a/arch/arm/boot/dts/lpc4337-ciaa.dts
>> +++ b/arch/arm/boot/dts/lpc4337-ciaa.dts
>> @@ -155,6 +155,10 @@
>>       };
>>  };
>>
>> +&eeprom {
>> +     status = "okay";
>> +};
>> +

I already had a patch that enabled the eeprom for all boards that
included the lpc4357.dtsi from when I tested your driver. So opted to
use that patch instead of this one. This is really a SoC device
feature that doesn't require any board resources so I think it is
better to have this entry in lpc4357.dtsi.


regards,
Joachim Eastwood

^ permalink raw reply	[flat|nested] 32+ messages in thread
* Re: [PATCH 1/3] ARM: dts: lpc4337-ciaa: enable EEPROM memory
  2015-12-14 22:52             ` Joachim Eastwood
@ 2015-12-16 12:53                 ` Ariel D'Alessandro
  -1 siblings, 0 replies; 32+ messages in thread
From: Ariel D'Alessandro @ 2015-12-16 12:53 UTC (permalink / raw)
  To: Joachim Eastwood
  Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	devicetree-u79uwXL29TY76Z2rM5mHXA, Rob Herring, Pawel Moll,
	Mark Rutland, Ian Campbell, Kumar Gala, Russell King - ARM Linux

Joachim,

El 14/12/15 a las 19:52, Joachim Eastwood escribió:
> Hi Ariel,
> 
> On 25 November 2015 at 13:28, Ariel D'Alessandro
> <ariel-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org> wrote:
>> (+To: Joachim)
>>
>> El 23/11/15 a las 09:37, Ariel D'Alessandro escribió:
>>> The CIAA-NXP board has a NXP LPC4337 SoC that includes a 16 KiB
>>> EEPROM memory.
>>>
>>> Signed-off-by: Ariel D'Alessandro <ariel-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org>
>>> ---
>>>  arch/arm/boot/dts/lpc4337-ciaa.dts | 4 ++++
>>>  1 file changed, 4 insertions(+)
>>>
>>> diff --git a/arch/arm/boot/dts/lpc4337-ciaa.dts b/arch/arm/boot/dts/lpc4337-ciaa.dts
>>> index 5f500c1..03bdf77 100644
>>> --- a/arch/arm/boot/dts/lpc4337-ciaa.dts
>>> +++ b/arch/arm/boot/dts/lpc4337-ciaa.dts
>>> @@ -155,6 +155,10 @@
>>>       };
>>>  };
>>>
>>> +&eeprom {
>>> +     status = "okay";
>>> +};
>>> +
> 
> I already had a patch that enabled the eeprom for all boards that
> included the lpc4357.dtsi from when I tested your driver. So opted to
> use that patch instead of this one. This is really a SoC device
> feature that doesn't require any board resources so I think it is
> better to have this entry in lpc4357.dtsi.

Yeah, you're right. Fine for me, thanks.

-- 
Ariel D'Alessandro, VanguardiaSur
www.vanguardiasur.com.ar
--
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

^ permalink raw reply	[flat|nested] 32+ messages in thread
* [PATCH 1/3] ARM: dts: lpc4337-ciaa: enable EEPROM memory
@ 2015-12-16 12:53                 ` Ariel D'Alessandro
  0 siblings, 0 replies; 32+ messages in thread
From: Ariel D'Alessandro @ 2015-12-16 12:53 UTC (permalink / raw)
  To: linux-arm-kernel

Joachim,

El 14/12/15 a las 19:52, Joachim Eastwood escribi?:
> Hi Ariel,
> 
> On 25 November 2015 at 13:28, Ariel D'Alessandro
> <ariel@vanguardiasur.com.ar> wrote:
>> (+To: Joachim)
>>
>> El 23/11/15 a las 09:37, Ariel D'Alessandro escribi?:
>>> The CIAA-NXP board has a NXP LPC4337 SoC that includes a 16 KiB
>>> EEPROM memory.
>>>
>>> Signed-off-by: Ariel D'Alessandro <ariel@vanguardiasur.com.ar>
>>> ---
>>>  arch/arm/boot/dts/lpc4337-ciaa.dts | 4 ++++
>>>  1 file changed, 4 insertions(+)
>>>
>>> diff --git a/arch/arm/boot/dts/lpc4337-ciaa.dts b/arch/arm/boot/dts/lpc4337-ciaa.dts
>>> index 5f500c1..03bdf77 100644
>>> --- a/arch/arm/boot/dts/lpc4337-ciaa.dts
>>> +++ b/arch/arm/boot/dts/lpc4337-ciaa.dts
>>> @@ -155,6 +155,10 @@
>>>       };
>>>  };
>>>
>>> +&eeprom {
>>> +     status = "okay";
>>> +};
>>> +
> 
> I already had a patch that enabled the eeprom for all boards that
> included the lpc4357.dtsi from when I tested your driver. So opted to
> use that patch instead of this one. This is really a SoC device
> feature that doesn't require any board resources so I think it is
> better to have this entry in lpc4357.dtsi.

Yeah, you're right. Fine for me, thanks.

-- 
Ariel D'Alessandro, VanguardiaSur
www.vanguardiasur.com.ar

^ permalink raw reply	[flat|nested] 32+ messages in thread
* Re: [PATCH 2/3] ARM: dts: lpc4337-ciaa: enable SCT-PWM
  2015-12-14 22:48             ` Joachim Eastwood
@ 2015-12-16 12:55                 ` Ariel D'Alessandro
  -1 siblings, 0 replies; 32+ messages in thread
From: Ariel D'Alessandro @ 2015-12-16 12:55 UTC (permalink / raw)
  To: Joachim Eastwood
  Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	devicetree-u79uwXL29TY76Z2rM5mHXA, Rob Herring, Pawel Moll,
	Mark Rutland, Ian Campbell, Kumar Gala, Russell King - ARM Linux

Joachim,

El 14/12/15 a las 19:48, Joachim Eastwood escribió:
> Hi Ariel,
> 
> Sorry for the late reply. I have just started to put together my DT
> for 4.5 branch now.
> 
> On 25 November 2015 at 13:28, Ariel D'Alessandro
> <ariel-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org> wrote:
>> (+To: Joachim)
>>
>> El 23/11/15 a las 09:37, Ariel D'Alessandro escribió:
>>> Enable the PWM based on the State Configurable Timer (SCT) included in
>>> the LPC4337 SoC of the CIAA-NXP board.
>>>
>>> Signed-off-by: Ariel D'Alessandro <ariel-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org>
>>> ---
>>>  arch/arm/boot/dts/lpc4337-ciaa.dts | 4 ++++
>>>  1 file changed, 4 insertions(+)
>>>
>>> diff --git a/arch/arm/boot/dts/lpc4337-ciaa.dts b/arch/arm/boot/dts/lpc4337-ciaa.dts
>>> index 03bdf77..1a23589 100644
>>> --- a/arch/arm/boot/dts/lpc4337-ciaa.dts
>>> +++ b/arch/arm/boot/dts/lpc4337-ciaa.dts
>>> @@ -170,6 +170,10 @@
>>>       pinctrl-0 = <&enet_rmii_pins>;
>>>  };
>>>
>>> +&sct_pwm {
>>> +     status = "okay";
>>> +};
>>> +
> 
> Isn't this a bit useless without any pinmux setup?

Yes, it might be kind of useless. Do you think it shouldn't be enabled then?

-- 
Ariel D'Alessandro, VanguardiaSur
www.vanguardiasur.com.ar
--
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

^ permalink raw reply	[flat|nested] 32+ messages in thread
* [PATCH 2/3] ARM: dts: lpc4337-ciaa: enable SCT-PWM
@ 2015-12-16 12:55                 ` Ariel D'Alessandro
  0 siblings, 0 replies; 32+ messages in thread
From: Ariel D'Alessandro @ 2015-12-16 12:55 UTC (permalink / raw)
  To: linux-arm-kernel

Joachim,

El 14/12/15 a las 19:48, Joachim Eastwood escribi?:
> Hi Ariel,
> 
> Sorry for the late reply. I have just started to put together my DT
> for 4.5 branch now.
> 
> On 25 November 2015 at 13:28, Ariel D'Alessandro
> <ariel@vanguardiasur.com.ar> wrote:
>> (+To: Joachim)
>>
>> El 23/11/15 a las 09:37, Ariel D'Alessandro escribi?:
>>> Enable the PWM based on the State Configurable Timer (SCT) included in
>>> the LPC4337 SoC of the CIAA-NXP board.
>>>
>>> Signed-off-by: Ariel D'Alessandro <ariel@vanguardiasur.com.ar>
>>> ---
>>>  arch/arm/boot/dts/lpc4337-ciaa.dts | 4 ++++
>>>  1 file changed, 4 insertions(+)
>>>
>>> diff --git a/arch/arm/boot/dts/lpc4337-ciaa.dts b/arch/arm/boot/dts/lpc4337-ciaa.dts
>>> index 03bdf77..1a23589 100644
>>> --- a/arch/arm/boot/dts/lpc4337-ciaa.dts
>>> +++ b/arch/arm/boot/dts/lpc4337-ciaa.dts
>>> @@ -170,6 +170,10 @@
>>>       pinctrl-0 = <&enet_rmii_pins>;
>>>  };
>>>
>>> +&sct_pwm {
>>> +     status = "okay";
>>> +};
>>> +
> 
> Isn't this a bit useless without any pinmux setup?

Yes, it might be kind of useless. Do you think it shouldn't be enabled then?

-- 
Ariel D'Alessandro, VanguardiaSur
www.vanguardiasur.com.ar

^ permalink raw reply	[flat|nested] 32+ messages in thread
* Re: [PATCH 2/3] ARM: dts: lpc4337-ciaa: enable SCT-PWM
  2015-12-16 12:55                 ` Ariel D'Alessandro
@ 2015-12-17 22:55                     ` Joachim Eastwood
  -1 siblings, 0 replies; 32+ messages in thread
From: Joachim Eastwood @ 2015-12-17 22:55 UTC (permalink / raw)
  To: Ariel D'Alessandro
  Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	devicetree-u79uwXL29TY76Z2rM5mHXA, Rob Herring, Pawel Moll,
	Mark Rutland, Ian Campbell, Kumar Gala, Russell King - ARM Linux

On 16 December 2015 at 13:55, Ariel D'Alessandro
<ariel-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org> wrote:
> Joachim,
>
> El 14/12/15 a las 19:48, Joachim Eastwood escribió:
>> Hi Ariel,
>>
>> Sorry for the late reply. I have just started to put together my DT
>> for 4.5 branch now.
>>
>> On 25 November 2015 at 13:28, Ariel D'Alessandro
>> <ariel-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org> wrote:
>>> (+To: Joachim)
>>>
>>> El 23/11/15 a las 09:37, Ariel D'Alessandro escribió:
>>>> Enable the PWM based on the State Configurable Timer (SCT) included in
>>>> the LPC4337 SoC of the CIAA-NXP board.
>>>>
>>>> Signed-off-by: Ariel D'Alessandro <ariel-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org>
>>>> ---
>>>>  arch/arm/boot/dts/lpc4337-ciaa.dts | 4 ++++
>>>>  1 file changed, 4 insertions(+)
>>>>
>>>> diff --git a/arch/arm/boot/dts/lpc4337-ciaa.dts b/arch/arm/boot/dts/lpc4337-ciaa.dts
>>>> index 03bdf77..1a23589 100644
>>>> --- a/arch/arm/boot/dts/lpc4337-ciaa.dts
>>>> +++ b/arch/arm/boot/dts/lpc4337-ciaa.dts
>>>> @@ -170,6 +170,10 @@
>>>>       pinctrl-0 = <&enet_rmii_pins>;
>>>>  };
>>>>
>>>> +&sct_pwm {
>>>> +     status = "okay";
>>>> +};
>>>> +
>>
>> Isn't this a bit useless without any pinmux setup?
>
> Yes, it might be kind of useless. Do you think it shouldn't be enabled then?

It would be nice to have a board with sct-pwm enabled. Would it
possible for you to assign a pin to it?


regards,
Joachim Eastwood
--
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

^ permalink raw reply	[flat|nested] 32+ messages in thread
* [PATCH 2/3] ARM: dts: lpc4337-ciaa: enable SCT-PWM
@ 2015-12-17 22:55                     ` Joachim Eastwood
  0 siblings, 0 replies; 32+ messages in thread
From: Joachim Eastwood @ 2015-12-17 22:55 UTC (permalink / raw)
  To: linux-arm-kernel

On 16 December 2015 at 13:55, Ariel D'Alessandro
<ariel@vanguardiasur.com.ar> wrote:
> Joachim,
>
> El 14/12/15 a las 19:48, Joachim Eastwood escribi?:
>> Hi Ariel,
>>
>> Sorry for the late reply. I have just started to put together my DT
>> for 4.5 branch now.
>>
>> On 25 November 2015 at 13:28, Ariel D'Alessandro
>> <ariel@vanguardiasur.com.ar> wrote:
>>> (+To: Joachim)
>>>
>>> El 23/11/15 a las 09:37, Ariel D'Alessandro escribi?:
>>>> Enable the PWM based on the State Configurable Timer (SCT) included in
>>>> the LPC4337 SoC of the CIAA-NXP board.
>>>>
>>>> Signed-off-by: Ariel D'Alessandro <ariel@vanguardiasur.com.ar>
>>>> ---
>>>>  arch/arm/boot/dts/lpc4337-ciaa.dts | 4 ++++
>>>>  1 file changed, 4 insertions(+)
>>>>
>>>> diff --git a/arch/arm/boot/dts/lpc4337-ciaa.dts b/arch/arm/boot/dts/lpc4337-ciaa.dts
>>>> index 03bdf77..1a23589 100644
>>>> --- a/arch/arm/boot/dts/lpc4337-ciaa.dts
>>>> +++ b/arch/arm/boot/dts/lpc4337-ciaa.dts
>>>> @@ -170,6 +170,10 @@
>>>>       pinctrl-0 = <&enet_rmii_pins>;
>>>>  };
>>>>
>>>> +&sct_pwm {
>>>> +     status = "okay";
>>>> +};
>>>> +
>>
>> Isn't this a bit useless without any pinmux setup?
>
> Yes, it might be kind of useless. Do you think it shouldn't be enabled then?

It would be nice to have a board with sct-pwm enabled. Would it
possible for you to assign a pin to it?


regards,
Joachim Eastwood

^ permalink raw reply	[flat|nested] 32+ messages in thread
end of thread, other threads:[~2015-12-17 22:55 UTC | newest]

Thread overview: 32+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-11-23 12:37 [PATCH 0/3] LPC4337 CIAA-NXP DTS changes Ariel D'Alessandro
2015-11-23 12:37 ` Ariel D'Alessandro
     [not found] ` <1448282225-2635-1-git-send-email-ariel-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org>
2015-11-23 12:37   ` [PATCH 1/3] ARM: dts: lpc4337-ciaa: enable EEPROM memory Ariel D'Alessandro
2015-11-23 12:37     ` Ariel D'Alessandro
     [not found]     ` <1448282225-2635-2-git-send-email-ariel-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org>
2015-11-25 12:28       ` Ariel D'Alessandro
2015-11-25 12:28         ` Ariel D'Alessandro
     [not found]         ` <5655A950.3000003-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org>
2015-12-14 22:52           ` Joachim Eastwood
2015-12-14 22:52             ` Joachim Eastwood
     [not found]             ` <CAGhQ9VzdWSoPbXv-pJUuz7GkM7PuYSz8+Np7Gz1h4MkXVzDeEA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-12-16 12:53               ` Ariel D'Alessandro
2015-12-16 12:53                 ` Ariel D'Alessandro
2015-11-23 12:37   ` [PATCH 2/3] ARM: dts: lpc4337-ciaa: enable SCT-PWM Ariel D'Alessandro
2015-11-23 12:37     ` Ariel D'Alessandro
     [not found]     ` <1448282225-2635-3-git-send-email-ariel-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org>
2015-11-25 12:28       ` Ariel D'Alessandro
2015-11-25 12:28         ` Ariel D'Alessandro
     [not found]         ` <5655A977.1040408-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org>
2015-12-14 22:48           ` Joachim Eastwood
2015-12-14 22:48             ` Joachim Eastwood
     [not found]             ` <CAGhQ9Vxqpe4QQn1+d6bOCQH9q6evignVR_+U4NK7HqSNhBg4qA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-12-16 12:55               ` Ariel D'Alessandro
2015-12-16 12:55                 ` Ariel D'Alessandro
     [not found]                 ` <56715F4B.7050109-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org>
2015-12-17 22:55                   ` Joachim Eastwood
2015-12-17 22:55                     ` Joachim Eastwood
2015-11-23 12:37   ` [PATCH 3/3] ARM: dts: lpc4337-ciaa: add i2c0 and devices Ariel D'Alessandro
2015-11-23 12:37     ` Ariel D'Alessandro
     [not found]     ` <1448282225-2635-4-git-send-email-ariel-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org>
2015-11-25 12:28       ` Ariel D'Alessandro
2015-11-25 12:28         ` Ariel D'Alessandro
2015-11-23 12:43   ` [PATCH 0/3] LPC4337 CIAA-NXP DTS changes Ariel D'Alessandro
2015-11-23 12:43     ` Ariel D'Alessandro
     [not found]     ` <565309F3.1040306-30ULvvUtt6G51wMPkGsGjgyUoB5FGQPZ@public.gmane.org>
2015-11-23 18:17       ` Joachim Eastwood
2015-11-23 18:17         ` Joachim Eastwood
     [not found]         ` <CAGhQ9Vz-fpqKBtaHY5bDZqXEMrpat=UWz4KofGBWMNdNVWw4Cg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-11-25 12:22           ` Ariel D'Alessandro
2015-11-25 12:22             ` Ariel D'Alessandro
2015-11-25 12:27   ` Ariel D'Alessandro
2015-11-25 12:27     ` Ariel D'Alessandro

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.