* [PATCH 1/3] ASoC: SDCA: add support for HIDE entity properties and HID descriptor/report
@ 2025-06-16 11:47 shumingf
2025-06-16 11:50 ` Shuming [范書銘]
0 siblings, 1 reply; 5+ messages in thread
From: shumingf @ 2025-06-16 11:47 UTC (permalink / raw)
To: broonie, lgirdwood
Cc: linux-sound, lars, flove, oder_chiou, jack.yu, derek.fang,
ckeepax, bard.liao, Shuming Fan
From: Shuming Fan <shumingf@realtek.com>
Add support for parsing the HIDE entity descriptor and HID descriptor/report
Signed-off-by: Shuming Fan <shumingf@realtek.com>
---
include/sound/sdca_function.h | 29 ++++++++++++
sound/soc/sdca/sdca_functions.c | 83 +++++++++++++++++++++++++++++++++
2 files changed, 112 insertions(+)
diff --git a/include/sound/sdca_function.h b/include/sound/sdca_function.h
index eaedb54a8322..856b0f40ce5e 100644
--- a/include/sound/sdca_function.h
+++ b/include/sound/sdca_function.h
@@ -11,6 +11,7 @@
#include <linux/bits.h>
#include <linux/types.h>
+#include <linux/hid.h>
struct device;
struct sdca_entity;
@@ -1040,6 +1041,32 @@ struct sdca_entity_ge {
int num_modes;
};
+/**
+ * struct sdca_entity_hide - information specific to HIDE Entities
+ * @hid: HID device structure
+ * @hidtx_ids: HIDTx Report ID
+ * @num_hidtx_ids: number of HIDTx Report ID
+ * @hidrx_ids: HIDRx Report ID
+ * @num_hidrx_ids: number of HIDRx Report ID
+ * @hide_reside_function_num: indicating which Audio Function Numbers within this Device
+ * @max_delay: the maximum time in microseconds allowed for the Device to change the ownership from Device to Host
+ * @af_number_list: which Audio Function Numbers within this Device are sending/receiving the messages in this HIDE
+ * @hid_desc: HID descriptor for the HIDE Entity
+ * @hid_report_desc: HID Report Descriptor for the HIDE Entity
+ */
+struct sdca_entity_hide {
+ struct hid_device *hid;
+ unsigned int *hidtx_ids;
+ int num_hidtx_ids;
+ unsigned int *hidrx_ids;
+ int num_hidrx_ids;
+ unsigned int hide_reside_function_num;
+ unsigned int max_delay;
+ unsigned int af_number_list[SDCA_MAX_FUNCTION_COUNT];
+ struct hid_descriptor hid_desc;
+ unsigned char *hid_report_desc;
+};
+
/**
* struct sdca_entity - information for one SDCA Entity
* @label: String such as "OT 12".
@@ -1055,6 +1082,7 @@ struct sdca_entity_ge {
* @cs: Clock Source specific Entity properties.
* @pde: Power Domain Entity specific Entity properties.
* @ge: Group Entity specific Entity properties.
+ * @hide: HIDE Entity specific Entity properties.
*/
struct sdca_entity {
const char *label;
@@ -1071,6 +1099,7 @@ struct sdca_entity {
struct sdca_entity_cs cs;
struct sdca_entity_pde pde;
struct sdca_entity_ge ge;
+ struct sdca_entity_hide hide;
};
};
diff --git a/sound/soc/sdca/sdca_functions.c b/sound/soc/sdca/sdca_functions.c
index 64ac26443890..4a89067dcf76 100644
--- a/sound/soc/sdca/sdca_functions.c
+++ b/sound/soc/sdca/sdca_functions.c
@@ -1220,6 +1220,86 @@ static int find_sdca_entity_ge(struct device *dev,
return -EINVAL;
}
+static int
+find_sdca_entity_hide(struct device *dev, struct fwnode_handle *function_node,
+ struct fwnode_handle *entity_node, struct sdca_entity *entity)
+{
+ struct sdca_entity_hide *hide = &entity->hide;
+ unsigned int delay, *af_list = hide->af_number_list;
+ int nval, ret;
+ unsigned char *report_desc = NULL;
+
+ ret = fwnode_property_read_u32(entity_node,
+ "mipi-sdca-RxUMP-ownership-transition-maxdelay", &delay);
+ if (!ret)
+ hide->max_delay = delay;
+
+ nval = fwnode_property_count_u32(entity_node, "mipi-sdca-HIDTx-supported-report-ids");
+ if (nval > 0) {
+ hide->num_hidtx_ids = nval;
+ hide->hidtx_ids = devm_kcalloc(dev, hide->num_hidtx_ids,
+ sizeof(*hide->hidtx_ids), GFP_KERNEL);
+ if (!hide->hidtx_ids)
+ return -ENOMEM;
+
+ ret = fwnode_property_read_u32_array(entity_node,
+ "mipi-sdca-HIDTx-supported-report-ids",
+ hide->hidtx_ids,
+ hide->num_hidtx_ids);
+ if (ret < 0)
+ return ret;
+ }
+
+ nval = fwnode_property_count_u32(entity_node, "mipi-sdca-HIDRx-supported-report-ids");
+ if (nval > 0) {
+ hide->num_hidrx_ids = nval;
+ hide->hidrx_ids = devm_kcalloc(dev, hide->num_hidrx_ids,
+ sizeof(*hide->hidrx_ids), GFP_KERNEL);
+ if (!hide->hidrx_ids)
+ return -ENOMEM;
+
+ ret = fwnode_property_read_u32_array(entity_node,
+ "mipi-sdca-HIDRx-supported-report-ids",
+ hide->hidrx_ids,
+ hide->num_hidrx_ids);
+ if (ret < 0)
+ return ret;
+ }
+
+ nval = fwnode_property_count_u32(entity_node, "mipi-sdca-hide-related-audio-function-list");
+ if (nval <= 0) {
+ dev_err(dev, "%pfwP: audio function numbers list missing: %d\n",
+ entity_node, nval);
+ return -EINVAL;
+ } else if (nval > SDCA_MAX_FUNCTION_COUNT) {
+ dev_err(dev, "%pfwP: maximum number of audio function exceeded\n", entity_node);
+ return -EINVAL;
+ }
+
+ hide->hide_reside_function_num = nval;
+ fwnode_property_read_u32_array(entity_node,
+ "mipi-sdca-hide-related-audio-function-list", af_list, nval);
+
+ nval = fwnode_property_count_u8(function_node, "mipi-sdca-hid-descriptor");
+ if (nval)
+ fwnode_property_read_u8_array(function_node, "mipi-sdca-hid-descriptor",
+ (u8 *)&hide->hid_desc, nval);
+
+ if (hide->hid_desc.bNumDescriptors) {
+ nval = fwnode_property_count_u8(function_node, "mipi-sdca-report-descriptor");
+ if (nval) {
+ report_desc = devm_kzalloc(dev, nval, GFP_KERNEL);
+ if (!report_desc)
+ return -ENOMEM;
+ hide->hid_report_desc = report_desc;
+ fwnode_property_read_u8_array(function_node, "mipi-sdca-report-descriptor",
+ report_desc, nval);
+ }
+ }
+
+ return 0;
+}
+
static int find_sdca_entity(struct device *dev,
struct fwnode_handle *function_node,
struct fwnode_handle *entity_node,
@@ -1261,6 +1341,9 @@ static int find_sdca_entity(struct device *dev,
case SDCA_ENTITY_TYPE_GE:
ret = find_sdca_entity_ge(dev, entity_node, entity);
break;
+ case SDCA_ENTITY_TYPE_HIDE:
+ ret = find_sdca_entity_hide(dev, function_node, entity_node, entity);
+ break;
default:
break;
}
--
2.34.1
^ permalink raw reply related [flat|nested] 5+ messages in thread* RE: [PATCH 1/3] ASoC: SDCA: add support for HIDE entity properties and HID descriptor/report
2025-06-16 11:47 [PATCH 1/3] ASoC: SDCA: add support for HIDE entity properties and HID descriptor/report shumingf
@ 2025-06-16 11:50 ` Shuming [范書銘]
0 siblings, 0 replies; 5+ messages in thread
From: Shuming [范書銘] @ 2025-06-16 11:50 UTC (permalink / raw)
To: broonie@kernel.org, lgirdwood@gmail.com
Cc: linux-sound@vger.kernel.org, lars@metafoo.de, Flove(HsinFu),
Oder Chiou, Jack Yu, Derek [方德義],
ckeepax@opensource.cirrus.com, bard.liao@intel.com
Ignore this one, sorry.
> Subject: [PATCH 1/3] ASoC: SDCA: add support for HIDE entity properties and
> HID descriptor/report
>
> From: Shuming Fan <shumingf@realtek.com>
>
> Add support for parsing the HIDE entity descriptor and HID descriptor/report
>
> Signed-off-by: Shuming Fan <shumingf@realtek.com>
> ---
> include/sound/sdca_function.h | 29 ++++++++++++
> sound/soc/sdca/sdca_functions.c | 83
> +++++++++++++++++++++++++++++++++
> 2 files changed, 112 insertions(+)
>
> diff --git a/include/sound/sdca_function.h b/include/sound/sdca_function.h
> index eaedb54a8322..856b0f40ce5e 100644
> --- a/include/sound/sdca_function.h
> +++ b/include/sound/sdca_function.h
> @@ -11,6 +11,7 @@
>
> #include <linux/bits.h>
> #include <linux/types.h>
> +#include <linux/hid.h>
>
> struct device;
> struct sdca_entity;
> @@ -1040,6 +1041,32 @@ struct sdca_entity_ge {
> int num_modes;
> };
>
> +/**
> + * struct sdca_entity_hide - information specific to HIDE Entities
> + * @hid: HID device structure
> + * @hidtx_ids: HIDTx Report ID
> + * @num_hidtx_ids: number of HIDTx Report ID
> + * @hidrx_ids: HIDRx Report ID
> + * @num_hidrx_ids: number of HIDRx Report ID
> + * @hide_reside_function_num: indicating which Audio Function Numbers
> +within this Device
> + * @max_delay: the maximum time in microseconds allowed for the Device
> +to change the ownership from Device to Host
> + * @af_number_list: which Audio Function Numbers within this Device are
> +sending/receiving the messages in this HIDE
> + * @hid_desc: HID descriptor for the HIDE Entity
> + * @hid_report_desc: HID Report Descriptor for the HIDE Entity */
> +struct sdca_entity_hide {
> + struct hid_device *hid;
> + unsigned int *hidtx_ids;
> + int num_hidtx_ids;
> + unsigned int *hidrx_ids;
> + int num_hidrx_ids;
> + unsigned int hide_reside_function_num;
> + unsigned int max_delay;
> + unsigned int af_number_list[SDCA_MAX_FUNCTION_COUNT];
> + struct hid_descriptor hid_desc;
> + unsigned char *hid_report_desc;
> +};
> +
> /**
> * struct sdca_entity - information for one SDCA Entity
> * @label: String such as "OT 12".
> @@ -1055,6 +1082,7 @@ struct sdca_entity_ge {
> * @cs: Clock Source specific Entity properties.
> * @pde: Power Domain Entity specific Entity properties.
> * @ge: Group Entity specific Entity properties.
> + * @hide: HIDE Entity specific Entity properties.
> */
> struct sdca_entity {
> const char *label;
> @@ -1071,6 +1099,7 @@ struct sdca_entity {
> struct sdca_entity_cs cs;
> struct sdca_entity_pde pde;
> struct sdca_entity_ge ge;
> + struct sdca_entity_hide hide;
> };
> };
>
> diff --git a/sound/soc/sdca/sdca_functions.c
> b/sound/soc/sdca/sdca_functions.c index 64ac26443890..4a89067dcf76
> 100644
> --- a/sound/soc/sdca/sdca_functions.c
> +++ b/sound/soc/sdca/sdca_functions.c
> @@ -1220,6 +1220,86 @@ static int find_sdca_entity_ge(struct device *dev,
> return -EINVAL;
> }
>
> +static int
> +find_sdca_entity_hide(struct device *dev, struct fwnode_handle
> *function_node,
> + struct fwnode_handle *entity_node, struct sdca_entity
> *entity)
> +{
> + struct sdca_entity_hide *hide = &entity->hide;
> + unsigned int delay, *af_list = hide->af_number_list;
> + int nval, ret;
> + unsigned char *report_desc = NULL;
> +
> + ret = fwnode_property_read_u32(entity_node,
> +
> "mipi-sdca-RxUMP-ownership-transition-maxdelay", &delay);
> + if (!ret)
> + hide->max_delay = delay;
> +
> + nval = fwnode_property_count_u32(entity_node,
> "mipi-sdca-HIDTx-supported-report-ids");
> + if (nval > 0) {
> + hide->num_hidtx_ids = nval;
> + hide->hidtx_ids = devm_kcalloc(dev, hide->num_hidtx_ids,
> + sizeof(*hide->hidtx_ids), GFP_KERNEL);
> + if (!hide->hidtx_ids)
> + return -ENOMEM;
> +
> + ret = fwnode_property_read_u32_array(entity_node,
> + "mipi-sdca-HIDTx-supported-report-ids",
> + hide->hidtx_ids,
> + hide->num_hidtx_ids);
> + if (ret < 0)
> + return ret;
> + }
> +
> + nval = fwnode_property_count_u32(entity_node,
> "mipi-sdca-HIDRx-supported-report-ids");
> + if (nval > 0) {
> + hide->num_hidrx_ids = nval;
> + hide->hidrx_ids = devm_kcalloc(dev, hide->num_hidrx_ids,
> + sizeof(*hide->hidrx_ids), GFP_KERNEL);
> + if (!hide->hidrx_ids)
> + return -ENOMEM;
> +
> + ret = fwnode_property_read_u32_array(entity_node,
> + "mipi-sdca-HIDRx-supported-report-ids",
> + hide->hidrx_ids,
> + hide->num_hidrx_ids);
> + if (ret < 0)
> + return ret;
> + }
> +
> + nval = fwnode_property_count_u32(entity_node,
> "mipi-sdca-hide-related-audio-function-list");
> + if (nval <= 0) {
> + dev_err(dev, "%pfwP: audio function numbers list missing: %d\n",
> + entity_node, nval);
> + return -EINVAL;
> + } else if (nval > SDCA_MAX_FUNCTION_COUNT) {
> + dev_err(dev, "%pfwP: maximum number of audio function
> exceeded\n", entity_node);
> + return -EINVAL;
> + }
> +
> + hide->hide_reside_function_num = nval;
> + fwnode_property_read_u32_array(entity_node,
> + "mipi-sdca-hide-related-audio-function-list",
> af_list,
> +nval);
> +
> + nval = fwnode_property_count_u8(function_node,
> "mipi-sdca-hid-descriptor");
> + if (nval)
> + fwnode_property_read_u8_array(function_node,
> "mipi-sdca-hid-descriptor",
> + (u8 *)&hide->hid_desc, nval);
> +
> + if (hide->hid_desc.bNumDescriptors) {
> + nval = fwnode_property_count_u8(function_node,
> "mipi-sdca-report-descriptor");
> + if (nval) {
> + report_desc = devm_kzalloc(dev, nval, GFP_KERNEL);
> + if (!report_desc)
> + return -ENOMEM;
> + hide->hid_report_desc = report_desc;
> + fwnode_property_read_u8_array(function_node,
> "mipi-sdca-report-descriptor",
> + report_desc, nval);
> + }
> + }
> +
> + return 0;
> +}
> +
> static int find_sdca_entity(struct device *dev,
> struct fwnode_handle *function_node,
> struct fwnode_handle *entity_node, @@ -1261,6 +1341,9
> @@ static int find_sdca_entity(struct device *dev,
> case SDCA_ENTITY_TYPE_GE:
> ret = find_sdca_entity_ge(dev, entity_node, entity);
> break;
> + case SDCA_ENTITY_TYPE_HIDE:
> + ret = find_sdca_entity_hide(dev, function_node, entity_node, entity);
> + break;
> default:
> break;
> }
> --
> 2.34.1
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 1/3] ASoC: SDCA: add support for HIDE entity properties and HID descriptor/report
@ 2025-06-11 10:25 shumingf
2025-06-11 10:54 ` Charles Keepax
2025-06-12 12:52 ` Charles Keepax
0 siblings, 2 replies; 5+ messages in thread
From: shumingf @ 2025-06-11 10:25 UTC (permalink / raw)
To: broonie, lgirdwood
Cc: linux-sound, lars, flove, oder_chiou, jack.yu, derek.fang,
ckeepax, bard.liao, Shuming Fan
From: Shuming Fan <shumingf@realtek.com>
Add support for parsing the HIDE entity descriptor and HID descriptor/report
Signed-off-by: Shuming Fan <shumingf@realtek.com>
---
include/sound/sdca_function.h | 29 ++++++++++++
sound/soc/sdca/sdca_functions.c | 83 +++++++++++++++++++++++++++++++++
2 files changed, 112 insertions(+)
diff --git a/include/sound/sdca_function.h b/include/sound/sdca_function.h
index eaedb54a8322..856b0f40ce5e 100644
--- a/include/sound/sdca_function.h
+++ b/include/sound/sdca_function.h
@@ -11,6 +11,7 @@
#include <linux/bits.h>
#include <linux/types.h>
+#include <linux/hid.h>
struct device;
struct sdca_entity;
@@ -1040,6 +1041,32 @@ struct sdca_entity_ge {
int num_modes;
};
+/**
+ * struct sdca_entity_hide - information specific to HIDE Entities
+ * @hid: HID device structure
+ * @hidtx_ids: HIDTx Report ID
+ * @num_hidtx_ids: number of HIDTx Report ID
+ * @hidrx_ids: HIDRx Report ID
+ * @num_hidrx_ids: number of HIDRx Report ID
+ * @hide_reside_function_num: indicating which Audio Function Numbers within this Device
+ * @max_delay: the maximum time in microseconds allowed for the Device to change the ownership from Device to Host
+ * @af_number_list: which Audio Function Numbers within this Device are sending/receiving the messages in this HIDE
+ * @hid_desc: HID descriptor for the HIDE Entity
+ * @hid_report_desc: HID Report Descriptor for the HIDE Entity
+ */
+struct sdca_entity_hide {
+ struct hid_device *hid;
+ unsigned int *hidtx_ids;
+ int num_hidtx_ids;
+ unsigned int *hidrx_ids;
+ int num_hidrx_ids;
+ unsigned int hide_reside_function_num;
+ unsigned int max_delay;
+ unsigned int af_number_list[SDCA_MAX_FUNCTION_COUNT];
+ struct hid_descriptor hid_desc;
+ unsigned char *hid_report_desc;
+};
+
/**
* struct sdca_entity - information for one SDCA Entity
* @label: String such as "OT 12".
@@ -1055,6 +1082,7 @@ struct sdca_entity_ge {
* @cs: Clock Source specific Entity properties.
* @pde: Power Domain Entity specific Entity properties.
* @ge: Group Entity specific Entity properties.
+ * @hide: HIDE Entity specific Entity properties.
*/
struct sdca_entity {
const char *label;
@@ -1071,6 +1099,7 @@ struct sdca_entity {
struct sdca_entity_cs cs;
struct sdca_entity_pde pde;
struct sdca_entity_ge ge;
+ struct sdca_entity_hide hide;
};
};
diff --git a/sound/soc/sdca/sdca_functions.c b/sound/soc/sdca/sdca_functions.c
index 64ac26443890..4a89067dcf76 100644
--- a/sound/soc/sdca/sdca_functions.c
+++ b/sound/soc/sdca/sdca_functions.c
@@ -1220,6 +1220,86 @@ static int find_sdca_entity_ge(struct device *dev,
return -EINVAL;
}
+static int
+find_sdca_entity_hide(struct device *dev, struct fwnode_handle *function_node,
+ struct fwnode_handle *entity_node, struct sdca_entity *entity)
+{
+ struct sdca_entity_hide *hide = &entity->hide;
+ unsigned int delay, *af_list = hide->af_number_list;
+ int nval, ret;
+ unsigned char *report_desc = NULL;
+
+ ret = fwnode_property_read_u32(entity_node,
+ "mipi-sdca-RxUMP-ownership-transition-maxdelay", &delay);
+ if (!ret)
+ hide->max_delay = delay;
+
+ nval = fwnode_property_count_u32(entity_node, "mipi-sdca-HIDTx-supported-report-ids");
+ if (nval > 0) {
+ hide->num_hidtx_ids = nval;
+ hide->hidtx_ids = devm_kcalloc(dev, hide->num_hidtx_ids,
+ sizeof(*hide->hidtx_ids), GFP_KERNEL);
+ if (!hide->hidtx_ids)
+ return -ENOMEM;
+
+ ret = fwnode_property_read_u32_array(entity_node,
+ "mipi-sdca-HIDTx-supported-report-ids",
+ hide->hidtx_ids,
+ hide->num_hidtx_ids);
+ if (ret < 0)
+ return ret;
+ }
+
+ nval = fwnode_property_count_u32(entity_node, "mipi-sdca-HIDRx-supported-report-ids");
+ if (nval > 0) {
+ hide->num_hidrx_ids = nval;
+ hide->hidrx_ids = devm_kcalloc(dev, hide->num_hidrx_ids,
+ sizeof(*hide->hidrx_ids), GFP_KERNEL);
+ if (!hide->hidrx_ids)
+ return -ENOMEM;
+
+ ret = fwnode_property_read_u32_array(entity_node,
+ "mipi-sdca-HIDRx-supported-report-ids",
+ hide->hidrx_ids,
+ hide->num_hidrx_ids);
+ if (ret < 0)
+ return ret;
+ }
+
+ nval = fwnode_property_count_u32(entity_node, "mipi-sdca-hide-related-audio-function-list");
+ if (nval <= 0) {
+ dev_err(dev, "%pfwP: audio function numbers list missing: %d\n",
+ entity_node, nval);
+ return -EINVAL;
+ } else if (nval > SDCA_MAX_FUNCTION_COUNT) {
+ dev_err(dev, "%pfwP: maximum number of audio function exceeded\n", entity_node);
+ return -EINVAL;
+ }
+
+ hide->hide_reside_function_num = nval;
+ fwnode_property_read_u32_array(entity_node,
+ "mipi-sdca-hide-related-audio-function-list", af_list, nval);
+
+ nval = fwnode_property_count_u8(function_node, "mipi-sdca-hid-descriptor");
+ if (nval)
+ fwnode_property_read_u8_array(function_node, "mipi-sdca-hid-descriptor",
+ (u8 *)&hide->hid_desc, nval);
+
+ if (hide->hid_desc.bNumDescriptors) {
+ nval = fwnode_property_count_u8(function_node, "mipi-sdca-report-descriptor");
+ if (nval) {
+ report_desc = devm_kzalloc(dev, nval, GFP_KERNEL);
+ if (!report_desc)
+ return -ENOMEM;
+ hide->hid_report_desc = report_desc;
+ fwnode_property_read_u8_array(function_node, "mipi-sdca-report-descriptor",
+ report_desc, nval);
+ }
+ }
+
+ return 0;
+}
+
static int find_sdca_entity(struct device *dev,
struct fwnode_handle *function_node,
struct fwnode_handle *entity_node,
@@ -1261,6 +1341,9 @@ static int find_sdca_entity(struct device *dev,
case SDCA_ENTITY_TYPE_GE:
ret = find_sdca_entity_ge(dev, entity_node, entity);
break;
+ case SDCA_ENTITY_TYPE_HIDE:
+ ret = find_sdca_entity_hide(dev, function_node, entity_node, entity);
+ break;
default:
break;
}
--
2.34.1
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH 1/3] ASoC: SDCA: add support for HIDE entity properties and HID descriptor/report
2025-06-11 10:25 shumingf
@ 2025-06-11 10:54 ` Charles Keepax
2025-06-12 12:52 ` Charles Keepax
1 sibling, 0 replies; 5+ messages in thread
From: Charles Keepax @ 2025-06-11 10:54 UTC (permalink / raw)
To: shumingf
Cc: broonie, lgirdwood, linux-sound, lars, flove, oder_chiou, jack.yu,
derek.fang, bard.liao, pierre-louis.bossart
On Wed, Jun 11, 2025 at 06:25:49PM +0800, shumingf@realtek.com wrote:
Adding Pierre.
> From: Shuming Fan <shumingf@realtek.com>
>
> Add support for parsing the HIDE entity descriptor and HID descriptor/report
>
> Signed-off-by: Shuming Fan <shumingf@realtek.com>
> ---
> include/sound/sdca_function.h | 29 ++++++++++++
> sound/soc/sdca/sdca_functions.c | 83 +++++++++++++++++++++++++++++++++
> 2 files changed, 112 insertions(+)
>
> diff --git a/include/sound/sdca_function.h b/include/sound/sdca_function.h
> index eaedb54a8322..856b0f40ce5e 100644
> --- a/include/sound/sdca_function.h
> +++ b/include/sound/sdca_function.h
> @@ -11,6 +11,7 @@
>
> #include <linux/bits.h>
> #include <linux/types.h>
> +#include <linux/hid.h>
>
> struct device;
> struct sdca_entity;
> @@ -1040,6 +1041,32 @@ struct sdca_entity_ge {
> int num_modes;
> };
>
> +/**
> + * struct sdca_entity_hide - information specific to HIDE Entities
> + * @hid: HID device structure
> + * @hidtx_ids: HIDTx Report ID
> + * @num_hidtx_ids: number of HIDTx Report ID
> + * @hidrx_ids: HIDRx Report ID
> + * @num_hidrx_ids: number of HIDRx Report ID
> + * @hide_reside_function_num: indicating which Audio Function Numbers within this Device
> + * @max_delay: the maximum time in microseconds allowed for the Device to change the ownership from Device to Host
> + * @af_number_list: which Audio Function Numbers within this Device are sending/receiving the messages in this HIDE
> + * @hid_desc: HID descriptor for the HIDE Entity
> + * @hid_report_desc: HID Report Descriptor for the HIDE Entity
> + */
> +struct sdca_entity_hide {
> + struct hid_device *hid;
> + unsigned int *hidtx_ids;
> + int num_hidtx_ids;
> + unsigned int *hidrx_ids;
> + int num_hidrx_ids;
> + unsigned int hide_reside_function_num;
> + unsigned int max_delay;
> + unsigned int af_number_list[SDCA_MAX_FUNCTION_COUNT];
> + struct hid_descriptor hid_desc;
> + unsigned char *hid_report_desc;
> +};
> +
> /**
> * struct sdca_entity - information for one SDCA Entity
> * @label: String such as "OT 12".
> @@ -1055,6 +1082,7 @@ struct sdca_entity_ge {
> * @cs: Clock Source specific Entity properties.
> * @pde: Power Domain Entity specific Entity properties.
> * @ge: Group Entity specific Entity properties.
> + * @hide: HIDE Entity specific Entity properties.
> */
> struct sdca_entity {
> const char *label;
> @@ -1071,6 +1099,7 @@ struct sdca_entity {
> struct sdca_entity_cs cs;
> struct sdca_entity_pde pde;
> struct sdca_entity_ge ge;
> + struct sdca_entity_hide hide;
> };
> };
>
> diff --git a/sound/soc/sdca/sdca_functions.c b/sound/soc/sdca/sdca_functions.c
> index 64ac26443890..4a89067dcf76 100644
> --- a/sound/soc/sdca/sdca_functions.c
> +++ b/sound/soc/sdca/sdca_functions.c
> @@ -1220,6 +1220,86 @@ static int find_sdca_entity_ge(struct device *dev,
> return -EINVAL;
> }
>
> +static int
> +find_sdca_entity_hide(struct device *dev, struct fwnode_handle *function_node,
> + struct fwnode_handle *entity_node, struct sdca_entity *entity)
> +{
> + struct sdca_entity_hide *hide = &entity->hide;
> + unsigned int delay, *af_list = hide->af_number_list;
> + int nval, ret;
> + unsigned char *report_desc = NULL;
> +
> + ret = fwnode_property_read_u32(entity_node,
> + "mipi-sdca-RxUMP-ownership-transition-maxdelay", &delay);
> + if (!ret)
> + hide->max_delay = delay;
> +
> + nval = fwnode_property_count_u32(entity_node, "mipi-sdca-HIDTx-supported-report-ids");
> + if (nval > 0) {
> + hide->num_hidtx_ids = nval;
> + hide->hidtx_ids = devm_kcalloc(dev, hide->num_hidtx_ids,
> + sizeof(*hide->hidtx_ids), GFP_KERNEL);
> + if (!hide->hidtx_ids)
> + return -ENOMEM;
> +
> + ret = fwnode_property_read_u32_array(entity_node,
> + "mipi-sdca-HIDTx-supported-report-ids",
> + hide->hidtx_ids,
> + hide->num_hidtx_ids);
> + if (ret < 0)
> + return ret;
> + }
> +
> + nval = fwnode_property_count_u32(entity_node, "mipi-sdca-HIDRx-supported-report-ids");
> + if (nval > 0) {
> + hide->num_hidrx_ids = nval;
> + hide->hidrx_ids = devm_kcalloc(dev, hide->num_hidrx_ids,
> + sizeof(*hide->hidrx_ids), GFP_KERNEL);
> + if (!hide->hidrx_ids)
> + return -ENOMEM;
> +
> + ret = fwnode_property_read_u32_array(entity_node,
> + "mipi-sdca-HIDRx-supported-report-ids",
> + hide->hidrx_ids,
> + hide->num_hidrx_ids);
> + if (ret < 0)
> + return ret;
> + }
> +
> + nval = fwnode_property_count_u32(entity_node, "mipi-sdca-hide-related-audio-function-list");
> + if (nval <= 0) {
> + dev_err(dev, "%pfwP: audio function numbers list missing: %d\n",
> + entity_node, nval);
> + return -EINVAL;
> + } else if (nval > SDCA_MAX_FUNCTION_COUNT) {
> + dev_err(dev, "%pfwP: maximum number of audio function exceeded\n", entity_node);
> + return -EINVAL;
> + }
> +
> + hide->hide_reside_function_num = nval;
> + fwnode_property_read_u32_array(entity_node,
> + "mipi-sdca-hide-related-audio-function-list", af_list, nval);
> +
> + nval = fwnode_property_count_u8(function_node, "mipi-sdca-hid-descriptor");
> + if (nval)
> + fwnode_property_read_u8_array(function_node, "mipi-sdca-hid-descriptor",
> + (u8 *)&hide->hid_desc, nval);
> +
> + if (hide->hid_desc.bNumDescriptors) {
> + nval = fwnode_property_count_u8(function_node, "mipi-sdca-report-descriptor");
> + if (nval) {
> + report_desc = devm_kzalloc(dev, nval, GFP_KERNEL);
> + if (!report_desc)
> + return -ENOMEM;
> + hide->hid_report_desc = report_desc;
> + fwnode_property_read_u8_array(function_node, "mipi-sdca-report-descriptor",
> + report_desc, nval);
> + }
> + }
> +
> + return 0;
> +}
> +
> static int find_sdca_entity(struct device *dev,
> struct fwnode_handle *function_node,
> struct fwnode_handle *entity_node,
> @@ -1261,6 +1341,9 @@ static int find_sdca_entity(struct device *dev,
> case SDCA_ENTITY_TYPE_GE:
> ret = find_sdca_entity_ge(dev, entity_node, entity);
> break;
> + case SDCA_ENTITY_TYPE_HIDE:
> + ret = find_sdca_entity_hide(dev, function_node, entity_node, entity);
> + break;
> default:
> break;
> }
> --
> 2.34.1
>
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH 1/3] ASoC: SDCA: add support for HIDE entity properties and HID descriptor/report
2025-06-11 10:25 shumingf
2025-06-11 10:54 ` Charles Keepax
@ 2025-06-12 12:52 ` Charles Keepax
1 sibling, 0 replies; 5+ messages in thread
From: Charles Keepax @ 2025-06-12 12:52 UTC (permalink / raw)
To: shumingf
Cc: broonie, lgirdwood, linux-sound, lars, flove, oder_chiou, jack.yu,
derek.fang, bard.liao
On Wed, Jun 11, 2025 at 06:25:49PM +0800, shumingf@realtek.com wrote:
> From: Shuming Fan <shumingf@realtek.com>
>
> Add support for parsing the HIDE entity descriptor and HID descriptor/report
>
> Signed-off-by: Shuming Fan <shumingf@realtek.com>
> ---
Reviewed-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Thanks,
Charles
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2025-06-16 11:50 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-06-16 11:47 [PATCH 1/3] ASoC: SDCA: add support for HIDE entity properties and HID descriptor/report shumingf
2025-06-16 11:50 ` Shuming [范書銘]
-- strict thread matches above, loose matches on Subject: below --
2025-06-11 10:25 shumingf
2025-06-11 10:54 ` Charles Keepax
2025-06-12 12:52 ` Charles Keepax
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox