Message ID | 20210222130514.2167-3-bmeng.cn@gmail.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | hw/arm: zynqmp: Implement a CSU DMA model and connect it with GQSPI | expand |
On Mon, Feb 22, 2021 at 09:05:11PM +0800, Bin Meng wrote: > From: Xuzhou Cheng <xuzhou.cheng@windriver.com> > > There are some coding convention warnings in xlnx-zynqmp.c and > xlnx-zynqmp.h, as reported by: > > $ ./scripts/checkpatch.pl include/hw/arm/xlnx-zynqmp.h > $ ./scripts/checkpatch.pl hw/arm/xlnx-zynqmp.c > > Let's clean them up. > > Signed-off-by: Xuzhou Cheng <xuzhou.cheng@windriver.com> > Signed-off-by: Bin Meng <bin.meng@windriver.com> Reviewed-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com> > > --- > > Changes in v4: > - remove one change that is not a checkpatch warning > > include/hw/arm/xlnx-zynqmp.h | 3 ++- > hw/arm/xlnx-zynqmp.c | 9 ++++++--- > 2 files changed, 8 insertions(+), 4 deletions(-) > > diff --git a/include/hw/arm/xlnx-zynqmp.h b/include/hw/arm/xlnx-zynqmp.h > index 6f45387a17..be15cc8814 100644 > --- a/include/hw/arm/xlnx-zynqmp.h > +++ b/include/hw/arm/xlnx-zynqmp.h > @@ -60,7 +60,8 @@ OBJECT_DECLARE_SIMPLE_TYPE(XlnxZynqMPState, XLNX_ZYNQMP) > > #define XLNX_ZYNQMP_GIC_REGIONS 6 > > -/* ZynqMP maps the ARM GIC regions (GICC, GICD ...) at consecutive 64k offsets > +/* > + * ZynqMP maps the ARM GIC regions (GICC, GICD ...) at consecutive 64k offsets > * and under-decodes the 64k region. This mirrors the 4k regions to every 4k > * aligned address in the 64k region. To implement each GIC region needs a > * number of memory region aliases. > diff --git a/hw/arm/xlnx-zynqmp.c b/hw/arm/xlnx-zynqmp.c > index 881847255b..49465a2794 100644 > --- a/hw/arm/xlnx-zynqmp.c > +++ b/hw/arm/xlnx-zynqmp.c > @@ -301,11 +301,13 @@ static void xlnx_zynqmp_realize(DeviceState *dev, Error **errp) > > ram_size = memory_region_size(s->ddr_ram); > > - /* Create the DDR Memory Regions. User friendly checks should happen at > + /* > + * Create the DDR Memory Regions. User friendly checks should happen at > * the board level > */ > if (ram_size > XLNX_ZYNQMP_MAX_LOW_RAM_SIZE) { > - /* The RAM size is above the maximum available for the low DDR. > + /* > + * The RAM size is above the maximum available for the low DDR. > * Create the high DDR memory region as well. > */ > assert(ram_size <= XLNX_ZYNQMP_MAX_RAM_SIZE); > @@ -526,7 +528,8 @@ static void xlnx_zynqmp_realize(DeviceState *dev, Error **errp) > SysBusDevice *sbd = SYS_BUS_DEVICE(&s->sdhci[i]); > Object *sdhci = OBJECT(&s->sdhci[i]); > > - /* Compatible with: > + /* > + * Compatible with: > * - SD Host Controller Specification Version 3.00 > * - SDIO Specification Version 3.0 > * - eMMC Specification Version 4.51 > -- > 2.25.1 >
diff --git a/include/hw/arm/xlnx-zynqmp.h b/include/hw/arm/xlnx-zynqmp.h index 6f45387a17..be15cc8814 100644 --- a/include/hw/arm/xlnx-zynqmp.h +++ b/include/hw/arm/xlnx-zynqmp.h @@ -60,7 +60,8 @@ OBJECT_DECLARE_SIMPLE_TYPE(XlnxZynqMPState, XLNX_ZYNQMP) #define XLNX_ZYNQMP_GIC_REGIONS 6 -/* ZynqMP maps the ARM GIC regions (GICC, GICD ...) at consecutive 64k offsets +/* + * ZynqMP maps the ARM GIC regions (GICC, GICD ...) at consecutive 64k offsets * and under-decodes the 64k region. This mirrors the 4k regions to every 4k * aligned address in the 64k region. To implement each GIC region needs a * number of memory region aliases. diff --git a/hw/arm/xlnx-zynqmp.c b/hw/arm/xlnx-zynqmp.c index 881847255b..49465a2794 100644 --- a/hw/arm/xlnx-zynqmp.c +++ b/hw/arm/xlnx-zynqmp.c @@ -301,11 +301,13 @@ static void xlnx_zynqmp_realize(DeviceState *dev, Error **errp) ram_size = memory_region_size(s->ddr_ram); - /* Create the DDR Memory Regions. User friendly checks should happen at + /* + * Create the DDR Memory Regions. User friendly checks should happen at * the board level */ if (ram_size > XLNX_ZYNQMP_MAX_LOW_RAM_SIZE) { - /* The RAM size is above the maximum available for the low DDR. + /* + * The RAM size is above the maximum available for the low DDR. * Create the high DDR memory region as well. */ assert(ram_size <= XLNX_ZYNQMP_MAX_RAM_SIZE); @@ -526,7 +528,8 @@ static void xlnx_zynqmp_realize(DeviceState *dev, Error **errp) SysBusDevice *sbd = SYS_BUS_DEVICE(&s->sdhci[i]); Object *sdhci = OBJECT(&s->sdhci[i]); - /* Compatible with: + /* + * Compatible with: * - SD Host Controller Specification Version 3.00 * - SDIO Specification Version 3.0 * - eMMC Specification Version 4.51