On 21.03.2017 06:10, Inki Dae wrote:
>
> 2017년 03월 15일 20:20에 Andrzej Hajda 이(가) 쓴 글:
>> DSI forwards te-gpios interrupts to display controller, but if display
>> controller works in HW-TRIGGER mode this interrupt is not necessary.
>> Making te-gpios property optional allows to avoid generating spare
>> interrupts.
>> With this patch we can get rid of 60 interrupt callbacks per second.
> As I said before like below,
> "If panel device node of command mode panel device doesn't provide te-gpios property then now the panel driver will fail to probe.
> Seems this patch makes it to allow the panel driver probing is always ok even if gpio is invalid."
>
> This patch will fix critical issue that the panel driver fails to probe if the panel device node has no te-gpios property.
> I think adding above sentence to this description would be good. So I will merge this patch after adding the description. I can do this.
Thanks, it seems I have misunderstood your previous post.
Regards
Andrzej
>
> Thanks,
> Inki Dae
>
>> Signed-off-by: Andrzej Hajda <a.hajda@xxxxxxxxxxx>
>> ---
>> drivers/gpu/drm/exynos/exynos_drm_dsi.c | 5 ++++-
>> 1 file changed, 4 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/gpu/drm/exynos/exynos_drm_dsi.c b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
>> index 812e2ec..822094d 100644
>> --- a/drivers/gpu/drm/exynos/exynos_drm_dsi.c
>> +++ b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
>> @@ -1348,9 +1348,12 @@ static int exynos_dsi_register_te_irq(struct exynos_dsi *dsi)
>> int te_gpio_irq;
>>
>> dsi->te_gpio = of_get_named_gpio(dsi->panel_node, "te-gpios", 0);
>> + if (dsi->te_gpio == -ENOENT)
>> + return 0;
>> +
>> if (!gpio_is_valid(dsi->te_gpio)) {
>> - dev_err(dsi->dev, "no te-gpios specified\n");
>> ret = dsi->te_gpio;
>> + dev_err(dsi->dev, "cannot get te-gpios, %d\n", ret);
>> goto out;
>> }
>>
>>
>
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel
