Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Robin Gong <yibin.gong@nxp.com>
To:     Adam Ford <aford173@gmail.com>, Shawn Guo <shawnguo@kernel.org>
CC:     arm-soc <linux-arm-kernel@lists.infradead.org>,
        Rob Herring <robh+dt@kernel.org>,
        Sascha Hauer <s.hauer@pengutronix.de>,
        Pengutronix Kernel Team <kernel@pengutronix.de>,
        Fabio Estevam <festevam@gmail.com>,
        dl-linux-imx <linux-imx@nxp.com>,
        devicetree <devicetree@vger.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH 1/2] arm64: dts: imx8mn: Add spba1 bus
Thread-Topic: [PATCH 1/2] arm64: dts: imx8mn: Add spba1 bus
Thread-Index: AQHXKoTrm+1dEO4gCEi87d5cyZOFDqrdyocAgACF5YCAABIPoA==
Date:   Tue, 11 May 2021 12:20:42 +0000
Message-ID: <VE1PR04MB6688CD4AA4826EEEBBA2651689539@VE1PR04MB6688.eurprd04.prod.outlook.com>
References: <20210406013344.124255-1-aford173@gmail.com>
 <20210511024604.GE3425@dragon>
 <CAHCN7xLFpL=9BF9M5gUA6sMhc2ZZMNz+GP0OLmLfpJAWdD7W-w@mail.gmail.com>
In-Reply-To: <CAHCN7xLFpL=9BF9M5gUA6sMhc2ZZMNz+GP0OLmLfpJAWdD7W-w@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: d2b78ae8-fcc7-4b04-efa5-08d91477327b
X-MS-Exchange-CrossTenant-originalarrivaltime: 11 May 2021 12:20:42.9343
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: h12iimvlxOzSSz3otYUDzifZnFBu70L84bV48DhpmMddl0jYpwrAJn/+ZwZo3gKyZqSwGONOTXzM1HyEmhNrIg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB4062
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 2021/05/11 18:45 Adam Ford <aford173@gmail.com> wrote:
> > Also may I ask if you have a real use case for this bus node?
>=20
> The reference manual shows the SPBA bus tells the DMA controller which
> peripherals are associated with it.  Nearly all the i.MX boards use this.=
  The
> boards I support have Bluetooth devices connected to a UART running high
> speeds, and if the DMA driver isn't loaded, I can see a performance chang=
e.
Compare PIO with DMA on UART, but not w/o this  'spba bus node ' patch?=20

> In fact, if the DMA firmware isn't loaded, I often get transfer errors.
UART use SDMA ROM firmware instead of RAM firmware, so it should work
even without sdma RAM firmware loaded.  Still curious what really happen in
your board without this patch.=20


=20