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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> -----Original Message-----
> From: Tushar Behera [mailto:tushar.behera@xxxxxxxxxx]
> Sent: Tuesday, July 17, 2012 12:33 PM
> To: Kukjin Kim
> Cc: 'Sachin Kamat'; linux-samsung-soc@xxxxxxxxxxxxxxx; patches@xxxxxxxxxx;
> a.sim@xxxxxxxxxxx; jiun.yu@xxxxxxxxxxx
> 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@xxxxxxxxxx>
> >>
> >> 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@xxxxxxxxxx>
> >> Signed-off-by: Sachin Kamat <sachin.kamat@xxxxxxxxxx>
> >> ---
> >>  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@xxxxxxxxxxx>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux