回复: [PATCH v1 1/4] spi: nxp_xspi: Add new driver for NXP XSPI controller

[Alice Guo (OSS)]

> -----邮件原件-----
> 发件人: Marek Vasut <marek.vasut at mailbox.org>
> 发送时间: 2025年10月14日 19:27
> 收件人: Alice Guo (OSS) <alice.guo at oss.nxp.com>; u-boot at lists.denx.de;
> dl-uboot-imx <uboot-imx at nxp.com>; trini at konsulko.com;
> sbabic at nabladev.com; festevam at gmail.com; jagan at amarulasolutions.com
> 抄送: Ye Li <ye.li at nxp.com>; Alice Guo <alice.guo at nxp.com>; Peng Fan
> <peng.fan at nxp.com>; Jacky Bai <ping.bai at nxp.com>; sjg at chromium.org;
> mkorpershoek at kernel.org
> 主题: Re: [PATCH v1 1/4] spi: nxp_xspi: Add new driver for NXP XSPI controller
> 
> On 10/14/25 1:17 PM, alice.guo at oss.nxp.com wrote:
> 
> Hi,
> 
> > +++ b/drivers/spi/nxp_xspi.c
> > @@ -0,0 +1,889 @@
> > +// SPDX-License-Identifier: GPL-2.0+
> > +/*
> > + * Copyright 2025 NXP
> > + */
> > +
> > +#include <asm/arch/clock.h>
> > +#include <asm/io.h>
> > +#include <clk.h>
> > +#include <dm.h>
> > +#include <dm/device_compat.h>
> > +#include <linux/bitops.h>
> > +#include <linux/bug.h>
> > +#include <linux/err.h>
> > +#include <linux/iopoll.h>
> > +#include <linux/kernel.h>
> > +#include <linux/sizes.h>
> > +#include <log.h>
> > +#include <malloc.h>
> > +#include <spi.h>
> > +#include <spi-mem.h>
> > +
> > +#include "nxp_xspi.h"
> > +
> > +static inline void xspi_writel(struct nxp_xspi *x, u32 val, u32 addr)
> > +{
> > +	void __iomem *_addr = (void __iomem *)(uintptr_t)addr;
> > +
> > +	if (x->devtype_data->little_endian)
> > +		out_le32(_addr, val);
> > +	else
> > +		out_be32(_addr, val);
> > +};
> Is the BE mode ever used ?

Hi Marek,

As far as I know, the i.MX943 operates in little-endian mode, and the BE mode is not used. Do you think it is necessary to remove the big-endian related code?

Best regards,
Alice Guo