Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Fri, 04 May 2018 11:40:58 +0200
Message-ID: <s5hmuxflpw5.wl-tiwai@suse.de>
From:   Takashi Iwai <tiwai@suse.de>
To:     "Connor McAdams" <conmanx360@gmail.com>
Cc:     <alsa-devel@alsa-project.org>,
        =?UTF-8?B?IkrDqXLDqW15IExlZmF1cmUi?= <jeremy.lefaure@lse.epita.fr>,
        "Jaroslav Kysela" <perex@perex.cz>, <o-takashi@sakamocchi.jp>,
        <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 1/9] R3Di and SBZ quirk entires + alt firmware loading
In-Reply-To: <1525407594-25644-1-git-send-email-conmanx360@gmail.com>
References: <1525407594-25644-1-git-send-email-conmanx360@gmail.com>
User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka)
 FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.8 Emacs/25.3
 (x86_64-suse-linux-gnu) MULE/6.0 (HANACHIRUSATO)
MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka")
Content-Type: text/plain; charset=US-ASCII
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Fri, 04 May 2018 06:19:44 +0200,
Connor McAdams wrote:
> 
> This patch adds PCI quirk ID's for the Sound Blaster Z and Recon3Di.
> Only the currently tested ID's have been added.
> 
> This patch also adds the ability to load alternative firmwares for each
> card, the firmwares can be obtained from within the Windows driver.
> The Recon3Di uses "ctefx-r3di.bin" and the Sound Blaster Z uses
> "ctefx-sbz.bin". If the alternative firmware for the given quirk is not
> found, the original ctefx.bin will be used. This has been confirmed to
> work for both the R3Di and the SBZ.
> 
> This patch also makes the character array *dirstr a const.
> 
> Signed-off-by: Connor McAdams <conmanx360@gmail.com>

I reply here as the patch series lacks of a cover letter, as already
Sakamoto-san mentioned.

I like the patches in general.  It's a great effort for the long-time
PITA.  I already replied one minor thing I stumbled on.  But the whole
changes are specific to ca0132, so there is no big danger by the patch
series itself per se.

However, one thing to be fixed is the style you took in the series,
"put some changes commented out for the next patch".  I guess you made
it in that way for ease of splitting patches.  But of course it's
messy when you follow the patch history as a reader.

So please clean up them, move the new code to the new patch, and
resubmit as v3.  And, please give also a proper subject prefix and a
cover letter, too :)


Thanks!

Takashi