On Wed Jun 25, 2025 at 4:38 PM CEST, Konrad Dybcio wrote:
> On 6/25/25 11:23 AM, Luca Weiss wrote:
>> Add a devicetree for The Fairphone (Gen. 6) smartphone, which is based
>> on the SM7635 SoC.
>
> [...]
>
>> + /* Dummy panel for simple-framebuffer dimension info */
>> + panel: panel {
>> + compatible = "boe,bj631jhm-t71-d900";
>> + width-mm = <65>;
>> + height-mm = <146>;
>> + };
>
> I haven't ran through all the prerequisite-xx-id, but have
> you submitted a binding for this?
Actually not, kind of forgot about this. I believe I can create a
(mostly?) complete binding for the panel, but this simple description
for only width-mm & height-mm will differ from the final one, which will
have the DSI port, pinctrl, reset-gpios and various supplies.
I think I'll just drop it from v2 and keep it locally only, to get the
simpledrm scaling right.
>
> [...]
>
>> + reserved-memory {
>> + /*
>> + * ABL is powering down display and controller if this node is
>> + * not named exactly "splash_region".
>> + */
>> + splash_region@e3940000 {
>> + reg = <0x0 0xe3940000 0x0 0x2b00000>;
>> + no-map;
>> + };
>> + };
>
> :/ maybe we can convince ABL not to do it..
Yes, we talked about that. I will look into getting "splash-region" and
"splash" also into the ABL (edk2) build for the phone. Still won't
resolve that for any other brand of devices.
>
> [...]
>
>> + vreg_l12b: ldo12 {
>> + regulator-name = "vreg_l12b";
>> + /*
>> + * Skip voltage voting for UFS VCC.
>> + */
>
> Why so?
