Received: by 2002:ac0:a5a6:0:0:0:0:0 with SMTP id m35-v6csp441937imm; Thu, 13 Sep 2018 02:18:22 -0700 (PDT) X-Google-Smtp-Source: ANB0VdaIp8st5d5AclQmZwQ6R0k8YS0mZCq+Nq/OLovwuoAvT2C9LVH/g/Ik4+FR8EllaIvNLm+q X-Received: by 2002:a63:dc17:: with SMTP id s23-v6mr6263064pgg.40.1536830302562; Thu, 13 Sep 2018 02:18:22 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1536830302; cv=none; d=google.com; s=arc-20160816; b=VxlTz1cSlXrNjLN9xtEZnJo8/DGzrir/wgFH6rb++ctjxwsXZt7+7Dy7vnkJCT0B1W pQcyWXIxsEOnXHMtkubferFt8QjxlMfMVUx17UgttPCp5hpR2O3bto3ju78aL0ElzlCj XI8pXAMT6kMObTV1toU0SbO6rxUTsVt1kvdsRCAZAPedROXQ5kPlkYShMZ7MYWOR+t1V ZLPr0TLcNI7Ql9e3rBcJWl6/+zPQe28GGSxDefhosWtC0GCgBQEZWhHjxLSulW4caeYQ 3lQxIKQ1AxE5VSWmhrxDWScQwsYpb+M1O/iVXZXRvUoBfSU63m96gaOV6vA4beC+Iuij u5Ag== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-transfer-encoding:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=9n8a+xz+Oz2i5BEr3vB5jMf6qCXGeM6zAfVFMP1APJg=; b=XtSrGHbLb5/3DhlcPKxpPUfLIwfpj8fBPWhDmFkyD4+vBnFrbouKp0ZNwobXjZGfBN lXeaynceCZyBjVbq7qeQkuudJIPFbYTYc1Ibv1LTjpuORllKl+agr2e5MCRncb5IIHfM 5rxMIt9KdGGbMcAjVVg97zhWnlwZBGQHkTtYW/LC+aAC7oF/WhbBcQaHAEmXWA2CE32I J4T5c1+UeP9bZDWb8vSLsmu3hihy0i/L51GHwYqhO1meHre7s8bL1PdKYe+2Y0jK2nDn 0LkKmtg50NIsgR18wSEPG6QV4n8Hp6E+Z1U7wrojTUuITC1bCZHPbSYpm0coQtVaaS/x 6NSg== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@gmail.com header.s=20161025 header.b=pVX6O5Jk; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id z71-v6si3523369pff.223.2018.09.13.02.18.07; Thu, 13 Sep 2018 02:18:22 -0700 (PDT) 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; Authentication-Results: mx.google.com; dkim=fail header.i=@gmail.com header.s=20161025 header.b=pVX6O5Jk; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727685AbeIMO0d (ORCPT + 99 others); Thu, 13 Sep 2018 10:26:33 -0400 Received: from mail-lj1-f193.google.com ([209.85.208.193]:36184 "EHLO mail-lj1-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727150AbeIMO0d (ORCPT ); Thu, 13 Sep 2018 10:26:33 -0400 Received: by mail-lj1-f193.google.com with SMTP id v26-v6so4013993ljj.3; Thu, 13 Sep 2018 02:17:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:date:from:to:cc:subject:message-id:references:mime-version :content-disposition:content-transfer-encoding:in-reply-to :user-agent; bh=9n8a+xz+Oz2i5BEr3vB5jMf6qCXGeM6zAfVFMP1APJg=; b=pVX6O5Jkqa9Yjmm9POcJglHOgs2fYLSBnpz14Dpx/MLJNQNQ9XSnN2XQC4qkVwLgrw 6q08ayb6OflqcemjVEnSzEAyFSZ7W6Atqdrsz/68tgG0yl++BIyzv41vONmm308P5Yte +ETj3A82LIFaPWB2CC19W4g15LO/9r2gNy6TaV+DWaaT7ACDlsSnwfKJyQbcrgrWgVq+ TtayA2XjF9cSwq8W4KCpH81OeKcAsznFXMBgMiq8wFxwKtR5O6LMjForbhQ30+qzKUFs HhRZPLD8g13n6a0ZnN9WziTu6m/ZwuSFsIkVeCsgdgXkJz9wyabTzAFREPuAOFDaX8I4 2ufg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:date:from:to:cc:subject:message-id :references:mime-version:content-disposition :content-transfer-encoding:in-reply-to:user-agent; bh=9n8a+xz+Oz2i5BEr3vB5jMf6qCXGeM6zAfVFMP1APJg=; b=FjIsfeYpgphi9uyW7rJQBrofH8CGTrkHj67E1bhAgeENGa9VGQJhf5aWWmUuPZ50cI byDRET+XkXdCkxPgVwb2NDbsmjmRws96cVhWPrLzWtZo+WbRX9saSM6ezu4ZBqHDIcy2 7mHg9JlgbgPKqjQhh6Zwz2kFWDFQMsQBdwqq1fkpgqr3s75ob2e2FK5zPkJmmSjyktzr CbM7nuIDE0OWOohLIhE2RAgLVCjDDBZzXozOVG3foWt18nZvvx7ZO9DA5QE/idWwcb6P SFE+BmsafBFDTJOt47VtYFE1ttiKQ8vWKP06CuoW8nYduck1EOPSdkRjAPdUYdajOsuM X85g== X-Gm-Message-State: APzg51AsQq8BlcGbHla+Qebk9Dq3dWV91fF4GIgladSssqGgoKGK5jgj TqYdTJkPBpA1CaVou2ttYi4= X-Received: by 2002:a2e:2f19:: with SMTP id v25-v6mr3829634ljv.113.1536830274306; Thu, 13 Sep 2018 02:17:54 -0700 (PDT) Received: from xi.terra (c-74bee655.07-184-6d6c6d4.bbcust.telenor.se. [85.230.190.116]) by smtp.gmail.com with ESMTPSA id g30-v6sm618576lfi.84.2018.09.13.02.17.53 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 13 Sep 2018 02:17:53 -0700 (PDT) Received: from johan by xi.terra with local (Exim 4.91) (envelope-from ) id 1g0Nl3-0008WU-Hn; Thu, 13 Sep 2018 11:17:54 +0200 Date: Thu, 13 Sep 2018 11:17:53 +0200 From: Johan Hovold To: =?iso-8859-1?Q?Bj=F8rn?= Mork Cc: Dan Williams , Lars Melin , Kristian Evensen , Johan Hovold , linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] option: Improve Quectel EP06 detection Message-ID: <20180913091753.GA3443@localhost> References: <20180908125754.1947-1-kristian.evensen@gmail.com> <20180910103022.GR1089@localhost> <8ce10eef-f332-a1c0-e16e-fb1c2055131a@gmail.com> <87y3c6a1zw.fsf@miraculix.mork.no> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <87y3c6a1zw.fsf@miraculix.mork.no> User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Sep 12, 2018 at 10:34:43PM +0200, Bj?rn Mork wrote: > Dan Williams writes: > > > The fact that the firmware implementation has the ability to change the > > endpoints is unrelated to Kristian's case, and that alone is > > justification for this to be quirked in the driver. People other than > > Kristian will undoubtedly use the functionality, on platforms less > > limited. > > FWIW, I agree with Dan and Kristian on this. It's a documented feature, > and it will be used. The reasons are irrelevant. The firmware > implementation is inconvenient, but we should still strive to make it > Just Work in Linux. Kristian's solution does that. > > > Also most Huawei modems have the ability to change their layout and > > configuration just like the EP06 via the U2DIAG and SETPORT commands. > > Yes, but they are nice enough to use unique class/subclass/protocol > triplets for their functions so it's easy to support the changing > layout. At least as long as they use their own VID and not some laptop > vendor's.. > > The Sierra Wireless strategy, using fixed interface numbers leaving > "holes" is another fine solution to the problem. Or they could have > allocated unique VIDs per function combination, as long as the number of > valid combinations are low. But they didn't. It's not like it's the > first bad firmware design we've had to deal with. Let's just work > around it, like we always do. No need to make life difficult for end > users just because Quectel makes life difficult for us. Ok, thanks for everyone for your input. As Lars I was sceptical about this, but if this is contained to Quectel and we have a solutions which hopefully covers all permutations for their other devices, let's go along with this. I'd prefer seeing this contained in the device-id table as far as possible rather than maintaining a second table of product ids in probe() so I've cooked up a patch (on top of this one) adding a new device-id match flag. Kristian would you mind giving it a try? Oh, also note that I dropped the RSVD(5) for the ADB interface in your patch since it uses a different subclass anyway. I'll submit both patches in a series. Thanks, Johan