From: Paul Cercueil <paul@crapouillou.net>
To: James Hogan <jhogan@kernel.org>
Cc: PrasannaKumar Muralidharan <prasannatsmkumar@gmail.com>,
herbert@gondor.apana.org.au, robh+dt@kernel.org,
ralf@linux-mips.org, davem@davemloft.net,
linux-crypto@vger.kernel.org, linux-mips@linux-mips.org,
malat@debian.org, noloader@gmail.com
Subject: Re: [PATCH v3 3/4] crypto: jz4780-rng: Add RNG node to jz4780.dtsi
Subject:
Date: Wed, 07 Mar 2018 00:01:25 +0100 [thread overview]
Message-ID: <788bf0bf0d0aaa97f59bc908ebf34ebf@crapouillou.net> (raw)
In-Reply-To: <20180306091932.GM4197@saruman>
Le 2018-03-06 10:32, James Hogan a écrit :
> On Mon, Sep 18, 2017 at 07:32:40PM +0530, PrasannaKumar Muralidharan
> wrote:
>> Add RNG node to jz4780 dtsi. This driver uses registers that are part
>> of
>> the register set used by Ingenic CGU driver. Use regmap in RNG driver
>> to
>> access its register. Create 'simple-bus' node, make CGU and RNG node
>> as
>> child of it so that both the nodes are visible without changing CGU
>> driver code.
>>
>> Signed-off-by: PrasannaKumar Muralidharan <prasannatsmkumar@gmail.com>
>
> Better late than never:
> Acked-by: James Hogan <jhogan@kernel.org>
>
> (I presume its okay for the reg ranges to overlap, ISTR that being an
> issue a few years ago, but maybe thats fixed now).
>
> Cheers
> James
What bothers me is that the CGU code has not been modified to use
regmap, so the
registers area is actually mapped twice (once in the CGU driver, once
with regmap).
Besides, regmap would be useful if the RNG registers were actually
located in the
middle of the register area used by the CGU driver, which is not the
case here.
The CGU block does have some registers after the RNG ones on the X1000
SoC, but
I don't think they will ever be used (and if they are it won't be by the
CGU driver).
Regards,
-Paul
next prev parent reply other threads:[~2018-03-06 23:01 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-18 14:02 [PATCH v3 0/4] crypto: Add driver for JZ4780 PRNG PrasannaKumar Muralidharan
2017-09-18 14:02 ` [PATCH v3 1/4] crypto: jz4780-rng: Add JZ4780 PRNG devicetree binding documentation PrasannaKumar Muralidharan
2017-09-18 14:02 ` [PATCH v3 2/4] crypto: jz4780-rng: Add Ingenic JZ4780 hardware PRNG driver PrasannaKumar Muralidharan
2017-09-18 14:02 ` [PATCH v3 3/4] crypto: jz4780-rng: Add RNG node to jz4780.dtsi PrasannaKumar Muralidharan
2018-03-06 9:32 ` James Hogan
2018-03-06 13:55 ` Rob Herring
2018-03-06 13:55 ` Rob Herring
2018-03-07 14:54 ` PrasannaKumar Muralidharan
2018-03-07 14:54 ` PrasannaKumar Muralidharan
2018-03-07 20:25 ` Rob Herring
2018-03-07 20:25 ` Rob Herring
2018-03-06 23:01 ` Paul Cercueil [this message]
2018-03-07 14:51 ` PrasannaKumar Muralidharan
2018-03-07 14:51 ` PrasannaKumar Muralidharan
2018-03-07 20:23 ` [PATCH v3 3/4] crypto: jz4780-rng: Add RNG node to jz4780.dtsi, Paul Cercueil
2018-03-07 20:23 ` Paul Cercueil
2017-09-18 14:02 ` [PATCH v3 4/4] crypto: jz4780-rng: Enable PRNG support in CI20 defconfig PrasannaKumar Muralidharan
2017-10-12 14:30 ` [PATCH v3 0/4] crypto: Add driver for JZ4780 PRNG Herbert Xu
2017-10-12 17:29 ` PrasannaKumar Muralidharan
2017-10-12 17:29 ` PrasannaKumar Muralidharan
2017-10-12 17:29 ` PrasannaKumar Muralidharan
2017-11-25 14:46 ` PrasannaKumar Muralidharan
2017-11-25 14:46 ` PrasannaKumar Muralidharan
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=788bf0bf0d0aaa97f59bc908ebf34ebf@crapouillou.net \
--to=paul@crapouillou.net \
--cc=davem@davemloft.net \
--cc=herbert@gondor.apana.org.au \
--cc=jhogan@kernel.org \
--cc=linux-crypto@vger.kernel.org \
--cc=linux-mips@linux-mips.org \
--cc=malat@debian.org \
--cc=noloader@gmail.com \
--cc=prasannatsmkumar@gmail.com \
--cc=ralf@linux-mips.org \
--cc=robh+dt@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.