On Fri, 22 Dec 2023 at 12:36, Sumit Garg <sumit.g...@linaro.org> wrote: > > On Thu, 21 Dec 2023 at 21:08, Caleb Connolly <caleb.conno...@linaro.org> > wrote: > > > > > > > > On 20/12/2023 13:07, Sumit Garg wrote: > > > Hi Caleb, > > > > Hi Sumit, > > > > [...] > > > > >> > > >> == Upstream DT == > > >> > > >> All Qualcomm boards have had their devicetree files replaced with the > > >> upstream versions. Previous patch series made the necessary driver > > >> adjustments to fully support the upstream DT format. All future > > >> Qualcomm boards should use upstream DTS by default. > > >> > > >> Once Sumit's work to import dt-rebasing has been merged, we will drop > > >> the imported DT and bindings again. > > > > > > Since there is consensus building in the U-Boot community to use DT > > > rebasing subtree, I would suggest you rebase your work on top of [1]. > > > You can take an amlogic SoC patch [2] as a reference and see > > > corresponding value-add. I don't see the point of unnecessary churn if > > > there is consensus we agree upon. > > > > > > I designed this so that we can just delete the imported stuff once the > > dt-rebasing series is merged. Importing a few files and deleting them > > later is not really a big deal, so please let's not block fixing > > Qualcomm support on this tangentially related series. > > Fair enough, I think I can live with that for the time being. Let me > give this series a spin on QCS404.
Just to update here, this series doesn't boot on QCS404. Internally I am working with Caleb to fix issues seen. One of the major issues was to not map reserved memory ranges in U-Boot as mapping then leads to a hang on QCS404 (see [1] for details). [1] https://github.com/u-boot/u-boot/commit/881338a0c6686bbcf043275714d43080b746b17e -Sumit > > > > > Thanks, > > > > > > > > [1] > > > https://lore.kernel.org/all/20231214135103.1606605-1-sumit.g...@linaro.org/ > > > [2] > > > https://lore.kernel.org/all/20231214135103.1606605-8-sumit.g...@linaro.org/ > > > > > >> > > > > -- > > // Caleb (they/them)
