Re: [PATCH v8 0/9] RZN1 DMA support

From: Miquel Raynal <miquel.raynal@bootlin.com>
To: Stephen Boyd <sboyd@kernel.org>
Cc: Gareth Williams <gareth.williams.jx@renesas.com>,
	Geert Uytterhoeven <geert@linux-m68k.org>,
	Magnus Damm <magnus.damm@gmail.com>,
	Phil Edworthy <phil.edworthy@renesas.com>,
	Vinod Koul <vkoul@kernel.org>,
	linux-renesas-soc@vger.kernel.org, dmaengine@vger.kernel.org,
	Milan Stevanovic <milan.stevanovic@se.com>,
	Jimmy Lalande <jimmy.lalande@se.com>,
	Pascal Eberhard <pascal.eberhard@se.com>,
	Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
	Herve Codina <herve.codina@bootlin.com>,
	Clement Leger <clement.leger@bootlin.com>,
	Michael Turquette <mturquette@baylibre.com>,
	linux-clk@vger.kernel.org, Viresh Kumar <vireshk@kernel.org>,
	Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
	Ilpo Jarvinen <ilpo.jarvinen@linux.intel.com>,
	Rob Herring <robh@kernel.org>,
	devicetree@vger.kernel.org
Subject: Re: [PATCH v8 0/9] RZN1 DMA support
Date: Thu, 7 Apr 2022 10:16:05 +0200	[thread overview]
Message-ID: <20220407101605.7d2a17cc@xps13> (raw)
In-Reply-To: <20220407004511.3A6D1C385A3@smtp.kernel.org>

Hi Stephen,

sboyd@kernel.org wrote on Wed, 06 Apr 2022 17:45:09 -0700:

> Quoting Miquel Raynal (2022-04-06 09:18:47)
> > Hello,
> > 
> > Here is a first series bringing DMA support to RZN1 platforms. Soon a
> > second series will come with changes made to the UART controller
> > driver, in order to interact with the RZN1 DMA controller.
> > 
> > Stephen acked the sysctrl patch (in the clk driver) but somehow I feel
> > like it would be good to have this patch applied on both sides
> > (dmaengine and clk) because more changes will depend on the addition of
> > this helper, that are not related to DMA at all. I'll let you folks
> > figure out what is best.  
> 
> Are you sending more patches in the next 7 weeks or so that will touch
> the same area? If so, then it sounds like I'll need to take the clk
> patch through clk tree. I don't know what is best because I don't have
> the information about what everyone plans to do in that file.

This series brings DMA support and needs to access the dmamux registers
that are in the sysctrl area.

I've sent an RTC series which needs to access this area as well, but
it is not fully ready yet as it was advised to go for a reset
controller in this case. The reset controller would be registered by
the clock driver, so yes it would touch the same file.

Finally, there is an USB series that is coming soon, I don't know if
it will be ready for merge for 5.19, but it needs to access a specific
register in this area as well (h2mode).

So provided that we are able to contribute this reset driver quickly
enough, I would argue that it is safer to merge the clk changes in the
clk tree.

Thanks,
Miquèl