RE: [PATCH 0/7] Add FFT Support for R-Car Gen3 devices

From: Fabrizio Castro <fabrizio.castro.jz@renesas.com>
To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Rob Herring <robh+dt@kernel.org>, Arnd Bergmann <arnd@arndb.de>,
	Geert Uytterhoeven <geert+renesas@glider.be>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	"linux-renesas-soc@vger.kernel.org" 
	<linux-renesas-soc@vger.kernel.org>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>,
	"linux-api@vger.kernel.org" <linux-api@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Will Deacon <will@kernel.org>,
	Chris Paterson <Chris.Paterson2@renesas.com>,
	Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@bp.renesas.com>,
	Phil Edworthy <phil.edworthy@renesas.com>,
	"REE dirk.behme@de.bosch.com" <dirk.behme@de.bosch.com>,
	Peter Erben <Peter.Erben@de.bosch.com>
Subject: RE: [PATCH 0/7] Add FFT Support for R-Car Gen3 devices
Date: Mon, 1 Mar 2021 14:50:32 +0000	[thread overview]
Message-ID: <OSAPR01MB2737459B1C15430613E84CACC29A9@OSAPR01MB2737.jpnprd01.prod.outlook.com> (raw)
In-Reply-To: <YDjnoGDOCXm86ffW@pendragon.ideasonboard.com>

Hi Laurent,

Thank you for your feedback!

> From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Sent: 26 February 2021 12:21
> Subject: Re: [PATCH 0/7] Add FFT Support for R-Car Gen3 devices
> 
> Hi Fabrizio,
> 
> On Thu, Feb 25, 2021 at 10:51:40PM +0000, Fabrizio Castro wrote:
> > The DAB hardware accelerator found on R-Car E3 (a.k.a. r8a77990)
> > and R-Car M3-N (a.k.a. r8a77965) devices is a hardware accelerator
> > for software DAB demodulators.
> > It consists of one FFT (Fast Fourier Transform) module and one
> > decoder module, compatible with DAB specification (ETSI EN 300 401
> > and ETSI TS 102 563).
> > The decoder module can perform FIC decoding and MSC decoding
> > processing from de-puncture to final decoded result.
> >
> > This series adds FFT support only for R-Car E3 and R-Car M3-N,
> > FIC and MSC support will be added later on.
> 
> Out of curiosity, could the FFT module be used as an accelerator for 2D
> FFT on images ?

I haven't tried using it that way but I don't think so.

Thanks,
Fab 

> 
> > Fabrizio Castro (7):
> >   clk: renesas: r8a77990: Add DAB clock
> >   clk: renesas: r8a77965: Add DAB clock
> >   dt-bindings: misc: Add binding for R-Car DAB
> >   misc: Add driver for DAB IP found on Renesas R-Car devices
> >   arm64: dts: renesas: r8a77990: Add DAB support
> >   arm64: dts: renesas: r8a77965: Add DAB support
> >   arm64: configs: Add R-Car DAB support
> >
> >  .../devicetree/bindings/misc/renesas,dab.yaml |  75 ++++++++
> >  MAINTAINERS                                   |   7 +
> >  arch/arm64/boot/dts/renesas/r8a77965.dtsi     |  12 ++
> >  arch/arm64/boot/dts/renesas/r8a77990.dtsi     |  12 ++
> >  arch/arm64/configs/defconfig                  |   1 +
> >  drivers/clk/renesas/r8a77965-cpg-mssr.c       |   1 +
> >  drivers/clk/renesas/r8a77990-cpg-mssr.c       |   1 +
> >  drivers/misc/Kconfig                          |   1 +
> >  drivers/misc/Makefile                         |   1 +
> >  drivers/misc/rcar_dab/Kconfig                 |  11 ++
> >  drivers/misc/rcar_dab/Makefile                |   8 +
> >  drivers/misc/rcar_dab/rcar_dev.c              | 176 ++++++++++++++++++
> >  drivers/misc/rcar_dab/rcar_dev.h              | 116 ++++++++++++
> >  drivers/misc/rcar_dab/rcar_fft.c              | 160 ++++++++++++++++
> >  include/uapi/linux/rcar_dab.h                 |  35 ++++
> >  15 files changed, 617 insertions(+)
> >  create mode 100644
> Documentation/devicetree/bindings/misc/renesas,dab.yaml
> >  create mode 100644 drivers/misc/rcar_dab/Kconfig
> >  create mode 100644 drivers/misc/rcar_dab/Makefile
> >  create mode 100644 drivers/misc/rcar_dab/rcar_dev.c
> >  create mode 100644 drivers/misc/rcar_dab/rcar_dev.h
> >  create mode 100644 drivers/misc/rcar_dab/rcar_fft.c
> >  create mode 100644 include/uapi/linux/rcar_dab.h
> 
> --
> Regards,
> 
> Laurent Pinchart