[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <ZtkgCCAWt+Oof9xI@lizhi-Precision-Tower-5810>
Date: Wed, 4 Sep 2024 23:05:44 -0400
From: Frank Li <Frank.li@....com>
To: Bough Chen <haibo.chen@....com>
Cc: Fabio Estevam <festevam@...il.com>, Han Xu <han.xu@....com>,
"yogeshgaur.83@...il.com" <yogeshgaur.83@...il.com>,
"broonie@...nel.org" <broonie@...nel.org>,
"robh@...nel.org" <robh@...nel.org>,
"krzk+dt@...nel.org" <krzk+dt@...nel.org>,
"conor+dt@...nel.org" <conor+dt@...nel.org>,
"shawnguo@...nel.org" <shawnguo@...nel.org>,
"s.hauer@...gutronix.de" <s.hauer@...gutronix.de>,
"kernel@...gutronix.de" <kernel@...gutronix.de>,
"singh.kuldeep87k@...il.com" <singh.kuldeep87k@...il.com>,
"hs@...x.de" <hs@...x.de>,
"linux-spi@...r.kernel.org" <linux-spi@...r.kernel.org>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"imx@...ts.linux.dev" <imx@...ts.linux.dev>,
"linux-arm-kernel@...ts.infradead.org" <linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH v2 2/3] spi: fspi: add support for imx8ulp
On Thu, Sep 05, 2024 at 01:49:03AM +0000, Bough Chen wrote:
>
> > -----Original Message-----
> > From: Fabio Estevam <festevam@...il.com>
> > Sent: 2024年9月5日 9:38
> > To: Bough Chen <haibo.chen@....com>
> > Cc: Han Xu <han.xu@....com>; yogeshgaur.83@...il.com;
> > broonie@...nel.org; robh@...nel.org; krzk+dt@...nel.org;
> > conor+dt@...nel.org; shawnguo@...nel.org; s.hauer@...gutronix.de;
> > kernel@...gutronix.de; singh.kuldeep87k@...il.com; hs@...x.de;
> > linux-spi@...r.kernel.org; devicetree@...r.kernel.org;
> > linux-kernel@...r.kernel.org; imx@...ts.linux.dev;
> > linux-arm-kernel@...ts.infradead.org
> > Subject: Re: [PATCH v2 2/3] spi: fspi: add support for imx8ulp
> >
> > Hi Haibo,
> >
> > On Wed, Sep 4, 2024 at 10:26 PM <haibo.chen@....com> wrote:
> >
> > > @@ -373,6 +367,16 @@ static struct nxp_fspi_devtype_data imx8dxl_data =
> > {
> > > .txfifo = SZ_1K, /* (128 * 64 bits) */
> > > .ahb_buf_size = SZ_2K, /* (256 * 64 bits) */
> > > .quirks = FSPI_QUIRK_USE_IP_ONLY,
> > > + .lut_num = 32,
> > > + .little_endian = true, /* little-endian */
> >
> > The addition of little_endian to imx8dl should be a separate patch.
>
> Hi Fabio,
>
> I only add the .lut_num here, do not touch .lettle_endian here, and I add another 8ulp in the follow line, but format patch show in this way. Anything I can do to adjust how to format patch?
you can try below diff option.
--patience
Generate a diff using the "patience diff" algorithm.
--histogram
Generate a diff using the "histogram diff" algorithm.
--anchored=<text>
Generate a diff using the "anchored diff" algorithm.
I suggest split to two patches, one patch convert to hardcode 16 to
lut_num, no function change. The other one is add 7ulp part.
Frank
>
> Best Regards
> Haibo Chen
> >
> > The comment is pointless too.
Powered by blists - more mailing lists