linux-kernel - Re: [PATCH 5/5] ASoC: rsnd: dma: use extended audio dmac registers when available

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <87d0mvgtnc.wl-kuninori.morimoto.gx@renesas.com>
Date:   13 Mar 2019 15:57:25 +0900
From:   Kuninori Morimoto <kuninori.morimoto.gx@...esas.com>
To:     <horms@...ge.net.au>
Cc:     Jiada Wang <jiada_wang@...tor.com>, <magnus.damm@...il.com>,
        <robh+dt@...nel.org>, <mark.rutland@....com>,
        <lgirdwood@...il.com>, <broonie@...nel.org>, <perex@...ex.cz>,
        <tiwai@...e.com>, <geert@...ux-m68k.org>,
        <linux-renesas-soc@...r.kernel.org>, <devicetree@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>, <alsa-devel@...a-project.org>
Subject: Re: [PATCH 5/5] ASoC: rsnd: dma: use extended audio dmac registers when available


Hi Simon

> > 2nd, in my understanding, our conclusion at Renesas-ML
> > is that we don't need to think about basic/extend DMAC register.
> > Because extend area is 100% covering basic area.
> > In other words, it is compatible.
> > Driver side don't need to think about it.
> > 
> I am a little confused,
> because latest comment received from simon, suggests to let driver to
> decide which register set to use.
> 
> for me, I think it's not necessary, if extended register set is available,
> driver shall always use it.

I can agree to have both basic/extend register
if driver need to switch its behavior.
But this case, there is nothing to do on driver side.
In other words, SoC always need to use extend
register if it has.
I don't know why datasheet is indicating both area.
Maybe it is because for Gen3 all-in ? I'm not sure.

Anyway, Simon, can you agree about it ?
Having both basic/extend register is just noise for driver.

Best regards
---
Kuninori Morimoto