On Mon, 2011-08-29 at 11:44 +0530, mythr...@ti.com wrote:
> From: Mythri P K <mythr...@ti.com>
> 
> As the pll and the video configuration info are part of the ip_data those
> structures are moved to the ip_data strtucure.Also the functions are modified
> accordingly to take care of this movement.

structure typoed, and use a space after period.

> Signed-off-by: Mythri P K <mythr...@ti.com>
> ---
>  drivers/video/omap2/dss/hdmi.c |   34 +++++++++++++++-------------------
>  drivers/video/omap2/dss/hdmi.h |   18 ++++++++++--------
>  2 files changed, 25 insertions(+), 27 deletions(-)
> 
> diff --git a/drivers/video/omap2/dss/hdmi.c b/drivers/video/omap2/dss/hdmi.c
> index 544f93e..084a47e 100644
> --- a/drivers/video/omap2/dss/hdmi.c
> +++ b/drivers/video/omap2/dss/hdmi.c
> @@ -59,7 +59,6 @@ static struct {
>       u8 edid[HDMI_EDID_MAX_LENGTH];
>       u8 edid_set;
>       bool custom_set;
> -     struct hdmi_config cfg;
>  
>       struct clk *sys_clk;
>  } hdmi;
> @@ -230,11 +229,11 @@ int hdmi_init_display(struct omap_dss_device *dssdev)
>  }
>  
>  static int hdmi_pll_init(struct hdmi_ip_data *ip_data,
> -             enum hdmi_clk_refsel refsel, int dcofreq,
> -             struct hdmi_pll_info *fmt, u16 sd)
> +                     enum hdmi_clk_refsel refsel)
>  {
>       u32 r;
>       void __iomem *pll_base = hdmi_pll_base(ip_data);
> +     struct hdmi_pll_info *fmt = &ip_data->pll_data;
>  
>       /* PLL start always use manual mode */
>       REG_FLD_MOD(pll_base, PLLCTRL_PLL_CONTROL, 0x0, 0, 0);
> @@ -251,9 +250,9 @@ static int hdmi_pll_init(struct hdmi_ip_data *ip_data,
>       r = FLD_MOD(r, 0x1, 13, 13); /* PLL_REFEN */
>       r = FLD_MOD(r, 0x0, 14, 14); /* PHY_CLKINEN de-assert during locking */
>  
> -     if (dcofreq) {
> +     if (fmt->dcofreq) {
>               /* divider programming for frequency beyond 1000Mhz */
> -             REG_FLD_MOD(pll_base, PLLCTRL_CFG3, sd, 17, 10);
> +             REG_FLD_MOD(pll_base, PLLCTRL_CFG3, fmt->regsd, 17, 10);
>               r = FLD_MOD(r, 0x4, 3, 1); /* 1000MHz and 2000MHz */
>       } else {
>               r = FLD_MOD(r, 0x2, 3, 1); /* 500MHz and 1000MHz */
> @@ -379,8 +378,7 @@ static int hdmi_phy_init(struct hdmi_ip_data *ip_data)
>       return 0;
>  }
>  
> -static int hdmi_pll_program(struct hdmi_ip_data *ip_data,
> -                             struct hdmi_pll_info *fmt)
> +static int hdmi_pll_program(struct hdmi_ip_data *ip_data)
>  {
>       u16 r = 0;
>       enum hdmi_clk_refsel refsel;
> @@ -399,7 +397,7 @@ static int hdmi_pll_program(struct hdmi_ip_data *ip_data,
>  
>       refsel = HDMI_REFSEL_SYSCLK;
>  
> -     r = hdmi_pll_init(ip_data, refsel, fmt->dcofreq, fmt, fmt->regsd);
> +     r = hdmi_pll_init(ip_data, refsel);

I don't think I quite understood why refsel is not part of the pll info.
And if it has to be hardcoded, you could as well do that in
hdmi_pll_init().

 Tomi


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

Reply via email to