RE: [PATCH] drm/amd/display: Align dcn314_smu logging with other DCNs

2022-11-15 Thread Rizvi, Saaem
ander.deuc...@amd.com>; Wentland, Harry<mailto:harry.wentl...@amd.com>; Rizvi, Saaem<mailto:syedsaaem.ri...@amd.com> Subject: RE: [PATCH] drm/amd/display: Align dcn314_smu logging with other DCNs [Public] > -Original Message- > From: Li, Roman > Sent: Tuesd

RE: [PATCH] drm/amd/display: Align dcn314_smu logging with other DCNs

2022-11-15 Thread Limonciello, Mario
; > > Deucher, Alexander ; Wentland, Harry > > ; Rizvi, Saaem > > Cc: Li, Roman > > Subject: RE: [PATCH] drm/amd/display: Align dcn314_smu logging with > other > > DCNs > > > > [Public] > > > > Conceptually makes sense to me, but please see

RE: [PATCH] drm/amd/display: Align dcn314_smu logging with other DCNs

2022-11-15 Thread Li, Roman
Roman > Subject: RE: [PATCH] drm/amd/display: Align dcn314_smu logging with other > DCNs > > [Public] > > Conceptually makes sense to me, but please see below comments: > > > -Original Message- > > From: roman...@amd.com > > Sent: Monday, November 1

RE: [PATCH] drm/amd/display: Align dcn314_smu logging with other DCNs

2022-11-14 Thread Limonciello, Mario
izvi, Saaem > Cc: Li, Roman > Subject: [PATCH] drm/amd/display: Align dcn314_smu logging with other > DCNs > > From: Roman Li > > [Why] > Assert on non-OK response from SMU is unnecessary. > It was replaced with respective log message on other asics > in the pa

[PATCH] drm/amd/display: Align dcn314_smu logging with other DCNs

2022-11-14 Thread Roman.Li
From: Roman Li [Why] Assert on non-OK response from SMU is unnecessary. It was replaced with respective log message on other asics in the past with commit: "drm/amd/display: Removing assert statements for Linux" [How] Remove asert and add dbg logging as on other DCNs. CC: Saaem Rizvi Signed-of