Re: [PATCH 3/4] arm64: dts: qcom: Add HAMOA-IOT-SOM platform

[Yijie Yang]

On 2025-07-18 17:26, Dmitry Baryshkov wrote:
> On Fri, Jul 18, 2025 at 02:33:50PM +0800, Yijie Yang wrote:
> > On 2025-07-18 02:52, Dmitry Baryshkov wrote:
> > > On Wed, Jul 16, 2025 at 05:08:41PM +0800, Yijie Yang wrote:
> > > > The HAMOA-IOT-SOM is a compact computing module that integrates a System
> > > > on Chip (SoC) — specifically the x1e80100 — along with essential
> > > > components optimized for IoT applications. It is designed to be mounted on
> > > > carrier boards, enabling the development of complete embedded systems.
> > > >
> > > > This change enables and overlays the following components:
> > > > - Regulators on the SOM
> > > > - Reserved memory regions
> > > > - PCIe6a and its PHY
> > > > - PCIe4 and its PHY
> > > > - USB0 through USB6 and their PHYs
> > > > - ADSP, CDSP
> > > > - WLAN, Bluetooth (M.2 interface)
> > > >
> > > > Written with contributions from Yingying Tang (added PCIe4 and its PHY to
> > > > enable WLAN).
> > > >
> > > > Signed-off-by: Yijie Yang <yijie.yang@xxxxxxxxxxxxxxxx>
> > > > ---
> > > >    arch/arm64/boot/dts/qcom/hamoa-iot-som.dtsi | 607 ++++++++++++++++++++++++++++
> > > >    1 file changed, 607 insertions(+)
> > >
> > > > +&remoteproc_adsp {
> > > > +	firmware-name = "qcom/hamoa-iot/adsp.mbn",
> > > > +			"qcom/hamoa-iot/adsp_dtb.mbn";
> > >
> > > Is there a significant difference qcom/x1e80100/adsp.mbn ? If not, can
> > > we use that firmware?
> >
> > I believe there are differences in firmware between it and the EVK, even if
> > they’re minor. Therefore, it's better to maintain a dedicated folder for
> > each board and move the code to the carrier board.
>
> Then it's not a 'hamoa-iot'. It should be 'qcom/hamoa/iot-board-name'.
> Please submit the firmware to linux-firmware and also move existing
> x1e80100 firmware to the 'hamoa' subdir, maintaining the compatibility
> x1e80100 -> hamoa symlink.

After looking into the firmware, it appears this board can use the same 
one as the others. I’ll keep the path consistent and avoid making major 
changes in this patch set.

> > > > +
> > > > +	status = "okay";
> > > > +};
> > > > +
> > > > +&remoteproc_cdsp {
> > > > +	firmware-name = "qcom/hamoa-iot/cdsp.mbn",
> > > > +			"qcom/hamoa-iot/cdsp_dtb.mbn";
> > > > +
> > > > +	status = "okay";
> > > > +};
> > > > +
> >
> > --
> > Best Regards,
> > Yijie

--
Best Regards,
Yijie