Re: [PATCH 5/5] firmware: arm_scmi: Relocate atomic_threshold to scmi_desc

From: Dan Carpenter
Date: Wed Oct 23 2024 - 09:21:21 EST


On Fri, Oct 18, 2024 at 09:06:02AM +0100, Cristian Marussi wrote:
> @@ -2959,7 +2952,7 @@ static struct scmi_debug_info *scmi_debugfs_common_setup(struct scmi_info *info)
> (char **)&dbg->name);
>
> debugfs_create_u32("atomic_threshold_us", 0400, top_dentry,
> - &info->atomic_threshold);
> + (u32 *)&info->desc->atomic_threshold);

This cast is unnecessary.

>
> debugfs_create_str("type", 0400, trans, (char **)&dbg->type);
>
> @@ -3071,6 +3064,13 @@ static const struct scmi_desc *scmi_transport_setup(struct device *dev)
> trans->desc->max_rx_timeout_ms, trans->desc->max_msg_size,
> trans->desc->max_msg);
>
> + /* System wide atomic threshold for atomic ops .. if any */
> + if (!of_property_read_u32(dev->of_node, "atomic-threshold-us",
> + &trans->desc->atomic_threshold))
> + dev_info(dev,
> + "SCMI System wide atomic threshold set to %d us\n",
^^
%u for unsigned int.

> + trans->desc->atomic_threshold);
> +
> return trans->desc;
> }

regards,
dan carpenter