From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f46.google.com (mail-wr1-f46.google.com [209.85.221.46]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id F27C4363096 for ; Wed, 11 Mar 2026 16:27:23 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.46 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773246445; cv=none; b=aCmAP2MPYYNUM9Qb4FRMbXVdipNxTO+IZJkNjFyiwvRI6Jgfd46uVEViM5UUzVMvBFQze5NDpBxdS6oa0Z2Lv3n8ermwlK5OrpkcRSrKKesrVw7/29AeZIRty4FggJqUNPK598dYZVceBzsvDsgBgmtSJiJ5c9/DxKzT5vYKZ5c= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773246445; c=relaxed/simple; bh=EnU7CaNkIwUFhTfWBEiVDCVcCFLfPHjoZFzkqbjaQgo=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=qQ2FHtUHmasu/aQ06hq+j5JXQN+pqwuuHjkIjsRIbgp1nju054YDiV1VkUQd70bpbXDjd55HwZ8zHI33t5cZbqTOydpsV/zrr8AGdu18ZGJ+7CfEVg1UGJWJ+hy7AfwGwpSv92mfAbwVYrdkQCiIpn6+dSAqgajvanCXXsnqlRw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=nkQXkh67; arc=none smtp.client-ip=209.85.221.46 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="nkQXkh67" Received: by mail-wr1-f46.google.com with SMTP id ffacd0b85a97d-4327790c4e9so48162f8f.2 for ; Wed, 11 Mar 2026 09:27:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1773246442; x=1773851242; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=ykuNTX9WARes7OQ8thK8bu9wdsPTSrhLs2vPDVOR02Q=; b=nkQXkh67YrKaW6y+dW/cdoDXLpXXxM8oUREPguMfQKe0dbt1pT9BxS+D3sJ9MCLeNW 2qeCH4q1ppPBBQOR1WK3SM95pFtSHeoA5niF6bGtsOKBsfeF8gk47wrXqaSDlJre6Zq7 3aNux/ermBSy0RgPRD72xEMlAsxlczI6byj7BxgqR6guywADOKfwb4UiTu7fVq19NMIR RTtSdS2+oJ2Q6i3zU+BG4qoG4sNvdpipc5z0THcodVGOlpjK4QzZKEcc7aOWIwEJtP31 d82nzSa7QvS4OHDM4DeDlBcVJYBvJQZPTPthXYO46YgwbzpvgIJMNYbCidfwrU68YZR/ jSMA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1773246442; x=1773851242; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=ykuNTX9WARes7OQ8thK8bu9wdsPTSrhLs2vPDVOR02Q=; b=pWFyaF0s1WcbvPslxZzkq1c0aTjrldJVxL7rx7CdG9O1ntYOAmOjsmSzSpuOWRHoCV eKwJKlPWGSqhMCc1rMuMFuvxdfhvc3nnotC+EuxEbjFFFC3A6CWn1GAy3xSaR8r0tP0v H44GP/rnxF6QcB3m5XlZGlEMmyOYqxoYS8u6SqdJIYEZ5dEoZsQ4oRR2HT4Kj/G/UQEb IkMBrKnCEHdIG6NhgD6glaDKNwjmovBdzgkr8MJrIZvCbIWxXiD/jBfiqn0H9DsK7FJV k8lW17iFPD0t3NM61CZCprOlzANdbuGZUMA4TfOIIAqueuYRjseqK4rJZckZP9VCBFhA YyUg== X-Forwarded-Encrypted: i=1; AJvYcCWyExnpis9QHqGe/ND+aV5mX30D4G93VIzqnfrLCBWr6aUo+TrFYVzjwetutMyh4OTM2N52Y8U=@vger.kernel.org X-Gm-Message-State: AOJu0Yyt4Mp605qsQXyIst2pLf1SCnyiqJRSWjMhPoVJI0AamvSMlux6 4791EVma1KlSRg13nT/G+OYZRf4fRkZQkIlC2onFtwDcuSiMu02heSb+ X-Gm-Gg: ATEYQzx4CDnMW5efHRWcxhrnqF0vTjIvWRzGDebYWSidHGeJmUBpBEit3rO5TRMP73f fN1Dl5NAamZBTF0Ox/Kg2zY0QX0uWKI+syAdF62WEozwpSLKjOn8BwitmUa4vpeOwVgAaTOzbn+ zVTwNdouA747lQ/3c27aawNOBJu+on2RpaGEQ06Q40z8XDAQgwHWa6Tm1ChGxw3YxryKj6zJNFf vfIva2qLfKKOcxebpcA7aeYRYYPNcnJkuq56AQkz7EwNkL8ELm9n0BY4jxXvt3l+SLjlejevL5h zSHpoztbRKFhJwBdiE+Ux5h/HfBjdtnQ3o6gO1X5EWejAzFCAI4khXAaa4hleMLGwSDIa7BgD92 x+HZ2s3HjaFtlQiSxN6NhglwjxDpOKKrllKU/OsCN9atBoWbq9X7kQ3DI9gAFYiNhCTvvqox0qA KQ5naNycZvKUGL8bIZgRFP2e9XG8Up8GvPKkLlLao= X-Received: by 2002:a05:600c:1f10:b0:485:419c:4eba with SMTP id 5b1f17b1804b1-4854b0a5471mr60344935e9.1.1773246442177; Wed, 11 Mar 2026 09:27:22 -0700 (PDT) Received: from egonzo (82-64-73-52.subs.proxad.net. [82.64.73.52]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-4854b0fe6cfsm21409105e9.31.2026.03.11.09.27.21 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 11 Mar 2026 09:27:21 -0700 (PDT) Date: Wed, 11 Mar 2026 17:27:20 +0100 From: Dave Penkler To: Johan Hovold Cc: Greg Kroah-Hartman , linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org Subject: Re: [PATCH v2 1/2] gpib: lpvo_usb: fix unintended binding of FTDI 8U232AM devices Message-ID: References: <20260305151729.10501-1-johan@kernel.org> <20260305151729.10501-2-johan@kernel.org> <2026031131-abdominal-surgery-9b98@gregkh> Precedence: bulk X-Mailing-List: stable@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Wed, Mar 11, 2026 at 04:28:50PM +0100, Johan Hovold wrote: > On Wed, Mar 11, 2026 at 03:21:52PM +0100, Greg Kroah-Hartman wrote: > > On Thu, Mar 05, 2026 at 04:17:28PM +0100, Johan Hovold wrote: > > > The LPVO USB GPIB adapter apparently uses an FTDI 8U232AM with the > > > default PID, but this device id is already handled by the ftdi_sio > > > serial driver. > > > > > > Stop binding to the default PID to avoid breaking existing setups with > > > FTDI 8U232AM. > > > > > > Anyone using this driver should blacklist the ftdi_sio driver and add > > > the device id manually through sysfs (e.g. using udev rules). > > > > @@ -38,8 +38,10 @@ MODULE_DESCRIPTION("GPIB driver for LPVO usb devices"); > > > /* > > > * Table of devices that work with this driver. > > > * > > > - * Currently, only one device is known to be used in the > > > - * lpvo_usb_gpib adapter (FTDI 0403:6001). > > > + * Currently, only one device is known to be used in the lpvo_usb_gpib > > > + * adapter (FTDI 0403:6001) but as this device id is already handled by the > > > + * ftdi_sio USB serial driver the LPVO driver must not bind to it by default. > > > + * > > > * If your adapter uses a different chip, insert a line > > > * in the following table with proper , . > > > * > > > @@ -50,7 +52,6 @@ MODULE_DESCRIPTION("GPIB driver for LPVO usb devices"); > > > */ > > > > > > static const struct usb_device_id skel_table[] = { > > > - { USB_DEVICE(0x0403, 0x6001) }, > > > > With this change, the driver now "does nothing". Should we just mark it > > as CONFIG_BROKEN as well? > > That would prevent people with this device from using the driver by > manually adding the device id through sysfs. > > Some FTDI devices can be programmed with product specific PIDs, but not > sure about 8U232AM. Or if whoever built this is still around to care > enough. We are testing udev rules to bind the driver in sysfs for the lpvo_usb_gpib driver for inclusion in the user space package. Removing the binding for this driver is the right thing to do. Belatedly: Acked-by: Dave Penkler > > When I saw the skeleton driver included verbatim, including the > example minor number (which has been reserved for something else, but I > guess that's less of an issue these days), and that it was binding to > the default FTDI PID, my initial thought was just to drop the driver (or > move it back to staging, but then we'd still need to drop the device > id). > > Johan