Quoting Dmitry Baryshkov (2022-08-22 11:46:39) > On 12/07/2022 02:16, Rob Herring wrote: > > > > But this is the wrong location for 'data-lanes'. It belongs in an > > endpoint node. > > I rechecked the existing device trees (sc7280-herobrine.dtsi). The > data-lanes are already inside the main dp controller node. I'll take a > glance on fixing the driver to check the dp_out endpoint (and update > existing DT to move the property to the endpoint node), but to make > schema compatible with the existing device trees we'd probably still > need this property (which can be marked as deprecated once the proper > endpoint property is supported). Does that sound plausible? It would be nice if drm_of_get_data_lanes_count() took some port and endpoint number instead of a node pointer directly. Then you couldn't mess this up as easily.
