From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ej1-f42.google.com (mail-ej1-f42.google.com [209.85.218.42]) (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 D87D7197 for ; Tue, 3 May 2022 01:10:00 +0000 (UTC) Received: by mail-ej1-f42.google.com with SMTP id dk23so30768115ejb.8 for ; Mon, 02 May 2022 18:10:00 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=Koda+Kmf6rnMD/zj5Q4WlT42MyjML9+QDJTCHNl5ArY=; b=lgXpaChoki38r237kEaHScwsef9q+RhL9t3q/EGcsmq1Nle+XST7WCWWkEQI2W7Gh9 a875yHgmv5a6UmRrnyGQVo4tIqR3NhkEXHqcM2BwsVfgljT+5wsxZkJ4vDidcBmIQASp 82E10En8ezVJwvZkCDTTdq1Pn4Hq0PqwzS+9g= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=Koda+Kmf6rnMD/zj5Q4WlT42MyjML9+QDJTCHNl5ArY=; b=Bl9WAWvz8673JRZusKHoE41Sq9zUU1W1Y5HP2TUGR/96tyokivgVw2kiVrdYT1BPtn hIiEHjcAAiKp9y7IAnfRwD0yqD2HZBmy6m4Sx5i19jK5eDbjWiQh5BEWt8K9iR0bKl+T /8e7WlwIeRHOG79AFgfze3Y3gXJr5QLJP4wGPD2I8XLJePSVO7c+tysJWblUBdyx/kMJ ckMuw/ujMuCyQ4bCoHqKyVDLZdzMSEy5BQOJlEkpPhUGcj3vAj9c211GwiEVbIm9EgH2 PPHTBTXTOyswB9IDIE0U8Tn01XkwIXATOHjVV9U6yOCpPzdUPMkOnUx7ecNlX/HreE1E W0Fg== X-Gm-Message-State: AOAM531V4b1cIQ/LED/4rWHMxv8VN2W7Jkr7p5yEsFKoKeAYO5mzR+Rj tVXXqc6580J+AdQbdecDw5NNU6cB9R/fNr/l X-Google-Smtp-Source: ABdhPJyuiQ5NxAj3PCWUu2Nn/d5Sgn4Hn3ngQosiZDz2hGVzKMLtrC22dPPxoHG11MasRoZYtSB4Bw== X-Received: by 2002:a17:906:86d3:b0:6f3:be23:7ad5 with SMTP id j19-20020a17090686d300b006f3be237ad5mr13585822ejy.264.1651540198910; Mon, 02 May 2022 18:09:58 -0700 (PDT) Received: from mail-wm1-f42.google.com (mail-wm1-f42.google.com. [209.85.128.42]) by smtp.gmail.com with ESMTPSA id gx19-20020a1709068a5300b006f3ef214e58sm4088388ejc.190.2022.05.02.18.09.57 for (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 02 May 2022 18:09:58 -0700 (PDT) Received: by mail-wm1-f42.google.com with SMTP id m2-20020a1ca302000000b003943bc63f98so469173wme.4 for ; Mon, 02 May 2022 18:09:57 -0700 (PDT) X-Received: by 2002:a05:600c:4f08:b0:391:fe3c:40e6 with SMTP id l8-20020a05600c4f0800b00391fe3c40e6mr1254978wmq.34.1651540197013; Mon, 02 May 2022 18:09:57 -0700 (PDT) Precedence: bulk X-Mailing-List: chrome-platform@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 References: <20220429233112.2851665-1-swboyd@chromium.org> <20220429233112.2851665-2-swboyd@chromium.org> In-Reply-To: From: Doug Anderson Date: Mon, 2 May 2022 18:09:45 -0700 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v2 1/2] dt-bindings: google,cros-ec-keyb: Introduce switches only compatible To: Stephen Boyd Cc: Dmitry Torokhov , LKML , patches@lists.linux.dev, chrome-platform@lists.linux.dev, Krzysztof Kozlowski , Rob Herring , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , Benson Leung , Guenter Roeck , Hsin-Yi Wang , "Joseph S. Barrera III" Content-Type: text/plain; charset="UTF-8" Hi, On Mon, May 2, 2022 at 1:41 PM Stephen Boyd wrote: > > Quoting Dmitry Torokhov (2022-05-02 10:43:06) > > On Mon, May 2, 2022 at 10:00 AM Doug Anderson wrote: > > > > > > That goes against the recently landed commit 4352e23a7ff2 ("Input: > > > cros-ec-keyb - only register keyboard if rows/columns exist") but > > > perhaps we should just _undo_ that that since it landed pretty > > > recently and say that the truly supported way to specify that you only > > > have keyboards/switches is with the compatible. > > > > > > What do you think? > > > > I am sorry, I am still confused on what exactly we are trying to solve > > here? Having a device with the new device tree somehow run an older > > kernel and fail? Why exactly do we care about this case? > > Yes, we're trying to solve the problem where a new device tree is used > with an older kernel because it doesn't have the driver patch to only > create an input device for the matrix when rows/columns properties are > present. Otherwise applying that devicetree patch to an older kernel > will break bisection. I mean, we can also just say that we don't care about breaking bisections and just say that the solution we already landed is fine. It would certainly be less work at this point. > > > We have > > implemented the notion that without rows/columns properties we will > > not be creating input device for the matrix portion, all older devices > > should have it defined, so the newer driver is compatible with them... > > > > Agreed, that solves half the problem. This new compatible eases > integration so that devicetrees can say they're compatible with the old > binding that _requires_ the rows/column properties. By making the driver > change we loosened that requirement, but the binding should have been > making the properties required at the start because it fails to bind > otherwise. > > My interpretation of what Doug is saying is that we should maintain that > requirement that rows/columns exists if the original compatible > google,cros-ec-keyb is present and use the new compatible to indicate > that there are switches. Combining the two compatibles means there's > switches and a matrix keyboard, having only the switches compatible > means only switches, and having only the keyboard compatible means only > matrix keyboard. That's not quite what I was saying. See my response to patch #2.