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...
_______________________________________________
kvmarm mailing list
kvmarm@xxxxxxxxxxxxxxxxxxxxx
https://lists.cs.columbia.edu/mailman/listinfo/kvmarm
