Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756459AbbBGQSb (ORCPT ); Sat, 7 Feb 2015 11:18:31 -0500 Received: from mail-qa0-f52.google.com ([209.85.216.52]:44828 "EHLO mail-qa0-f52.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755349AbbBGQS3 (ORCPT ); Sat, 7 Feb 2015 11:18:29 -0500 MIME-Version: 1.0 In-Reply-To: References: <1423234272-7348-1-git-send-email-chris@rorvick.com> <1423234272-7348-4-git-send-email-chris@rorvick.com> Date: Sat, 7 Feb 2015 10:18:28 -0600 X-Google-Sender-Auth: 2jFrtC5BkuKXBNgYA_tjSrDl9do Message-ID: Subject: Re: [PATCH 3/4] ALSA: line6: Pass driver name to line6_probe() From: Chris Rorvick To: Takashi Iwai Cc: alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org, Stefan Hajnoczi Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2090 Lines: 48 On Fri, Feb 6, 2015 at 11:23 AM, Takashi Iwai wrote: > At Fri, 06 Feb 2015 16:04:42 +0100, Takashi Iwai wrote: >> >> At Fri, 6 Feb 2015 08:51:11 -0600, Chris Rorvick wrote: >> > >> > Provide a descriptive name for each driver instead of calling all of >> > them "line6usb". >> >> This needs to be done carefully. This string is referred in alsa-lib >> to pick up the the configuration file. So, this change shall break >> the compatibility. >> >> If we ever want to pick up a different alsa-lib configuration >> depending on each line6 driver type, then yes, we should give the >> individual driver name. If we want to keep rather the common >> configuration file (so far there is none, but if any in furture), then >> we should keep the common driver name. >> >> And, the decision must be done now. From now on, basically we are not >> allowed to break the user-space compatibility. That is, this is the >> very last chance to do it. >> >> If your patch is supposed to do it with these consideration, I'm >> willing to take. But, I guess it's not, because you chose the string >> like "Line 6 POD". This is usually not ideal as a driver name; think >> of $DRIVER.conf file that is used for alsa-lib configuration. >> >> So, from that POV, "line6usb" is no bad name string. If we want >> differentiate per driver, a name like "Line6-Pod" or just "Pod" would >> be more appropriate. > > Just to make sure: I'm not against giving own driver name for each > line6 driver. I myself think it'd be rather better than the single > common name for long term. But, the name string should be more usable > as a file name, i.e. without space and just two words or so. OK, thanks for clarifying. I did not realize this was used to look for a config file name. I will resubmit using the /Line6-\w+/ convention. Regards, Chris -- 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/