[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 04/10] aspeed: Clean up includes
From: |
Peter Maydell |
Subject: |
Re: [PATCH 04/10] aspeed: Clean up includes |
Date: |
Fri, 26 Jan 2024 15:38:56 +0000 |
On Fri, 26 Jan 2024 at 15:26, Zhao Liu <zhao1.liu@intel.com> wrote:
>
> On Thu, Jan 25, 2024 at 04:34:02PM +0000, Peter Maydell wrote:
> > Date: Thu, 25 Jan 2024 16:34:02 +0000
> > From: Peter Maydell <peter.maydell@linaro.org>
> > Subject: [PATCH 04/10] aspeed: Clean up includes
> > X-Mailer: git-send-email 2.34.1
> >
> > This commit was created with scripts/clean-includes.
> >
> > All .c should include qemu/osdep.h first. The script performs three
> > related cleanups:
> >
> > * Ensure .c files include qemu/osdep.h first.
> > * Including it in a .h is redundant, since the .c already includes
> > it. Drop such inclusions.
> > * Likewise, including headers qemu/osdep.h includes is redundant.
> > Drop these, too.
> >
> > Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> > ---
> > hw/arm/aspeed_eeprom.h | 1 -
> > tests/qtest/qtest_aspeed.h | 2 --
> > hw/arm/aspeed_eeprom.c | 1 +
> > 3 files changed, 1 insertion(+), 3 deletions(-)
> >
> > diff --git a/hw/arm/aspeed_eeprom.h b/hw/arm/aspeed_eeprom.h
> > index bbf9e54365b..f08c16ef506 100644
> > --- a/hw/arm/aspeed_eeprom.h
> > +++ b/hw/arm/aspeed_eeprom.h
> > @@ -7,7 +7,6 @@
> > #ifndef ASPEED_EEPROM_H
> > #define ASPEED_EEPROM_H
> >
> > -#include "qemu/osdep.h"
> >
> > extern const uint8_t tiogapass_bmc_fruid[];
> > extern const size_t tiogapass_bmc_fruid_len;
> > diff --git a/tests/qtest/qtest_aspeed.h b/tests/qtest/qtest_aspeed.h
> > index 235dfaa186a..d35b0c7cba5 100644
> > --- a/tests/qtest/qtest_aspeed.h
> > +++ b/tests/qtest/qtest_aspeed.h
> > @@ -12,8 +12,6 @@
> > #ifndef QTEST_ASPEED_H
> > #define QTEST_ASPEED_H
> >
> > -#include <stdint.h>
> > -
> > #include "libqtest.h"
> >
> > #define AST2600_ASPEED_I2C_BASE_ADDR 0x1e78a000
> > diff --git a/hw/arm/aspeed_eeprom.c b/hw/arm/aspeed_eeprom.c
> > index ace5266cec9..daa3d329d10 100644
> > --- a/hw/arm/aspeed_eeprom.c
> > +++ b/hw/arm/aspeed_eeprom.c
> > @@ -4,6 +4,7 @@
> > * SPDX-License-Identifier: GPL-2.0-only
> > */
> >
> > +#include "qemu/osdep.h"
>
> Just a question, as the code style, should we add a blank line after
> this header?
No, we don't generally include blank lines in the #include
block. (But all these patches are autogenerated, so you get
what the script generates :-))
thanks
-- PMM
- [PATCH 00/10] Clean up includes, Peter Maydell, 2024/01/25
- [PATCH 03/10] disas/riscv: Clean up includes, Peter Maydell, 2024/01/25
- [PATCH 04/10] aspeed: Clean up includes, Peter Maydell, 2024/01/25
- [PATCH 01/10] scripts/clean-includes: Update exclude list, Peter Maydell, 2024/01/25
- [PATCH 06/10] m68k: Clean up includes, Peter Maydell, 2024/01/25
- [PATCH 02/10] hyperv: Clean up includes, Peter Maydell, 2024/01/25
- [PATCH 09/10] riscv: Clean up includes, Peter Maydell, 2024/01/25