On 4/27/20 4:00 PM, Patrick DELAUNAY wrote:
> Dear Marek,
>
>> From: Marek Vasut <ma...@denx.de>
>> Sent: lundi 27 avril 2020 12:30
>>
>> On 4/24/20 4:31 PM, Patrick DELAUNAY wrote:
>>> Dear Marek,
>>>
>>>> From: Marek Vasut <ma...@denx.de>
>>>> Sent: mardi 31 mars 2020 19:52
>>>>
>>>> The AV96 is in fact an assembly of DH Electronics DHCOR SoM on top of
>>>> an
>>>> AV96 reference board. Split the DTs to reflect that and make sure to
>>>> DHCOR SoM can be reused on other boards easily.
>>>>
>>>> It is also highly recommended to configure the board for the DHCOM
>>>> make stm32mp15_dhcom_basic_defconfig make
>>>> DEVICE_TREE=stm32mp15xx-dhcor-
>>>> avenger96
>>>> as that permits reusing the board code for the DH components, like
>>>> accessing and reading out the ethernet MAC from EEPROM.
>>>>
>>>> Signed-off-by: Marek Vasut <ma...@denx.de>
>>>> Cc: Manivannan Sadhasivam <manivannan.sadhasi...@linaro.org>
>>>> Cc: Patrick Delaunay <patrick.delau...@st.com>
>>>> Cc: Patrice Chotard <patrice.chot...@st.com>
>>>> ---
>>>> V2: No change
>>>> V3: No change
>>>> ---
>>>
>>> Applied to u-boot-stm/master, with few modifications:
>>>
>>> arch/arm/dts/stm32mp15xx-dhcor-u-boot.dtsi
>>>
>>> -
>>> -&v3v3 {
>>> - regulator-always-on;
>>> -};
>>
>> It seems this patch was applied incorrectly and the 1V8 and 3V3
>> differentiation
>> between the SoMs is completely missing from mainline. I will send a
>> subsequent
>> patch to correct that.
>
> Strange, I don't remember conflict on this patch....
>
> And 1.8V was present for the dhcor.dtsi in initial patch,
> I miss something I thing when I apply the patch:
>
> +++ b/arch/arm/dts/stm32mp15xx-dhcor.dtsi
> @@ -0,0 +1,231 @@
> +
> + /* Enpirion EP3A8LQI U2 on the DHCOR */
> + vdd_io: regulator-buck-io {
> + compatible = "regulator-fixed";
> + regulator-name = "buck-io";
> + regulator-min-microvolt = <1800000>;
> + regulator-max-microvolt = <1800000>;
> + regulator-always-on;
> + regulator-boot-on;
> + vin-supply = <&vdd>;
> + };
> [...]
> +
> +&pwr_regulators {
> + vdd-supply = <&vdd_io>;
> + vdd_3v3_usbfs-supply = <&vdd_usb>;
> +};
>
> Anyway thanks to the check and the correction.
The thing is, there are two versions of the SoM that's soldered on the
bottom of the AV96. The one on AV96 has that regulator populated, but
that is optional, so for other devices you can just opt for 3V3 IO and
the regulator won't be there. That's why there are the two DTs (io1v8
and io3v3).
