Re: [PATCH 6/7] iommu: dart: Support different variants with different registers

From: Sven Peter
Date: Thu Jan 05 2023 - 04:46:52 EST


Hi,

On Thu, Jan 5, 2023, at 06:16, Hector Martin wrote:
> On 2023/01/04 22:43, Sven Peter wrote:
>> On Wed, Jan 4, 2023, at 12:00, Hector Martin wrote:
>>> + u64 enable_streams;
>>> + u64 disable_streams;
>> I don't think disable_streams is used anywhere. I assume you just left it in
>> here to document it?
>
> Yeah, we don't use this field ever, so we might as well drop it. I'll
> leave the #define for T8110 in though, as documentation.
>
>>> + u64 ttbr;
>>> + u64 ttbr_valid;
>>> + u64 ttbr_addr_off;
>>
>> This name confused me a bit since off sounds like offset to me while
>> this is actually another shift. Can't really think of a better name
>> right now though. I'd at least a comment here to describe it.
>
> How about `ttbr_addr_field_shift`?

Sounds good to me!


Sven