From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751817AbeEDJlB (ORCPT ); Fri, 4 May 2018 05:41:01 -0400 Received: from mx2.suse.de ([195.135.220.15]:41430 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750707AbeEDJlA (ORCPT ); Fri, 4 May 2018 05:41:00 -0400 Date: Fri, 04 May 2018 11:40:58 +0200 Message-ID: From: Takashi Iwai To: "Connor McAdams" Cc: , =?UTF-8?B?IkrDqXLDqW15IExlZmF1cmUi?= , "Jaroslav Kysela" , , 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 List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 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