[PATCH] clk: mediatek: reset: Fix device registration kernel-doc
AngeloGioacchino Del Regno
angelogioacchino.delregno at collabora.com
Thu Sep 10 02:23:58 PDT 2026
On 9/10/26 05:30, Karl Mehltretter wrote:
> The comment for mtk_register_reset_controller_with_dev() names the
> older helper and documents np rather than dev. The new helper takes
> a struct device and obtains its device node internally.
>
> Use the actual function and parameter names and clarify that failures
> return negative error codes.
>
> Fixes: 761bc64003e3 ("clk: mediatek: reset: Add new register reset function with device")
> Assisted-by: LLM
> Signed-off-by: Karl Mehltretter <kmehltretter at gmail.com>
> ---
> drivers/clk/mediatek/reset.h | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/clk/mediatek/reset.h b/drivers/clk/mediatek/reset.h
> index 562ffd290a22..b09cdcab0c4b 100644
> --- a/drivers/clk/mediatek/reset.h
> +++ b/drivers/clk/mediatek/reset.h
> @@ -60,11 +60,11 @@ struct mtk_clk_rst_data {
> };
>
> /**
> - * mtk_register_reset_controller - Register mediatek clock reset controller with device
> - * @np: Pointer to device.
> + * mtk_register_reset_controller_with_dev - Register a MediaTek reset controller
> + * @dev: Device used to manage the reset controller resources.
> * @desc: Constant pointer to description of clock reset.
> *
> - * Return: 0 on success and errorno otherwise.
> + * Return: 0 on success, negative errno otherwise.
> */
> int mtk_register_reset_controller_with_dev(struct device *dev,
> const struct mtk_clk_rst_desc *desc);
>
> base-commit: 50d05c7c76c96b90462f24debacca971d2e86713
Hey, thanks for the patch!
It's almost perfect - just change the first line with:
* mtk_register_reset_controller_with_dev() - Register a MediaTek reset controller
After which,
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno at collabora.com>
Cheers,
Angelo
More information about the Linux-mediatek
mailing list