From: Jaime Arrocha <jarrocha@kerneldev.net>
To: greg@kroah.com
Cc: abbotti@mev.co.uk, hsweeten@visionengravers.com,
devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org
Subject: Re:[PATCH] staging: comedi: drivers: das16m1.c: checkpatch error fix
Date: Thu, 07 May 2015 09:31:15 -0500 [thread overview]
Message-ID: <554B7733.3070600@kerneldev.net> (raw)
In-Reply-To: <CAFGHVq3-FGaEy3NEOprLpAfWddwdVKPYpvXMXb21iHReBh85Wg@mail.gmail.com>
On 05/07/2015 09:27 AM, Jaime Arrocha wrote:
> ---------- Forwarded message ----------
> From: Jaime Arrocha <jarr@kerneldev.net>
> Date: Wed, May 6, 2015 at 4:08 PM
> Subject: [PATCH] staging: comedi: drivers: das16m1.c: checkpatch error fix
> To: greg@kroah.com
> Cc: abbotti@mev.co.uk, hsweeten@visionengravers.com,
> devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org
>
>
> Errors found by checkpatch.pl.
> ERROR: code indent should use tabs where possible
> /drivers/staging/comedi/drivers/das16m1.c:49
> /drivers/staging/comedi/drivers/das16m1.c:50
>
> Signed-off-by: Jaime Arrocha <jarr@kerneldev.net>
> ---
> drivers/staging/comedi/drivers/das16m1.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/staging/comedi/drivers/das16m1.c
> b/drivers/staging/comedi/drivers/das16m1.c
> index 1adf6a7..a18a887 100644
> --- a/drivers/staging/comedi/drivers/das16m1.c
> +++ b/drivers/staging/comedi/drivers/das16m1.c
> @@ -46,8 +46,8 @@ list has 2 or more channels in it, then two conditions
> must be satisfied:
> (2) - the list must have an even number of entries.
>
> Options:
> - [0] - base io address
> - [1] - irq (optional, but you probably want it)
> + [0] - base io address
> + [1] - irq (optional, but you probably want it)
>
> irq can be omitted, although the cmd interface will not work without it.
> */
> --
> 1.7.10.4
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>
This patch should not be applied.
prev parent reply other threads:[~2015-05-07 14:47 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-06 21:08 [PATCH] staging: comedi: drivers: das16m1.c: checkpatch error fix Jaime Arrocha
[not found] ` <CAFGHVq3-FGaEy3NEOprLpAfWddwdVKPYpvXMXb21iHReBh85Wg@mail.gmail.com>
2015-05-07 14:31 ` Jaime Arrocha [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=554B7733.3070600@kerneldev.net \
--to=jarrocha@kerneldev.net \
--cc=abbotti@mev.co.uk \
--cc=devel@driverdev.osuosl.org \
--cc=greg@kroah.com \
--cc=hsweeten@visionengravers.com \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.