[PATCH 4/6] drm/amd/display: Add DMUB support for DCN316

Harry Wentland harry.wentland at amd.com
Wed Feb 16 19:46:52 UTC 2022


On 2022-02-15 16:44, Alex Deucher wrote:
> From: Leo Li <sunpeng.li at amd.com>
> 
> Initialize DMUB for DCN316. Use same funcs as DCN31 for
> DCN316.
> 
> Acked-by: Prike Liang <Prike.Liang at amd.com>
> Signed-off-by: Leo Li <sunpeng.li at amd.com>
> Signed-off-by: Alex Deucher <alexander.deucher at amd.com>

Reviewed-by: Harry Wentland <harry.wentland at amd.com>

Harry

> ---
>  .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c |  2 +-
>  drivers/gpu/drm/amd/display/dmub/dmub_srv.h   |  1 +
>  drivers/gpu/drm/amd/display/dmub/src/Makefile |  2 +
>  .../drm/amd/display/dmub/src/dmub_dcn316.c    | 62 +++++++++++++++++++
>  .../drm/amd/display/dmub/src/dmub_dcn316.h    | 33 ++++++++++
>  .../gpu/drm/amd/display/dmub/src/dmub_srv.c   |  4 ++
>  6 files changed, 103 insertions(+), 1 deletion(-)
>  create mode 100644 drivers/gpu/drm/amd/display/dmub/src/dmub_dcn316.c
>  create mode 100644 drivers/gpu/drm/amd/display/dmub/src/dmub_dcn316.h
> 
> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> index a4254357bf5f..4b2cb9bdc3d2 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> @@ -1920,7 +1920,7 @@ static int dm_dmub_sw_init(struct amdgpu_device *adev)
>  		fw_name_dmub = FIRMWARE_YELLOW_CARP_DMUB;
>  		break;
>  	case IP_VERSION(3, 1, 6):
> -		dmub_asic = DMUB_ASIC_DCN31B;
> +		dmub_asic = DMUB_ASIC_DCN316;
>  		fw_name_dmub = FIRMWARE_DCN316_DMUB;
>  		break;
>  
> diff --git a/drivers/gpu/drm/amd/display/dmub/dmub_srv.h b/drivers/gpu/drm/amd/display/dmub/dmub_srv.h
> index 83855b8a32e9..821d3fb1f824 100644
> --- a/drivers/gpu/drm/amd/display/dmub/dmub_srv.h
> +++ b/drivers/gpu/drm/amd/display/dmub/dmub_srv.h
> @@ -98,6 +98,7 @@ enum dmub_asic {
>  	DMUB_ASIC_DCN303,
>  	DMUB_ASIC_DCN31,
>  	DMUB_ASIC_DCN31B,
> +	DMUB_ASIC_DCN316,
>  	DMUB_ASIC_MAX,
>  };
>  
> diff --git a/drivers/gpu/drm/amd/display/dmub/src/Makefile b/drivers/gpu/drm/amd/display/dmub/src/Makefile
> index 0495bcdd3463..21e818862072 100644
> --- a/drivers/gpu/drm/amd/display/dmub/src/Makefile
> +++ b/drivers/gpu/drm/amd/display/dmub/src/Makefile
> @@ -22,7 +22,9 @@
>  
>  DMUB = dmub_srv.o dmub_srv_stat.o dmub_reg.o dmub_dcn20.o dmub_dcn21.o
>  DMUB += dmub_dcn30.o dmub_dcn301.o dmub_dcn302.o dmub_dcn303.o
> +
>  DMUB += dmub_dcn31.o
> +DMUB += dmub_dcn316.o
>  
>  AMD_DAL_DMUB = $(addprefix $(AMDDALPATH)/dmub/src/,$(DMUB))
>  
> diff --git a/drivers/gpu/drm/amd/display/dmub/src/dmub_dcn316.c b/drivers/gpu/drm/amd/display/dmub/src/dmub_dcn316.c
> new file mode 100644
> index 000000000000..c43d1e3819e0
> --- /dev/null
> +++ b/drivers/gpu/drm/amd/display/dmub/src/dmub_dcn316.c
> @@ -0,0 +1,62 @@
> +/*
> + * Copyright 2021 Advanced Micro Devices, Inc.
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a
> + * copy of this software and associated documentation files (the "Software"),
> + * to deal in the Software without restriction, including without limitation
> + * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> + * and/or sell copies of the Software, and to permit persons to whom the
> + * Software is furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice shall be included in
> + * all copies or substantial portions of the Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
> + * THE COPYRIGHT HOLDER(S) OR AUTHOR(S) BE LIABLE FOR ANY CLAIM, DAMAGES OR
> + * OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
> + * ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
> + * OTHER DEALINGS IN THE SOFTWARE.
> + *
> + * Authors: AMD
> + *
> + */
> +
> +#include "../dmub_srv.h"
> +#include "dmub_reg.h"
> +#include "dmub_dcn316.h"
> +
> +#include "dcn/dcn_3_1_6_offset.h"
> +#include "dcn/dcn_3_1_6_sh_mask.h"
> +
> +#define DCN_BASE__INST0_SEG0                       0x00000012
> +#define DCN_BASE__INST0_SEG1                       0x000000C0
> +#define DCN_BASE__INST0_SEG2                       0x000034C0
> +#define DCN_BASE__INST0_SEG3                       0x00009000
> +#define DCN_BASE__INST0_SEG4                       0x02403C00
> +#define DCN_BASE__INST0_SEG5                       0
> +
> +#define BASE_INNER(seg) DCN_BASE__INST0_SEG##seg
> +#define CTX dmub
> +#define REGS dmub->regs_dcn31
> +#define REG_OFFSET_EXP(reg_name) (BASE(reg##reg_name##_BASE_IDX) + reg##reg_name)
> +
> +/* Registers. */
> +
> +const struct dmub_srv_dcn31_regs dmub_srv_dcn316_regs = {
> +#define DMUB_SR(reg) REG_OFFSET_EXP(reg),
> +	{
> +		DMUB_DCN31_REGS()
> +		DMCUB_INTERNAL_REGS()
> +	},
> +#undef DMUB_SR
> +
> +#define DMUB_SF(reg, field) FD_MASK(reg, field),
> +	{ DMUB_DCN31_FIELDS() },
> +#undef DMUB_SF
> +
> +#define DMUB_SF(reg, field) FD_SHIFT(reg, field),
> +	{ DMUB_DCN31_FIELDS() },
> +#undef DMUB_SF
> +};
> diff --git a/drivers/gpu/drm/amd/display/dmub/src/dmub_dcn316.h b/drivers/gpu/drm/amd/display/dmub/src/dmub_dcn316.h
> new file mode 100644
> index 000000000000..9e7d109129ea
> --- /dev/null
> +++ b/drivers/gpu/drm/amd/display/dmub/src/dmub_dcn316.h
> @@ -0,0 +1,33 @@
> +/*
> + * Copyright 2021 Advanced Micro Devices, Inc.
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a
> + * copy of this software and associated documentation files (the "Software"),
> + * to deal in the Software without restriction, including without limitation
> + * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> + * and/or sell copies of the Software, and to permit persons to whom the
> + * Software is furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice shall be included in
> + * all copies or substantial portions of the Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
> + * THE COPYRIGHT HOLDER(S) OR AUTHOR(S) BE LIABLE FOR ANY CLAIM, DAMAGES OR
> + * OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
> + * ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
> + * OTHER DEALINGS IN THE SOFTWARE.
> + *
> + * Authors: AMD
> + *
> + */
> +
> +#ifndef _DMUB_DCN316_H_
> +#define _DMUB_DCN316_H_
> +
> +#include "dmub_dcn31.h"
> +
> +extern const struct dmub_srv_dcn31_regs dmub_srv_dcn316_regs;
> +
> +#endif /* _DMUB_DCN316_H_ */
> diff --git a/drivers/gpu/drm/amd/display/dmub/src/dmub_srv.c b/drivers/gpu/drm/amd/display/dmub/src/dmub_srv.c
> index 9280f2abd973..db2802a14f95 100644
> --- a/drivers/gpu/drm/amd/display/dmub/src/dmub_srv.c
> +++ b/drivers/gpu/drm/amd/display/dmub/src/dmub_srv.c
> @@ -32,6 +32,7 @@
>  #include "dmub_dcn302.h"
>  #include "dmub_dcn303.h"
>  #include "dmub_dcn31.h"
> +#include "dmub_dcn316.h"
>  #include "os_types.h"
>  /*
>   * Note: the DMUB service is standalone. No additional headers should be
> @@ -220,7 +221,10 @@ static bool dmub_srv_hw_setup(struct dmub_srv *dmub, enum dmub_asic asic)
>  
>  	case DMUB_ASIC_DCN31:
>  	case DMUB_ASIC_DCN31B:
> +	case DMUB_ASIC_DCN316:
>  		dmub->regs_dcn31 = &dmub_srv_dcn31_regs;
> +		if (asic == DMUB_ASIC_DCN316)
> +			dmub->regs_dcn31 = &dmub_srv_dcn316_regs;
>  		funcs->reset = dmub_dcn31_reset;
>  		funcs->reset_release = dmub_dcn31_reset_release;
>  		funcs->backdoor_load = dmub_dcn31_backdoor_load;



More information about the amd-gfx mailing list