diff mbox

[v4,09/13] ARM: dts: Add SPI1 controller and s5c73m3 sensor node for TRATS2

Message ID 1374671845-30373-10-git-send-email-s.nawrocki@samsung.com (mailing list archive)
State New, archived
Headers show

Commit Message

From: Andrzej Hajda <a.hajda@samsung.com>

This patch add dts entries required for the SPI bus used for
firware upload by the S5C73M3 camera.

Signed-off-by: Andrzej Hajda <a.hajda@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
---
 arch/arm/boot/dts/exynos4412-trats2.dts |   16 ++++++++++++++++
 1 file changed, 16 insertions(+)

Comments

Kim Kukjin Aug. 5, 2013, 5:53 p.m. UTC | #1
On 07/24/13 22:17, Sylwester Nawrocki wrote:
> From: Andrzej Hajda<a.hajda@samsung.com>
>
> This patch add dts entries required for the SPI bus used for
> firware upload by the S5C73M3 camera.
>
> Signed-off-by: Andrzej Hajda<a.hajda@samsung.com>
> Signed-off-by: Kyungmin Park<kyungmin.park@samsung.com>

Sylwester, probably you missed your sign here?

- Kukjin
On 08/05/2013 07:53 PM, Kukjin Kim wrote:
> On 07/24/13 22:17, Sylwester Nawrocki wrote:
>> From: Andrzej Hajda<a.hajda@samsung.com>
>>
>> This patch add dts entries required for the SPI bus used for
>> firware upload by the S5C73M3 camera.
>>
>> Signed-off-by: Andrzej Hajda<a.hajda@samsung.com>
>> Signed-off-by: Kyungmin Park<kyungmin.park@samsung.com>
> 
> Sylwester, probably you missed your sign here?

Yes, I think it should be added, since I was sending out the patch.
Sorry about this omission and typo in patch 05/13.

Thank you for applying this series.

--
Regards,
Sylwester
Kim Kukjin Aug. 5, 2013, 6:16 p.m. UTC | #3
On 08/06/13 03:00, Sylwester Nawrocki wrote:
> On 08/05/2013 07:53 PM, Kukjin Kim wrote:
>> On 07/24/13 22:17, Sylwester Nawrocki wrote:
>>> From: Andrzej Hajda<a.hajda@samsung.com>
>>>
>>> This patch add dts entries required for the SPI bus used for
>>> firware upload by the S5C73M3 camera.
>>>
>>> Signed-off-by: Andrzej Hajda<a.hajda@samsung.com>
>>> Signed-off-by: Kyungmin Park<kyungmin.park@samsung.com>
>>
>> Sylwester, probably you missed your sign here?
>
> Yes, I think it should be added, since I was sending out the patch.
> Sorry about this omission and typo in patch 05/13.
>
No problem, and if you want let me amend on this for your sign. I 
couldn't it without your permission because it is a legal issue ;)

> Thank you for applying this series.

You're welcome, thanks.
- Kukjin
On 08/05/2013 08:16 PM, Kukjin Kim wrote:
> On 08/06/13 03:00, Sylwester Nawrocki wrote:
>> On 08/05/2013 07:53 PM, Kukjin Kim wrote:
>>> On 07/24/13 22:17, Sylwester Nawrocki wrote:
>>>> From: Andrzej Hajda<a.hajda@samsung.com>
>>>>
>>>> This patch add dts entries required for the SPI bus used for
>>>> firware upload by the S5C73M3 camera.
>>>>
>>>> Signed-off-by: Andrzej Hajda<a.hajda@samsung.com>
>>>> Signed-off-by: Kyungmin Park<kyungmin.park@samsung.com>
>>>
>>> Sylwester, probably you missed your sign here?
>>
>> Yes, I think it should be added, since I was sending out the patch.
>> Sorry about this omission and typo in patch 05/13.
>>
> No problem, and if you want let me amend on this for your sign. I 
> couldn't it without your permission because it is a legal issue ;)

Sure, please go ahead. Thanks again for correcting this.

--
Regards,
Sylwester
Kim Kukjin Aug. 6, 2013, 10:14 p.m. UTC | #5
On 08/06/13 03:50, Sylwester Nawrocki wrote:
> On 08/05/2013 08:16 PM, Kukjin Kim wrote:
>> On 08/06/13 03:00, Sylwester Nawrocki wrote:
>>> On 08/05/2013 07:53 PM, Kukjin Kim wrote:
>>>> On 07/24/13 22:17, Sylwester Nawrocki wrote:
>>>>> From: Andrzej Hajda<a.hajda@samsung.com>
>>>>>
>>>>> This patch add dts entries required for the SPI bus used for
>>>>> firware upload by the S5C73M3 camera.
>>>>>
>>>>> Signed-off-by: Andrzej Hajda<a.hajda@samsung.com>
>>>>> Signed-off-by: Kyungmin Park<kyungmin.park@samsung.com>
>>>>
>>>> Sylwester, probably you missed your sign here?
>>>
>>> Yes, I think it should be added, since I was sending out the patch.
>>> Sorry about this omission and typo in patch 05/13.
>>>
>> No problem, and if you want let me amend on this for your sign. I
>> couldn't it without your permission because it is a legal issue ;)
>
> Sure, please go ahead. Thanks again for correcting this.
>
I did, thanks.

- Kukjin
diff mbox

Patch

diff --git a/arch/arm/boot/dts/exynos4412-trats2.dts b/arch/arm/boot/dts/exynos4412-trats2.dts
index 5558e5a..fb7b9ae 100644
--- a/arch/arm/boot/dts/exynos4412-trats2.dts
+++ b/arch/arm/boot/dts/exynos4412-trats2.dts
@@ -482,6 +482,22 @@ 
 		};
 	};
 
+	spi_1: spi@13930000 {
+		pinctrl-names = "default";
+		pinctrl-0 = <&spi1_bus>;
+		status = "okay";
+
+		s5c73m3_spi: s5c73m3 {
+			compatible = "samsung,s5c73m3";
+			spi-max-frequency = <50000000>;
+			reg = <0>;
+			controller-data {
+				cs-gpio = <&gpb 5 0>;
+				samsung,spi-feedback-delay = <2>;
+			};
+		};
+	};
+
 	camera {
 		pinctrl-0 = <&cam_port_b_clk_active>;
 		pinctrl-names = "default";