Hi,
On 5/17/22 17:52, Marek Vasut wrote:
On 5/17/22 17:32, Patrick DELAUNAY wrote:
Hi,
+static int board_get_regulator_buck3_nvm_uv_av96(int *uv)
+{
+ const void *fdt = gd->fdt_blob;
+ struct udevice *dev;
+ u8 bucks_vout = 0;
+ const char *prop;
+ int len, ret;
+
+ /* Check whether this is Avenger96 board. */
+ prop = fdt_getprop(fdt, 0, "compatible", &len);
This API is not compatible with CONFIG_OF_LIVE
consider replacement with ofnode_read_prop or with
of_machine_is_compatible, for example
if (!of_machine_is_compatible(prop, "arrow,stm32mp15xx-avenger96"))
return -EINVAL;
I explicitly don't want to use of_machine_is_compatible, I need to
search for the avenger96 substring in the machine compatible,
since the stm32mp15xx can be anything (15{1,3,7}{a,d}) and it
might not even have the arrow prefix.
Ok, I check on existing DT and I found only on compatible with
"avenger96" string.
arch/arm/dts/stm32mp15xx-dhcor-avenger96.dts:15:
compatible = "arrow,stm32mp15xx-avenger96", "st,stm32mp15x";
But you can forget my proposal with of_machine_is_compatible,
consider:
prop = ofnode_read_prop(ofnode_root(), "compatible", &len);
But it is just minor remark.
There are multiple other fdt_getprop() calls in both board/st/ and
board/dh , maybe if you want to do OF_LIVE enablement (do you?),
let's convert them all at once ?
Ok. It is more a minor remark linked to checkpatch warning for all
the added code:
"Use the livetree API (dev_read_...)"
I let you handle OF_LIVE for your boards (code & defconfig),
FYI: it is already enable for STMicroelectronics boards (defconfig
and in driver, mach-stm32mp and board code).
But look:
c387e626147 ("Merge branch '2022-05-11-Kconfig-cleanups-etc'")
with this patch:
u-boot$ git grep fdt_getprop.*comp board/dhelectronics/dh_stm32mp1/*c
board/st/stm32mp1/*c
board/dhelectronics/dh_stm32mp1/board.c: fdt_compat =
fdt_getprop(gd->fdt_blob, 0, "compatible",
board/dhelectronics/dh_stm32mp1/board.c: compat =
fdt_getprop(gd->fdt_blob, 0, "compatible", NULL);
board/dhelectronics/dh_stm32mp1/board.c: fdt_compat =
fdt_getprop(gd->fdt_blob, 0, "compatible",
board/st/stm32mp1/stm32mp1.c: fdt_compat = fdt_getprop(gd->fdt_blob,
0, "compatible",
board/st/stm32mp1/stm32mp1.c: fdt_compat =
fdt_getprop(gd->fdt_blob, 0, "compatible",
Am I missing a patch or something ?
No, it is my fault, I forget these references....
I prepare a serie to solve this issue in STMicroelectronics Boards code.
Do you expect this patch for next pull-request for v2022.07 or for
next ?
Ideally 2022.07.
Ok, I plan that.
Thank you
regards
Patrick
