Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1554127

Re: [PATCH v7 3/4] drm/panel: Add support for S6E3HA2 panel driver on TM2 board

Show all headers | View raw

On 06.01.2017 09:36, Inki Dae wrote:
>
> 2017년 01월 06일 17:18에 Andi Shyti 이(가) 쓴 글:
>> Hi Inki,
>>
>> Thanks for the reply, but...
>>
>>>>> +static const struct drm_display_mode default_mode = {
>>>>> +	.clock = 222372,
>>>>> +	.hdisplay = 1440,
>>>>> +	.hsync_start = 1440 + 1,
>>>>> +	.hsync_end = 1440 + 1 + 1,
>>>>> +	.htotal = 1440 + 1 + 1 + 1,
>>>>> +	.vdisplay = 2560,
>>>>> +	.vsync_start = 2560 + 1,
>>>>> +	.vsync_end = 2560 + 1 + 1,
>>>>> +	.vtotal = 2560 + 1 + 1 + 15,
>>>>> +	.vrefresh = 60,
>>>>> +	.flags = 0,
>>>>> +};
>>>> how is this working with tm2e? Are these values valid for both
>>>> the boards?
>>> We don't need to consider tm2e board with two reasones,
>>> 1. there is no tm2e board support in mainline
>>> 2. the panel on tm2 would be a little bit different from one on tm2e
>> ... this display in the Tizen Kernel is supported by both:
>> tm2 [1] and tm2e [2]. The only differences are:
> Why tm2e dts file is in mainline? Seems communication miss with Chanwoo. :( 
>
>> TM2:
>>    clock-frequency = <14874444>;
>>    hactive = <1440>;
>>
>> TM2E:
>>    clock-frequency = <16523724>;
>>    hactive = <1600>;
>>
>> I don't know much about the differences you mention in point 2,
>> but it's a pity to drop support only because we don't want to put
>> in the dts the 'hactive', and 'clock-frequency' properties.
> Anyway, tm2e board is already in mainline so Panel driver may need to identify what kinds of panel is probed to decide porch values. I think there are relevant registers in MCU of the Panel device to check version or similar thing.

I think we can safely use different compatible string for tm2e - it uses
different display IC controller - s6e3hf2, driver will provide timings
based on it.
As far as I examined available specs/docs there is no reliable register
which can be used to safely distinguish it on runtime, but the docs I
have are far from completeness.

Regards
Andrzej

>
> Thanks.
>
>> Andi
>>
>> [1] https://git.tizen.org/cgit/platform/kernel/linux-exynos/tree/arch/arm64/boot/dts/exynos/exynos5433-tm2.dts?h=tizen#n1284
>> [2] https://git.tizen.org/cgit/platform/kernel/linux-exynos/tree/arch/arm64/boot/dts/exynos/exynos5433-tm2e.dts?h=tizen#n1270
>> --
>> To unsubscribe from this list: send the line "unsubscribe devicetree" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>
>> .
>>
>

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v7 3/4] drm/panel: Add support for S6E3HA2 panel driver on TM2  board Hoegeun Kwon <hoegeun.kwon@samsung.com> - 2017-01-05 11:30 +0100
  Re: [PATCH v7 3/4] drm/panel: Add support for S6E3HA2 panel driver on  TM2 board Krzysztof Kozlowski <krzk@kernel.org> - 2017-01-05 19:20 +0100
  Re: [PATCH v7 3/4] drm/panel: Add support for S6E3HA2 panel driver on  TM2 board Andi Shyti <andi.shyti@samsung.com> - 2017-01-06 06:30 +0100
    Re: [PATCH v7 3/4] drm/panel: Add support for S6E3HA2 panel driver on  TM2 board Inki Dae <inki.dae@samsung.com> - 2017-01-06 06:50 +0100
      Re: [PATCH v7 3/4] drm/panel: Add support for S6E3HA2 panel driver on  TM2 board Andi Shyti <andi.shyti@samsung.com> - 2017-01-06 09:20 +0100
        Re: [PATCH v7 3/4] drm/panel: Add support for S6E3HA2 panel driver on  TM2 board Inki Dae <inki.dae@samsung.com> - 2017-01-06 09:40 +0100
          Re: [PATCH v7 3/4] drm/panel: Add support for S6E3HA2 panel driver on  TM2 board Andrzej Hajda <a.hajda@samsung.com> - 2017-01-09 08:40 +0100
            Re: [PATCH v7 3/4] drm/panel: Add support for S6E3HA2 panel driver on  TM2 board Inki Dae <inki.dae@samsung.com> - 2017-01-09 10:30 +0100
              Re: [PATCH v7 3/4] drm/panel: Add support for S6E3HA2 panel driver on  TM2 board Andrzej Hajda <a.hajda@samsung.com> - 2017-01-09 11:00 +0100
                Re: [PATCH v7 3/4] drm/panel: Add support for S6E3HA2 panel driver on  TM2 board Inki Dae <inki.dae@samsung.com> - 2017-01-10 02:20 +0100

csiph-web