Thank you On Tue, Apr 9, 2019 at 10:12 PM, Aruna Jayasena <aruna...@cse.mrt.ac.lk> wrote:
> Yeah this is a stand alone patch. > > On Tue, Apr 9, 2019 at 10:10 PM, Peter Maydell <peter.mayd...@linaro.org> > wrote: > >> On Tue, 9 Apr 2019 at 16:58, Aruna Jayasena <aruna...@cse.mrt.ac.lk> >> wrote: >> > >> > Removed unwanted includes from cpu-common.h >> > This task was under https://wiki.qemu.org/Contribute/BiteSizedTasks >> > >> > Signed-off-by: Aruna Jayasena <aruna...@cse.mrt.ac.lk> >> > >> > --- >> > include/exec/cpu-common.h | 2 -- >> > 1 file changed, 2 deletions(-) >> > >> > diff --git a/include/exec/cpu-common.h b/include/exec/cpu-common.h >> > index cef8b88a2a..a3594f3f50 100644 >> > --- a/include/exec/cpu-common.h >> > +++ b/include/exec/cpu-common.h >> > @@ -7,8 +7,6 @@ >> > #include "exec/hwaddr.h" >> > #endif >> > >> > -#include "qemu/bswap.h" >> > -#include "qemu/queue.h" >> > #include "qemu/fprintf-fn.h" >> > >> >> I was surprised that no .c files are implicitly relying >> on these includes, but apparently none are, so we can >> safely remove them. >> >> A couple of notes: >> * this doesn't need to go to qemu-stable as it's just a >> code cleanup >> * did something go wrong with your sending of this patch? >> Its subject is "PATCH 0/4" which would usually indicate >> a cover letter for a four-patch series, but this seems >> to be a stand-alone patch. >> >> Reviewed-by: Peter Maydell <peter.mayd...@linaro.org> >> >> thanks >> -- PMM >> > -- > *Aruna* Jayasena > Embedded Systems and Computer Science Engineering Undergraduate > University of Moratuwa > *✆ **+94719872152 * > ✉ <aruna...@cse.mrt.ac.lk> ⌨ <http://archfx.github.io> in > <https://www.linkedin.com/in/arunajayasena/> > > > -- *Aruna* Jayasena Embedded Systems and Computer Science Engineering Undergraduate University of Moratuwa *✆ **+94719872152 * ✉ <aruna...@cse.mrt.ac.lk> ⌨ <http://archfx.github.io> in <https://www.linkedin.com/in/arunajayasena/>