From: Konrad Dybcio <konrad.dybcio@somainline.org> To: Rob Herring <robh@kernel.org> Cc: ~postmarketos/upstreaming@lists.sr.ht, martin.botka@somainline.org, angelogioacchino.delregno@somainline.org, marijn.suijten@somainline.org, jamipkettunen@somainline.org, Thierry Reding <thierry.reding@gmail.com>, Sam Ravnborg <sam@ravnborg.org>, David Airlie <airlied@linux.ie>, Daniel Vetter <daniel@ffwll.ch>, dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/2] dt-bindings: Add SONY Synaptics JDI panel Date: Sat, 5 Jun 2021 13:14:59 +0200 [thread overview] Message-ID: <f68df9d6-7d17-12d0-5d4f-686482b9185e@somainline.org> (raw) In-Reply-To: <20210604201841.GA3834184@robh.at.kernel.org> >> + >> +maintainers: >> + - Konrad Dybcio <konrad.dybcio@somainline.org> >> + >> +description: |+ > Do you need the formatting? If not, drop '|+'. I'm not sure whether I do, so I guess dropping it is fine. >> + This panel seems to only be found in SONY Xperia >> + X, X Performance, X Compact, XZ and XZs >> + smartphones and we have no straightforward way of >> + actually getting the correct model number, >> + as no schematics are released publicly. > Odd choice of line break length. 80 char please. Right. >> + >> + preset-gpio: >> + description: panel reset pin > What's reset-gpios then? reset-gpios is an overlook on my side :) >> + >> + pvddio-gpio: >> + description: panel vddio pin >> + >> + treset-gpio: >> + description: touch reset pin > Use '-gpios' > > And need to define how many (maxItems: 1). Right. >> + #include <dt-bindings/gpio/gpio.h> >> + >> + dsi { >> + #address-cells = <1>; >> + #size-cells = <0>; >> + panel: panel@0 { >> + reg = <0>; >> + >> + pvddio-gpio = <&tlmm 51 GPIO_ACTIVE_HIGH>; >> + preset-gpio = <&tlmm 8 GPIO_ACTIVE_HIGH>; >> + treset-gpio = <&tlmm 89 GPIO_ACTIVE_HIGH>; >> + >> + vddio-supply = <&pm8994_s4>; >> + avdd-supply = <&pm8994_l2>; >> + tvdd-supply = <&panel_tvdd>; >> + >> + backlight = <&pmi8994_wled>; >> + >> + port { >> + panel_in: endpoint { >> + remote-endpoint = <&dsi0_out>; >> + }; > Consistent indentation please. 4 spaces is good. Another mistake, ack. I'll send a v2 shortly. Konrad
WARNING: multiple messages have this Message-ID (diff)
From: Konrad Dybcio <konrad.dybcio@somainline.org> To: Rob Herring <robh@kernel.org> Cc: devicetree@vger.kernel.org, David Airlie <airlied@linux.ie>, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, jamipkettunen@somainline.org, Thierry Reding <thierry.reding@gmail.com>, martin.botka@somainline.org, ~postmarketos/upstreaming@lists.sr.ht, angelogioacchino.delregno@somainline.org, marijn.suijten@somainline.org, Sam Ravnborg <sam@ravnborg.org> Subject: Re: [PATCH 1/2] dt-bindings: Add SONY Synaptics JDI panel Date: Sat, 5 Jun 2021 13:14:59 +0200 [thread overview] Message-ID: <f68df9d6-7d17-12d0-5d4f-686482b9185e@somainline.org> (raw) In-Reply-To: <20210604201841.GA3834184@robh.at.kernel.org> >> + >> +maintainers: >> + - Konrad Dybcio <konrad.dybcio@somainline.org> >> + >> +description: |+ > Do you need the formatting? If not, drop '|+'. I'm not sure whether I do, so I guess dropping it is fine. >> + This panel seems to only be found in SONY Xperia >> + X, X Performance, X Compact, XZ and XZs >> + smartphones and we have no straightforward way of >> + actually getting the correct model number, >> + as no schematics are released publicly. > Odd choice of line break length. 80 char please. Right. >> + >> + preset-gpio: >> + description: panel reset pin > What's reset-gpios then? reset-gpios is an overlook on my side :) >> + >> + pvddio-gpio: >> + description: panel vddio pin >> + >> + treset-gpio: >> + description: touch reset pin > Use '-gpios' > > And need to define how many (maxItems: 1). Right. >> + #include <dt-bindings/gpio/gpio.h> >> + >> + dsi { >> + #address-cells = <1>; >> + #size-cells = <0>; >> + panel: panel@0 { >> + reg = <0>; >> + >> + pvddio-gpio = <&tlmm 51 GPIO_ACTIVE_HIGH>; >> + preset-gpio = <&tlmm 8 GPIO_ACTIVE_HIGH>; >> + treset-gpio = <&tlmm 89 GPIO_ACTIVE_HIGH>; >> + >> + vddio-supply = <&pm8994_s4>; >> + avdd-supply = <&pm8994_l2>; >> + tvdd-supply = <&panel_tvdd>; >> + >> + backlight = <&pmi8994_wled>; >> + >> + port { >> + panel_in: endpoint { >> + remote-endpoint = <&dsi0_out>; >> + }; > Consistent indentation please. 4 spaces is good. Another mistake, ack. I'll send a v2 shortly. Konrad
next prev parent reply other threads:[~2021-06-05 11:15 UTC|newest] Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-05-25 11:31 [PATCH 1/2] dt-bindings: Add SONY Synaptics JDI panel Konrad Dybcio 2021-05-25 11:31 ` Konrad Dybcio 2021-05-25 11:31 ` [PATCH 2/2] drm/panel: Add support for SONY JDI Synaptics panel Konrad Dybcio 2021-05-25 11:31 ` Konrad Dybcio 2021-06-04 20:18 ` [PATCH 1/2] dt-bindings: Add SONY Synaptics JDI panel Rob Herring 2021-06-04 20:18 ` Rob Herring 2021-06-05 11:14 ` Konrad Dybcio [this message] 2021-06-05 11:14 ` Konrad Dybcio
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=f68df9d6-7d17-12d0-5d4f-686482b9185e@somainline.org \ --to=konrad.dybcio@somainline.org \ --cc=airlied@linux.ie \ --cc=angelogioacchino.delregno@somainline.org \ --cc=daniel@ffwll.ch \ --cc=devicetree@vger.kernel.org \ --cc=dri-devel@lists.freedesktop.org \ --cc=jamipkettunen@somainline.org \ --cc=linux-kernel@vger.kernel.org \ --cc=marijn.suijten@somainline.org \ --cc=martin.botka@somainline.org \ --cc=robh@kernel.org \ --cc=sam@ravnborg.org \ --cc=thierry.reding@gmail.com \ --cc=~postmarketos/upstreaming@lists.sr.ht \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.