lkml.org 
[lkml]   [2021]   [Mar]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: [RFC PATCH 1/3] dt-bindings: display: simple: Add the panel on sc7180-trogdor-pompom
    Hi,

    On Fri, Mar 26, 2021 at 5:38 AM Thierry Reding <thierry.reding@gmail.com> wrote:
    >
    > The point remains that unless we describe exactly which panel we're
    > dealing with, we ultimately have no way of properly quirking anything if
    > we ever have to.

    Just to clarify here: with my initial proposal we actually could still
    quirk things if we had to. If the quirk needed to be applied before
    power on we'd just have to apply the quirk to the whole board (which
    we'd have to do anyway). After the panel was powered on then we could
    read the EDID and apply a quirk based on what the EDID tells us,
    right?


    > Also, once we allow this kind of wildcard we can
    > suddenly get into a situation where people might want to reuse this on
    > something that's not at all a google-pompom board because the same
    > particular power sequence happens to work on on some other board.

    That's a legit concern. Of course, people could already do that with
    existing panels right? One would also hope that if they reused this
    they also used the "more specific to least specific" rule, so someone
    could reuse (without any problems) with:

    compatible = "some-other-company,some-other-board-panel", "google,pompom-panel"

    That doesn't seem like it would be terrible.


    > Similarly I can imagine a situation where we could now have the same
    > panel supported by multiple different wildcard compatible strings. How
    > is that supposed to be any cleaner than what we have now?

    I'm tempted to call this (same panel supported by multiple different
    compatible strings) a feature, actually. Specifically:

    Even if the exact same hardware is shipped with more than one board,
    it may have a different EDID programmed into it. From what I've seen
    the timings used on a panel may need to be adjusted based on the SoC
    used (and what clock rates it can provide / features of the underlying
    display driver), EMI concerns, and power consumption concerns. Once a
    different EDID is programmed in it then it sorta becomes a "different"
    panel, right? I think sometimes (?) panel vendors assign a slightly
    different model number per board, but I'm not sure.


    -Doug

    \
     
     \ /
      Last update: 2021-03-29 18:09    [W:5.748 / U:0.040 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site