Re: [PATCH 2/7] soc: st: add RISAB dump debug driver

[Krzysztof Kozlowski <krzk@kernel.org>]

On 09/02/2026 15:59, Gatien Chevallier wrote:
> +
> +static int stm32_risab_register_debugfs(struct risab_pdata *pdata)
> +{
> +	struct dentry *root = NULL;
> +
> +	root = debugfs_lookup("stm32_firewall", NULL);
> +	if (!root)
> +		root = debugfs_create_dir("stm32_firewall", NULL);
> +
> +	if (IS_ERR(root))
> +		return PTR_ERR(root);
> +
> +	pdata->dbg_entry = debugfs_create_file(dev_name(pdata->dev), 0444,
> +					       root, pdata, &stm32_risab_conf_dump_fops);

soc drivers should not have any user-space interfaces. This was
requested by Arnd many times in the past, nothing new, including about
debugfs. Otherwise it opens cans of worms, because soc platform
maintainer can dump here whatever the subsystem rejected.

Find suitable subsystem and get its approval for such debugfs entry.

Your commit msg is so vague it is another reason this is not mergeable.
Explains nothing about the feature, nothing about the risk or impact on
the system when user does (while (1); read stm32_risab_conf_dump_fops).

Not mentioning that it does not explain WHY do we want this driver in
the first place...

Best regards,
Krzysztof