Re: [Patch v2 2/3] drm/mst: Refactor the flow for payload allocation/removement

2023-09-13 Thread Imre Deak
; > > amd-gfx@lists.freedesktop.org; jani.nik...@intel.com; imre.d...@intel.com; > > Wentland, Harry ; Zuo, Jerry > > ; ville.syrj...@linux.intel.com > > Subject: Re: [Patch v2 2/3] drm/mst: Refactor the flow for payload > > allocation/removement > > > > On Thu, A

RE: [Patch v2 2/3] drm/mst: Refactor the flow for payload allocation/removement

2023-09-04 Thread Lin, Wayne
mre.d...@intel.com; > Wentland, Harry ; Zuo, Jerry > ; ville.syrj...@linux.intel.com > Subject: Re: [Patch v2 2/3] drm/mst: Refactor the flow for payload > allocation/removement > > On Thu, Aug 31, 2023 at 6:45 PM Lyude Paul wrote: > > > > On Thu, 2023-08-24 at 04:12 +,

Re: [Patch v2 2/3] drm/mst: Refactor the flow for payload allocation/removement

2023-09-01 Thread Alex Deucher
> > -Original Message- > > > > > From: Lyude Paul > > > > > Sent: Friday, August 18, 2023 6:17 AM > > > > > To: Lin, Wayne ; dri-de...@lists.freedesktop.org; > > > > > amd-gfx@lists.freedesktop.org > > > > > Cc: jani.nik

Re: [Patch v2 2/3] drm/mst: Refactor the flow for payload allocation/removement

2023-08-31 Thread Lyude Paul
; ville.syrj...@linux.intel.com; > > imre.d...@intel.com; > > Wentland, Harry ; Zuo, Jerry > > > > Subject: Re: [Patch v2 2/3] drm/mst: Refactor the flow for payload > > allocation/removement > > > > Sure - you're also welcome to push the first two patches af

RE: [Patch v2 2/3] drm/mst: Refactor the flow for payload allocation/removement

2023-08-23 Thread Lin, Wayne
esktop.org > Cc: jani.nik...@intel.com; ville.syrj...@linux.intel.com; imre.d...@intel.com; > Wentland, Harry ; Zuo, Jerry > > Subject: Re: [Patch v2 2/3] drm/mst: Refactor the flow for payload > allocation/removement > > Sure - you're also welcome to push the first two pa

Re: [Patch v2 2/3] drm/mst: Refactor the flow for payload allocation/removement

2023-08-23 Thread Lyude Paul
> > From: Lyude Paul > > Sent: Friday, August 18, 2023 6:17 AM > > To: Lin, Wayne ; dri-de...@lists.freedesktop.org; > > amd-gfx@lists.freedesktop.org > > Cc: jani.nik...@intel.com; ville.syrj...@linux.intel.com; > > imre.d...@intel.com; > > Wentland,

RE: [Patch v2 2/3] drm/mst: Refactor the flow for payload allocation/removement

2023-08-22 Thread Lin, Wayne
el.com; ville.syrj...@linux.intel.com; imre.d...@intel.com; > Wentland, Harry ; Zuo, Jerry > > Subject: Re: [Patch v2 2/3] drm/mst: Refactor the flow for payload > allocation/removement > > Two small comments: > > On Mon, 2023-08-07 at 10:56 +0800, Wayne Lin wrote: > > [W

Re: [Patch v2 2/3] drm/mst: Refactor the flow for payload allocation/removement

2023-08-17 Thread Lyude Paul
Two small comments: On Mon, 2023-08-07 at 10:56 +0800, Wayne Lin wrote: > [Why] > Today, the allocation/deallocation steps and status is a bit unclear. > > For instance, payload->vc_start_slot = -1 stands for "the failure of > updating DPCD payload ID table" and can also represent as "payload is

RE: [Patch v2 2/3] drm/mst: Refactor the flow for payload allocation/removement

2023-08-07 Thread Lin, Wayne
el.com; ville.syrj...@linux.intel.com; imre.d...@intel.com; > Wentland, Harry ; Zuo, Jerry > > Subject: Re: [Patch v2 2/3] drm/mst: Refactor the flow for payload > allocation/removement > > Oo! This is a wonderful idea so far - keeping track of the status of > allocat

Re: [Patch v2 2/3] drm/mst: Refactor the flow for payload allocation/removement

2023-08-07 Thread Lyude Paul
Oo! This is a wonderful idea so far - keeping track of the status of allocations like this solves a lot of problems, especially with regards to the fact this actually seems to make it possible for us to have much better handling of payload failures in drivers - especially in situations like

[Patch v2 2/3] drm/mst: Refactor the flow for payload allocation/removement

2023-08-06 Thread Wayne Lin
[Why] Today, the allocation/deallocation steps and status is a bit unclear. For instance, payload->vc_start_slot = -1 stands for "the failure of updating DPCD payload ID table" and can also represent as "payload is not allocated yet". These two cases should be handled differently and hence better