From: John F Leach <jfleach@jfleach.com>
To: Takashi Iwai <tiwai@suse.de>
Cc: alsa-devel <alsa-devel@alsa-project.org>,
clemens <clemens@ladisch.de>, Daniel Mack <zonque@gmail.com>
Subject: Re: [PATCH] [ALSA] usb-audio - Support for Roland GAIA SH-01 Synthesizer
Date: Sun, 27 Nov 2011 17:15:30 -0500 [thread overview]
Message-ID: <1322432130.19532.7.camel@darkside> (raw)
In-Reply-To: <s5h8vn1xy6h.wl%tiwai@suse.de>
> Yes, I can fix it by myself.
> But, the patch doesn't pass checkpatch.pl test, so I postpone for
> now.
>
> John, please fix warnings scripts/checkaptch.pl gives as much as
> possible. Some warnings can be ignored (e.g. over 80 chars), but
> space issues should be fixed at least.
Takashi,
I replaced the spaces with tabs, I didn't realize I was supposed to run checkpatch.pl before submitting.
How do I fix the ERROR: space prohibited after that '&' (ctx:WxW)? All of the other table entries look like this.
$ scripts/checkpatch.pl 0001-ALSA-usb-audio-Support-for-Roland-GAIA-SH-01-Synthes.patch
ERROR: space prohibited after that '&' (ctx:WxW)
#28: FILE: sound/usb/quirks-table.h:1638:
+ .driver_info = (unsigned long) & (const struct snd_usb_audio_quirk) {
^
WARNING: line over 80 characters
#45: FILE: sound/usb/quirks-table.h:1655:
+ .data = & (const struct snd_usb_midi_endpoint_info) {
ERROR: space prohibited after that '&' (ctx:WxW)
#45: FILE: sound/usb/quirks-table.h:1655:
+ .data = & (const struct snd_usb_midi_endpoint_info) {
^
total: 2 errors, 1 warnings, 37 lines checked
0001-ALSA-usb-audio-Support-for-Roland-GAIA-SH-01-Synthes.patch has style problems, please review.
If any of these errors are false positives, please report
them to the maintainer, see CHECKPATCH in MAINTAINERS.
Regards,
John F Leach
next prev parent reply other threads:[~2011-11-27 22:15 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-27 14:34 [PATCH] [ALSA] usb-audio - Support for Roland GAIA SH-01 Synthesizer John F Leach
2011-11-27 15:21 ` Daniel Mack
2011-11-27 16:03 ` [PATCH] [ALSA] usb-audio - Support for Roland GAIA SH-01 Synthesizer - fix list order and wrapped line John F Leach
2011-11-27 16:10 ` [PATCH] [ALSA] usb-audio - Support for Roland GAIA SH-01 Synthesizer John F Leach
2011-11-27 16:42 ` Daniel Mack
2011-11-27 17:39 ` Takashi Iwai
2011-11-27 22:15 ` John F Leach [this message]
2011-11-28 9:48 ` Takashi Iwai
2011-11-29 0:41 ` John F Leach
2011-11-29 7:24 ` Takashi Iwai
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=1322432130.19532.7.camel@darkside \
--to=jfleach@jfleach.com \
--cc=alsa-devel@alsa-project.org \
--cc=clemens@ladisch.de \
--cc=tiwai@suse.de \
--cc=zonque@gmail.com \
/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.