Skip to content

Commit

Permalink
aspeed/smc: Use max number of CE instead of 'num_cs'
Browse files Browse the repository at this point in the history
The Aspeed SMC model uses the 'num_cs' field to allocate resources
fitting the number of devices of the machine. This is a small
optimization without real need in the controller. Simplify modelling
and use the max_peripherals field instead.

Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20220307071856.1410731-2-clg@kaod.org>
Signed-off-by: Cédric Le Goater <clg@kaod.org>
  • Loading branch information
legoater committed Mar 8, 2022
1 parent 05e6e40 commit dcacfa9
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions hw/ssi/aspeed_smc.c
Expand Up @@ -693,7 +693,7 @@ static void aspeed_smc_reset(DeviceState *d)
}

/* Unselect all peripherals */
for (i = 0; i < s->num_cs; ++i) {
for (i = 0; i < asc->max_peripherals; ++i) {
s->regs[s->r_ctrl0 + i] |= CTRL_CE_STOP_ACTIVE;
qemu_set_irq(s->cs_lines[i], true);
}
Expand Down Expand Up @@ -1042,7 +1042,7 @@ static void aspeed_smc_write(void *opaque, hwaddr addr, uint64_t data,
addr < s->r_timings + asc->nregs_timings) ||
addr == s->r_ce_ctrl) {
s->regs[addr] = value;
} else if (addr >= s->r_ctrl0 && addr < s->r_ctrl0 + s->num_cs) {
} else if (addr >= s->r_ctrl0 && addr < s->r_ctrl0 + asc->max_peripherals) {
int cs = addr - s->r_ctrl0;
aspeed_smc_flash_update_ctrl(&s->flashes[cs], value);
} else if (addr >= R_SEG_ADDR0 &&
Expand Down Expand Up @@ -1139,9 +1139,9 @@ static void aspeed_smc_realize(DeviceState *dev, Error **errp)
s->spi = ssi_create_bus(dev, "spi");

/* Setup cs_lines for peripherals */
s->cs_lines = g_new0(qemu_irq, s->num_cs);
s->cs_lines = g_new0(qemu_irq, asc->max_peripherals);

for (i = 0; i < s->num_cs; ++i) {
for (i = 0; i < asc->max_peripherals; ++i) {
sysbus_init_irq(sbd, &s->cs_lines[i]);
}

Expand Down

0 comments on commit dcacfa9

Please sign in to comment.