Commit 2f39835c authored by Nicholas Kazlauskas's avatar Nicholas Kazlauskas Committed by Alex Deucher
Browse files

drm/amd/display: Add shared DMCUB/driver firmware state cache window 



[Why]
Scratch registers are limited on the DMCUB and we have an expanding
list of state to track between driver and DMCUB.

[How]
Place shared state in cache window 6. The cache window size is aligned
to the size of the cache line on the DMCUB to make it easy to
invalidate.

The shared state is intended to be read only from driver side so
it's been marked as const.

The use of volatile is intentional. The memory for the shared firmware
state is memory mapped from the framebuffer memory. The DMCUB will
flush its cache after modifying the region. There's no way for x86
to known whether this data is stale or not so we want to intentionally
disable optimization to force the read at every access.

Signed-off-by: default avatarNicholas Kazlauskas <nicholas.kazlauskas@amd.com>
Reviewed-by: default avatarTony Cheng <Tony.Cheng@amd.com>
Acked-by: default avatarLeo Li <sunpeng.li@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent d4bbcecb

drivers/gpu/drm/amd/display/dmub/inc/dmub_fw_state.h

0 → 100644
+73 −0
Original line number Diff line number Diff line 
/*

 * Copyright 2019 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_FW_STATE_H_

#define _DMUB_FW_STATE_H_



#include "dmub_types.h"



#pragma pack(push, 1)



struct dmub_fw_state {

	/**

	 * @phy_initialized_during_fw_boot:

	 *

	 * Detects if VBIOS/VBL has ran before firmware boot.

	 * A value of 1 will usually mean S0i3 boot.

	 */

	uint8_t phy_initialized_during_fw_boot;



	/**

	 * @intialized_phy:

	 *

	 * Bit vector of initialized PHY.

	 */

	uint8_t initialized_phy;



	/**

	 * @enabled_phy:

	 *

	 * Bit vector of enabled PHY for DP alt mode switch tracking.

	 */

	uint8_t enabled_phy;



	/**

	 * @dmcu_fw_loaded:

	 *

	 * DMCU auto load state.

	 */

	uint8_t dmcu_fw_loaded;



	/**

	 * @psr_state:

	 *

	 * PSR state tracking.

	 */

	uint8_t psr_state;

};



#pragma pack(pop)



#endif /* _DMUB_FW_STATE_H_ */

drivers/gpu/drm/amd/display/dmub/inc/dmub_srv.h

+6 −2
Original line number Diff line number Diff line
@@ -67,6 +67,7 @@ 
#include "dmub_types.h"

#include "dmub_cmd.h"

#include "dmub_rb.h"

#include "dmub_fw_state.h"



#if defined(__cplusplus)

extern "C" {
@@ -102,7 +103,7 @@ enum dmub_window_id { 
	DMUB_WINDOW_3_VBIOS,

	DMUB_WINDOW_4_MAILBOX,

	DMUB_WINDOW_5_TRACEBUFF,

	DMUB_WINDOW_6_RESERVED,

	DMUB_WINDOW_6_FW_STATE,

	DMUB_WINDOW_7_RESERVED,

	DMUB_WINDOW_TOTAL,

};
@@ -241,7 +242,8 @@ struct dmub_srv_hw_funcs { 
			      const struct dmub_window *cw2,

			      const struct dmub_window *cw3,

			      const struct dmub_window *cw4,

				  const struct dmub_window *cw5);

			      const struct dmub_window *cw5,

			      const struct dmub_window *cw6);



	void (*setup_mailbox)(struct dmub_srv *dmub,

			      const struct dmub_region *inbox1);
