| Message ID | 77a6b65b-5f3f-4a21-a837-7a4a7e09b099@stanley.mountain (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | remoteproc: sysmon: Update qcom_add_sysmon_subdev() comment | expand |
On Fri, Mar 21, 2025 at 05:35:44PM +0300, Dan Carpenter wrote: > The comment says the qcom_add_sysmon_subdev() returns NULL on error but > it actually returns error pointers. > > Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org> > --- > drivers/remoteproc/qcom_sysmon.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
diff --git a/drivers/remoteproc/qcom_sysmon.c b/drivers/remoteproc/qcom_sysmon.c index c24e4a882873..660ac6fc4082 100644 --- a/drivers/remoteproc/qcom_sysmon.c +++ b/drivers/remoteproc/qcom_sysmon.c @@ -619,7 +619,7 @@ static irqreturn_t sysmon_shutdown_interrupt(int irq, void *data) * @name: name of this subdev, to use in SSR * @ssctl_instance: instance id of the ssctl QMI service * - * Return: A new qcom_sysmon object, or NULL on failure + * Return: A new qcom_sysmon object, or an error pointer on failure */ struct qcom_sysmon *qcom_add_sysmon_subdev(struct rproc *rproc, const char *name,
The comment says the qcom_add_sysmon_subdev() returns NULL on error but it actually returns error pointers. Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org> --- drivers/remoteproc/qcom_sysmon.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)