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=-10.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=ham 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 6886DC48BCF for ; Wed, 9 Jun 2021 11:05:23 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 4B47E6139A for ; Wed, 9 Jun 2021 11:05:23 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236875AbhFILHQ (ORCPT ); Wed, 9 Jun 2021 07:07:16 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49448 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232165AbhFILHP (ORCPT ); Wed, 9 Jun 2021 07:07:15 -0400 Received: from mail-lj1-x22d.google.com (mail-lj1-x22d.google.com [IPv6:2a00:1450:4864:20::22d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5290EC061574 for ; Wed, 9 Jun 2021 04:05:21 -0700 (PDT) Received: by mail-lj1-x22d.google.com with SMTP id c11so31125924ljd.6 for ; Wed, 09 Jun 2021 04:05:21 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=VePCktjSiUwy7zF0X3i1kiD5uKEWitdsZcvT2ZeLJmk=; b=lCebyaKzT4o+/UcvZ1XICOmUN7/NHBX1soF6VpaCp/TCvgmk56jULiWBAvNk3i6Loj z5wJnqwqEEUANW+u3ygdh5BEaWLy9N8xhJu5CTPM9oqJnYdl30CwZZWEux+H+KOeGDDI ZwxIVZKMULaK7VsCRoihYTZ7UtCWnM65PGo6NHphuP4WS/bf4zw9cd5uz3z8exwr9+E+ E0IPZfYfPD1onxv89/78rZD4LBFvbPBKfdvrtQUSz1tXKO9FIUkI8KHkdhFzDrpC0hZk 00RJQFVtqyf2oOP2/m8/mM9pqWN6bbJzWLMo5fac27BiLh4WA2TgtZvcRfZEggy9XQoF IdAA== 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=VePCktjSiUwy7zF0X3i1kiD5uKEWitdsZcvT2ZeLJmk=; b=K9EdSbZ6Rpt9IcTeGvswtepOjRDdpdkqO33xSLcTBFcM2cmxhZtyhp5fYYd7QPzAkN 7nPatuO1xO6fxytVuYB5lOU3kWLfCRRQKA2UNYwSfUxz6FWdJ7bE7m/ubyn+jj5DHqy0 LQFSKUqBY4v+ujGhYJCjDGlKtAqGN5xXX2CHJmqT8F61ncYY//3aty4bSIHXbrjE9aSw 6UDPfCdmMqQxibpjqDIP7NCJKKctQJGOfgpeHxAijAD5bvlzoVUMxp7JbBVZuu/QImDR dWEQFpZBl2mJ2o4kwS8g+QDDzT+Xw5t75aFLJPUI7I0GKrZLH1vKxpXjZajeCXmXdPz0 tN9Q== X-Gm-Message-State: AOAM53165KwOyAy8/NIa86e70PObNmgV64ezNdkmfVTzvSiCgP8bvaHJ o/xM61CA2c2Qh8qc4EZWjFqkpkDTQacDaOhIZkoSOA== X-Google-Smtp-Source: ABdhPJybvThxOzTqKAq2tM0a2YQzhSX2mOBRONdPJcNN5UbQix2w4ogjAjPGxmyqXPR5egn25taCVXJw2O2j5xJPZ+4= X-Received: by 2002:a05:651c:4c6:: with SMTP id e6mr21901557lji.326.1623236719658; Wed, 09 Jun 2021 04:05:19 -0700 (PDT) MIME-Version: 1.0 References: <20210607170555.4006050-1-dianders@chromium.org> <20210607100234.v9.2.Id3c048d22e72a9f90084a543b5b4e3f43bc9ab62@changeid> In-Reply-To: <20210607100234.v9.2.Id3c048d22e72a9f90084a543b5b4e3f43bc9ab62@changeid> From: Linus Walleij Date: Wed, 9 Jun 2021 13:05:08 +0200 Message-ID: Subject: Re: [PATCH v9 02/11] dt-bindings: drm: Introduce the DP AUX bus To: Douglas Anderson Cc: Andrzej Hajda , Neil Armstrong , Laurent Pinchart , Jonas Karlman , Sam Ravnborg , Rob Clark , Maarten Lankhorst , Stanislav Lisovskiy , Steev Klimaszewski , MSM , Bjorn Andersson , Thierry Reding , Lyude Paul , "open list:DRM PANEL DRIVERS" , Stephen Boyd , Rob Herring , Daniel Vetter , David Airlie , Rob Herring , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , linux-kernel Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org On Mon, Jun 7, 2021 at 7:06 PM Douglas Anderson wrote: > We want to be able to list an eDP panel as a child of an eDP > controller node to represent the fact that the panel is connected to > the controller's DP AUX bus. Though the panel and the controller are > connected in several ways, the DP AUX bus is the primary control > interface between the two and thus makes the most sense to model in > device tree hierarchy. > > Listing a panel in this way makes it possible for the panel driver to > easily get access to the DP AUX bus that it resides on, which can be > useful to help in auto-detecting the panel and for turning on various > bits. > > NOTE: historically eDP panels were _not_ listed under their controller > but were listed at the top level of the device tree. This will still > be supported for backward compatibility (and while DP controller > drivers are adapted to support the new DT syntax) but should be > considered deprecated since there is no downside to listing the panel > under the controller. > > For now, the DP AUX bus bindings will only support an eDP panel > underneath. It's possible it could be extended to allow having a DP > connector under it in the future. > > NOTE: there is no "Example" in this bindings file. Yikes! This avoids > duplicating the same example lots of places. See users of the aux bus > (like ti-sn65dsi86) for examples. > > The idea for this bus's design was hashed out over IRC [1]. > > [1] https://people.freedesktop.org/~cbrill/dri-log/?channel=dri-devel&date=2021-05-11 > > Signed-off-by: Douglas Anderson > Reviewed-by: Rob Herring I overall like this approach a lot: Reviewed-by: Linus Walleij Yours, Linus Walleij