On Wed, Dec 13, 2023 at 3:40 AM Wayne Lin <Wayne.Lin@xxxxxxx> wrote:
>
> From: Alvin Lee <alvin.lee2@xxxxxxx>
>
> [Description]
> Revert commit 8c5660987ee1 ("drm/amd/display: Use channel_width = 2 for vram table 3.0")
> Because the issue is being fixed from VBIOS side.
>
> Reviewed-by: Samson Tam <samson.tam@xxxxxxx>
> Acked-by: Wayne Lin <wayne.lin@xxxxxxx>
> Signed-off-by: Alvin Lee <alvin.lee2@xxxxxxx>
Acked-by: Alex Deucher <alexander.deucher@xxxxxxx>
> ---
> drivers/gpu/drm/amd/display/dc/bios/bios_parser2.c | 8 +-------
> 1 file changed, 1 insertion(+), 7 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/display/dc/bios/bios_parser2.c b/drivers/gpu/drm/amd/display/dc/bios/bios_parser2.c
> index 875a064bb9a5..fcd65a2057ad 100644
> --- a/drivers/gpu/drm/amd/display/dc/bios/bios_parser2.c
> +++ b/drivers/gpu/drm/amd/display/dc/bios/bios_parser2.c
> @@ -2386,13 +2386,7 @@ static enum bp_result get_vram_info_v30(
> return BP_RESULT_BADBIOSTABLE;
>
> info->num_chans = info_v30->channel_num;
> - /* As suggested by VBIOS we should always use
> - * dram_channel_width_bytes = 2 when using VRAM
> - * table version 3.0. This is because the channel_width
> - * param in the VRAM info table is changed in 7000 series and
> - * no longer represents the memory channel width.
> - */
> - info->dram_channel_width_bytes = 2;
> + info->dram_channel_width_bytes = (1 << info_v30->channel_width) / 8;
>
> return result;
> }
> --
> 2.37.3
>
