Re: L3 custom error on OMAP4460 (AES/DES)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 15 April 2014 16:01, Nishanth Menon <nm@xxxxxx> wrote:
> On 22:36-20140414, Joachim Eastwood wrote:
>> On 14 April 2014 21:19, Nishanth Menon <nm@xxxxxx> wrote:
>> > On 04/14/2014 02:15 PM, Joachim Eastwood wrote:
>> >> On 14 April 2014 15:38, Santosh Shilimkar <santosh.shilimkar@xxxxxx> wrote:
>> >>> On Saturday 12 April 2014 05:06 PM, Joachim Eastwood wrote:
>> >>>> Hi,
>> >>>>
>> >>>> I getting the following error on Linus master right now.
>> >>>>
>> >>>> [ 2.166320] WARNING: CPU: 0 PID: 0 at drivers/bus/omap_l3_noc.c:113
>> >>>> l3_interrupt_handler+0xf4/0x154()
>> >>>> [ 2.166320] L3 custom error: MASTER:MPU TARGET:L4 PER2
>> >>>> [ 2.166320] Modules linked in:
>> >
>> > [...]
>> >>>> The hardware is a VAR-STK-OM44 dev kit. I got one patch on top of
>> >>>> Linus master which is the DT support patch which I posted a couple of
>> >>>> hours ago.
>> >>>>
>> >>> Have you tried removing AES from the build ? Probably worth a
>> >>> try.
>> >>
>> >> Removing the aes driver makes the warning from omap_l3_noc disappear.
>> >>
>> >> I also tried tried the omap_l3_noc patches from Nishanth Menon that
>> >> was just posted but it had the same error when the aes driver was
>> >> built-in.
>> > Yeah - it better :).. Good to know that driver continues to report bad
>> > accesses by drivers to targets that are not active yet. will be great
>> > if you could provide a "Tested-by" tag on my series :).
>>
>> Sure. The patch set booted fine on my VAR-STK-OM44 (OMAP4460).
>> Tested-by: Joachim Eastwood <manabian@xxxxxxxxx>
>>
>> > Btw, I just finished testing on PandaBoard ES and SDP4430 and I dont
>> > see the error that you see on var som. I wonder why?
>> >
>> > pandaboard-es:  Boot PASS: http://slexy.org/raw/s21Hsq97mt
>> > sdp4430:  Boot PASS: http://slexy.org/raw/s2ZQJUqjrP
>>
>> hmm. Maybe it's a combination of different config options.
>>
>> I have attached the complete dmesg and config from my board. Maybe you
>> could try the config out.
>>
>> I noticed from your PandaBoard ES log that you only get this line:
>> [ 0.555480] platform 4b501000.aes: Cannot lookup hwmod 'aes'
>>
>> While on my board I first get the "Cannot lookup" message and then
>> some other messages:
>> [ 0.512847] platform 4b501000.aes: Cannot lookup hwmod 'aes'
>> [ 2.341125] omap-aes 4b501000.aes: _od_fail_runtime_resume: FIXME:
>> missing hwmod/omap_dev info
>> [ 2.350219] omap-aes 4b501000.aes: omap_aes_probe: failed to get_sync(-19)
>> [ 2.357482] omap-aes 4b501000.aes: initialization failed.
>
> Yep, I was able to reproduce the error:
> pandaboard-es-before: http://slexy.org/raw/s21rGPFnKl
> pandaboard-es-after:  http://slexy.org/raw/s2A4UFQVna
>
> Key was "omap-des 480a5000.des: OMAP DES hw accel rev: 0.0" in the log.
>
> Please try the following patch:
> --8<--
> From a5c1b5645eef74c88ea22f6ec99054ec0e9a2502 Mon Sep 17 00:00:00 2001
> From: Nishanth Menon <nm@xxxxxx>
> Date: Tue, 15 Apr 2014 08:55:44 -0500
> Subject: [PATCH] crypto: omap-des - handle error of pm_runtime_get_sync
>
> pm_runtime_get_sync may not always succeed depending on SoC involved. So
> handle the error appropriately.
>
> Signed-off-by: Nishanth Menon <nm@xxxxxx>
> ---
>
> based on v3.15-rc1
>
>  drivers/crypto/omap-des.c |   18 ++++++++++++++----
>  1 file changed, 14 insertions(+), 4 deletions(-)

After this patch the warnings from omap_l3_noc disappear. Thanks Nishanth.
Tested-by: Joachim Eastwood <manabian@xxxxxxxxx>

I have some other issues on OMAP4460, unrelated to this. I'll send out
an email to the list later.

best regards,
Joachim Eastwood
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux