[PATCH 03/25] drm/amd/display: Revert " drm/amd/display: Use channel_width = 2 for vram table 3.0"

Alex Deucher alexdeucher at gmail.com
Wed Dec 13 14:19:07 UTC 2023


On Wed, Dec 13, 2023 at 3:40 AM Wayne Lin <Wayne.Lin at amd.com> wrote:
>
> From: Alvin Lee <alvin.lee2 at amd.com>
>
> [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 at amd.com>
> Acked-by: Wayne Lin <wayne.lin at amd.com>
> Signed-off-by: Alvin Lee <alvin.lee2 at amd.com>

Acked-by: Alex Deucher <alexander.deucher at amd.com>

> ---
>  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
>


More information about the amd-gfx mailing list