On Thu, Sep 4, 2014 at 7:19 PM, Mathieu Poirier
<mathieu.poir...@linaro.org> wrote:
> [Me]:
>> I would rewrite the last function like this:
>>
>> static inline unsigned int etm_readl(struct etm_drvdata *drvdata, u32 off)
>> {
>> u32 val;
>>
>> if (drvdata->use_cp14)
>> asm_volatile()...
>> else
>> val = readl_relaxed(drvdata->base + off);
>> return val;
>> }
>
> That unfortunately won't work. "u32 off" is a numerical value and it
> works well in the case were CP14 accesses aren't needed. On the flip
> side that numerical value isn't sufficient to deduce all 3 arguments
> (crn, crm, op2) required by instructions mcr/mrc for the right access
> to happen - there is simply no correlation between the offset of an
> APB bus memory mapped address and the corresponding CP14 access.
You're right, I oversimplified it in my head grossly...
Jens Wiklander pointed me to the macros used in OP-TEE for CP15
access with the same unpredictable patterns:
https://github.com/OP-TEE/optee_os/tree/master/core/arch/arm32/include
Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
