Re: [PATCH 3/8] OMAP4 : DSS : HDMI: HDMI specific display controller and dss change.

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

 



Hi Tomi,

On Sun, Feb 27, 2011 at 2:53 PM, Tomi Valkeinen <tomi.valkeinen@xxxxxx> wrote:
> On Fri, 2011-02-25 at 08:21 -0600, K, Mythri P wrote:
>> Adding changes to set gamma table bit for TV interface and function to select
>> between VENC and HDMI.
>>
>> Signed-off-by: Mythri P K <mythripk@xxxxxx>
>> ---
>>  drivers/video/omap2/dss/dispc.c |    5 +++++
>>  drivers/video/omap2/dss/dss.c   |    7 +++++++
>>  drivers/video/omap2/dss/dss.h   |    2 ++
>>  3 files changed, 14 insertions(+), 0 deletions(-)
>>
>> diff --git a/drivers/video/omap2/dss/dispc.c b/drivers/video/omap2/dss/dispc.c
>> index 6d9bb17..16f1106 100644
>> --- a/drivers/video/omap2/dss/dispc.c
>> +++ b/drivers/video/omap2/dss/dispc.c
>> @@ -1213,6 +1213,11 @@ void dispc_enable_zorder(enum omap_plane plane, bool enable)
>>       dispc_write_reg(dispc_reg_att[plane], val);
>>  }
>>
>> +void dispc_enable_gamma_table(bool enable)
>> +{
>> +     REG_FLD_MOD(DISPC_CONFIG, enable, 9, 9);
>> +}
>> +
>>  static void _dispc_set_vid_color_conv(enum omap_plane plane, bool enable)
>>  {
>>       u32 val;
>> diff --git a/drivers/video/omap2/dss/dss.c b/drivers/video/omap2/dss/dss.c
>> index 99de4e1..127d42e 100644
>> --- a/drivers/video/omap2/dss/dss.c
>> +++ b/drivers/video/omap2/dss/dss.c
>> @@ -559,6 +559,13 @@ void dss_set_dac_pwrdn_bgz(bool enable)
>>       REG_FLD_MOD(DSS_CONTROL, enable, 5, 5); /* DAC Power-Down Control */
>>  }
>>
>> +void dss_select_hdmi_venc(bool hdmi)
>> +{
>> +     REG_FLD_MOD(DSS_CONTROL, hdmi, 15, 15); /* 0x1 for HDMI, 0x0 VENC */
>> +     if (hdmi)
>> +             REG_FLD_MOD(DSS_CONTROL, 0, 9, 8);
>> +}
>
> bool argument for this function is very confusing. Which one is true?
> Perhaps this could use an enum.
I have added a comment , 1 is for HDMI and 0 for VENC.
>
> I've tried to keep the habit of having the name of the register field in
> comments to make the code easier to read. For example, /*
> VENC_HDMI_SWITCH */ for the bit 15.
I can add this.
>
> The bits 8 and 9 are FCK_CLK_SWITCH. Why do you set it here? Looks
> rather dangerous to change the clock source like that.
>
This is to select what would be the clock source to DISPC, HDMI would
need the DISPC
to run at the rate of pixel clock (TMDS if deep color is set ) required by HDMI.
Where as DSI does not have any such constraint( correct me if i am wrong).
I suppose we can have a patch to select the correct clock source to
DISPC dynamically
based on what panels are connected later on ?

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


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux