[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <502899ee-07a7-cebb-b730-7c5888de2ce9@intel.com>
Date: Thu, 1 Sep 2022 13:54:30 -0700
From: "Ceraolo Spurio, Daniele" <daniele.ceraolospurio@...el.com>
To: Tomas Winkler <tomas.winkler@...el.com>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
David Airlie <airlied@...ux.ie>,
Daniel Vetter <daniel@...ll.ch>
CC: <intel-gfx@...ts.freedesktop.org>,
Alexander Usyskin <alexander.usyskin@...el.com>,
<linux-kernel@...r.kernel.org>,
Rodrigo Vivi <rodrigo.vivi@...el.com>,
Vitaly Lubart <vitaly.lubart@...el.com>
Subject: Re: [Intel-gfx] [PATCH v7 09/15] mei: bus: export common mkhi
definitions into a separate header
On 8/6/2022 5:26 AM, Tomas Winkler wrote:
> From: Vitaly Lubart <vitaly.lubart@...el.com>
>
> Exported common mkhi definitions from bus-fixup.c into a separate
> header file mkhi.h for other driver usage.
>
> Signed-off-by: Vitaly Lubart <vitaly.lubart@...el.com>
> Signed-off-by: Tomas Winkler <tomas.winkler@...el.com>
> Signed-off-by: Alexander Usyskin <alexander.usyskin@...el.com>
> ---
> drivers/misc/mei/bus-fixup.c | 31 +------------------------
> drivers/misc/mei/mkhi.h | 45 ++++++++++++++++++++++++++++++++++++
> 2 files changed, 46 insertions(+), 30 deletions(-)
> create mode 100644 drivers/misc/mei/mkhi.h
>
> diff --git a/drivers/misc/mei/bus-fixup.c b/drivers/misc/mei/bus-fixup.c
> index 24e91a9ea558..811c94ebf250 100644
> --- a/drivers/misc/mei/bus-fixup.c
> +++ b/drivers/misc/mei/bus-fixup.c
> @@ -15,6 +15,7 @@
>
> #include "mei_dev.h"
> #include "client.h"
> +#include "mkhi.h"
>
> #define MEI_UUID_NFC_INFO UUID_LE(0xd2de1625, 0x382d, 0x417d, \
> 0x48, 0xa4, 0xef, 0xab, 0xba, 0x8a, 0x12, 0x06)
> @@ -89,20 +90,6 @@ struct mei_os_ver {
> u8 reserved2;
> } __packed;
>
> -#define MKHI_FEATURE_PTT 0x10
> -
> -struct mkhi_rule_id {
> - __le16 rule_type;
> - u8 feature_id;
> - u8 reserved;
> -} __packed;
> -
> -struct mkhi_fwcaps {
> - struct mkhi_rule_id id;
> - u8 len;
> - u8 data[];
> -} __packed;
> -
> struct mkhi_fw_ver_block {
> u16 minor;
> u8 major;
> @@ -115,22 +102,6 @@ struct mkhi_fw_ver {
> struct mkhi_fw_ver_block ver[MEI_MAX_FW_VER_BLOCKS];
> } __packed;
>
> -#define MKHI_FWCAPS_GROUP_ID 0x3
> -#define MKHI_FWCAPS_SET_OS_VER_APP_RULE_CMD 6
> -#define MKHI_GEN_GROUP_ID 0xFF
> -#define MKHI_GEN_GET_FW_VERSION_CMD 0x2
> -struct mkhi_msg_hdr {
> - u8 group_id;
> - u8 command;
> - u8 reserved;
> - u8 result;
> -} __packed;
> -
> -struct mkhi_msg {
> - struct mkhi_msg_hdr hdr;
> - u8 data[];
> -} __packed;
> -
> #define MKHI_OSVER_BUF_LEN (sizeof(struct mkhi_msg_hdr) + \
> sizeof(struct mkhi_fwcaps) + \
> sizeof(struct mei_os_ver))
> diff --git a/drivers/misc/mei/mkhi.h b/drivers/misc/mei/mkhi.h
> new file mode 100644
> index 000000000000..27a9b476904e
> --- /dev/null
> +++ b/drivers/misc/mei/mkhi.h
> @@ -0,0 +1,45 @@
> +/* SPDX-License-Identifier: GPL-2.0 */
> +/*
> + * Copyright (c) 2003-2020, Intel Corporation. All rights reserved.
2022 (can fix when merging)
> + * Intel Management Engine Interface (Intel MEI) Linux driver
> + */
> +
> +#ifndef _MEI_MKHI_H_
> +#define _MEI_MKHI_H_
> +
> +#include "mei_dev.h"
Any reason you need the full mei_dev.h and not just linux/types.h ?
> +
> +#define MKHI_FEATURE_PTT 0x10
> +
> +#define MKHI_FWCAPS_GROUP_ID 0x3
> +#define MKHI_FWCAPS_SET_OS_VER_APP_RULE_CMD 6
> +#define MKHI_GEN_GROUP_ID 0xFF
> +#define MKHI_GEN_GET_FW_VERSION_CMD 0x2
> +
> +#define MCHI_GROUP_ID 0xA
This is new and doesn't seem to be used in follow-up patches. Any reason
for adding it in?
Daniele
> +
> +struct mkhi_rule_id {
> + __le16 rule_type;
> + u8 feature_id;
> + u8 reserved;
> +} __packed;
> +
> +struct mkhi_fwcaps {
> + struct mkhi_rule_id id;
> + u8 len;
> + u8 data[];
> +} __packed;
> +
> +struct mkhi_msg_hdr {
> + u8 group_id;
> + u8 command;
> + u8 reserved;
> + u8 result;
> +} __packed;
> +
> +struct mkhi_msg {
> + struct mkhi_msg_hdr hdr;
> + u8 data[];
> +} __packed;
> +
> +#endif /* _MEI_MKHI_H_ */
Powered by blists - more mailing lists