lkml.org 
[lkml]   [2021]   [Dec]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v6 11/28] ARM: tegra: Add device-tree for ASUS Transformer Infinity TF700T
    On Sun, Dec 12, 2021 at 12:13:55AM +0300, Dmitry Osipenko wrote:
    [...]
    > + i2c@1 {
    > + reg = <1>;
    > + #address-cells = <1>;
    > + #size-cells = <0>;
    > +
    > + dsi-bridge@7 {
    > + compatible = "toshiba,tc358768";
    > + reg = <0x7>;
    > +
    > + #address-cells = <1>;
    > + #size-cells = <0>;
    > +
    > + clocks = <&tc358768_osc>;
    > + clock-names = "refclk";
    > +
    > + reset-gpios = <&gpio TEGRA_GPIO(N, 6) GPIO_ACTIVE_LOW>;
    > +
    > + vddc-supply = <&vdd_1v2_mipi>;
    > + vddio-supply = <&vdd_1v8_vio>;
    > + vddmipi-supply = <&vdd_1v2_mipi>;
    > +
    > + ports {
    > + #address-cells = <1>;
    > + #size-cells = <0>;
    > +
    > + port@0 {
    > + reg = <0>;
    > +
    > + bridge_input: endpoint {
    > + remote-endpoint = <&dpi_output>;
    > + data-lines = <24>;
    > + };
    > + };
    > +
    > + port@1 {
    > + reg = <1>;
    > +
    > + bridge_output: endpoint {
    > + remote-endpoint = <&panel_input>;
    > + };
    > + };
    > + };
    > +
    > + /*
    > + * Panasonic VVX10F004B00 or HYDIS HV101WU1-1E1
    > + * LCD SuperIPS+ Full HD panel.
    > + */
    > + panel@1 {
    > + compatible = "panasonic,vvx10f004b00";
    > + reg = <1>;
    > +
    > + power-supply = <&vdd_pnl>;
    > + backlight = <&backlight>;
    > +
    > + port {
    > + panel_input: endpoint {
    > + remote-endpoint = <&bridge_output>;
    > + };
    > + };
    > + };

    make dtbs_check complains about this and says that panel@1 (as well as
    #address-cells and #size-cells) are not allowed here. And indeed the
    binding for the Toshiba bridge doesn't mention them here.

    Do we need this here or should this be moved to the top level to fix
    those warnings? I guess what you're doing above is describe a DSI bus
    created by the DSI bridge, which also makes sense, so another
    alternative would be to fix up the binding and let it accept those
    properties.

    Thierry
    [unhandled content-type:application/pgp-signature]
    \
     
     \ /
      Last update: 2021-12-15 15:02    [W:2.272 / U:0.348 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site