[PATCH 2/8] dt-bindings: spi: fsl-qspi: support SpacemiT K1
Conor Dooley
conor at kernel.org
Mon Oct 20 11:41:46 PDT 2025
On Mon, Oct 20, 2025 at 01:06:50PM -0500, Alex Elder wrote:
> On 10/20/25 12:41 PM, Conor Dooley wrote:
> > On Mon, Oct 20, 2025 at 11:51:45AM -0500, Alex Elder wrote:
> > > Add the SpacemiT K1 SoC QSPI IP to the list of supported hardware.
> >
> > Also, you should really explain why this spacemit device is the first
> > one to be in what's been an fsl-specific binding for now in the commit
> > message.
>
> I'm not sure how much of an explanation you're looking for, but
> yes, I agree with you, it stands out that it's the first one, so
> I at least should have acknowledged that. I'll add something
> here in the next version.
Even just mentioning that the register interface etc is nigh identical.
Otherwise this just looks like picking a random file to put the
compatible in. Remember, this is independent from the driver change and
must be justified in its own commit message.
>
> -Alex
>
> > pw-bot: changes-requested
> >
> > >
> > > Signed-off-by: Alex Elder <elder at riscstar.com>
> > > ---
> > > Documentation/devicetree/bindings/spi/fsl,spi-fsl-qspi.yaml | 1 +
> > > 1 file changed, 1 insertion(+)
> > >
> > > diff --git a/Documentation/devicetree/bindings/spi/fsl,spi-fsl-qspi.yaml b/Documentation/devicetree/bindings/spi/fsl,spi-fsl-qspi.yaml
> > > index 0315a13fe319a..5bbda4bc33350 100644
> > > --- a/Documentation/devicetree/bindings/spi/fsl,spi-fsl-qspi.yaml
> > > +++ b/Documentation/devicetree/bindings/spi/fsl,spi-fsl-qspi.yaml
> > > @@ -22,6 +22,7 @@ properties:
> > > - fsl,imx6ul-qspi
> > > - fsl,ls1021a-qspi
> > > - fsl,ls2080a-qspi
> > > + - spacemit,k1-qspi
> > > - items:
> > > - enum:
> > > - fsl,ls1043a-qspi
> > > --
> > > 2.48.1
> > >
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-riscv/attachments/20251020/681f8472/attachment.sig>
More information about the linux-riscv
mailing list