2022-06-08 13:29:59

by Xiang wangx

[permalink] [raw]
Subject: [PATCH] remoteproc: omap_remoteproc: Fix typo in comment

Delete the redundant word 'The'.
Delete the redundant word 'to'.

Signed-off-by: Xiang wangx <[email protected]>
---
drivers/remoteproc/omap_remoteproc.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/remoteproc/omap_remoteproc.c b/drivers/remoteproc/omap_remoteproc.c
index 32a588fefbdc..430fab0266ed 100644
--- a/drivers/remoteproc/omap_remoteproc.c
+++ b/drivers/remoteproc/omap_remoteproc.c
@@ -243,7 +243,7 @@ static inline int omap_rproc_get_timer_irq(struct omap_rproc_timer *timer)
* omap_rproc_ack_timer_irq() - acknowledge a timer irq
* @timer: handle to a OMAP rproc timer
*
- * This function is used to clear the irq associated with a watchdog timer. The
+ * This function is used to clear the irq associated with a watchdog timer.
* The function is called by the OMAP remoteproc upon a watchdog event on the
* remote processor to clear the interrupt status of the watchdog timer.
*/
@@ -303,7 +303,7 @@ static irqreturn_t omap_rproc_watchdog_isr(int irq, void *data)
* @configure: boolean flag used to acquire and configure the timer handle
*
* This function is used primarily to enable the timers associated with
- * a remoteproc. The configure flag is provided to allow the driver to
+ * a remoteproc. The configure flag is provided to allow the driver
* to either acquire and start a timer (during device initialization) or
* to just start a timer (during a resume operation).
*
@@ -443,7 +443,7 @@ static int omap_rproc_enable_timers(struct rproc *rproc, bool configure)
* @configure: boolean flag used to release the timer handle
*
* This function is used primarily to disable the timers associated with
- * a remoteproc. The configure flag is provided to allow the driver to
+ * a remoteproc. The configure flag is provided to allow the driver
* to either stop and release a timer (during device shutdown) or to just
* stop a timer (during a suspend operation).
*
--
2.36.1


2022-06-14 23:35:22

by Mathieu Poirier

[permalink] [raw]
Subject: Re: [PATCH] remoteproc: omap_remoteproc: Fix typo in comment

On Wed, 8 Jun 2022 at 07:04, Xiang wangx <[email protected]> wrote:
>
> Delete the redundant word 'The'.
> Delete the redundant word 'to'.
>
> Signed-off-by: Xiang wangx <[email protected]>
> ---
> drivers/remoteproc/omap_remoteproc.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>

I have applied this patch.

Thanks,
Mathieu

> diff --git a/drivers/remoteproc/omap_remoteproc.c b/drivers/remoteproc/omap_remoteproc.c
> index 32a588fefbdc..430fab0266ed 100644
> --- a/drivers/remoteproc/omap_remoteproc.c
> +++ b/drivers/remoteproc/omap_remoteproc.c
> @@ -243,7 +243,7 @@ static inline int omap_rproc_get_timer_irq(struct omap_rproc_timer *timer)
> * omap_rproc_ack_timer_irq() - acknowledge a timer irq
> * @timer: handle to a OMAP rproc timer
> *
> - * This function is used to clear the irq associated with a watchdog timer. The
> + * This function is used to clear the irq associated with a watchdog timer.
> * The function is called by the OMAP remoteproc upon a watchdog event on the
> * remote processor to clear the interrupt status of the watchdog timer.
> */
> @@ -303,7 +303,7 @@ static irqreturn_t omap_rproc_watchdog_isr(int irq, void *data)
> * @configure: boolean flag used to acquire and configure the timer handle
> *
> * This function is used primarily to enable the timers associated with
> - * a remoteproc. The configure flag is provided to allow the driver to
> + * a remoteproc. The configure flag is provided to allow the driver
> * to either acquire and start a timer (during device initialization) or
> * to just start a timer (during a resume operation).
> *
> @@ -443,7 +443,7 @@ static int omap_rproc_enable_timers(struct rproc *rproc, bool configure)
> * @configure: boolean flag used to release the timer handle
> *
> * This function is used primarily to disable the timers associated with
> - * a remoteproc. The configure flag is provided to allow the driver to
> + * a remoteproc. The configure flag is provided to allow the driver
> * to either stop and release a timer (during device shutdown) or to just
> * stop a timer (during a suspend operation).
> *
> --
> 2.36.1
>