@@ -296,11 +298,13 @@ struct dmub_srv_hw_params { 
 * @asic: dmub asic identifier

 * @user_ctx: user provided context for the dmub_srv

 * @is_virtual: false if hardware support only

 * @fw_state: dmub firmware state pointer

 */

struct dmub_srv {

	enum dmub_asic asic;

	void *user_ctx;

	bool is_virtual;

	volatile const struct dmub_fw_state *fw_state;



	/* private: internal use only */

	struct dmub_srv_base_funcs funcs;

drivers/gpu/drm/amd/display/dmub/src/dmub_dcn20.c

+9 −1
Original line number Diff line number Diff line
@@ -76,7 +76,8 @@ void dmub_dcn20_setup_windows(struct dmub_srv *dmub, 
			      const struct dmub_window *cw2,

			      const struct dmub_window *cw3,

			      const struct dmub_window *cw4,

				  const struct dmub_window *cw5)

			      const struct dmub_window *cw5,

			      const struct dmub_window *cw6)

{

	REG_WRITE(DMCUB_REGION3_CW2_OFFSET, cw2->offset.u.low_part);

	REG_WRITE(DMCUB_REGION3_CW2_OFFSET_HIGH, cw2->offset.u.high_part);
@@ -106,6 +107,13 @@ void dmub_dcn20_setup_windows(struct dmub_srv *dmub, 
	REG_SET_2(DMCUB_REGION3_CW5_TOP_ADDRESS, 0,

		  DMCUB_REGION3_CW5_TOP_ADDRESS, cw5->region.top,

		  DMCUB_REGION3_CW5_ENABLE, 1);



	REG_WRITE(DMCUB_REGION3_CW6_OFFSET, cw6->offset.u.low_part);

	REG_WRITE(DMCUB_REGION3_CW6_OFFSET_HIGH, cw6->offset.u.high_part);

	REG_WRITE(DMCUB_REGION3_CW6_BASE_ADDRESS, cw6->region.base);

	REG_SET_2(DMCUB_REGION3_CW6_TOP_ADDRESS, 0,

		  DMCUB_REGION3_CW6_TOP_ADDRESS, cw6->region.top,

		  DMCUB_REGION3_CW6_ENABLE, 1);

}



void dmub_dcn20_setup_mailbox(struct dmub_srv *dmub,

drivers/gpu/drm/amd/display/dmub/src/dmub_dcn20.h

+2 −1
Original line number Diff line number Diff line
@@ -46,7 +46,8 @@ void dmub_dcn20_setup_windows(struct dmub_srv *dmub, 
			      const struct dmub_window *cw2,

			      const struct dmub_window *cw3,

			      const struct dmub_window *cw4,

				  const struct dmub_window *cw5);

			      const struct dmub_window *cw5,

			      const struct dmub_window *cw6);



void dmub_dcn20_setup_mailbox(struct dmub_srv *dmub,

			      const struct dmub_region *inbox1);

drivers/gpu/drm/amd/display/dmub/src/dmub_dcn21.c

+11 −1
Original line number Diff line number Diff line
@@ -78,7 +78,8 @@ void dmub_dcn21_setup_windows(struct dmub_srv *dmub, 
			      const struct dmub_window *cw2,

			      const struct dmub_window *cw3,

			      const struct dmub_window *cw4,

				  const struct dmub_window *cw5)

			      const struct dmub_window *cw5,

			      const struct dmub_window *cw6)

{

	union dmub_addr offset;

	uint64_t fb_base = dmub->fb_base, fb_offset = dmub->fb_offset;
@@ -118,6 +119,15 @@ void dmub_dcn21_setup_windows(struct dmub_srv *dmub, 
	REG_SET_2(DMCUB_REGION3_CW5_TOP_ADDRESS, 0,

		  DMCUB_REGION3_CW5_TOP_ADDRESS, cw5->region.top,

		  DMCUB_REGION3_CW5_ENABLE, 1);



	dmub_dcn21_translate_addr(&cw6->offset, fb_base, fb_offset, &offset);



	REG_WRITE(DMCUB_REGION3_CW6_OFFSET, offset.u.low_part);

	REG_WRITE(DMCUB_REGION3_CW6_OFFSET_HIGH, offset.u.high_part);

	REG_WRITE(DMCUB_REGION3_CW6_BASE_ADDRESS, cw6->region.base);

	REG_SET_2(DMCUB_REGION3_CW6_TOP_ADDRESS, 0,

		  DMCUB_REGION3_CW6_TOP_ADDRESS, cw6->region.top,

		  DMCUB_REGION3_CW6_ENABLE, 1);

}



bool dmub_dcn21_is_auto_load_done(struct dmub_srv *dmub)

CRA Git | Maintained and supported by SUSTech CRA and CCSE