Thread (21 messages) 21 messages, 4 authors, 2018-06-21

[PATCH v2 0/5] crypto: ccree: cleanup, fixes and R-Car enabling

From: gilad@benyossef.com (Gilad Ben-Yossef)
Date: 2018-06-21 05:01:04
Also in: linux-clk, linux-crypto, linux-devicetree, linux-renesas-soc, lkml

On Wed, Jun 20, 2018 at 11:51 AM, Simon Horman [off-list ref] wrote:
On Tue, Jun 19, 2018 at 04:57:15PM +0300, Gilad Ben-Yossef wrote:
quoted
On Tue, Jun 19, 2018 at 3:58 PM, Geert Uytterhoeven
[off-list ref] wrote:
quoted
Hi Gilad,

On Thu, May 24, 2018 at 4:19 PM Gilad Ben-Yossef [off-list ref] wrote:
quoted
The patch set enables the use of CryptoCell found in some Renesas R-Car
Salvator-X boards and fixes some driver issues uncovered that prevented
to work properly.
With DEBUG enabled on R-Car H3, I see lots of

    ccree e6601000.crypto: IRR includes unknown cause bits (0x00000098)
    ccree e6601000.crypto: IRR includes unknown cause bits (0x000000C0)
    ccree e6601000.crypto: IRR includes unknown cause bits (0x000000D0)
    ccree e6601000.crypto: IRR includes unknown cause bits (0x000000D8)
    ccree e6601000.crypto: IRR includes unknown cause bits (0x000000E0)
    ccree e6601000.crypto: IRR includes unknown cause bits (0x000000F0)
    ccree e6601000.crypto: IRR includes unknown cause bits (0x000000F8)

during boot. Is that expected?
Yes. The condition itself it is reporting is not necessarily bad. It
means that driver
did not act on certain HW notification during interrupts and that's
OK, we don't act on all of them
depending on configuration - e.g. if you have CONFIG_FIPS enabled and
an active TEE module or not.

I can rate_limit the message if it bothers you but other than that it
is a harmless debug print.
Rate limiting sounds like an excellent idea to me.
Will do.

Thanks!
Gilad



-- 
Gilad Ben-Yossef
Chief Coffee Drinker

values of ? will give rise to dom!
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help