On Tue, Mar 11, 2025 at 03:27:25PM +0000, Caleb Connolly wrote: > Hi Varadarajan, > > Thanks for the series, and apologies for the slow reply, i've been out sick > and still finding my feet again.
Oops. Sorry to have troubled you. > On 3/4/25 11:01, Varadarajan Narayanan wrote: > > Introduce a defconfig for the Qualcomm IPQ5424 SoC based RDPs. > > Presently supports eMMC. > > > > Per the flash memory layout, U-Boot size cannot exceed 756KB. With this > > defconfig, u-boot.mbn size is ~480KB. > > What are the differences between this and the other new IPQ platform? Can > they share a defconfig? They are very similar. At this point, CONFIG_DEFAULT_DEVICE_TREE, CONFIG_CLK_QCOM_xxx, CONFIG_PINCTRL_QCOM_xxx, CONFIG_DEBUG_UART_xxx and memory map are different. > Alternatively, could most of the things here be de-duplicated so there is a > "base" config which they both #include (similarly to qcm6490_defconfig) this > way we can nicely keep track of what parts of board specific. > > If you'd prefer to keep it this way that's fine too. Would having an ipq_defconfig and including that in ipq9574 and ipq5424 defconfigs be a good approach? > Should I be expecting more platforms like this? Yes. Thanks Varada [ . . . ]

