[PATCH 1/4] ARM: SAMSUNG: Set HDMI platform data for Exynos4x12 SoCs

[PATCH 1/4] ARM: SAMSUNG: Set HDMI platform data for Exynos4x12 SoCs

[Sachin Kamat]

Adds support for setting HDMI platform data for Exynos4X12 SoCs.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 arch/arm/plat-samsung/devs.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/arch/arm/plat-samsung/devs.c b/arch/arm/plat-samsung/devs.c
index 1d214cb..07b5ac6 100644
--- a/arch/arm/plat-samsung/devs.c
+++ b/arch/arm/plat-samsung/devs.c
@@ -747,7 +747,8 @@ void __init s5p_i2c_hdmiphy_set_platdata(struct s3c2410_platform_i2c *pd)
 	if (!pd) {
 		pd = &default_i2c_data;
 
-		if (soc_is_exynos4210())
+		if (soc_is_exynos4210() || soc_is_exynos4212() ||
+							soc_is_exynos4412())
 			pd->bus_num = 8;
 		else if (soc_is_s5pv210())
 			pd->bus_num = 3;
-- 
1.7.4.1

[PATCH 2/4] ARM: SAMSUNG: Add API to set platform data for s5p-tv driver

[Sachin Kamat]

From: Tushar Behera <tushar.behera@linaro.org>

Commit 350f2f4dad64 ("[media] v4l: s5p-tv: hdmi: add support for
platform data") makes the presence of platform data mandatory for s5p-tv
driver. Adding an API to plat-samsung for this purpose.

Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 arch/arm/plat-samsung/devs.c              |   26 ++++++++++++++++++++++++++
 arch/arm/plat-samsung/include/plat/hdmi.h |   16 ++++++++++++++++
 2 files changed, 42 insertions(+), 0 deletions(-)
 create mode 100644 arch/arm/plat-samsung/include/plat/hdmi.h

diff --git a/arch/arm/plat-samsung/devs.c b/arch/arm/plat-samsung/devs.c
index 07b5ac6..921ed88 100644
--- a/arch/arm/plat-samsung/devs.c
+++ b/arch/arm/plat-samsung/devs.c
@@ -32,6 +32,8 @@
 #include <linux/platform_data/s3c-hsudc.h>
 #include <linux/platform_data/s3c-hsotg.h>
 
+#include <media/s5p_hdmi.h>
+
 #include <asm/irq.h>
 #include <asm/pmu.h>
 #include <asm/mach/arch.h>
@@ -759,6 +761,30 @@ void __init s5p_i2c_hdmiphy_set_platdata(struct s3c2410_platform_i2c *pd)
 	npd = s3c_set_platdata(pd, sizeof(struct s3c2410_platform_i2c),
 			       &s5p_device_i2c_hdmiphy);
 }
+
+struct s5p_hdmi_platform_data s5p_hdmi_def_platdata;
+
+void __init s5p_hdmi_set_platdata(struct i2c_board_info *hdmiphy_info,
+				struct i2c_board_info *mhl_info, int mhl_bus)
+{
+	struct s5p_hdmi_platform_data *pd = &s5p_hdmi_def_platdata;
+
+	if (soc_is_exynos4210() || soc_is_exynos4212() ||
+						soc_is_exynos4412())
+		pd->hdmiphy_bus = 8;
+	else if (soc_is_s5pv210())
+		pd->hdmiphy_bus = 3;
+	else
+		pd->hdmiphy_bus = 0;
+
+	pd->hdmiphy_info = hdmiphy_info;
+	pd->mhl_info = mhl_info;
+	pd->mhl_bus = mhl_bus;
+
+	s3c_set_platdata(pd, sizeof(struct s5p_hdmi_platform_data),
+				&s5p_device_hdmi);
+}
+
 #endif /* CONFIG_S5P_DEV_I2C_HDMIPHY */
 
 /* I2S */
diff --git a/arch/arm/plat-samsung/include/plat/hdmi.h b/arch/arm/plat-samsung/include/plat/hdmi.h
new file mode 100644
index 0000000..5dc7347
--- /dev/null
+++ b/arch/arm/plat-samsung/include/plat/hdmi.h
@@ -0,0 +1,16 @@
+/*
+ * Copyright (C) 2012 Samsung Electronics Co.Ltd
+ *
+ * This program is free software; you can redistribute  it and/or modify it
+ * under  the terms of  the GNU General  Public License as published by the
+ * Free Software Foundation;  either version 2 of the  License, or (at your
+ * option) any later version.
+ */
+
+#ifndef __PLAT_SAMSUNG_HDMI_H
+#define __PLAT_SAMSUNG_HDMI_H __FILE__
+
+extern void s5p_hdmi_set_platdata(struct i2c_board_info *hdmiphy_info,
+				struct i2c_board_info *mhl_info, int mhl_bus);
+
+#endif /* __PLAT_SAMSUNG_HDMI_H */
-- 
1.7.4.1

[PATCH 3/4] ARM: EXYNOS: Set HDMI platform data in SMDKV310

[Sachin Kamat]

Sets HDMI platform data in SMDKV310 board.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 arch/arm/mach-exynos/mach-smdkv310.c |    7 +++++++
 1 files changed, 7 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-exynos/mach-smdkv310.c b/arch/arm/mach-exynos/mach-smdkv310.c
index 95ce9f6..1841aad 100644
--- a/arch/arm/mach-exynos/mach-smdkv310.c
+++ b/arch/arm/mach-exynos/mach-smdkv310.c
@@ -40,6 +40,7 @@
 #include <plat/mfc.h>
 #include <plat/ehci.h>
 #include <plat/clock.h>
+#include <plat/hdmi.h>
 
 #include <mach/map.h>
 #include <mach/ohci.h>
@@ -347,6 +348,11 @@ static struct platform_pwm_backlight_data smdkv310_bl_data = {
 	.pwm_period_ns  = 1000,
 };
 
+/* I2C module and id for HDMIPHY */
+static struct i2c_board_info hdmiphy_info = {
+	I2C_BOARD_INFO("hdmiphy-exynos4210", 0x38),
+};
+
 static void s5p_tv_setup(void)
 {
 	/* direct HPD to HDMI chip */
@@ -381,6 +387,7 @@ static void __init smdkv310_machine_init(void)
 
 	s5p_tv_setup();
 	s5p_i2c_hdmiphy_set_platdata(NULL);
+	s5p_hdmi_set_platdata(&hdmiphy_info, NULL, 0);
 
 	samsung_keypad_set_platdata(&smdkv310_keypad_data);
 
-- 
1.7.4.1

[PATCH 4/4] ARM: EXYNOS: Set HDMI platform data in Origen board

[Sachin Kamat]

From: Tushar Behera <tushar.behera@linaro.org>

Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 arch/arm/mach-exynos/mach-origen.c |    7 +++++++
 1 files changed, 7 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-exynos/mach-origen.c b/arch/arm/mach-exynos/mach-origen.c
index 580cc77..83d4e96 100644
--- a/arch/arm/mach-exynos/mach-origen.c
+++ b/arch/arm/mach-exynos/mach-origen.c
@@ -41,6 +41,7 @@
 #include <plat/backlight.h>
 #include <plat/fb.h>
 #include <plat/mfc.h>
+#include <plat/hdmi.h>
 
 #include <mach/ohci.h>
 #include <mach/map.h>
@@ -705,6 +706,11 @@ static void __init origen_bt_setup(void)
 	s3c_gpio_setpull(EXYNOS4_GPX2(2), S3C_GPIO_PULL_NONE);
 }
 
+/* I2C module and id for HDMIPHY */
+static struct i2c_board_info hdmiphy_info = {
+	I2C_BOARD_INFO("hdmiphy-exynos4210", 0x38),
+};
+
 static void s5p_tv_setup(void)
 {
 	/* Direct HPD to HDMI chip */
@@ -753,6 +759,7 @@ static void __init origen_machine_init(void)
 
 	s5p_tv_setup();
 	s5p_i2c_hdmiphy_set_platdata(NULL);
+	s5p_hdmi_set_platdata(&hdmiphy_info, NULL, 0);
 
 #ifdef CONFIG_DRM_EXYNOS
 	s5p_device_fimd0.dev.platform_data = &drm_fimd_pdata;
-- 
1.7.4.1

RE: [PATCH 2/4] ARM: SAMSUNG: Add API to set platform data for s5p-tv driver

[Kukjin Kim]

Sachin Kamat wrote:
> 
> From: Tushar Behera <tushar.behera@linaro.org>
> 
> Commit 350f2f4dad64 ("[media] v4l: s5p-tv: hdmi: add support for
> platform data") makes the presence of platform data mandatory for s5p-tv
> driver. Adding an API to plat-samsung for this purpose.
> 
> Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
> ---
>  arch/arm/plat-samsung/devs.c              |   26
++++++++++++++++++++++++++
>  arch/arm/plat-samsung/include/plat/hdmi.h |   16 ++++++++++++++++
>  2 files changed, 42 insertions(+), 0 deletions(-)
>  create mode 100644 arch/arm/plat-samsung/include/plat/hdmi.h
> 
> diff --git a/arch/arm/plat-samsung/devs.c b/arch/arm/plat-samsung/devs.c
> index 07b5ac6..921ed88 100644
> --- a/arch/arm/plat-samsung/devs.c
> +++ b/arch/arm/plat-samsung/devs.c
> @@ -32,6 +32,8 @@
>  #include <linux/platform_data/s3c-hsudc.h>
>  #include <linux/platform_data/s3c-hsotg.h>
> 
> +#include <media/s5p_hdmi.h>
> +
>  #include <asm/irq.h>
>  #include <asm/pmu.h>
>  #include <asm/mach/arch.h>
> @@ -759,6 +761,30 @@ void __init s5p_i2c_hdmiphy_set_platdata(struct
> s3c2410_platform_i2c *pd)
>  	npd = s3c_set_platdata(pd, sizeof(struct s3c2410_platform_i2c),
>  			       &s5p_device_i2c_hdmiphy);
>  }
> +
> +struct s5p_hdmi_platform_data s5p_hdmi_def_platdata;
> +
> +void __init s5p_hdmi_set_platdata(struct i2c_board_info *hdmiphy_info,
> +				struct i2c_board_info *mhl_info, int
mhl_bus)
> +{
> +	struct s5p_hdmi_platform_data *pd = &s5p_hdmi_def_platdata;
> +
> +	if (soc_is_exynos4210() || soc_is_exynos4212() ||
> +						soc_is_exynos4412())
> +		pd->hdmiphy_bus = 8;
> +	else if (soc_is_s5pv210())
> +		pd->hdmiphy_bus = 3;
> +	else
> +		pd->hdmiphy_bus = 0;
> +
> +	pd->hdmiphy_info = hdmiphy_info;
> +	pd->mhl_info = mhl_info;
> +	pd->mhl_bus = mhl_bus;
> +
> +	s3c_set_platdata(pd, sizeof(struct s5p_hdmi_platform_data),
> +				&s5p_device_hdmi);
> +}

(Cc'ed Ayoung Sim and Jiun Yu in my team)

Totally I wonder, do we really need s5p_i2c_hdmiphy_set_platdata and this
adding together? You updated s5p_i2c_hdmiphy_set_platdata in your 1/4 patch
so I don't see why we should keep the bus number in duplicated...

Thanks.

Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.


> +
>  #endif /* CONFIG_S5P_DEV_I2C_HDMIPHY */
> 
>  /* I2S */
> diff --git a/arch/arm/plat-samsung/include/plat/hdmi.h b/arch/arm/plat-
> samsung/include/plat/hdmi.h
> new file mode 100644
> index 0000000..5dc7347
> --- /dev/null
> +++ b/arch/arm/plat-samsung/include/plat/hdmi.h
> @@ -0,0 +1,16 @@
> +/*
> + * Copyright (C) 2012 Samsung Electronics Co.Ltd
> + *
> + * This program is free software; you can redistribute  it and/or modify
> it
> + * under  the terms of  the GNU General  Public License as published by
> the
> + * Free Software Foundation;  either version 2 of the  License, or (at
> your
> + * option) any later version.
> + */
> +
> +#ifndef __PLAT_SAMSUNG_HDMI_H
> +#define __PLAT_SAMSUNG_HDMI_H __FILE__
> +
> +extern void s5p_hdmi_set_platdata(struct i2c_board_info *hdmiphy_info,
> +				struct i2c_board_info *mhl_info, int
mhl_bus);
> +
> +#endif /* __PLAT_SAMSUNG_HDMI_H */
> --
> 1.7.4.1

Re: [PATCH 2/4] ARM: SAMSUNG: Add API to set platform data for s5p-tv driver

[Tushar Behera]

On 07/17/2012 08:11 AM, Kukjin Kim wrote:
> Sachin Kamat wrote:
>>
>> From: Tushar Behera <tushar.behera@linaro.org>
>>
>> Commit 350f2f4dad64 ("[media] v4l: s5p-tv: hdmi: add support for
>> platform data") makes the presence of platform data mandatory for s5p-tv
>> driver. Adding an API to plat-samsung for this purpose.
>>
>> Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
>> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
>> ---
>>  arch/arm/plat-samsung/devs.c              |   26
> ++++++++++++++++++++++++++
>>  arch/arm/plat-samsung/include/plat/hdmi.h |   16 ++++++++++++++++
>>  2 files changed, 42 insertions(+), 0 deletions(-)
>>  create mode 100644 arch/arm/plat-samsung/include/plat/hdmi.h
>>
>> diff --git a/arch/arm/plat-samsung/devs.c b/arch/arm/plat-samsung/devs.c
>> index 07b5ac6..921ed88 100644
>> --- a/arch/arm/plat-samsung/devs.c
>> +++ b/arch/arm/plat-samsung/devs.c
>> @@ -32,6 +32,8 @@
>>  #include <linux/platform_data/s3c-hsudc.h>
>>  #include <linux/platform_data/s3c-hsotg.h>
>>
>> +#include <media/s5p_hdmi.h>
>> +
>>  #include <asm/irq.h>
>>  #include <asm/pmu.h>
>>  #include <asm/mach/arch.h>
>> @@ -759,6 +761,30 @@ void __init s5p_i2c_hdmiphy_set_platdata(struct
>> s3c2410_platform_i2c *pd)
>>  	npd = s3c_set_platdata(pd, sizeof(struct s3c2410_platform_i2c),
>>  			       &s5p_device_i2c_hdmiphy);
>>  }
>> +
>> +struct s5p_hdmi_platform_data s5p_hdmi_def_platdata;
>> +
>> +void __init s5p_hdmi_set_platdata(struct i2c_board_info *hdmiphy_info,
>> +				struct i2c_board_info *mhl_info, int
> mhl_bus)
>> +{
>> +	struct s5p_hdmi_platform_data *pd = &s5p_hdmi_def_platdata;
>> +
>> +	if (soc_is_exynos4210() || soc_is_exynos4212() ||
>> +						soc_is_exynos4412())
>> +		pd->hdmiphy_bus = 8;
>> +	else if (soc_is_s5pv210())
>> +		pd->hdmiphy_bus = 3;
>> +	else
>> +		pd->hdmiphy_bus = 0;
>> +
>> +	pd->hdmiphy_info = hdmiphy_info;
>> +	pd->mhl_info = mhl_info;
>> +	pd->mhl_bus = mhl_bus;
>> +
>> +	s3c_set_platdata(pd, sizeof(struct s5p_hdmi_platform_data),
>> +				&s5p_device_hdmi);
>> +}
> 
> (Cc'ed Ayoung Sim and Jiun Yu in my team)
> 
> Totally I wonder, do we really need s5p_i2c_hdmiphy_set_platdata and this
> adding together? You updated s5p_i2c_hdmiphy_set_platdata in your 1/4 patch
> so I don't see why we should keep the bus number in duplicated...
> 

s5p_i2c_hdmiphy_set_platdata sets platform data for
s5p_device_i2c_hdmiphy. 'bus_num' assigned in that structure is not
accessible in s5p-tv driver. But s5p-tv driver needs the bus number to
get appropriate i2c adapter. Hence we need to update the bus number in
both the places.


> Thanks.
> 
> Best regards,
> Kgene.
> --
> Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
> SW Solution Development Team, Samsung Electronics Co., Ltd.
> 
> 
>> +
>>  #endif /* CONFIG_S5P_DEV_I2C_HDMIPHY */
>>
>>  /* I2S */
>> diff --git a/arch/arm/plat-samsung/include/plat/hdmi.h b/arch/arm/plat-
>> samsung/include/plat/hdmi.h
>> new file mode 100644
>> index 0000000..5dc7347
>> --- /dev/null
>> +++ b/arch/arm/plat-samsung/include/plat/hdmi.h
>> @@ -0,0 +1,16 @@
>> +/*
>> + * Copyright (C) 2012 Samsung Electronics Co.Ltd
>> + *
>> + * This program is free software; you can redistribute  it and/or modify
>> it
>> + * under  the terms of  the GNU General  Public License as published by
>> the
>> + * Free Software Foundation;  either version 2 of the  License, or (at
>> your
>> + * option) any later version.
>> + */
>> +
>> +#ifndef __PLAT_SAMSUNG_HDMI_H
>> +#define __PLAT_SAMSUNG_HDMI_H __FILE__
>> +
>> +extern void s5p_hdmi_set_platdata(struct i2c_board_info *hdmiphy_info,
>> +				struct i2c_board_info *mhl_info, int
> mhl_bus);
>> +
>> +#endif /* __PLAT_SAMSUNG_HDMI_H */
>> --
>> 1.7.4.1
> 


-- 
Tushar Behera

RE: [PATCH 2/4] ARM: SAMSUNG: Add API to set platform data for s5p-tv driver

[Kukjin Kim]

> -----Original Message-----
> From: Tushar Behera [mailto:tushar.behera@linaro.org]
> Sent: Tuesday, July 17, 2012 12:33 PM
> To: Kukjin Kim
> Cc: 'Sachin Kamat'; linux-samsung-soc@vger.kernel.org; patches@linaro.org;
> a.sim@samsung.com; jiun.yu@samsung.com
> Subject: Re: [PATCH 2/4] ARM: SAMSUNG: Add API to set platform data for
> s5p-tv driver
> 
> On 07/17/2012 08:11 AM, Kukjin Kim wrote:
> > Sachin Kamat wrote:
> >>
> >> From: Tushar Behera <tushar.behera@linaro.org>
> >>
> >> Commit 350f2f4dad64 ("[media] v4l: s5p-tv: hdmi: add support for
> >> platform data") makes the presence of platform data mandatory for s5p-
> tv
> >> driver. Adding an API to plat-samsung for this purpose.
> >>
> >> Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
> >> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
> >> ---
> >>  arch/arm/plat-samsung/devs.c              |   26
> > ++++++++++++++++++++++++++
> >>  arch/arm/plat-samsung/include/plat/hdmi.h |   16 ++++++++++++++++
> >>  2 files changed, 42 insertions(+), 0 deletions(-)
> >>  create mode 100644 arch/arm/plat-samsung/include/plat/hdmi.h
> >>
> >> diff --git a/arch/arm/plat-samsung/devs.c b/arch/arm/plat-
> samsung/devs.c
> >> index 07b5ac6..921ed88 100644
> >> --- a/arch/arm/plat-samsung/devs.c
> >> +++ b/arch/arm/plat-samsung/devs.c
> >> @@ -32,6 +32,8 @@
> >>  #include <linux/platform_data/s3c-hsudc.h>
> >>  #include <linux/platform_data/s3c-hsotg.h>
> >>
> >> +#include <media/s5p_hdmi.h>
> >> +
> >>  #include <asm/irq.h>
> >>  #include <asm/pmu.h>
> >>  #include <asm/mach/arch.h>
> >> @@ -759,6 +761,30 @@ void __init s5p_i2c_hdmiphy_set_platdata(struct
> >> s3c2410_platform_i2c *pd)
> >>  	npd = s3c_set_platdata(pd, sizeof(struct s3c2410_platform_i2c),
> >>  			       &s5p_device_i2c_hdmiphy);
> >>  }
> >> +
> >> +struct s5p_hdmi_platform_data s5p_hdmi_def_platdata;
> >> +
> >> +void __init s5p_hdmi_set_platdata(struct i2c_board_info *hdmiphy_info,
> >> +				struct i2c_board_info *mhl_info, int
> > mhl_bus)
> >> +{
> >> +	struct s5p_hdmi_platform_data *pd = &s5p_hdmi_def_platdata;
> >> +
> >> +	if (soc_is_exynos4210() || soc_is_exynos4212() ||
> >> +						soc_is_exynos4412())
> >> +		pd->hdmiphy_bus = 8;
> >> +	else if (soc_is_s5pv210())
> >> +		pd->hdmiphy_bus = 3;
> >> +	else
> >> +		pd->hdmiphy_bus = 0;
> >> +
> >> +	pd->hdmiphy_info = hdmiphy_info;
> >> +	pd->mhl_info = mhl_info;
> >> +	pd->mhl_bus = mhl_bus;
> >> +
> >> +	s3c_set_platdata(pd, sizeof(struct s5p_hdmi_platform_data),
> >> +				&s5p_device_hdmi);
> >> +}
> >
> > (Cc'ed Ayoung Sim and Jiun Yu in my team)
> >
> > Totally I wonder, do we really need s5p_i2c_hdmiphy_set_platdata and
> this
> > adding together? You updated s5p_i2c_hdmiphy_set_platdata in your 1/4
> patch
> > so I don't see why we should keep the bus number in duplicated...
> >
> 
> s5p_i2c_hdmiphy_set_platdata sets platform data for
> s5p_device_i2c_hdmiphy. 'bus_num' assigned in that structure is not
> accessible in s5p-tv driver. But s5p-tv driver needs the bus number to
> get appropriate i2c adapter. Hence we need to update the bus number in
> both the places.
> 
Yeah, I know the situation, but I don't know why we should having the same
i2c channel is included in different platform_data. I need to talk about
this approach to hdmi guys in my team, I'll let you know as soon as
possible.

Thanks.

Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.

Re: [PATCH 2/4] ARM: SAMSUNG: Add API to set platform data for s5p-tv driver

[Tushar Behera]

On 07/20/2012 04:26 AM, Kukjin Kim wrote:
> 
>> -----Original Message-----
>> From: Tushar Behera [mailto:tushar.behera@linaro.org]
>> Sent: Tuesday, July 17, 2012 12:33 PM
>> To: Kukjin Kim
>> Cc: 'Sachin Kamat'; linux-samsung-soc@vger.kernel.org; patches@linaro.org;
>> a.sim@samsung.com; jiun.yu@samsung.com
>> Subject: Re: [PATCH 2/4] ARM: SAMSUNG: Add API to set platform data for
>> s5p-tv driver
>>
>> On 07/17/2012 08:11 AM, Kukjin Kim wrote:
>>> Sachin Kamat wrote:
>>>>
>>>> From: Tushar Behera <tushar.behera@linaro.org>
>>>>
>>>> Commit 350f2f4dad64 ("[media] v4l: s5p-tv: hdmi: add support for
>>>> platform data") makes the presence of platform data mandatory for s5p-
>> tv
>>>> driver. Adding an API to plat-samsung for this purpose.
>>>>
>>>> Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
>>>> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
>>>> ---
>>>>  arch/arm/plat-samsung/devs.c              |   26
>>> ++++++++++++++++++++++++++
>>>>  arch/arm/plat-samsung/include/plat/hdmi.h |   16 ++++++++++++++++
>>>>  2 files changed, 42 insertions(+), 0 deletions(-)
>>>>  create mode 100644 arch/arm/plat-samsung/include/plat/hdmi.h
>>>>
>>>> diff --git a/arch/arm/plat-samsung/devs.c b/arch/arm/plat-
>> samsung/devs.c
>>>> index 07b5ac6..921ed88 100644
>>>> --- a/arch/arm/plat-samsung/devs.c
>>>> +++ b/arch/arm/plat-samsung/devs.c
>>>> @@ -32,6 +32,8 @@
>>>>  #include <linux/platform_data/s3c-hsudc.h>
>>>>  #include <linux/platform_data/s3c-hsotg.h>
>>>>
>>>> +#include <media/s5p_hdmi.h>
>>>> +
>>>>  #include <asm/irq.h>
>>>>  #include <asm/pmu.h>
>>>>  #include <asm/mach/arch.h>
>>>> @@ -759,6 +761,30 @@ void __init s5p_i2c_hdmiphy_set_platdata(struct
>>>> s3c2410_platform_i2c *pd)
>>>>  	npd = s3c_set_platdata(pd, sizeof(struct s3c2410_platform_i2c),
>>>>  			       &s5p_device_i2c_hdmiphy);
>>>>  }
>>>> +
>>>> +struct s5p_hdmi_platform_data s5p_hdmi_def_platdata;
>>>> +
>>>> +void __init s5p_hdmi_set_platdata(struct i2c_board_info *hdmiphy_info,
>>>> +				struct i2c_board_info *mhl_info, int
>>> mhl_bus)
>>>> +{
>>>> +	struct s5p_hdmi_platform_data *pd = &s5p_hdmi_def_platdata;
>>>> +
>>>> +	if (soc_is_exynos4210() || soc_is_exynos4212() ||
>>>> +						soc_is_exynos4412())
>>>> +		pd->hdmiphy_bus = 8;
>>>> +	else if (soc_is_s5pv210())
>>>> +		pd->hdmiphy_bus = 3;
>>>> +	else
>>>> +		pd->hdmiphy_bus = 0;
>>>> +
>>>> +	pd->hdmiphy_info = hdmiphy_info;
>>>> +	pd->mhl_info = mhl_info;
>>>> +	pd->mhl_bus = mhl_bus;
>>>> +
>>>> +	s3c_set_platdata(pd, sizeof(struct s5p_hdmi_platform_data),
>>>> +				&s5p_device_hdmi);
>>>> +}
>>>
>>> (Cc'ed Ayoung Sim and Jiun Yu in my team)
>>>
>>> Totally I wonder, do we really need s5p_i2c_hdmiphy_set_platdata and
>> this
>>> adding together? You updated s5p_i2c_hdmiphy_set_platdata in your 1/4
>> patch
>>> so I don't see why we should keep the bus number in duplicated...
>>>
>>
>> s5p_i2c_hdmiphy_set_platdata sets platform data for
>> s5p_device_i2c_hdmiphy. 'bus_num' assigned in that structure is not
>> accessible in s5p-tv driver. But s5p-tv driver needs the bus number to
>> get appropriate i2c adapter. Hence we need to update the bus number in
>> both the places.
>>
> Yeah, I know the situation, but I don't know why we should having the same
> i2c channel is included in different platform_data. I need to talk about
> this approach to hdmi guys in my team, I'll let you know as soon as
> possible.
> 

While configuring all the i2c channels, we need to pass the channel
number. Since the channel number is not fixed for hdmiphy, it was
updated as per respective CPUs.

The same would be done for other I2C channels also, we pass channel
number in platform data (s3c_i2c0_set_platdata) and then specify the bus
number as the first parameter in i2c_register_board_info(...) while
adding i2c devices.

It would be great if we can find a resolution on this before the merge
window. HDMI support in mainline kernel is broken since 3.4-rc1 and we
would love to get this fixed during 3.6.

> Thanks.
> 
> Best regards,
> Kgene.
> --
> Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
> SW Solution Development Team, Samsung Electronics Co., Ltd.
> 


-- 
Tushar Behera

RE: [PATCH 2/4] ARM: SAMSUNG: Add API to set platform data for s5p-tv driver

[Kukjin Kim]

Tushar Behera wrote:
> 
> On 07/20/2012 04:26 AM, Kukjin Kim wrote:
> >

[...]

> >>
> > Yeah, I know the situation, but I don't know why we should having the
> same
> > i2c channel is included in different platform_data. I need to talk about
> > this approach to hdmi guys in my team, I'll let you know as soon as
> > possible.
> >
> 
> While configuring all the i2c channels, we need to pass the channel
> number. Since the channel number is not fixed for hdmiphy, it was
> updated as per respective CPUs.
> 
> The same would be done for other I2C channels also, we pass channel
> number in platform data (s3c_i2c0_set_platdata) and then specify the bus
> number as the first parameter in i2c_register_board_info(...) while
> adding i2c devices.
> 
> It would be great if we can find a resolution on this before the merge
> window. HDMI support in mainline kernel is broken since 3.4-rc1 and we
> would love to get this fixed during 3.6.
> 
OK, firstly let's fix its working. But I'm still thinking we need to sort
this out next time.

Will apply this series, thanks.

Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.