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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id D801EC71153 for ; Mon, 4 Sep 2023 08:19:53 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1350555AbjIDITz (ORCPT ); Mon, 4 Sep 2023 04:19:55 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:38956 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229883AbjIDITz (ORCPT ); Mon, 4 Sep 2023 04:19:55 -0400 Received: from mail-ej1-x636.google.com (mail-ej1-x636.google.com [IPv6:2a00:1450:4864:20::636]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 493F4FD for ; Mon, 4 Sep 2023 01:19:51 -0700 (PDT) Received: by mail-ej1-x636.google.com with SMTP id a640c23a62f3a-9a2a4a5472dso438597766b.1 for ; Mon, 04 Sep 2023 01:19:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fairphone.com; s=fair; t=1693815590; x=1694420390; darn=vger.kernel.org; h=in-reply-to:references:to:from:subject:cc:message-id:date :content-transfer-encoding:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=4SqCPlUACmzuR897Jw/2/k1NSEPnLNm8tSMQQYrDWME=; b=sCURjlIECG75NChDPAFgJtYUliKIkBlB5lHdZQsZ03Nz7U91Yl93d2PkC76eTQNQk/ y0slJPkhPVfgWNEAicJ2mbdQB0OnQhmscuPWuv5ky5okEf7xFID7n7oroxJrHDZBO0sq 3FsW5kDCZHGOb/9qkERTek0aJxrchdLiTtpNQL3rvxN9b9O2xGfZePu1+Jc6RufwuLlR DbA1aI0VzXXBXSQjBtLWvcSjhj7kDTNZFpHDoGX2YKkyIs43Wxps+LCXh7azfN0k0Zqr qHNbnaC0hiDziup3gy7Oq38fzCrnTcnC7TJyESBQxbhLMtqlj7Kj0IJPDvwzP2V4Ly4B n45w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1693815590; x=1694420390; h=in-reply-to:references:to:from:subject:cc:message-id:date :content-transfer-encoding:mime-version:x-gm-message-state:from:to :cc:subject:date:message-id:reply-to; bh=4SqCPlUACmzuR897Jw/2/k1NSEPnLNm8tSMQQYrDWME=; b=BqgXzzJ3x/hjzjFZIeJmwY0/AI16X3EuI5h4yDphbywidTwVlL2bwpj30D16gUsBOG Ra1dPMNCYwazGmUx3C/vzWXAWVV9s2+Gzz7H5JvWJNR8o3PRZmf/T7quHT3VDHfV/jlw 86897plP8GTK0Y32wDl9co6lMsV3MAc/0ZEl8bWtpK5Dq6VEq/u58f+uLYrWvnyWDcVF VsFLP87Ypyyu1Q+DVgDGoUlHWVAASbKRQom/1UMg5Xz93Afg8tQwOB6Q5p091l5SpzAT m67PE50awyGf9GamOtgkDdGabmvmslrNq+E98q0JPSj5clreqJLZHnwwDlEtxhCt8M7o nS0g== X-Gm-Message-State: AOJu0Yyuc2jAXzZ3CShgUyoGCJCswAr+A8rIhvzoR1EHrs6OXOYjcYTY zgCfa63akbGdeI1iutetwkURdWNIxtJLsOfnzWAkgA== X-Google-Smtp-Source: AGHT+IFdJLo8jZ3klMSQBPs2qIeMjRwZjzw3rOcgPXXM7tKUsmLDAMqJLlkkomMSQn9J8vCqgpfIIg== X-Received: by 2002:a17:906:76c9:b0:9a1:fc1e:19b8 with SMTP id q9-20020a17090676c900b009a1fc1e19b8mr9748830ejn.29.1693815589734; Mon, 04 Sep 2023 01:19:49 -0700 (PDT) Received: from localhost (144-178-202-138.static.ef-service.nl. [144.178.202.138]) by smtp.gmail.com with ESMTPSA id le18-20020a170906ae1200b0099bcbaa242asm5757648ejb.9.2023.09.04.01.19.49 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 04 Sep 2023 01:19:49 -0700 (PDT) Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Date: Mon, 04 Sep 2023 10:19:49 +0200 Message-Id: Cc: <~postmarketos/upstreaming@lists.sr.ht>, , , , , , Subject: Re: [PATCH 11/11] arm64: dts: qcom: qcm6490: Add device-tree for Fairphone 5 From: "Luca Weiss" To: "Konrad Dybcio" , , "Andy Gross" , "Bjorn Andersson" , "Rob Herring" , "Krzysztof Kozlowski" , "Conor Dooley" , "Srinivas Kandagatla" , "Linus Walleij" , "Rafael J. Wysocki" , "Viresh Kumar" X-Mailer: aerc 0.15.2 References: <20230830-fp5-initial-v1-0-5a954519bbad@fairphone.com> <20230830-fp5-initial-v1-11-5a954519bbad@fairphone.com> <72c2fd33-613b-49be-b394-0663f459f0c5@linaro.org> <5f05c76e-e74e-4954-a729-3ed2a1d33605@linaro.org> In-Reply-To: <5f05c76e-e74e-4954-a729-3ed2a1d33605@linaro.org> Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On Sat Sep 2, 2023 at 1:45 PM CEST, Konrad Dybcio wrote: > On 1.09.2023 16:27, Luca Weiss wrote: > > On Wed Aug 30, 2023 at 12:45 PM CEST, Konrad Dybcio wrote: > >> On 30.08.2023 11:58, Luca Weiss wrote: > >>> Add device tree for the Fairphone 5 smartphone which is based on > >>> the QCM6490 SoC. > >>> > >>> Supported features are, as of now: > >>> * Bluetooth > >>> * Debug UART > >>> * Display via simplefb > >>> * Flash/torch LED > >>> * Flip cover sensor > >>> * Power & volume buttons > >>> * RTC > >>> * SD card > >>> * USB > >>> * Various plumbing like regulators, i2c, spi, etc > >>> > >>> Signed-off-by: Luca Weiss > >>> --- > [...] > > >>> + panel: panel { > >>> + compatible =3D "boe,rm692e5"; > >> Undocumented compatbile > >=20 > > Not quite sure how to document this properly. This dummy panel node is > > used for the simplefb/simpledrm so that user space can get panel > > dimensions already. > >=20 > > This approach of a separate panel node was apparently preferred when th= e > > functionality was upstreamed but of course as can be seen this panel > > node is very barebones where many properties are not present which woul= d > > be needed with the full panel driver (once it's working). > >=20 > > Also for example for the bindings I think width-mm and height-mm won't > > be needed anymore since that information will be provided by the driver > > then. > >=20 > > Please let me know how/where to document this. > As discussed offline, the workflow here would be to oneOf: (wink) > > - wait until there's a proper driver and create a binding based on > what you know (because you have a working driver and can test it) > is necessary for it to function > > - create the binding for that display panel + driver ic combo in > advance and pray that whatever you put there will be enough when > you take upon yourself to write the driver > > I'd suggest dropping these properties (or keeping them downstream or > something) for now, the display should not be terribly hard to bring > up properly, let's hope that can be done soon! Yeah that sounds like a solution for now. But for other devices I hope something can be figured out so that these dummy panels can go into upstream dts. Otherwise the functionality added in the simplefb driver is kinda useless for pure upstream if noone's allowed to have incomplete bindings for providing the panel dimensions. > > > I'm aware, but will work on this later since it's not useful without > > USB-C role switching and working audio. > You can still peek at it in like sysfs or so, but up to you. Yes, later ;) Regards Luca > > > Konrad