* Reverting https://patchwork.freedesktop.org/patch/336850/
@ 2020-02-29 1:35 ` abhinavk
0 siblings, 0 replies; 6+ messages in thread
From: abhinavk @ 2020-02-29 1:35 UTC (permalink / raw
To: dri-devel, linux-arm-msm
Cc: robdclark, seanpaul, nganji, aravindh, chandanu, varar, treding,
daniel.vetter
Hi Thierry
For DP drivers using MSM chipsets, we are using drm_dp_link_configure,
drm_dp_link_power_up and drm_dp_link_power_down functions. Here is the
patch for reference:
https://patchwork.freedesktop.org/patch/343346/
We will be posting a newer version of the driver soon.
For that, we plan to revert
https://patchwork.freedesktop.org/patch/336850/ as now there are more
users of the functions and not just tegra.
Let us know if this is the right approach or shall we make the required
functions local to our drivers.
Thanks
Abhinav
^ permalink raw reply [flat|nested] 6+ messages in thread
* Reverting https://patchwork.freedesktop.org/patch/336850/
@ 2020-02-29 1:35 ` abhinavk
0 siblings, 0 replies; 6+ messages in thread
From: abhinavk @ 2020-02-29 1:35 UTC (permalink / raw
To: dri-devel, linux-arm-msm
Cc: treding, daniel.vetter, varar, seanpaul, aravindh, chandanu
Hi Thierry
For DP drivers using MSM chipsets, we are using drm_dp_link_configure,
drm_dp_link_power_up and drm_dp_link_power_down functions. Here is the
patch for reference:
https://patchwork.freedesktop.org/patch/343346/
We will be posting a newer version of the driver soon.
For that, we plan to revert
https://patchwork.freedesktop.org/patch/336850/ as now there are more
users of the functions and not just tegra.
Let us know if this is the right approach or shall we make the required
functions local to our drivers.
Thanks
Abhinav
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Reverting https://patchwork.freedesktop.org/patch/336850/
2020-02-29 1:35 ` abhinavk
@ 2020-03-02 8:09 ` Thierry Reding
-1 siblings, 0 replies; 6+ messages in thread
From: Thierry Reding @ 2020-03-02 8:09 UTC (permalink / raw
To: abhinavk
Cc: dri-devel, linux-arm-msm, treding, daniel.vetter, varar, seanpaul,
aravindh, chandanu
[-- Attachment #1: Type: text/plain, Size: 1066 bytes --]
On Fri, Feb 28, 2020 at 05:35:51PM -0800, abhinavk@codeaurora.org wrote:
> Hi Thierry
>
> For DP drivers using MSM chipsets, we are using drm_dp_link_configure,
> drm_dp_link_power_up and drm_dp_link_power_down functions. Here is the patch
> for reference:
>
> https://patchwork.freedesktop.org/patch/343346/
>
> We will be posting a newer version of the driver soon.
>
> For that, we plan to revert https://patchwork.freedesktop.org/patch/336850/
> as now there are more users of the functions and not just tegra.
>
> Let us know if this is the right approach or shall we make the required
> functions local to our drivers.
I think reimplementing these locally in you drivers is the preferred
way. It's not so much a lack of users (there were 5 or so in total when
I made the change to remove these helpers), but rather the general
approach that was being criticized by other driver maintainers. I'm not
aware of any of their opinions having changed on this subject, so I'd
recommend going down the path of least resistance.
Thierry
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Reverting https://patchwork.freedesktop.org/patch/336850/
@ 2020-03-02 8:09 ` Thierry Reding
0 siblings, 0 replies; 6+ messages in thread
From: Thierry Reding @ 2020-03-02 8:09 UTC (permalink / raw
To: abhinavk
Cc: aravindh, linux-arm-msm, dri-devel, varar, seanpaul,
daniel.vetter, treding, chandanu
[-- Attachment #1.1: Type: text/plain, Size: 1066 bytes --]
On Fri, Feb 28, 2020 at 05:35:51PM -0800, abhinavk@codeaurora.org wrote:
> Hi Thierry
>
> For DP drivers using MSM chipsets, we are using drm_dp_link_configure,
> drm_dp_link_power_up and drm_dp_link_power_down functions. Here is the patch
> for reference:
>
> https://patchwork.freedesktop.org/patch/343346/
>
> We will be posting a newer version of the driver soon.
>
> For that, we plan to revert https://patchwork.freedesktop.org/patch/336850/
> as now there are more users of the functions and not just tegra.
>
> Let us know if this is the right approach or shall we make the required
> functions local to our drivers.
I think reimplementing these locally in you drivers is the preferred
way. It's not so much a lack of users (there were 5 or so in total when
I made the change to remove these helpers), but rather the general
approach that was being criticized by other driver maintainers. I'm not
aware of any of their opinions having changed on this subject, so I'd
recommend going down the path of least resistance.
Thierry
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
[-- Attachment #2: Type: text/plain, Size: 160 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Reverting https://patchwork.freedesktop.org/patch/336850/
2020-03-02 8:09 ` Thierry Reding
@ 2020-03-02 20:41 ` abhinavk
-1 siblings, 0 replies; 6+ messages in thread
From: abhinavk @ 2020-03-02 20:41 UTC (permalink / raw
To: Thierry Reding
Cc: aravindh, linux-arm-msm, dri-devel, varar, seanpaul,
daniel.vetter, treding, chandanu
Hi Thierry
Thanks for the response.
We shall reimplement these locally in our drivers and push the next
patchset.
Thanks
Abhinav
On 2020-03-02 00:09, Thierry Reding wrote:
> On Fri, Feb 28, 2020 at 05:35:51PM -0800, abhinavk@codeaurora.org
> wrote:
>> Hi Thierry
>>
>> For DP drivers using MSM chipsets, we are using drm_dp_link_configure,
>> drm_dp_link_power_up and drm_dp_link_power_down functions. Here is the
>> patch
>> for reference:
>>
>> https://patchwork.freedesktop.org/patch/343346/
>>
>> We will be posting a newer version of the driver soon.
>>
>> For that, we plan to revert
>> https://patchwork.freedesktop.org/patch/336850/
>> as now there are more users of the functions and not just tegra.
>>
>> Let us know if this is the right approach or shall we make the
>> required
>> functions local to our drivers.
>
> I think reimplementing these locally in you drivers is the preferred
> way. It's not so much a lack of users (there were 5 or so in total when
> I made the change to remove these helpers), but rather the general
> approach that was being criticized by other driver maintainers. I'm not
> aware of any of their opinions having changed on this subject, so I'd
> recommend going down the path of least resistance.
>
> Thierry
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Reverting https://patchwork.freedesktop.org/patch/336850/
@ 2020-03-02 20:41 ` abhinavk
0 siblings, 0 replies; 6+ messages in thread
From: abhinavk @ 2020-03-02 20:41 UTC (permalink / raw
To: Thierry Reding
Cc: treding, linux-arm-msm, dri-devel, varar, seanpaul, daniel.vetter,
aravindh, chandanu
Hi Thierry
Thanks for the response.
We shall reimplement these locally in our drivers and push the next
patchset.
Thanks
Abhinav
On 2020-03-02 00:09, Thierry Reding wrote:
> On Fri, Feb 28, 2020 at 05:35:51PM -0800, abhinavk@codeaurora.org
> wrote:
>> Hi Thierry
>>
>> For DP drivers using MSM chipsets, we are using drm_dp_link_configure,
>> drm_dp_link_power_up and drm_dp_link_power_down functions. Here is the
>> patch
>> for reference:
>>
>> https://patchwork.freedesktop.org/patch/343346/
>>
>> We will be posting a newer version of the driver soon.
>>
>> For that, we plan to revert
>> https://patchwork.freedesktop.org/patch/336850/
>> as now there are more users of the functions and not just tegra.
>>
>> Let us know if this is the right approach or shall we make the
>> required
>> functions local to our drivers.
>
> I think reimplementing these locally in you drivers is the preferred
> way. It's not so much a lack of users (there were 5 or so in total when
> I made the change to remove these helpers), but rather the general
> approach that was being criticized by other driver maintainers. I'm not
> aware of any of their opinions having changed on this subject, so I'd
> recommend going down the path of least resistance.
>
> Thierry
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-03-02 20:41 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-02-29 1:35 Reverting https://patchwork.freedesktop.org/patch/336850/ abhinavk
2020-02-29 1:35 ` abhinavk
2020-03-02 8:09 ` Thierry Reding
2020-03-02 8:09 ` Thierry Reding
2020-03-02 20:41 ` abhinavk
2020-03-02 20:41 ` abhinavk
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.