Re: [PATCH] drm/amdgpu/dm/mst: Fix incorrect usage of drm_dp_add_payload_part2()

2022-10-19 Thread Lyude Paul
kita Lipski ; Mahfooz, > > Hamza ; Claudio Suarez ; Colin > > Ian King ; Jani Nikula ; open > > list > > Subject: Re: [PATCH] drm/amdgpu/dm/mst: Fix incorrect usage of > > drm_dp_add_payload_part2() > > > > On Tue, 2022-10-04 at 16:46 -0400, Rodrigo Siqueir

RE: [PATCH] drm/amdgpu/dm/mst: Fix incorrect usage of drm_dp_add_payload_part2()

2022-10-16 Thread Lin, Wayne
hristian > ; Pan, Xinhui ; David > Airlie ; Daniel Vetter ; Zuo, Jerry > ; Lin, Wayne ; Chen, Ian > ; Mikita Lipski ; Mahfooz, > Hamza ; Claudio Suarez ; Colin > Ian King ; Jani Nikula ; open > list > Subject: Re: [PATCH] drm/amdgpu/dm/mst: Fix incorrect usage of > drm_d

Re: [PATCH] drm/amdgpu/dm/mst: Fix incorrect usage of drm_dp_add_payload_part2()

2022-10-05 Thread Lyude Paul
On Tue, 2022-10-04 at 16:46 -0400, Rodrigo Siqueira Jordao wrote: > > On 2022-10-04 16:24, Lyude Paul wrote: > > Yikes, it appears somehow I totally made a mistake here. We're currently > > checking to see if drm_dp_add_payload_part2() returns a non-zero value to > > indicate success. That's

Re: [PATCH] drm/amdgpu/dm/mst: Fix incorrect usage of drm_dp_add_payload_part2()

2022-10-04 Thread Rodrigo Siqueira Jordao
On 2022-10-04 16:24, Lyude Paul wrote: Yikes, it appears somehow I totally made a mistake here. We're currently checking to see if drm_dp_add_payload_part2() returns a non-zero value to indicate success. That's totally wrong though, as this function only returns a zero value on success - not

[PATCH] drm/amdgpu/dm/mst: Fix incorrect usage of drm_dp_add_payload_part2()

2022-10-04 Thread Lyude Paul
Yikes, it appears somehow I totally made a mistake here. We're currently checking to see if drm_dp_add_payload_part2() returns a non-zero value to indicate success. That's totally wrong though, as this function only returns a zero value on success - not the other way around. So, fix that.