From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 188F11CD26 for ; Wed, 22 May 2024 16:00:24 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=170.10.133.124 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1716393627; cv=none; b=i0QSZAf1Jr875lZhZ9oYvbA0ehR4GYzFLkucnqoyWPVSWO4mm/yzPQEtL7Pex71xNwfVP74DisKBqw/g0Lh6IfriLXWR30bHKQM1XiaPMpzb2wicSjcwY1CtRD7ytJp3UUBLWfeUobWeyJossho2V5bhRv0Wy3XTPUykJf6dteg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1716393627; c=relaxed/simple; bh=2C9fNAU+Yg3t9+muOOfWc9qfe3e2oqVbk6lel/Tnsrg=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=ZF5onzRFAfKV0nBjnsGc/Nuxb/qrgqWjUsStRKQbp5+TVnANomfLOr7HDp652NhXnCh7gRWEhtP/vC0UkO4JOvWLlMTWMfekSo4V2mdv/Q7Zx14XYCFwYkwbH4LlOlQ0pZetLKMtejcFrWOTeakny0SJef9PIEz0k/auseEJqbs= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=redhat.com; spf=pass smtp.mailfrom=redhat.com; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b=doqFdIKP; arc=none smtp.client-ip=170.10.133.124 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=redhat.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=redhat.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="doqFdIKP" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1716393623; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=ajxqgjEKnMOrF68kO1CKH80QE6nlB/0C/WuHcK2TQAo=; b=doqFdIKPJFG72HwmhrDVBq3sgDqHs1rsMDcdJ97S2L8DYG0Kfo1r8tsuIvF+okiYKmuu5B ma2jQqUtQ/yHwFtOtHYtxuAuzPdtI5lW86SoPciny5+gtPhFcVc4NYhEZWSMSyM8iPxsoM MyqfmfR6Ai1EACavPw0Own2NsJsDvs0= Received: from mail-wr1-f69.google.com (mail-wr1-f69.google.com [209.85.221.69]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-548-_AdPuIsFN9yBXX3jdiFRUA-1; Wed, 22 May 2024 12:00:22 -0400 X-MC-Unique: _AdPuIsFN9yBXX3jdiFRUA-1 Received: by mail-wr1-f69.google.com with SMTP id ffacd0b85a97d-354cd52783eso2174853f8f.1 for ; Wed, 22 May 2024 09:00:22 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1716393621; x=1716998421; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=ajxqgjEKnMOrF68kO1CKH80QE6nlB/0C/WuHcK2TQAo=; b=BPATtKUZ6lnwzXNHx0fGuuykBTOB3NuUVrfw05BrJe32/KeGBMaKy/ne5YMike8xcL jzx83lzbQ1ovt9ABBUynYs9nXvC7jXiCJtVzhU0LLMYxvPikjK4XhPqERtbnt1b1WS48 Ni1RjNMAv3e+mpMgDL9br0OhVWzL040JBxfk8852kw/biMWBthyg0oKd1je485bccMes UovexIISxg2/9jlhgy43WxibGdSF7Od1BglkwPG44DbcLagqR8iqWF66dafCAMJX9VpD QbEDWB6FKZ5ozh8spTCHRXkx9tgWI7W++v5du6dRWIWkllHA7cCpxlq/X5L91u9uHBu7 AbUA== X-Gm-Message-State: AOJu0Yy2Y0KTqUizrSGRTM2flx1TscaA0fptjwfcg7PmDnyiFpA11USu aJZEfq4OE+3p/6XQCBM9iWW5CMayf4cMloq2VZmepT4zMOQ2f3dJG75jguQINKwPord/mBxyIsx kwMHEbOVcHeMLT5faSO+MzCafeW6zyKlxMV8ehRZha3iWBow5pOOZDra5l9dJ X-Received: by 2002:a05:6000:1b91:b0:34c:4d98:d6f7 with SMTP id ffacd0b85a97d-354d8cc565amr2110450f8f.26.1716393621157; Wed, 22 May 2024 09:00:21 -0700 (PDT) X-Google-Smtp-Source: AGHT+IEfuleeGHEI9v8Iu/zrWS0ge+CrdIoZVA0jmvF/ETV9r/10v9dT2bd72KAoHuP4ZL+cVSkHmQ== X-Received: by 2002:a05:6000:1b91:b0:34c:4d98:d6f7 with SMTP id ffacd0b85a97d-354d8cc565amr2110410f8f.26.1716393620727; Wed, 22 May 2024 09:00:20 -0700 (PDT) Received: from ?IPV6:2001:1c00:c32:7800:5bfa:a036:83f0:f9ec? (2001-1c00-0c32-7800-5bfa-a036-83f0-f9ec.cable.dynamic.v6.ziggo.nl. [2001:1c00:c32:7800:5bfa:a036:83f0:f9ec]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-a5a1787c63bsm1798036266b.51.2024.05.22.09.00.19 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 22 May 2024 09:00:20 -0700 (PDT) Message-ID: <3aef46dc-ab15-4f29-909e-bb7500b32cf9@redhat.com> Date: Wed, 22 May 2024 18:00:19 +0200 Precedence: bulk X-Mailing-List: linux-input@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 1/2] dt-bindings: input: document Novatek NVT touchscreen controller To: Krzysztof Kozlowski , joelselvaraj.oss@gmail.com, Dmitry Torokhov , Rob Herring , Krzysztof Kozlowski , Conor Dooley Cc: linux-input@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <20240521-nvt-ts-devicetree-regulator-support-v1-0-8d766c639dca@gmail.com> <20240521-nvt-ts-devicetree-regulator-support-v1-1-8d766c639dca@gmail.com> <6f22e42d-8a06-4c24-93bd-25b6ac141cea@kernel.org> Content-Language: en-US, nl From: Hans de Goede In-Reply-To: <6f22e42d-8a06-4c24-93bd-25b6ac141cea@kernel.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Hi, On 5/21/24 6:48 PM, Krzysztof Kozlowski wrote: > On 21/05/2024 14:09, Joel Selvaraj via B4 Relay wrote: >> From: Joel Selvaraj >> >> Document the Novatek NVT touchscreen driver which is used in devices like > > driver? or device? > >> the Xiaomi Poco F1 [1]. Also, include the devictree binding file in the >> MAINTAINERS file. >> >> [1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-tianma.dts?h=v6.9 >> >> Signed-off-by: Joel Selvaraj >> --- >> .../bindings/input/touchscreen/novatek,nvt-ts.yaml | 62 ++++++++++++++++++++++ >> MAINTAINERS | 1 + >> 2 files changed, 63 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/input/touchscreen/novatek,nvt-ts.yaml b/Documentation/devicetree/bindings/input/touchscreen/novatek,nvt-ts.yaml >> new file mode 100644 >> index 0000000000000..7839c6a028e4a >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/input/touchscreen/novatek,nvt-ts.yaml >> @@ -0,0 +1,62 @@ >> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/input/touchscreen/novatek,nvt-ts.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: Novatek NVT Touchscreen Controller >> + >> +maintainers: >> + - Hans de Goede >> + >> +allOf: >> + - $ref: touchscreen.yaml# >> + >> +properties: >> + compatible: >> + enum: >> + - novatek,nvt-ts > > That's too generic. Looking at your driver change, it is not even needed. I wrote the novatek-nvt-ts driver for an Acer Iconia One 7 B1-750 tablet, this is a x86 tablet which ships with Android as factory OS and because Android OS images use kernels where everything is hardcoded the ACPI tables do not describe this touchscreen. Instead the i2c_client for the touchscreen is manually instantiated by some x86 platform glue code. Since it is manually instantiated it uses i2c_device_id binding rather then OF/ACPI. The generic "NVT-ts" i2c_device_id comes from me not knowing the controller panel type back then. In the mean time I have learned that the B1-750 uses NVT-NT11205 controller. So what I think needs to happen here is add a preparation patch as first patch to this series which basically does this: diff --git a/drivers/input/touchscreen/novatek-nvt-ts.c b/drivers/input/touchscreen/novatek-nvt-ts.c index 1a797e410a3f..224fd112b25a 100644 --- a/drivers/input/touchscreen/novatek-nvt-ts.c +++ b/drivers/input/touchscreen/novatek-nvt-ts.c @@ -278,7 +278,7 @@ static int nvt_ts_probe(struct i2c_client *client) } static const struct i2c_device_id nvt_ts_i2c_id[] = { - { "NVT-ts" }, + { "NT11205-ts" }, { } }; MODULE_DEVICE_TABLE(i2c, nvt_ts_i2c_id); diff --git a/drivers/platform/x86/x86-android-tablets/other.c b/drivers/platform/x86/x86-android-tablets/other.c index eb0e55c69dfe..5ecee6e66fb4 100644 --- a/drivers/platform/x86/x86-android-tablets/other.c +++ b/drivers/platform/x86/x86-android-tablets/other.c @@ -40,7 +40,7 @@ static const struct x86_i2c_client_info acer_b1_750_i2c_clients[] __initconst = { /* Novatek NVT-ts touchscreen */ .board_info = { - .type = "NVT-ts", + .type = "NT11205-ts", .addr = 0x34, .dev_name = "NVT-ts", }, This solves the too-generic ID problema nd can then be merged together with the rest of the series through the input tree. I'll give my ack as drivers/platform/x86 subsys maintainer for merging the x86-android-tablets change this way. >> + - novatek,nt36672a-ts > > Eh, we have already panel. Why there is a need for touchscreen binding > (binding, not driver)? I believe that the nt36672a identifier is an identifier for a novatek display assembly which contains both a DSI display panel as well as an I2C touchscreen. Since I2C devices need to be children of the I2C controller we need a separate node in the device tree for the I2c touchscreen-controller and since it is a separate node it needs it own compatible I believe ? Regards, Hans