[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v16 78/99] target/arm: cpu-exceptions, cpu-exceptions-aa64: n
From: |
Peter Maydell |
Subject: |
Re: [PATCH v16 78/99] target/arm: cpu-exceptions, cpu-exceptions-aa64: new modules |
Date: |
Thu, 10 Jun 2021 10:01:10 +0100 |
On Thu, 10 Jun 2021 at 09:45, Claudio Fontana <cfontana@suse.de> wrote:
>
> On 6/5/21 10:50 PM, Richard Henderson wrote:
> > I don't see the point in the excessive replication of header files, for
> > exactly
> > one declaration. This is not the first example.
> >
> > What's wrong with internal.h?
>
> Doesn't mean anything and leads to a mess of unrelated code.
It is all related: it is "things that the target/arm code needs,
which are not needed by anything outside target/arm".
> I think we should be better at creating meaningful modules
target/arm is a meaningful grouping of code :-)
thanks
-- PMM
[PATCH v16 52/99] target/arm: rename handle_semihosting to tcg_handle_semihosting, Alex Bennée, 2021/06/04
[PATCH v16 94/99] target/arm: move CONFIG_V7M out of default-devices, Alex Bennée, 2021/06/04
[PATCH v16 26/99] target/arm: move physical address translation to cpu-mmu, Alex Bennée, 2021/06/04
[PATCH v16 70/99] target/arm: move TCG gt timer creation code in tcg/, Alex Bennée, 2021/06/04
[PATCH v16 32/99] target/arm: move cpsr_read, cpsr_write to cpu_common, Alex Bennée, 2021/06/04
[PATCH v16 18/99] target/arm: tcg: add sysemu and user subdirs, Alex Bennée, 2021/06/04