On 14/03/2023 05:48, Prasad Sodagudi wrote: > > > On 3/9/2023 3:47 PM, Konrad Dybcio wrote: >> >> >> On 10.03.2023 00:39, Eric Chanudet wrote: >>> ABL uses the __symbols__ section to process the DTB before passing it >>> forward. Without it, the bootstrap is interrupted. >>> >>> Signed-off-by: Eric Chanudet <echanude@xxxxxxxxxx> >>> --- >> Fix your ABL. > Hi Konrad, > > Apps boot-loader need __symbols__ for dynamic overlay operation. > Qualcomm firmware passes an overlay file to apps boot-loader to overlay > some of the nodes based on firmware configuration. Without __symbols__ > apps boot-loader is not able to overlay. > > Qualcomm hypervisor/gunyah would like to overlay arch timer node with > always-on property, So adding __symbols__ helps boot-loader to overlay. Since none of these overlays are upstream, we can argue that it is not a valid reason. If you decide to keep overlays downstream then also you can keep patch enabling the overlays there... Best regards, Krzysztof