Re: [PATCH] drm/buddy: Fix drm buddy info output format

2023-08-06 Thread Christian König
Am 04.08.23 um 08:56 schrieb Ma Jun: [1] Change pages to blocks to avoid confusion. [2] Fix output format to align the output info. Signed-off-by: Ma Jun Of hand looks good to me, but Arun should probably judge. Christian. --- drivers/gpu/drm/drm_buddy.c | 8 1 file changed, 4

RE: [PATCH] drm/amdgpu: add RAS fatal error handler for NBIO v7.9

2023-08-06 Thread Zhang, Hawking
[AMD Official Use Only - General] +static int nbio_v7_9_set_ras_err_event_athub_irq_state(struct amdgpu_device *adev, +static int nbio_v7_9_set_ras_controller_irq_state(struct amdgpu_device *adev, both functions could be left as dummy ones since by default it is vector #1 selected in bare-metal

[PATCH] drm/amdgpu/irq: Move irq resume to the beginning

2023-08-06 Thread Emily Deng
Need to move irq resume to the beginning of reset sriov, or if one interrupt occurs before irq resume, then the irq won't work anymore. Signed-off-by: Emily Deng --- drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd

[PATCH] drm/amdgpu: add RAS fatal error handler for NBIO v7.9

2023-08-06 Thread Tao Zhou
Register RAS fatal error interrupt and add handler. Signed-off-by: Tao Zhou --- drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c | 4 + drivers/gpu/drm/amd/amdgpu/nbio_v7_9.c | 219 drivers/gpu/drm/amd/amdgpu/nbio_v7_9.h | 1 + 3 files changed, 224 insertions(+) diff --git

[Patch v2 3/3] drm/mst: adjust the function drm_dp_remove_payload_part2()

2023-08-06 Thread Wayne Lin
[Why] Now in drm_dp_remove_payload_part2(), it utilizes the time slot number of the payload in old state to represent the one in the payload table at the moment. It would be better to clarify the idea by using the latest allocated time slot number for the port at the moment instead and which info

[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

[Patch v2 1/3] drm/mst: delete unnecessary case in drm_dp_add_payload_part2()

2023-08-06 Thread Wayne Lin
[Why] There is no need to consider payload->delete case since we won't call drm_dp_add_payload_part2() to create a payload when we're about to remove it. [How] Delete unnecessary case to simplify the code. Signed-off-by: Wayne Lin --- drivers/gpu/drm/display/drm_dp_mst_topology.c | 8 ++--

[Patch v2 0/3] Refactor and clean up codes of mst

2023-08-06 Thread Wayne Lin
This patch set is mainly trying to organize the mst code today a bit. Like to clarify and organize the sequence of mst payload allocation and removement.And also clean up some redundant codes today. The main refactor one is the patch "drm/mst: Refactor the flow for payload allocation/removement" w

RE: [PATCH 3/3] drm/mst: adjust the function drm_dp_remove_payload_part2()

2023-08-06 Thread Lin, Wayne
[AMD Official Use Only - General] > -Original Message- > From: Imre Deak > Sent: Friday, August 4, 2023 11:32 PM > To: Lin, Wayne > Cc: dri-de...@lists.freedesktop.org; amd-gfx@lists.freedesktop.org; > ly...@redhat.com; jani.nik...@intel.com; ville.syrj...@linux.intel.com; > Wentland, Ha