[PATCH v2 04/24] arm64: Move MAIR_EL1_SET to asm/memory.h
Marc Zyngier
maz at kernel.org
Wed Nov 25 08:33:22 EST 2020
On 2020-11-25 13:26, David Brazdil wrote:
>> I came up with the following patch on top of this series that seems to
>> compile without issue.
>
> That seems to have an implicit dependency of sysreg.h on memory.h,
> doesn't it?
> I had it the other way round initially. I also tried including memory.h
> in
> sysreg.h. That creates a circular dependency mmdebug.h -> bug.h -> ...
> ->
> sysreg.h -> memory.h -> mmdebug.h. Pretty annoying. I could try to fix
> that,
> or create a new header file... :(
I don't think we need this. Any low-level source using MAIR_ELx_SET is
bound
to require memory.h as well, one way or another. As this is all
#defines,
it won't break anything unless actively used.
And given that this is used in exactly *two* places, I don't believe
there
is a need for over-engineering this.
Thanks,
M.
--
Jazz is not dead. It just smells funny...
More information about the linux-arm-kernel
mailing list