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=-6.1 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,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 0E247C433F5 for ; Thu, 2 Sep 2021 22:33:23 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id D5A4B6109E for ; Thu, 2 Sep 2021 22:33:22 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1347560AbhIBWeV (ORCPT ); Thu, 2 Sep 2021 18:34:21 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:48612 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S239498AbhIBWeU (ORCPT ); Thu, 2 Sep 2021 18:34:20 -0400 Received: from mail-io1-xd2d.google.com (mail-io1-xd2d.google.com [IPv6:2607:f8b0:4864:20::d2d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id BCEB3C061760 for ; Thu, 2 Sep 2021 15:33:21 -0700 (PDT) Received: by mail-io1-xd2d.google.com with SMTP id y18so4577866ioc.1 for ; Thu, 02 Sep 2021 15:33:21 -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=vEOAhsHQM9G+tVvC78A/rhgecetN4byQdhrLLpo0P0g=; b=LDmJnOglDM6SRfrqk879nlvltfZEhwju/ts9Exu/2rE0XB+Lj74aGi7ZCLry1NSj0f btPQfuDa+Q2ntfjL+zK+k6leKrLg6ahv0/pjTn7Y3JJbbdq5v55q2XUGEdXr0VDSURWz zo/R1rY1OagPqlvLvu4Emr9cTHgcwBCjlU/VE= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=vEOAhsHQM9G+tVvC78A/rhgecetN4byQdhrLLpo0P0g=; b=NToPmxgR9sfTM6jCaxWvVk3CMt+p5cVpTR2k5/G+iI8sTEaadEyOsQ3YTeFam5eGdE wnnHtLhTeBCGmQZm0ySX6Rlp+nh3b22wDeCH6exHYtAp955bPltdODNFiwEhlmDQj3HP mLcU9XaG/ahCb39zReVjJad4OYIweXuj6FxnsHFHKn1ZEh8CvZ758kS4nB7SpFHhVkcQ CrFKu6Sbe+bio3UHDEFmg7FUtw448IG3DyojvdUsGUspD2wxNtY/PdzNqaDm2+z3HLOF vS7z7dOjXRTG5T4QmqdcJTaGhAAKu80o6jrSuBbq8I/leBkROOPlr4f1Irs+GDty9KYY T16Q== X-Gm-Message-State: AOAM531f5RIcRRkTd9tMF9RY6ZZJ9hJRJ2ebh83mr6AtBPV16GQQce44 iDKgZ7Qoo9XITaeCMbSum0gyhFUi4ZG9sA== X-Google-Smtp-Source: ABdhPJw8Ppy1OEkEk9FvrlyMmFjPnpNGbC+/lb5dJp/ARx7c246O4MKVQNjYO3GCOIF4ngkDFr604w== X-Received: by 2002:a6b:7519:: with SMTP id l25mr481064ioh.169.1630622000646; Thu, 02 Sep 2021 15:33:20 -0700 (PDT) Received: from mail-il1-f170.google.com (mail-il1-f170.google.com. [209.85.166.170]) by smtp.gmail.com with ESMTPSA id q15sm1676301ilm.60.2021.09.02.15.33.19 for (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 02 Sep 2021 15:33:20 -0700 (PDT) Received: by mail-il1-f170.google.com with SMTP id a1so3425024ilj.6 for ; Thu, 02 Sep 2021 15:33:19 -0700 (PDT) X-Received: by 2002:a92:cf4a:: with SMTP id c10mr329106ilr.269.1630621999435; Thu, 02 Sep 2021 15:33:19 -0700 (PDT) MIME-Version: 1.0 References: <20210901201934.1084250-1-dianders@chromium.org> In-Reply-To: From: Doug Anderson Date: Thu, 2 Sep 2021 15:33:07 -0700 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v3 00/16] eDP: Support probing eDP panels dynamically instead of hardcoding To: Andrzej Hajda Cc: Thierry Reding , Rob Herring , Sam Ravnborg , linux-arm-msm , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , Maxime Ripard , dri-devel , NXP Linux Team , Linux ARM , LKML , linux-mips , linux-omap , Linux-Renesas , linux-samsung-soc , linux-sunxi@lists.linux.dev, "open list:TEGRA ARCHITECTURE SUPPORT" , =?UTF-8?Q?=C5=81ukasz_Stelmach?= Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-samsung-soc@vger.kernel.org Hi, On Thu, Sep 2, 2021 at 3:10 PM Andrzej Hajda wrote: > > Removed most CC: SMTP server protested. Yeah, it was because of the dang defconfig patches. My general policy is to send the cover letter to everyone even if not everyone gets CCed on all patches, but it ended up as a lot... Speaking of which, I'd definitely be interested in your take on the defconfig topic: https://lore.kernel.org/r/CAD=FV=WPXAUyuAHb1jKx9F_aw+JGX4MWB3or=Eq5rXoKY=OQMw@mail.gmail.com Do you agree with Olof that I should set the "default" for the new config to be the same as the old config? It doesn't make sense going forward but it helps for anyone with old configs... > On 01.09.2021 22:19, Douglas Anderson wrote: > > The goal of this patch series is to move away from hardcoding exact > > eDP panels in device tree files. As discussed in the various patches > > in this series (I'm not repeating everything here), most eDP panels > > are 99% probable and we can get that last 1% by allowing two "power > > up" delays to be specified in the device tree file and then using the > > panel ID (found in the EDID) to look up additional power sequencing > > delays for the panel. > > > > This patch series is the logical contiunation of a previous patch > > series where I proposed solving this problem by adding a > > board-specific compatible string [1]. In the discussion that followed > > it sounded like people were open to something like the solution > > proposed in this new series. > > > > In version 2 I got rid of the idea that we could have a "fallback" > > compatible string that we'd use if we didn't recognize the ID in the > > EDID. This simplifies the bindings a lot and the implementation > > somewhat. As a result of not having a "fallback", though, I'm not > > confident in transitioning any existing boards over to this since > > we'll have to fallback to very conservative timings if we don't > > recognize the ID from the EDID and I can't guarantee that I've seen > > every panel that might have shipped on an existing product. The plan > > is to use "edp-panel" only on new boards or new revisions of old > > boards where we can guarantee that every EDID that ships out of the > > factory has an ID in the table. > > > > Version 3 of this series now splits out all eDP panels to their own > > driver and adds the generic eDP panel support to this new driver. I > > believe this is what Sam was looking for [2]. > > > > [1] https://lore.kernel.org/r/YFKQaXOmOwYyeqvM@google.com/ > > [2] https://lore.kernel.org/r/YRTsFNTn%2FT8fLxyB@ravnborg.org/ > > > I like the idea - if something can be configured dynamically lets do it. > But I have few questions: > 1. Have you read different real panels id's? In many cases (MIPI DSI, > LVDS with EDID) manufacturers often forgot to set proper id fields. I do > not know how EDID's ids are reliable in case of edp panels. I have read several and (so far) they have been quite reliable but I will admit that I haven't done an exhaustive sample. I guess my answer to whether we can trust it is: a) Presumably you'd only use this new "edp-panel" compatible on systems whose panel IDs are known to be reliable. Old systems can keep determining the panel by compatible string. The two schemes can co-exist. b) As we start using this new scheme then there will be more validation of panel ID strings and, presumably, they will become more reliable. It is still true that ID conflicts could exist. A vendor could ship two different panels with the same ID and maybe nobody would notice because they weren't ever hooked up to the same board. In that case we'd have a question of what to do upstream. If this really happens then I suppose (worst case) we could use the device tree to help differentiate and each board could say that "panel ID hooked up to this board is actually panel ". I hope we don't have to do that because it feels dirty, but at least it gives us _something_ if we get backed into a corner. > 2. You are working with edp panels - beside EDID they have DPCD which > contains things like IEEE_OUI and "Device Identification String", I > guess they could be also used for detecting panels, have you considered > it? I think DPCD Id should be assigned to EDP-Sink interface, and EDID > Id to the actual panel behind it. With this assumption one could > consider which timings should be property of which entity. This could be another way to help us if we're backed into a corner in the future. Right now the driver requires that you give access to a full eDP AUX bus to use the "generic eDP" panel support even though it currently only relies on the EDID, so it would be pretty easy to utilize this info in the future. So far the ID has been reliable for me though. -Doug