From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-3.6 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 57F6AC433DF for ; Sun, 23 Aug 2020 22:43:46 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 376A020720 for ; Sun, 23 Aug 2020 22:43:46 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=gmx.net header.i=@gmx.net header.b="DU452jkK" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726847AbgHWWnm (ORCPT ); Sun, 23 Aug 2020 18:43:42 -0400 Received: from mout.gmx.net ([212.227.17.21]:36209 "EHLO mout.gmx.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726737AbgHWWnk (ORCPT ); Sun, 23 Aug 2020 18:43:40 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gmx.net; s=badeba3b8450; t=1598222569; bh=PsnvQWRJcVbk0Axnfi67UKu0/qjaghfgJFgY4j2br1w=; h=X-UI-Sender-Class:Date:From:To:Cc:Subject:References:In-Reply-To; b=DU452jkKeR5UCthv13t86GkUF10DJgBTmZ4JepR2iUsiEU4rTbBJPiTJGEbI7CNN9 xy6khBALrH+zC44v9jtmEgn1r/4DdLnVaBoK7/Ndw+wsTCo6JTsASLs/xSFjCzdkfo x5XZ4zbf4ZSGjbVNFNSse+sEAxPbIJ70JliKtnhU= X-UI-Sender-Class: 01bb95c1-4bf8-414a-932a-4f6e2808ef9c Received: from longitude ([5.146.195.151]) by mail.gmx.com (mrgmx104 [212.227.17.168]) with ESMTPSA (Nemesis) id 1MZCfJ-1k5jYG2jz7-00V9WX; Mon, 24 Aug 2020 00:42:49 +0200 Date: Mon, 24 Aug 2020 00:42:36 +0200 From: Jonathan =?utf-8?Q?Neusch=C3=A4fer?= To: Andreas Kemnade Cc: Jonathan =?utf-8?Q?Neusch=C3=A4fer?= , linux-kernel@vger.kernel.org, Lee Jones , Rob Herring , Thierry Reding , Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= , Alessandro Zummo , Alexandre Belloni , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Sam Ravnborg , Linus Walleij , Heiko Stuebner , Stephan Gerhold , Lubomir Rintel , Mark Brown , allen , Mauro Carvalho Chehab , "David S. Miller" , devicetree@vger.kernel.org, linux-pwm@vger.kernel.org, linux-rtc@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Heiko Stuebner , Josua Mayer Subject: Re: [RFC PATCH 05/10] dt-bindings: pwm: Add bindings for PWM function in Netronix EC Message-ID: <20200823224236.GD103070@latitude> References: <20200620224222.1312520-1-j.neuschaefer@gmx.net> <20200620224222.1312520-4-j.neuschaefer@gmx.net> <20200621204123.6c761d98@aktux> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="tMbDGjvJuJijemkf" Content-Disposition: inline In-Reply-To: <20200621204123.6c761d98@aktux> X-Provags-ID: V03:K1:4M6N+P9avUy8KbKKUlA8iMg5xpQTk3JdbRV/3nYUHIHJQBaJETO TBsuuYAjXbbo+/cdiGz2wwz18Qv0FhZfI7KDNyURQq3W3pganq6EmBi/PeCOBUCvSPRZAMQ uFyrvJMALvK2ho5SQ3GBVDRF560iGPO610MjMc5CBjNfUPnAlTo2nba6lzuSnRhZgBuP/55 hwcmfjazvY/R6fdtBy7hQ== X-UI-Out-Filterresults: notjunk:1;V03:K0:CgZxIGWDXDA=:KJ1ztIQ8AjxuZUI0CT/wbV m1LOvjWkbI778+vtBMAheTqU6EVwjLWnsKK29jm1Qu6ngFa4WbTSVfTK7zJAyYvLY2Xqwwkt3 Pedk21HEbRRrP0zDSuFSWd3ver5/2Cvg29VLJjK1DVMrY3ymHVCYUmPlWYI7lfSoL7ge+tD6/ E6vnKja/wj8yVzX8GpELkWNrEb7AWRh2xdz5yHOhG+t9TFMiOou8PaBna1eSv36/j/hGREGUH 82ELJstkT45SN43t93N3F1sgjgvahDcqkj2pK7XCEl+MIg+i9JsDP8kDla1Ihsin8/3eRLY9z rqExUmGVTnqRBps8Ml1pXa5C77YBrvSyk/1ORItz4H6fFPPIkzKXA96CYkKGrHXz9SaMgJO+4 1xb+AyCB7daqS71Q8lmaoxtwmfkZ4cTrTWGNMjCktEP6v5r5r5ZSCag3XOR2DUmhcrob8NOe6 hjykLlmJZBvidWIXHVuTZQLqay72yEuPxrG5LY3bJlghNmJbwIgdHxHyA2izx+jfFgyKxDFcU Z+VQ2+0lsXUIo+9OJmW5dDBtDLTDd9qqb5XQEtMbZK1g74JWYcv5hPyO4OHpXN/uih/6V1rHS 2OPPMJ3e0dBNasW/a9aXx5Jo3d0tGu4Okhy8PbkzCQxMCtzkmUxWlWt5b8SQt4512vfQO/Jkp 6a73ul3b6qveTZvgfiFSFsNtSOrnSbkgL76CiXjD4FKx+zqxwl5TkMmHzb1Qkd8mbGod1vROL XP2EE7TmkANXqD9hUq/voa8yPuxra/qC6DsCfAe5vjiYD6/ThZsBD/bf+oSEtgjSRve4M9aof Lv2FkKTWgAdji5+1Maak7GFoTLVp11HLtO8Z9fCsXMlYuJQM2DfyGZGjRK9jyH6af21s/Q1NL YuTEKmoUyxCBRsd4EPF78VsyS70yljk0Q4u7OHj0M0LQCPPKyvxnjo3GmUX37oCiKhbzmsg+X YQR1XlZ1nC2CmrVPboKbeUM8M/6Rbi553J/fcp4W4i1qC2+odIUezVg4DDNi9R3tUoMnwGBxi L6BLc+2BcbN6rmYC20P9RLz4UkUdSfZYmUD/kD/ih03lnN+FfMOY9KmmW2nYQq7/8y+nCmZdF nEGj4sdqulcMoltI6uAvx02t8aYk2KBIbrTy1cx2Df5IzikP3BpcK3C4Xr50FJZMsH9YpsRVw k8V+EIPRCwwK8+FXbOOHbWVX2d6v4hzQgIwRj2TUkG1HpJtHnCLuvKcX8oIF7EXCDjIZnyZVE sQd1DKZik2FETabcrD7k3NzsWPKI63QeH3e+5UQLyFYmI6NdX75pYHvJ8zWzN9ZI2jrmaTpuE 55VhR7yqLB4VCjV4bjmjZf6ktj81JDrihgW+30Tedj20Fs5/rex46LGkbrOA3FLv45Cfu8zi7 Pu1hm0XkYHt1AVX7qdG01PHcEhATxrq1YjbJOKweAvvuFCTI9xZsa5AMD5ILpDsIPyDFN03YS 9OS4xYDDisPXPKkyEgbujSvBgwLtWUjXJDl8sQ+V9szqIVFmwjsziEtWroGdLhdQ5qJwz/Cwq UbeDOs1Wd6HHJSKpErW5PtJVaD9OaXlg1rL1MyfMKsJ71oJ9YKkXqXdKXx3doN1lrSccT0x+i AzOpVK9lwTvv+pWSkpCZuKasGQ9Ip3H0LFTmkuPAbMXwFl7CAjT1JnVPnw== Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --tMbDGjvJuJijemkf Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sun, Jun 21, 2020 at 08:41:23PM +0200, Andreas Kemnade wrote: > On Sun, 21 Jun 2020 00:42:16 +0200 > Jonathan Neusch=C3=A4fer wrote: >=20 > > The Netronix embedded controller as found in Kobo Aura and Tolino Shine > > supports one PWM channel, which is used to control the frontlight > > brightness on these devices. > >=20 > > Known problems: > > - `make dt_binding_check` shows the following warnings: > > Documentation/devicetree/bindings/mfd/netronix,ntxec.example.dts:49.1= 7-42: > > Warning (pwms_property): /example-0/backlight:pwms: cell 2 is not a > > phandle reference > > Documentation/devicetree/bindings/mfd/netronix,ntxec.example.dts:49.1= 7-42: > > Warning (pwms_property): /example-0/backlight:pwms: Could not get > > phandle node for (cell 2) > >=20 > In the tolino sources in ./drivers/misc/ntx-misc.c I find this line >=20 > if(4=3D=3DgptHWCFG->m_val.bFL_PWM) { >=20 > No idea what it does but I would expect to have a kind of translation to > a dt property? As far as I understand it, FL_PWM=3D4 means that there is a second PWM channel, in order to provide different backlight colors. I think it should be possible to simply extend the binding to list another available PWM channel, once we add support for such hardware. > > + ec_pwm: pwm { > > + compatible =3D "netronix,ntxec-pwm"; > > + #pwm-cells =3D <1>; > shouldn't that be 2? > > + }; > > }; > > }; > > + > > + backlight { > > + compatible =3D "pwm-backlight"; > > + pwms =3D <&ec_pwm 0 50000>; > since you have 2 values after the &ec_pwm=20 [...] > > +properties: > > + compatible: > > + const: netronix,ntxec-pwm > > + > > + "#pwm-cells": > > + const: 1 >=20 > shouln't that be 2? Right, I'll fix that. Thanks, Jonathan --tMbDGjvJuJijemkf Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCgAdFiEEvHAHGBBjQPVy+qvDCDBEmo7zX9sFAl9C8NQACgkQCDBEmo7z X9uNgRAAypyatRejZgWsIlkujRwQt4g+wZ9cH+A+BTje8tzoHEE5uOLhHZQrFOLU zA7FbHiGlovtkrA0ca1E8N7kuS9k4bfk7cQwyZmDDbuf6f8vTKgqlCS2TFuWgHWu lnTLO42WiM/G0eS0AUmqTuh445AH0xBw7KuQBq6/O39rW1In8qYg/9qdO4GsZRS1 uSE95Rn7nbmd1Nj0r0wrQy2O6E+a2JDv9koToSEszeuRMQPxBmHUx7YmS6EYe7+X AN/YKeQNtOmTQb6n6U46y/qH2KIp4+Ebnc0Ae5SiIrANkG6LEa5RJ0s++r4j1z4Y UMUiXKAeQL3XOX7GyNBIRK0ByX6BgPcO+ywO7Q7U3PW8mCHa3jnbTONkBT0mUwZe +xiu9mwz2u9vV6wTDiHZL0wSZ2GIcWqXNHjlxn1AkzPgAaA1Qg4BGCYBvTkkbivb fuCSusLSJHr4GHxK9Tk/nPJF/63j629ILcwp3h7ulwqc5ro+roDiMq4ADEnBTI2S GtmBnzSfvxOcN2emKrGTHqjY6MTPPXxLT5CS4z1ufASrExDu7hQ0RzDP4x4hDfCI 8FVkDFNyaUh2KhhT+0SkVqQ+Y6fdaIVVQIGfilpFiKZmeKrTZqwoYkp8tIhXePfK lx/HZuodLRdOJFh46yqADuYWZQlrYT1l+fBtdCO8fwa9L0r7bwM= =zbaI -----END PGP SIGNATURE----- --tMbDGjvJuJijemkf-- From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-3.5 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 5129FC433E1 for ; Sun, 23 Aug 2020 22:51:23 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 1476B206BE for ; Sun, 23 Aug 2020 22:51:23 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="Pfwgqdpr"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=gmx.net header.i=@gmx.net header.b="G/SeDkw+" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 1476B206BE Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=gmx.net Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Type:Cc: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=Djj2dfEyNItBFXC+T9DS81g0p9jeC8euZKZlR3+ZJG0=; b=PfwgqdprS9U1pTPefRgjI+ZnC 5fozT+mTbu6SkeUXl57Jk66PK8WW/b7OyoqhCmd8fp2iNiPDyPMYJxJl5cpBSNegiI12++AxSg+4b tVtZjrwR5t7w7jj4rYmR9FYZ/BhhEOu9/eBhp6YDTsg3KUfYdJw7AUfJyz+6wtfufMhHk+bV76Uw4 rEGX97qlHUpUAqRLH13jjQlWxwDIQYMSwPqh7JdS6+5xj+KzP3cGcCLREeVZaVoMyw8qjt5Ii2/3l OQx1yMQz6OL0bidzjnMeMZELgiJCHbkJo12xIJr7joSrcxPszvCOBZVF5KJ51DdndOQO0t8s8NQSi JRxwt0JtQ==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1k9yo4-0000eI-10; Sun, 23 Aug 2020 22:49:44 +0000 Received: from mout.gmx.net ([212.227.17.21]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1k9yo1-0000dk-MB for linux-arm-kernel@lists.infradead.org; Sun, 23 Aug 2020 22:49:42 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gmx.net; s=badeba3b8450; t=1598222966; bh=PsnvQWRJcVbk0Axnfi67UKu0/qjaghfgJFgY4j2br1w=; h=X-UI-Sender-Class:Date:From:To:Cc:Subject:References:In-Reply-To; b=G/SeDkw++FwcFRMamCh8jC2ZUGSibuHhNwaxnU4u09YAqGtg750t5AiJPVo+DO4RL vunOj73IG74L8Ql5r4sWDFkL+JGGVe1R9feAhxSGQKLlZkKM2PCE0P/vvRyrPj6FQL SO8UdYEzF2qIMb5xXP1VUjifg1PpfDrKJ96tVWRs= X-UI-Sender-Class: 01bb95c1-4bf8-414a-932a-4f6e2808ef9c Received: from longitude ([5.146.195.151]) by mail.gmx.com (mrgmx104 [212.227.17.168]) with ESMTPSA (Nemesis) id 1MZCfJ-1k5jYG2jz7-00V9WX; Mon, 24 Aug 2020 00:42:49 +0200 Date: Mon, 24 Aug 2020 00:42:36 +0200 From: Jonathan =?utf-8?Q?Neusch=C3=A4fer?= To: Andreas Kemnade Subject: Re: [RFC PATCH 05/10] dt-bindings: pwm: Add bindings for PWM function in Netronix EC Message-ID: <20200823224236.GD103070@latitude> References: <20200620224222.1312520-1-j.neuschaefer@gmx.net> <20200620224222.1312520-4-j.neuschaefer@gmx.net> <20200621204123.6c761d98@aktux> MIME-Version: 1.0 In-Reply-To: <20200621204123.6c761d98@aktux> X-Provags-ID: V03:K1:4M6N+P9avUy8KbKKUlA8iMg5xpQTk3JdbRV/3nYUHIHJQBaJETO TBsuuYAjXbbo+/cdiGz2wwz18Qv0FhZfI7KDNyURQq3W3pganq6EmBi/PeCOBUCvSPRZAMQ uFyrvJMALvK2ho5SQ3GBVDRF560iGPO610MjMc5CBjNfUPnAlTo2nba6lzuSnRhZgBuP/55 hwcmfjazvY/R6fdtBy7hQ== X-UI-Out-Filterresults: notjunk:1;V03:K0:nA1v/tZZtQw=:ZLsaiCskVvvTbmT4AvlYg5 rEOBVTxmNlQgVSQZxHty43THqcNdFgc51VjX4ipQAueFkfvDO3c524TncPg0MVqvOhiFWqJIS fqLGm4CWMS7UcV57fErCBzKU4+vPiQhrGzARo4cEn2e2UH+y+9rlmBRxCuYvUY9J/UrqgFQ3i 7YKFWipRaa39Ssjja0A8VUQPMcrOCychEck6GMZqhZ3kF3eppqpryGdjQ/JowSY9SQvJjxknh IpigE+dX13JMuT5cL+4eZ8MLYS+EQvf/5u+xfP61baBqyi1qNozpgBpOxKHPraQ0F1C4sIQv6 v6vvOwhgLh0OP7QmaANxGfFJmbODobsy+wjgaV2lv5SFQS5pwOHot/sXdwwV10r6p/ErjwPSN Knq1nh0oLb7z4FKN0n8wnl38hwtBNLaRnbJmzPXVdtp6YxuD1tLM0lbNMq0pFd+cPNAX5Sw57 AjyDeC/8QWmmyl3JHpny1Axh87ET9I/FZteLSFpE1wMeAbMf/2W0MUcOU+qt4WKjx2hFq7Imk 3yLV9UAQGZpkQzMxA0MmMVIyo7pMDMKfS6gZSbsy+l3lkCzd+qPn3dDh1+LgiVjsa+dtGImpU /UTziAx17a1G6cyazhum4ksioP/sjK7Bh0vgfHkjV5devKLlhBXZae9SCMs4H8MbeH5slxQju vtQ4OLk4wykoDJC+HLXMdM6sWN6HBX4iR18dmrcs1n/w/T+SZuKEs8MKrjHaeBv4qpuaCDciG pcRdQtqxl7f00lnZxcnlGX2T9z+U+wth+zPp2915P2ozxH7IX7kbu33wRqyG/qh3QIWR2qoyF 7nh2pt4NM8SF5PsFkohO/tRzI5iSMpXP3cSDbhp32opyf9JtWYcn54OAfv2fbpKnKrgYxcb/f lvgMRrae5fJ/uLOX+nNfkOXv7GwLPUTTyf3n4rdw8+BafOXRu+effhRYXSWSzbSW2IYTACWmz YQJVu5N1VLZCLNCEJw42DxlGIYJIKtV790I2Bi1j/U/bMX/WClN5Icc6tOHhLm6EmFZurRpl9 bBr1X5v+miWFdz1xVT1S1nG3cc1QNatnKSlrooJ7Lodz9Mfb+6s8AG+vklGPGEtJ5byn45HhT XxXwRTgsv7uWafF+2zTE8NCr7ROTikp2i5prBpt/DmTYD6SP6/HMNMTbZMGPjwxnppNXI3jth BpN6MJ8hN5/OFbb97BhprIHvEQdQ7EQHrrh6HLLgNIOcATnQEo3uT9iC2M5Pfl4PPiGz4W4oJ UNPL0wvJBqPoUV0CWQjn+BlLgykfKCJUmH8791RPM/yf4C+OINDiUree/ny79LdasIZYZIpmt mCIfR09sSfofm8HPcrFdyDBik9sdT26c41nxk24Hrg6lQIS6k4G5n9t7MalR9IrVLNJBxVukv HLHkZlxwM8l9qSG462x1OfP/U8Pn4c640fI4Tvc1PScVhsATlecU4tPiTN68yVWNtgw7z1RJY rLC84kHhj7Yh8I+iS3V+wbG0gEGY/XRN9VszwPhiFB9gHaXiei+jr8ykREvHxk8IXwTXXtTxM mzq9CnioTJCHsUE8K6MmIwNQipJ0FkLDAoUFFHOzGMKFLRA2Zu9cgNkLZPHUBLv6OfMlLEpqO EMZCODwEiywjgAM44stGyr/nrudPXfxNIoOdTwbBTJvmfkY4ccPbiGcKDg== X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200823_184941_969436_BE841E11 X-CRM114-Status: GOOD ( 24.00 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Alexandre Belloni , Heiko Stuebner , linux-pwm@vger.kernel.org, Linus Walleij , Thierry Reding , Fabio Estevam , linux-rtc@vger.kernel.org, Mauro Carvalho Chehab , Sam Ravnborg , NXP Linux Team , Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= , devicetree@vger.kernel.org, Stephan Gerhold , allen , Sascha Hauer , Jonathan =?utf-8?Q?Neusch=C3=A4fer?= , Lubomir Rintel , Rob Herring , Lee Jones , linux-arm-kernel@lists.infradead.org, Alessandro Zummo , linux-kernel@vger.kernel.org, Mark Brown , Pengutronix Kernel Team , Heiko Stuebner , Josua Mayer , Shawn Guo , "David S. Miller" Content-Type: multipart/mixed; boundary="===============2526880455188302101==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============2526880455188302101== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="tMbDGjvJuJijemkf" Content-Disposition: inline --tMbDGjvJuJijemkf Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sun, Jun 21, 2020 at 08:41:23PM +0200, Andreas Kemnade wrote: > On Sun, 21 Jun 2020 00:42:16 +0200 > Jonathan Neusch=C3=A4fer wrote: >=20 > > The Netronix embedded controller as found in Kobo Aura and Tolino Shine > > supports one PWM channel, which is used to control the frontlight > > brightness on these devices. > >=20 > > Known problems: > > - `make dt_binding_check` shows the following warnings: > > Documentation/devicetree/bindings/mfd/netronix,ntxec.example.dts:49.1= 7-42: > > Warning (pwms_property): /example-0/backlight:pwms: cell 2 is not a > > phandle reference > > Documentation/devicetree/bindings/mfd/netronix,ntxec.example.dts:49.1= 7-42: > > Warning (pwms_property): /example-0/backlight:pwms: Could not get > > phandle node for (cell 2) > >=20 > In the tolino sources in ./drivers/misc/ntx-misc.c I find this line >=20 > if(4=3D=3DgptHWCFG->m_val.bFL_PWM) { >=20 > No idea what it does but I would expect to have a kind of translation to > a dt property? As far as I understand it, FL_PWM=3D4 means that there is a second PWM channel, in order to provide different backlight colors. I think it should be possible to simply extend the binding to list another available PWM channel, once we add support for such hardware. > > + ec_pwm: pwm { > > + compatible =3D "netronix,ntxec-pwm"; > > + #pwm-cells =3D <1>; > shouldn't that be 2? > > + }; > > }; > > }; > > + > > + backlight { > > + compatible =3D "pwm-backlight"; > > + pwms =3D <&ec_pwm 0 50000>; > since you have 2 values after the &ec_pwm=20 [...] > > +properties: > > + compatible: > > + const: netronix,ntxec-pwm > > + > > + "#pwm-cells": > > + const: 1 >=20 > shouln't that be 2? Right, I'll fix that. Thanks, Jonathan --tMbDGjvJuJijemkf Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCgAdFiEEvHAHGBBjQPVy+qvDCDBEmo7zX9sFAl9C8NQACgkQCDBEmo7z X9uNgRAAypyatRejZgWsIlkujRwQt4g+wZ9cH+A+BTje8tzoHEE5uOLhHZQrFOLU zA7FbHiGlovtkrA0ca1E8N7kuS9k4bfk7cQwyZmDDbuf6f8vTKgqlCS2TFuWgHWu lnTLO42WiM/G0eS0AUmqTuh445AH0xBw7KuQBq6/O39rW1In8qYg/9qdO4GsZRS1 uSE95Rn7nbmd1Nj0r0wrQy2O6E+a2JDv9koToSEszeuRMQPxBmHUx7YmS6EYe7+X AN/YKeQNtOmTQb6n6U46y/qH2KIp4+Ebnc0Ae5SiIrANkG6LEa5RJ0s++r4j1z4Y UMUiXKAeQL3XOX7GyNBIRK0ByX6BgPcO+ywO7Q7U3PW8mCHa3jnbTONkBT0mUwZe +xiu9mwz2u9vV6wTDiHZL0wSZ2GIcWqXNHjlxn1AkzPgAaA1Qg4BGCYBvTkkbivb fuCSusLSJHr4GHxK9Tk/nPJF/63j629ILcwp3h7ulwqc5ro+roDiMq4ADEnBTI2S GtmBnzSfvxOcN2emKrGTHqjY6MTPPXxLT5CS4z1ufASrExDu7hQ0RzDP4x4hDfCI 8FVkDFNyaUh2KhhT+0SkVqQ+Y6fdaIVVQIGfilpFiKZmeKrTZqwoYkp8tIhXePfK lx/HZuodLRdOJFh46yqADuYWZQlrYT1l+fBtdCO8fwa9L0r7bwM= =zbaI -----END PGP SIGNATURE----- --tMbDGjvJuJijemkf-- --===============2526880455188302101== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============2526880455188302101==--