From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-dy1-f169.google.com (mail-dy1-f169.google.com [74.125.82.169]) (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 C2070318B96 for ; Wed, 1 Apr 2026 19:29:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.82.169 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775071759; cv=none; b=kfEpDQkdO2YyKOJjvSKJsNnRvHpohV/7Gdfm5kSHDzTPgFXGA0ZzlAK6eegOTvDgRvCaVIp0+oKVcRNkOosYz20KkLzQMlsi+2TDZ6Gu8+whYChTyJVc2Y2OR4L/1R8gWXe6SEe4HEVkfO9+XEt4Ev1t5cuVXIBZcjIds3OaosU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775071759; c=relaxed/simple; bh=2eTNHF0bZeLAiPU49Yh2C04Z3O5f2NeB1Pf6s2B287s=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=q7Rla7llw+kyWGexMvDnrmcMOm5eB7X1+iOVFqFP5kUzpqISc+W8CWplvJo6Ro3otYRrN65ysxcqxbj8If99hyeERipIWCHwWcRPRVIAcUX+zKbUary4z673a/qfsMWdSzepslkL7B5bHuRdtQWd3/F7v4z4UOY6mHYuZnTud+I= 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=b/kj1+rR; arc=none smtp.client-ip=74.125.82.169 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="b/kj1+rR" Received: by mail-dy1-f169.google.com with SMTP id 5a478bee46e88-2c7e5f38b37so268434eec.0 for ; Wed, 01 Apr 2026 12:29:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1775071758; x=1775676558; 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=aPnSoYguA2Jagx3CZ2LMOq6guAwJIRYuTtTKPzl1j+8=; b=b/kj1+rRlkEl831u+L8ZoJSSDMRBJovTNQ4wPnsMfgEpSU2uHpPJz6T9u67c8oiHU6 yLJv7ODsHYkFTrb+bZCxhhJus88lfYP2QmQyV/B8y8u2Gx3k6UnU7y6Jlvj48YcP5tFE 3thVJKzXYzxUMn3G1n++cBxMVP8CC4OsoHWVTkgReab4g8I+XHtEq6hcUrGLPFyH5TOj byFDWtvc5nV3612gCa6OlOlvF+ATECK3vFr0sNP9+yAg6sau1ckZ4u/rHm9BmvlHYUPM jQU9ELJ4til5qcouBwHtFClZPVJ48G+vNjDe+9o+4yrO2pHGU73xjvO7gkr9V42G/5XE f5pA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1775071758; x=1775676558; 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=aPnSoYguA2Jagx3CZ2LMOq6guAwJIRYuTtTKPzl1j+8=; b=d4w9XdlOmlYDhVQe16t+fF2eDGgoQodsx13f7emNSMMH4CYxgT7m+ag9idrGdIQ6cg 4eToVzOJkTkWyQDn0wRvMn37R6N46DWLAb3SWoIC6eeJmYBAjvGWj6uuNQfvOZp+wrzu s6M3NWvhE9BeXkd8a3SkQlUPcs/CAdVc+AdU3z0VzLa15jzwgzvejaCPC/mz32uYZmkd DHZIigHEfunF6NumPoeFJQMYoSlMwl5/bSfzadY/sj5upJJcwnn2kwS7No9pQOGcH90Y cWcrBsSLzavztSYq2b2ucKmiLpQmUi+sqNTx+M/q1F7OET7m8CTQLvPg7S510Lkxv9i+ MuEA== X-Forwarded-Encrypted: i=1; AJvYcCWMDLvgSSFIeRmTrJEabIuRdK3QXrrNYo6+p9g2hg3jqqtOZA+T7OP/m3Dn3VU3NZe2pURIUZ6BiwMDpOY=@vger.kernel.org X-Gm-Message-State: AOJu0YzkQq+ubP7HvO6h+dAaJe3zsbQq2BjA8stfcmOkGGs+CAhlTA8M xZlAERZ23xvQ3sQkwu0muCr2QpzknEvhNXrS8tzNlsm/UAPCz73eYMz6 X-Gm-Gg: ATEYQzxi3QvUsFENAJIzfpxgVCEGVC279oul0Q7RnK2IScHEc5NzpnOBoE+u7mvItoY DY0Hfw/12NqO5HU1MyabGx0H7d/QmkB6oTwO4uQSfNLAiLK6JyfpOfZMMug6C8E9prt6ylXQeuR eJo1XBlTecMHpiu+1aHtY4ltkwlY1Bg1laknsof06xEuMXgmilIEMQP5zmmM6B115mGSVx0UOeF ejO77BhUPkIUpclPhGEkrEh7RW+1SWgMvD9uQyWhtzRIuwZjUnx4sWRdXkmsjkMPUUA1KBxCvyJ Je6GsznoqJCdVBpdCcgko0EXvc/hdqcqjhrwfFMWHxvfulf4TnLZBUA2i9hYSZCeq+au1BChfe+ xeo3mP2Qunui4FWTNkkyqUXiO2LkQ9vbIK4Z8WO7WElY+E/3k0X4uSDmW3O0cXRDfd8skoAm5WP fxEpXh/sm1dM66fHAaNhYLKyzg1G+Cyu3lMUz3dLMMdVk4n7CAu4HaqcHO2sFck2YCxoTowwmzj vA= X-Received: by 2002:a05:7300:d50c:b0:2c5:704f:714a with SMTP id 5a478bee46e88-2c930c77e37mr2841716eec.8.1775071757905; Wed, 01 Apr 2026 12:29:17 -0700 (PDT) Received: from google.com ([2a00:79e0:2ebe:8:5f38:4c97:ec41:caa0]) by smtp.gmail.com with ESMTPSA id 5a478bee46e88-2ca7d00f5easm480206eec.29.2026.04.01.12.29.16 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 01 Apr 2026 12:29:17 -0700 (PDT) Date: Wed, 1 Apr 2026 12:29:13 -0700 From: Dmitry Torokhov To: Johan Hovold Cc: linux-input@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] Input: usbtouchscreen - refactor endpoint lookup Message-ID: References: <20260401082212.2180434-1-johan@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@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: <20260401082212.2180434-1-johan@kernel.org> On Wed, Apr 01, 2026 at 10:22:12AM +0200, Johan Hovold wrote: > Use the common USB helpers for looking up bulk and interrupt endpoints > (and determining endpoint numbers) instead of open coding. > > Note that the NEXIO data interface has two bulk endpoints (see commit > 5197424cdccc ("Input: usbtouchscreen - add NEXIO (or iNexio) support") > for the descriptors). > > The lookup in probe handles both bulk-in and interrupt-in endpoints and > was added to handle NEXIO devices. Replace the open coded lookup with a > lookup for the common interrupt endpoint and an explicit fallback > accepting a bulk endpoint. > > This iterates over the (two) endpoints twice for NEXIO devices but makes > it more clear what is going on. > > Signed-off-by: Johan Hovold Applied, thank you. -- Dmitry