From: Lucas Stach <l.stach@pengutronix.de>
To: "Lothar Waßmann" <LW@KARO-electronics.de>,
"Martin Kaiser" <martin@kaiser.cx>
Cc: Mark Rutland <mark.rutland@arm.com>,
devicetree@vger.kernel.org, Russell King <linux@armlinux.org.uk>,
linux-kernel@vger.kernel.org, Rob Herring <robh+dt@kernel.org>,
Sascha Hauer <kernel@pengutronix.de>,
Fabio Estevam <fabio.estevam@nxp.com>,
Shawn Guo <shawnguo@kernel.org>,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH] ARM: dts: i.MX25: define SSI FIFO depth
Date: Fri, 09 Mar 2018 10:27:12 +0100 [thread overview]
Message-ID: <1520587632.19507.3.camel@pengutronix.de> (raw)
In-Reply-To: <20180309093738.7cd5fded@karo-electronics.de>
Hi Lothar,
Am Freitag, den 09.03.2018, 09:37 +0100 schrieb Lothar Waßmann:
> Hi,
>
> On Thu, 8 Mar 2018 16:38:32 +0100 Martin Kaiser wrote:
> > Hi Lothar,
> >
> > > > Thus wrote Lothar Waßmann (LW@KARO-electronics.de):
> >
> > > > diff --git a/arch/arm/boot/dts/imx25.dtsi b/arch/arm/boot/dts/imx25.dtsi
> > > > index 9725705..cf70df2 100644
> > > > --- a/arch/arm/boot/dts/imx25.dtsi
> > > > +++ b/arch/arm/boot/dts/imx25.dtsi
> > > > @@ -269,6 +269,7 @@
> > > > > > > > dmas = <&sdma 24 1 0>,
> > > > > > > > <&sdma 25 1 0>;
> > > > > > > > dma-names = "rx", "tx";
> > > > > > > > + fsl,fifo-depth = <15>;
> > > > > > > > status = "disabled";
> > > > > > > > };
> > > > @@ -329,6 +330,7 @@
> > > > > > > > dmas = <&sdma 28 1 0>,
> > > > > > > > <&sdma 29 1 0>;
> > > > > > > > dma-names = "rx", "tx";
> > > > > > > > + fsl,fifo-depth = <15>;
> > > > > > > > status = "disabled";
> > > > };
> > > You are changing the global .dtsi file. Did you test this change with
> > > all devices that are affected by it?
> >
> > I changed the hardware description of the imx25 SSI to match the
> > reference manual.
> >
> > I did test this change on an imx25 board with audio playback. This uses
> > the SSI description I modified. I verified that the driver is actually
> > taking the modified setting into account and that this causes no
> > problems.
> >
> > As of today, this setting is used by the fsl_ssi driver to set the fifo
> > water level for dma requests.
> >
> > Of course, I don't have access to the enitre range of supported imx25
> > boards and I don't think this is required for submitting patches.
> >
> > Do you have any indication why this patch should not be merged?
> >
>
> Usually patches should not willy-nilly change the behaviour of existing
> configurations unless they fix any real life bugs.
With this logic we wouldn't be able to get most driver changes applied.
If it is matching the reference manual and has been tested on the
affected hardware it should be good to go.
If you know about any specific corner cases that might break with this
change, please speak up now. Don't reject patches based on the
overarching "it might break something" argument.
Regards,
Lucas
next prev parent reply other threads:[~2018-03-09 9:27 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-06 21:58 [PATCH] ARM: dts: i.MX25: define SSI FIFO depth Martin Kaiser
2018-03-06 22:04 ` Fabio Estevam
2018-03-08 15:11 ` Lothar Waßmann
2018-03-08 15:38 ` Martin Kaiser
2018-03-09 8:37 ` Lothar Waßmann
2018-03-09 9:25 ` Martin Kaiser
2018-03-09 9:27 ` Lucas Stach [this message]
2018-03-09 15:40 ` Lothar Waßmann
2018-03-12 7:24 ` Shawn Guo
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1520587632.19507.3.camel@pengutronix.de \
--to=l.stach@pengutronix.de \
--cc=LW@KARO-electronics.de \
--cc=devicetree@vger.kernel.org \
--cc=fabio.estevam@nxp.com \
--cc=kernel@pengutronix.de \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@armlinux.org.uk \
--cc=mark.rutland@arm.com \
--cc=martin@kaiser.cx \
--cc=robh+dt@kernel.org \
--cc=shawnguo@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).