X86 platform drivers
 help / color / mirror / Atom feed
From: "Ilpo Järvinen" <ilpo.jarvinen@linux.intel.com>
To: Arnd Bergmann <arnd@kernel.org>
Cc: Hans de Goede <hdegoede@redhat.com>,
	Arnd Bergmann <arnd@arndb.de>,
	 Naveen Krishna Chatradhi <naveenkrishna.chatradhi@amd.com>,
	 Carlos Bilbao <carlos.bilbao.osdev@gmail.com>,
	 Thomas Gleixner <tglx@linutronix.de>,
	Ingo Molnar <mingo@redhat.com>,  Borislav Petkov <bp@alien8.de>,
	Dave Hansen <dave.hansen@linux.intel.com>,
	 x86@kernel.org, "H. Peter Anvin" <hpa@zytor.com>,
	 Suma Hegde <suma.hegde@amd.com>,
	platform-driver-x86@vger.kernel.org,
	 LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] [v2] platform/x86/amd/hsmp: move hsmp_msg_desc_table[] to hsmp.c
Date: Thu, 31 Oct 2024 11:52:33 +0200 (EET)	[thread overview]
Message-ID: <582e3574-1823-0c1b-25a8-6a4a6bf73f63@linux.intel.com> (raw)
In-Reply-To: <20241030210019.2858358-1-arnd@kernel.org>

