Message ID | 1567782585-19854-1-git-send-email-bmeng.cn@gmail.com |
---|---|
State | Superseded |
Headers | show |
Series | riscv: hw: Remove duplicated "hw/hw.h" inclusion | expand |
On Fri, Sep 6, 2019 at 11:09 PM Bin Meng <bmeng.cn@gmail.com> wrote: > > Commit a27bd6c779ba ("Include hw/qdev-properties.h less") wrongly > added "hw/hw.h" to sifive_prci.c and sifive_test.c. > > Another inclusion of "hw/hw.h" was later added via > commit 650d103d3ea9 ("Include hw/hw.h exactly where needed"), that > resulted in duplicated inclusion of "hw/hw.h". > > Fixes: a27bd6c779ba ("Include hw/qdev-properties.h less") > Signed-off-by: Bin Meng <bmeng.cn@gmail.com> > --- > > hw/riscv/sifive_prci.c | 1 - > hw/riscv/sifive_test.c | 1 - > 2 files changed, 2 deletions(-) > Sigh, I just realized that this patch has inter-dependencies with the following patch series: riscv: sifive_test: Add reset functionality http://patchwork.ozlabs.org/patch/1158526/ and riscv: sifive_u: Improve the emulation fidelity of sifive_u machine http://patchwork.ozlabs.org/project/qemu-devel/list/?series=128443 Thus cannot be applied cleanly on top of qemu/master. If I create this patch on qemu/master, that means the other 2 series needs to be rebased again. Regards, Bin
Hello, On Fri, Sep 6, 2019 at 11:22 PM Bin Meng <bmeng.cn@gmail.com> wrote: > > On Fri, Sep 6, 2019 at 11:09 PM Bin Meng <bmeng.cn@gmail.com> wrote: > > > > Commit a27bd6c779ba ("Include hw/qdev-properties.h less") wrongly > > added "hw/hw.h" to sifive_prci.c and sifive_test.c. > > > > Another inclusion of "hw/hw.h" was later added via > > commit 650d103d3ea9 ("Include hw/hw.h exactly where needed"), that > > resulted in duplicated inclusion of "hw/hw.h". > > > > Fixes: a27bd6c779ba ("Include hw/qdev-properties.h less") > > Signed-off-by: Bin Meng <bmeng.cn@gmail.com> > > --- > > > > hw/riscv/sifive_prci.c | 1 - > > hw/riscv/sifive_test.c | 1 - > > 2 files changed, 2 deletions(-) > > > > Sigh, I just realized that this patch has inter-dependencies with the > following patch series: > > riscv: sifive_test: Add reset functionality > http://patchwork.ozlabs.org/patch/1158526/ > > and > > riscv: sifive_u: Improve the emulation fidelity of sifive_u machine > http://patchwork.ozlabs.org/project/qemu-devel/list/?series=128443 > > Thus cannot be applied cleanly on top of qemu/master. > > If I create this patch on qemu/master, that means the other 2 series > needs to be rebased again. > I've included this single patch to my v8 version of "riscv: sifive_u: Improve the emulation fidelity of sifive_u machine", to ease patch inter-dependencies, so that the whole v8 series can be applied on Palmer's RISC-V queue. See http://patchwork.ozlabs.org/patch/1159111/ So, please ignore this single patch. Regards, Bin
diff --git a/hw/riscv/sifive_prci.c b/hw/riscv/sifive_prci.c index 9837b61..562bc3d 100644 --- a/hw/riscv/sifive_prci.c +++ b/hw/riscv/sifive_prci.c @@ -19,7 +19,6 @@ */ #include "qemu/osdep.h" -#include "hw/hw.h" #include "hw/sysbus.h" #include "qemu/module.h" #include "target/riscv/cpu.h" diff --git a/hw/riscv/sifive_test.c b/hw/riscv/sifive_test.c index 3557e16..7117409 100644 --- a/hw/riscv/sifive_test.c +++ b/hw/riscv/sifive_test.c @@ -19,7 +19,6 @@ */ #include "qemu/osdep.h" -#include "hw/hw.h" #include "hw/sysbus.h" #include "qemu/module.h" #include "sysemu/runstate.h"
Commit a27bd6c779ba ("Include hw/qdev-properties.h less") wrongly added "hw/hw.h" to sifive_prci.c and sifive_test.c. Another inclusion of "hw/hw.h" was later added via commit 650d103d3ea9 ("Include hw/hw.h exactly where needed"), that resulted in duplicated inclusion of "hw/hw.h". Fixes: a27bd6c779ba ("Include hw/qdev-properties.h less") Signed-off-by: Bin Meng <bmeng.cn@gmail.com> --- hw/riscv/sifive_prci.c | 1 - hw/riscv/sifive_test.c | 1 - 2 files changed, 2 deletions(-)