[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 09/10] ide: collapse parameters to ide_init_drive
From: |
Paolo Bonzini |
Subject: |
[PULL 09/10] ide: collapse parameters to ide_init_drive |
Date: |
Wed, 28 Feb 2024 09:06:45 +0100 |
All calls to ide_init_drive comes from ide_dev_initfn. Just pass down the
IDEDevice (IDEState is kinda obsolete and should be merged into IDEDevice).
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
include/hw/ide/internal.h | 6 +-----
hw/ide/core.c | 40 ++++++++++++++++++---------------------
hw/ide/ide-dev.c | 5 +----
3 files changed, 20 insertions(+), 31 deletions(-)
diff --git a/include/hw/ide/internal.h b/include/hw/ide/internal.h
index 20dde37f458..0d64805da20 100644
--- a/include/hw/ide/internal.h
+++ b/include/hw/ide/internal.h
@@ -416,11 +416,7 @@ uint32_t ide_data_readw(void *opaque, uint32_t addr);
void ide_data_writel(void *opaque, uint32_t addr, uint32_t val);
uint32_t ide_data_readl(void *opaque, uint32_t addr);
-int ide_init_drive(IDEState *s, BlockBackend *blk, IDEDriveKind kind,
- const char *version, const char *serial, const char *model,
- uint64_t wwn,
- uint32_t cylinders, uint32_t heads, uint32_t secs,
- int chs_trans, Error **errp);
+int ide_init_drive(IDEState *s, IDEDevice *dev, IDEDriveKind kind, Error
**errp);
void ide_exit(IDEState *s);
void ide_bus_init_output_irq(IDEBus *bus, qemu_irq irq_out);
int ide_init_ioport(IDEBus *bus, ISADevice *isa, int iobase, int iobase2);
diff --git a/hw/ide/core.c b/hw/ide/core.c
index 9c4a8129028..3c42d72ac25 100644
--- a/hw/ide/core.c
+++ b/hw/ide/core.c
@@ -2589,24 +2589,20 @@ static const BlockDevOps ide_hd_block_ops = {
.resize_cb = ide_resize_cb,
};
-int ide_init_drive(IDEState *s, BlockBackend *blk, IDEDriveKind kind,
- const char *version, const char *serial, const char *model,
- uint64_t wwn,
- uint32_t cylinders, uint32_t heads, uint32_t secs,
- int chs_trans, Error **errp)
+int ide_init_drive(IDEState *s, IDEDevice *dev, IDEDriveKind kind, Error
**errp)
{
uint64_t nb_sectors;
- s->blk = blk;
+ s->blk = dev->conf.blk;
s->drive_kind = kind;
- blk_get_geometry(blk, &nb_sectors);
- s->cylinders = cylinders;
- s->heads = s->drive_heads = heads;
- s->sectors = s->drive_sectors = secs;
- s->chs_trans = chs_trans;
+ blk_get_geometry(s->blk, &nb_sectors);
+ s->cylinders = dev->conf.cyls;
+ s->heads = s->drive_heads = dev->conf.heads;
+ s->sectors = s->drive_sectors = dev->conf.secs;
+ s->chs_trans = dev->chs_trans;
s->nb_sectors = nb_sectors;
- s->wwn = wwn;
+ s->wwn = dev->wwn;
/* The SMART values should be preserved across power cycles
but they aren't. */
s->smart_enabled = 1;
@@ -2614,26 +2610,26 @@ int ide_init_drive(IDEState *s, BlockBackend *blk,
IDEDriveKind kind,
s->smart_errors = 0;
s->smart_selftest_count = 0;
if (kind == IDE_CD) {
- blk_set_dev_ops(blk, &ide_cd_block_ops, s);
+ blk_set_dev_ops(s->blk, &ide_cd_block_ops, s);
} else {
if (!blk_is_inserted(s->blk)) {
error_setg(errp, "Device needs media, but drive is empty");
return -1;
}
- if (!blk_is_writable(blk)) {
+ if (!blk_is_writable(s->blk)) {
error_setg(errp, "Can't use a read-only drive");
return -1;
}
- blk_set_dev_ops(blk, &ide_hd_block_ops, s);
+ blk_set_dev_ops(s->blk, &ide_hd_block_ops, s);
}
- if (serial) {
- pstrcpy(s->drive_serial_str, sizeof(s->drive_serial_str), serial);
+ if (dev->serial) {
+ pstrcpy(s->drive_serial_str, sizeof(s->drive_serial_str), dev->serial);
} else {
snprintf(s->drive_serial_str, sizeof(s->drive_serial_str),
"QM%05d", s->drive_serial);
}
- if (model) {
- pstrcpy(s->drive_model_str, sizeof(s->drive_model_str), model);
+ if (dev->model) {
+ pstrcpy(s->drive_model_str, sizeof(s->drive_model_str), dev->model);
} else {
switch (kind) {
case IDE_CD:
@@ -2648,14 +2644,14 @@ int ide_init_drive(IDEState *s, BlockBackend *blk,
IDEDriveKind kind,
}
}
- if (version) {
- pstrcpy(s->version, sizeof(s->version), version);
+ if (dev->version) {
+ pstrcpy(s->version, sizeof(s->version), dev->version);
} else {
pstrcpy(s->version, sizeof(s->version), qemu_hw_version());
}
ide_reset(s);
- blk_iostatus_enable(blk);
+ blk_iostatus_enable(s->blk);
return 0;
}
diff --git a/hw/ide/ide-dev.c b/hw/ide/ide-dev.c
index c8e2033469c..900f80faf19 100644
--- a/hw/ide/ide-dev.c
+++ b/hw/ide/ide-dev.c
@@ -118,10 +118,7 @@ void ide_dev_initfn(IDEDevice *dev, IDEDriveKind kind,
Error **errp)
return;
}
- if (ide_init_drive(s, dev->conf.blk, kind,
- dev->version, dev->serial, dev->model, dev->wwn,
- dev->conf.cyls, dev->conf.heads, dev->conf.secs,
- dev->chs_trans, errp) < 0) {
+ if (ide_init_drive(s, dev, kind, errp) < 0) {
return;
}
--
2.43.2
- [PULL 00/10] pc, target/i486 changes for 2024-02-27, Paolo Bonzini, 2024/02/28
- [PULL 01/10] vl, pc: turn -no-fd-bootchk into a machine property, Paolo Bonzini, 2024/02/28
- [PULL 03/10] target/i386: check validity of VMCB addresses, Paolo Bonzini, 2024/02/28
- [PULL 04/10] target/i386: introduce function to query MMU indices, Paolo Bonzini, 2024/02/28
- [PULL 02/10] target/i386: mask high bits of CR3 in 32-bit mode, Paolo Bonzini, 2024/02/28
- [PULL 05/10] target/i386: use separate MMU indexes for 32-bit accesses, Paolo Bonzini, 2024/02/28
- [PULL 07/10] target/i386: remove unnecessary/wrong application of the A20 mask, Paolo Bonzini, 2024/02/28
- [PULL 06/10] target/i386: Fix physical address truncation, Paolo Bonzini, 2024/02/28
- [PULL 08/10] target/i386: leave the A20 bit set in the final NPT walk, Paolo Bonzini, 2024/02/28
- [PULL 09/10] ide: collapse parameters to ide_init_drive,
Paolo Bonzini <=
- [PULL 10/10] ide, vl: turn -win2k-hack into a property on IDE devices, Paolo Bonzini, 2024/02/28
- Re: [PULL 00/10] pc, target/i486 changes for 2024-02-27, Peter Maydell, 2024/02/28