2019-04-04 01:25:02

by Hariprasad Kelam

Subject: [PATCH] staging: comedi: Fix spelling mistake

changes interupts --> interrupts to fix warning reported by checkpatch
tool

Signed-off-by: Hariprasad Kelam <[email protected]>
---
drivers/staging/comedi/drivers/dt2811.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/comedi/drivers/dt2811.c b/drivers/staging/comedi/drivers/dt2811.c
index 05207a5..8a1f9ef 100644
--- a/drivers/staging/comedi/drivers/dt2811.c
+++ b/drivers/staging/comedi/drivers/dt2811.c
@@ -52,7 +52,7 @@
#define DT2811_ADCSR_ADBUSY BIT(5) /* r 1=A/D busy */
#define DT2811_ADCSR_CLRERROR BIT(4)
#define DT2811_ADCSR_DMAENB BIT(3) /* r/w 1=dma ena */
-#define DT2811_ADCSR_INTENB BIT(2) /* r/w 1=interupts ena */
+#define DT2811_ADCSR_INTENB BIT(2) /* r/w 1=interrupts ena */
#define DT2811_ADCSR_ADMODE(x) (((x) & 0x3) << 0)

#define DT2811_ADGCR_REG 0x01 /* r/w A/D Gain/Channel */
--
2.7.4

2019-04-04 09:28:04

by Ian Abbott

Subject: Re: [PATCH] staging: comedi: Fix spelling mistake

On 04/04/2019 02:24, Hariprasad Kelam wrote:
> changes interupts --> interrupts to fix warning reported by checkpatch
> tool
>
> Signed-off-by: Hariprasad Kelam <[email protected]>
> ---
> drivers/staging/comedi/drivers/dt2811.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/comedi/drivers/dt2811.c b/drivers/staging/comedi/drivers/dt2811.c
> index 05207a5..8a1f9ef 100644
> --- a/drivers/staging/comedi/drivers/dt2811.c
> +++ b/drivers/staging/comedi/drivers/dt2811.c
> @@ -52,7 +52,7 @@
> #define DT2811_ADCSR_ADBUSY BIT(5) /* r 1=A/D busy */
> #define DT2811_ADCSR_CLRERROR BIT(4)
> #define DT2811_ADCSR_DMAENB BIT(3) /* r/w 1=dma ena */
> -#define DT2811_ADCSR_INTENB BIT(2) /* r/w 1=interupts ena */
> +#define DT2811_ADCSR_INTENB BIT(2) /* r/w 1=interrupts ena */
> #define DT2811_ADCSR_ADMODE(x) (((x) & 0x3) << 0)
>
> #define DT2811_ADGCR_REG 0x01 /* r/w A/D Gain/Channel */
>

Hi Hariprasad, could you mention dt2811 in the patch subject line like this:

staging: comedi: dt2811: Fix spelling mistake

It helps to narrow down the scope of the patch.

Apart from that, the patch looks fine. Thanks.

--
-=( Ian Abbott <[email protected]> || Web: http://www.mev.co.uk )=-
-=( MEV Ltd. is a company registered in England & Wales. )=-
-=( Registered number: 02862268. Registered address: )=-
-=( 15 West Park Road, Bramhall, STOCKPORT, SK7 3JZ, UK. )=-

2019-04-04 10:08:14

by Mukesh Ojha

Subject: Re: [PATCH] staging: comedi: Fix spelling mistake

On 4/4/2019 6:54 AM, Hariprasad Kelam wrote:
> changes interupts --> interrupts to fix warning reported by checkpatch
> tool
>
> Signed-off-by: Hariprasad Kelam <[email protected]>


Reviewed-by: Mukesh Ojha <[email protected]>

Cheers,
-Mukesh

> ---
> drivers/staging/comedi/drivers/dt2811.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/comedi/drivers/dt2811.c b/drivers/staging/comedi/drivers/dt2811.c
> index 05207a5..8a1f9ef 100644
> --- a/drivers/staging/comedi/drivers/dt2811.c
> +++ b/drivers/staging/comedi/drivers/dt2811.c
> @@ -52,7 +52,7 @@
> #define DT2811_ADCSR_ADBUSY BIT(5) /* r 1=A/D busy */
> #define DT2811_ADCSR_CLRERROR BIT(4)
> #define DT2811_ADCSR_DMAENB BIT(3) /* r/w 1=dma ena */
> -#define DT2811_ADCSR_INTENB BIT(2) /* r/w 1=interupts ena */
> +#define DT2811_ADCSR_INTENB BIT(2) /* r/w 1=interrupts ena */
> #define DT2811_ADCSR_ADMODE(x) (((x) & 0x3) << 0)
>
> #define DT2811_ADGCR_REG 0x01 /* r/w A/D Gain/Channel */