[-- Attachment #1: Type: text/plain, Size: 16968 bytes --]

On Wed, 30 Oct 2024, Arnd Bergmann wrote:

> From: Arnd Bergmann <arnd@arndb.de>
> 
> After the file got split, there are now W=1 warnings for users that
> include it without referencing hsmp_msg_desc_table:
> 
> In file included from arch/x86/include/asm/amd_hsmp.h:6,
>                  from drivers/platform/x86/amd/hsmp/plat.c:12:
> arch/x86/include/uapi/asm/amd_hsmp.h:91:35: error: 'hsmp_msg_desc_table' defined but not used [-Werror=unused-const-variable=]
>    91 | static const struct hsmp_msg_desc hsmp_msg_desc_table[] = {
>       |                                   ^~~~~~~~~~~~~~~~~~~
> 
> The array was never meant to be used by userspace code, but was
> left in the header as a reference for userspace programmers.
> 
> Move the contents of the array into the one file that actually needs
> it, and instead leave the URL of the new location in the uapi header
> in case anyone is looking for it.
> 
> Fixes: e47c018a0ee6 ("platform/x86/amd/hsmp: Move platform device specific code to plat.c")
> Suggested-by: Hans de Goede <hdegoede@redhat.com>
> Link: https://lore.kernel.org/lkml/046687d8-1e2d-435b-adcb-26897bfd29f7@redhat.com/
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>

I'll leave it into patchwork for a few days if somebody has some still 
unvoiced opinions on this.

--
 i.

> ---
>  arch/x86/include/uapi/asm/amd_hsmp.h | 255 ++-------------------------
>  drivers/platform/x86/amd/hsmp/hsmp.c | 245 +++++++++++++++++++++++++
>  2 files changed, 255 insertions(+), 245 deletions(-)
> 
> diff --git a/arch/x86/include/uapi/asm/amd_hsmp.h b/arch/x86/include/uapi/asm/amd_hsmp.h
> index e5d182c7373c..dfa6aa985ac4 100644
> --- a/arch/x86/include/uapi/asm/amd_hsmp.h
> +++ b/arch/x86/include/uapi/asm/amd_hsmp.h
> @@ -1,5 +1,15 @@
>  /* SPDX-License-Identifier: GPL-2.0 WITH Linux-syscall-note */
>  
> +/*
> + * See hsmp_msg_desc_table[] in:
> + * https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/platform/x86/amd/hsmp.c
> + *
> + * for some information on number of input- and output arguments
> + * for the various functions.
> + *
> + * Please find the supported list of messages and message definition
> + * in the HSMP chapter of respective family/model PPR.
> + */
>  #ifndef _UAPI_ASM_X86_AMD_HSMP_H_
>  #define _UAPI_ASM_X86_AMD_HSMP_H_
>  
> @@ -81,251 +91,6 @@ struct hsmp_msg_desc {
>  	enum hsmp_msg_type type;
>  };
>  
> -/*
> - * User may use these comments as reference, please find the
> - * supported list of messages and message definition in the
> - * HSMP chapter of respective family/model PPR.
> - *
> - * Not supported messages would return -ENOMSG.
> - */
> -static const struct hsmp_msg_desc hsmp_msg_desc_table[] = {
> -	/* RESERVED */
> -	{0, 0, HSMP_RSVD},
> -
> -	/*
> -	 * HSMP_TEST, num_args = 1, response_sz = 1
> -	 * input:  args[0] = xx
> -	 * output: args[0] = xx + 1
> -	 */
> -	{1, 1, HSMP_GET},
> -
> -	/*
> -	 * HSMP_GET_SMU_VER, num_args = 0, response_sz = 1
> -	 * output: args[0] = smu fw ver
> -	 */
> -	{0, 1, HSMP_GET},
> -
> -	/*
> -	 * HSMP_GET_PROTO_VER, num_args = 0, response_sz = 1
> -	 * output: args[0] = proto version
> -	 */
> -	{0, 1, HSMP_GET},
> -
> -	/*
> -	 * HSMP_GET_SOCKET_POWER, num_args = 0, response_sz = 1
> -	 * output: args[0] = socket power in mWatts
> -	 */
> -	{0, 1, HSMP_GET},
> -
> -	/*
> -	 * HSMP_SET_SOCKET_POWER_LIMIT, num_args = 1, response_sz = 0
> -	 * input: args[0] = power limit value in mWatts
> -	 */
> -	{1, 0, HSMP_SET},
> -
> -	/*
> -	 * HSMP_GET_SOCKET_POWER_LIMIT, num_args = 0, response_sz = 1
> -	 * output: args[0] = socket power limit value in mWatts
> -	 */
> -	{0, 1, HSMP_GET},
> -
> -	/*
> -	 * HSMP_GET_SOCKET_POWER_LIMIT_MAX, num_args = 0, response_sz = 1
> -	 * output: args[0] = maximuam socket power limit in mWatts
> -	 */
> -	{0, 1, HSMP_GET},
> -
> -	/*
> -	 * HSMP_SET_BOOST_LIMIT, num_args = 1, response_sz = 0
> -	 * input: args[0] = apic id[31:16] + boost limit value in MHz[15:0]
> -	 */
> -	{1, 0, HSMP_SET},
> -
> -	/*
> -	 * HSMP_SET_BOOST_LIMIT_SOCKET, num_args = 1, response_sz = 0
> -	 * input: args[0] = boost limit value in MHz
> -	 */
> -	{1, 0, HSMP_SET},
> -
> -	/*
> -	 * HSMP_GET_BOOST_LIMIT, num_args = 1, response_sz = 1
> -	 * input: args[0] = apic id
> -	 * output: args[0] = boost limit value in MHz
> -	 */
> -	{1, 1, HSMP_GET},
> -
> -	/*
> -	 * HSMP_GET_PROC_HOT, num_args = 0, response_sz = 1
> -	 * output: args[0] = proc hot status
> -	 */
> -	{0, 1, HSMP_GET},
> -
> -	/*
> -	 * HSMP_SET_XGMI_LINK_WIDTH, num_args = 1, response_sz = 0
> -	 * input: args[0] = min link width[15:8] + max link width[7:0]
> -	 */
> -	{1, 0, HSMP_SET},
> -
> -	/*
> -	 * HSMP_SET_DF_PSTATE, num_args = 1, response_sz = 0
> -	 * input: args[0] = df pstate[7:0]
> -	 */
> -	{1, 0, HSMP_SET},
> -
> -	/* HSMP_SET_AUTO_DF_PSTATE, num_args = 0, response_sz = 0 */
> -	{0, 0, HSMP_SET},
> -
> -	/*
> -	 * HSMP_GET_FCLK_MCLK, num_args = 0, response_sz = 2
> -	 * output: args[0] = fclk in MHz, args[1] = mclk in MHz
> -	 */
> -	{0, 2, HSMP_GET},
> -
> -	/*
> -	 * HSMP_GET_CCLK_THROTTLE_LIMIT, num_args = 0, response_sz = 1
> -	 * output: args[0] = core clock in MHz
> -	 */
> -	{0, 1, HSMP_GET},
> -
> -	/*
> -	 * HSMP_GET_C0_PERCENT, num_args = 0, response_sz = 1
> -	 * output: args[0] = average c0 residency
> -	 */
> -	{0, 1, HSMP_GET},
> -
> -	/*
> -	 * HSMP_SET_NBIO_DPM_LEVEL, num_args = 1, response_sz = 0
> -	 * input: args[0] = nbioid[23:16] + max dpm level[15:8] + min dpm level[7:0]
> -	 */
> -	{1, 0, HSMP_SET},
> -
> -	/*
> -	 * HSMP_GET_NBIO_DPM_LEVEL, num_args = 1, response_sz = 1
> -	 * input: args[0] = nbioid[23:16]
> -	 * output: args[0] = max dpm level[15:8] + min dpm level[7:0]
> -	 */
> -	{1, 1, HSMP_GET},
> -
> -	/*
> -	 * HSMP_GET_DDR_BANDWIDTH, num_args = 0, response_sz = 1
> -	 * output: args[0] = max bw in Gbps[31:20] + utilised bw in Gbps[19:8] +
> -	 * bw in percentage[7:0]
> -	 */
> -	{0, 1, HSMP_GET},
> -
> -	/*
> -	 * HSMP_GET_TEMP_MONITOR, num_args = 0, response_sz = 1
> -	 * output: args[0] = temperature in degree celsius. [15:8] integer part +
> -	 * [7:5] fractional part
> -	 */
> -	{0, 1, HSMP_GET},
> -
> -	/*
> -	 * HSMP_GET_DIMM_TEMP_RANGE, num_args = 1, response_sz = 1
> -	 * input: args[0] = DIMM address[7:0]
> -	 * output: args[0] = refresh rate[3] + temperature range[2:0]
> -	 */
> -	{1, 1, HSMP_GET},
> -
> -	/*
> -	 * HSMP_GET_DIMM_POWER, num_args = 1, response_sz = 1
> -	 * input: args[0] = DIMM address[7:0]
> -	 * output: args[0] = DIMM power in mW[31:17] + update rate in ms[16:8] +
> -	 * DIMM address[7:0]
> -	 */
> -	{1, 1, HSMP_GET},
> -
> -	/*
> -	 * HSMP_GET_DIMM_THERMAL, num_args = 1, response_sz = 1
> -	 * input: args[0] = DIMM address[7:0]
> -	 * output: args[0] = temperature in degree celsius[31:21] + update rate in ms[16:8] +
> -	 * DIMM address[7:0]
> -	 */
> -	{1, 1, HSMP_GET},
> -
> -	/*
> -	 * HSMP_GET_SOCKET_FREQ_LIMIT, num_args = 0, response_sz = 1
> -	 * output: args[0] = frequency in MHz[31:16] + frequency source[15:0]
> -	 */
> -	{0, 1, HSMP_GET},
> -
> -	/*
> -	 * HSMP_GET_CCLK_CORE_LIMIT, num_args = 1, response_sz = 1
> -	 * input: args[0] = apic id [31:0]
> -	 * output: args[0] = frequency in MHz[31:0]
> -	 */
> -	{1, 1, HSMP_GET},
> -
> -	/*
> -	 * HSMP_GET_RAILS_SVI, num_args = 0, response_sz = 1
> -	 * output: args[0] = power in mW[31:0]
> -	 */
> -	{0, 1, HSMP_GET},
> -
> -	/*
> -	 * HSMP_GET_SOCKET_FMAX_FMIN, num_args = 0, response_sz = 1
> -	 * output: args[0] = fmax in MHz[31:16] + fmin in MHz[15:0]
> -	 */
> -	{0, 1, HSMP_GET},
> -
> -	/*
> -	 * HSMP_GET_IOLINK_BANDWITH, num_args = 1, response_sz = 1
> -	 * input: args[0] = link id[15:8] + bw type[2:0]
> -	 * output: args[0] = io bandwidth in Mbps[31:0]
> -	 */
> -	{1, 1, HSMP_GET},
> -
> -	/*
> -	 * HSMP_GET_XGMI_BANDWITH, num_args = 1, response_sz = 1
> -	 * input: args[0] = link id[15:8] + bw type[2:0]
> -	 * output: args[0] = xgmi bandwidth in Mbps[31:0]
> -	 */
> -	{1, 1, HSMP_GET},
> -
> -	/*
> -	 * HSMP_SET_GMI3_WIDTH, num_args = 1, response_sz = 0
> -	 * input: args[0] = min link width[15:8] + max link width[7:0]
> -	 */
> -	{1, 0, HSMP_SET},
> -
> -	/*
> -	 * HSMP_SET_PCI_RATE, num_args = 1, response_sz = 1
> -	 * input: args[0] = link rate control value
> -	 * output: args[0] = previous link rate control value
> -	 */
> -	{1, 1, HSMP_SET},
> -
> -	/*
> -	 * HSMP_SET_POWER_MODE, num_args = 1, response_sz = 0
> -	 * input: args[0] = power efficiency mode[2:0]
> -	 */
> -	{1, 0, HSMP_SET},
> -
> -	/*
> -	 * HSMP_SET_PSTATE_MAX_MIN, num_args = 1, response_sz = 0
> -	 * input: args[0] = min df pstate[15:8] + max df pstate[7:0]
> -	 */
> -	{1, 0, HSMP_SET},
> -
> -	/*
> -	 * HSMP_GET_METRIC_TABLE_VER, num_args = 0, response_sz = 1
> -	 * output: args[0] = metrics table version
> -	 */
> -	{0, 1, HSMP_GET},
> -
> -	/*
> -	 * HSMP_GET_METRIC_TABLE, num_args = 0, response_sz = 0
> -	 */
> -	{0, 0, HSMP_GET},
> -
> -	/*
> -	 * HSMP_GET_METRIC_TABLE_DRAM_ADDR, num_args = 0, response_sz = 2
> -	 * output: args[0] = lower 32 bits of the address
> -	 * output: args[1] = upper 32 bits of the address
> -	 */
> -	{0, 2, HSMP_GET},
> -};
> -
>  /* Metrics table (supported only with proto version 6) */
>  struct hsmp_metric_table {
>  	__u32 accumulation_counter;
> diff --git a/drivers/platform/x86/amd/hsmp/hsmp.c b/drivers/platform/x86/amd/hsmp/hsmp.c
> index 82d8ba2e1204..5d21bc8b2fd7 100644
> --- a/drivers/platform/x86/amd/hsmp/hsmp.c
> +++ b/drivers/platform/x86/amd/hsmp/hsmp.c
> @@ -37,6 +37,251 @@
>  
>  static struct hsmp_plat_device hsmp_pdev;
>  
> +/*
> + * User may use these comments as reference, please find the
> + * supported list of messages and message definition in the
> + * HSMP chapter of respective family/model PPR.
> + *
> + * Not supported messages would return -ENOMSG.
> + */
> +static const struct hsmp_msg_desc hsmp_msg_desc_table[] = {
> +	/* RESERVED */
> +	{0, 0, HSMP_RSVD},
> +
> +	/*
> +	 * HSMP_TEST, num_args = 1, response_sz = 1
> +	 * input:  args[0] = xx
> +	 * output: args[0] = xx + 1
> +	 */
> +	{1, 1, HSMP_GET},
> +
> +	/*
> +	 * HSMP_GET_SMU_VER, num_args = 0, response_sz = 1
> +	 * output: args[0] = smu fw ver
> +	 */
> +	{0, 1, HSMP_GET},
> +
> +	/*
> +	 * HSMP_GET_PROTO_VER, num_args = 0, response_sz = 1
> +	 * output: args[0] = proto version
> +	 */
> +	{0, 1, HSMP_GET},
> +
> +	/*
> +	 * HSMP_GET_SOCKET_POWER, num_args = 0, response_sz = 1
> +	 * output: args[0] = socket power in mWatts
> +	 */
> +	{0, 1, HSMP_GET},
> +
> +	/*
> +	 * HSMP_SET_SOCKET_POWER_LIMIT, num_args = 1, response_sz = 0
> +	 * input: args[0] = power limit value in mWatts
> +	 */
> +	{1, 0, HSMP_SET},
> +
> +	/*
> +	 * HSMP_GET_SOCKET_POWER_LIMIT, num_args = 0, response_sz = 1
> +	 * output: args[0] = socket power limit value in mWatts
> +	 */
> +	{0, 1, HSMP_GET},
> +
> +	/*
> +	 * HSMP_GET_SOCKET_POWER_LIMIT_MAX, num_args = 0, response_sz = 1
> +	 * output: args[0] = maximuam socket power limit in mWatts
> +	 */
> +	{0, 1, HSMP_GET},
> +
> +	/*
> +	 * HSMP_SET_BOOST_LIMIT, num_args = 1, response_sz = 0
> +	 * input: args[0] = apic id[31:16] + boost limit value in MHz[15:0]
> +	 */
> +	{1, 0, HSMP_SET},
> +
> +	/*
> +	 * HSMP_SET_BOOST_LIMIT_SOCKET, num_args = 1, response_sz = 0
> +	 * input: args[0] = boost limit value in MHz
> +	 */
> +	{1, 0, HSMP_SET},
> +
> +	/*
> +	 * HSMP_GET_BOOST_LIMIT, num_args = 1, response_sz = 1
> +	 * input: args[0] = apic id
> +	 * output: args[0] = boost limit value in MHz
> +	 */
> +	{1, 1, HSMP_GET},
> +
> +	/*
> +	 * HSMP_GET_PROC_HOT, num_args = 0, response_sz = 1
> +	 * output: args[0] = proc hot status
> +	 */
> +	{0, 1, HSMP_GET},
> +
> +	/*
> +	 * HSMP_SET_XGMI_LINK_WIDTH, num_args = 1, response_sz = 0
> +	 * input: args[0] = min link width[15:8] + max link width[7:0]
> +	 */
> +	{1, 0, HSMP_SET},
> +
> +	/*
> +	 * HSMP_SET_DF_PSTATE, num_args = 1, response_sz = 0
> +	 * input: args[0] = df pstate[7:0]
> +	 */
> +	{1, 0, HSMP_SET},
> +
> +	/* HSMP_SET_AUTO_DF_PSTATE, num_args = 0, response_sz = 0 */
> +	{0, 0, HSMP_SET},
> +
> +	/*
> +	 * HSMP_GET_FCLK_MCLK, num_args = 0, response_sz = 2
> +	 * output: args[0] = fclk in MHz, args[1] = mclk in MHz
> +	 */
> +	{0, 2, HSMP_GET},
> +
> +	/*
> +	 * HSMP_GET_CCLK_THROTTLE_LIMIT, num_args = 0, response_sz = 1
> +	 * output: args[0] = core clock in MHz
> +	 */
> +	{0, 1, HSMP_GET},
> +
> +	/*
> +	 * HSMP_GET_C0_PERCENT, num_args = 0, response_sz = 1
> +	 * output: args[0] = average c0 residency
> +	 */
> +	{0, 1, HSMP_GET},
> +
> +	/*
> +	 * HSMP_SET_NBIO_DPM_LEVEL, num_args = 1, response_sz = 0
> +	 * input: args[0] = nbioid[23:16] + max dpm level[15:8] + min dpm level[7:0]
> +	 */
> +	{1, 0, HSMP_SET},
> +
> +	/*
> +	 * HSMP_GET_NBIO_DPM_LEVEL, num_args = 1, response_sz = 1
> +	 * input: args[0] = nbioid[23:16]
> +	 * output: args[0] = max dpm level[15:8] + min dpm level[7:0]
> +	 */
> +	{1, 1, HSMP_GET},
> +
> +	/*
> +	 * HSMP_GET_DDR_BANDWIDTH, num_args = 0, response_sz = 1
> +	 * output: args[0] = max bw in Gbps[31:20] + utilised bw in Gbps[19:8] +
> +	 * bw in percentage[7:0]
> +	 */
> +	{0, 1, HSMP_GET},
> +
> +	/*
> +	 * HSMP_GET_TEMP_MONITOR, num_args = 0, response_sz = 1
> +	 * output: args[0] = temperature in degree celsius. [15:8] integer part +
> +	 * [7:5] fractional part
> +	 */
> +	{0, 1, HSMP_GET},
> +
> +	/*
> +	 * HSMP_GET_DIMM_TEMP_RANGE, num_args = 1, response_sz = 1
> +	 * input: args[0] = DIMM address[7:0]
> +	 * output: args[0] = refresh rate[3] + temperature range[2:0]
> +	 */
> +	{1, 1, HSMP_GET},
> +
> +	/*
> +	 * HSMP_GET_DIMM_POWER, num_args = 1, response_sz = 1
> +	 * input: args[0] = DIMM address[7:0]
> +	 * output: args[0] = DIMM power in mW[31:17] + update rate in ms[16:8] +
> +	 * DIMM address[7:0]
> +	 */
> +	{1, 1, HSMP_GET},
> +
> +	/*
> +	 * HSMP_GET_DIMM_THERMAL, num_args = 1, response_sz = 1
> +	 * input: args[0] = DIMM address[7:0]
> +	 * output: args[0] = temperature in degree celsius[31:21] + update rate in ms[16:8] +
> +	 * DIMM address[7:0]
> +	 */
> +	{1, 1, HSMP_GET},
> +
> +	/*
> +	 * HSMP_GET_SOCKET_FREQ_LIMIT, num_args = 0, response_sz = 1
> +	 * output: args[0] = frequency in MHz[31:16] + frequency source[15:0]
> +	 */
> +	{0, 1, HSMP_GET},
> +
> +	/*
> +	 * HSMP_GET_CCLK_CORE_LIMIT, num_args = 1, response_sz = 1
> +	 * input: args[0] = apic id [31:0]
> +	 * output: args[0] = frequency in MHz[31:0]
> +	 */
> +	{1, 1, HSMP_GET},
> +
> +	/*
> +	 * HSMP_GET_RAILS_SVI, num_args = 0, response_sz = 1
> +	 * output: args[0] = power in mW[31:0]
> +	 */
> +	{0, 1, HSMP_GET},
> +
> +	/*
> +	 * HSMP_GET_SOCKET_FMAX_FMIN, num_args = 0, response_sz = 1
> +	 * output: args[0] = fmax in MHz[31:16] + fmin in MHz[15:0]
> +	 */
> +	{0, 1, HSMP_GET},
> +
> +	/*
> +	 * HSMP_GET_IOLINK_BANDWITH, num_args = 1, response_sz = 1
> +	 * input: args[0] = link id[15:8] + bw type[2:0]
> +	 * output: args[0] = io bandwidth in Mbps[31:0]
> +	 */
> +	{1, 1, HSMP_GET},
> +
> +	/*
> +	 * HSMP_GET_XGMI_BANDWITH, num_args = 1, response_sz = 1
> +	 * input: args[0] = link id[15:8] + bw type[2:0]
> +	 * output: args[0] = xgmi bandwidth in Mbps[31:0]
> +	 */
> +	{1, 1, HSMP_GET},
> +
> +	/*
> +	 * HSMP_SET_GMI3_WIDTH, num_args = 1, response_sz = 0
> +	 * input: args[0] = min link width[15:8] + max link width[7:0]
> +	 */
> +	{1, 0, HSMP_SET},
> +
> +	/*
> +	 * HSMP_SET_PCI_RATE, num_args = 1, response_sz = 1
> +	 * input: args[0] = link rate control value
> +	 * output: args[0] = previous link rate control value
> +	 */
> +	{1, 1, HSMP_SET},
> +
> +	/*
> +	 * HSMP_SET_POWER_MODE, num_args = 1, response_sz = 0
> +	 * input: args[0] = power efficiency mode[2:0]
> +	 */
> +	{1, 0, HSMP_SET},
> +
> +	/*
> +	 * HSMP_SET_PSTATE_MAX_MIN, num_args = 1, response_sz = 0
> +	 * input: args[0] = min df pstate[15:8] + max df pstate[7:0]
> +	 */
> +	{1, 0, HSMP_SET},
> +
> +	/*
> +	 * HSMP_GET_METRIC_TABLE_VER, num_args = 0, response_sz = 1
> +	 * output: args[0] = metrics table version
> +	 */
> +	{0, 1, HSMP_GET},
> +
> +	/*
> +	 * HSMP_GET_METRIC_TABLE, num_args = 0, response_sz = 0
> +	 */
> +	{0, 0, HSMP_GET},
> +
> +	/*
> +	 * HSMP_GET_METRIC_TABLE_DRAM_ADDR, num_args = 0, response_sz = 2
> +	 * output: args[0] = lower 32 bits of the address
> +	 * output: args[1] = upper 32 bits of the address
> +	 */
> +	{0, 2, HSMP_GET},
> +};
> +
>  /*
>   * Send a message to the HSMP port via PCI-e config space registers
>   * or by writing to MMIO space.
> 

  reply	other threads:[~2024-10-31  9:52 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-10-30 21:00 [PATCH] [v2] platform/x86/amd/hsmp: move hsmp_msg_desc_table[] to hsmp.c Arnd Bergmann
2024-10-31  9:52 ` Ilpo Järvinen [this message]
2024-11-04 10:45   ` Suma Hegde
2024-11-12 10:19     ` Ilpo Järvinen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=582e3574-1823-0c1b-25a8-6a4a6bf73f63@linux.intel.com \
    --to=ilpo.jarvinen@linux.intel.com \
    --cc=arnd@arndb.de \
    --cc=arnd@kernel.org \
    --cc=bp@alien8.de \
    --cc=carlos.bilbao.osdev@gmail.com \
    --cc=dave.hansen@linux.intel.com \
    --cc=hdegoede@redhat.com \
    --cc=hpa@zytor.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@redhat.com \
    --cc=naveenkrishna.chatradhi@amd.com \
    --cc=platform-driver-x86@vger.kernel.org \
    --cc=suma.hegde@amd.com \
    --cc=tglx@linutronix.de \
    --cc=x86@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox