diff options
| author | Rakesh Jeyasingh <rakeshjb010@gmail.com> | 2025-04-07 23:43:27 +0530 |
|---|---|---|
| committer | Paolo Bonzini <pbonzini@redhat.com> | 2025-04-23 10:35:23 +0200 |
| commit | 6d8c6dee3a767e7650e5d0640e13adb9f01fa37c (patch) | |
| tree | 66393e6a7ff62806039f236008a381d8a20fb5dd | |
| parent | efc5603292bfde97fd82fabcedce86310bedbc65 (diff) | |
| download | qemu-6d8c6dee3a767e7650e5d0640e13adb9f01fa37c.zip qemu-6d8c6dee3a767e7650e5d0640e13adb9f01fa37c.tar.gz qemu-6d8c6dee3a767e7650e5d0640e13adb9f01fa37c.tar.bz2 | |
rust/hw/char/pl011: Extract DR write logic into separate function
- Split `write()` DR case into `write_data_register()`
Signed-off-by: Rakesh Jeyasingh <rakeshjb010@gmail.com>
Link: https://lore.kernel.org/r/20250407181327.171563-3-rakeshjb010@gmail.com
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| -rw-r--r-- | rust/hw/char/pl011/src/device.rs | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/rust/hw/char/pl011/src/device.rs b/rust/hw/char/pl011/src/device.rs index 87153cd..bb2a0f2 100644 --- a/rust/hw/char/pl011/src/device.rs +++ b/rust/hw/char/pl011/src/device.rs @@ -221,12 +221,7 @@ impl PL011Registers { // eprintln!("write offset {offset} value {value}"); use RegisterOffset::*; match offset { - DR => { - // interrupts always checked - let _ = self.loopback_tx(value.into()); - self.int_level |= Interrupt::TX.0; - return true; - } + DR => return self.write_data_register(value), RSR => { self.receive_status_error_clear = 0.into(); } @@ -307,6 +302,13 @@ impl PL011Registers { u32::from(c) } + fn write_data_register(&mut self, value: u32) -> bool { + // interrupts always checked + let _ = self.loopback_tx(value.into()); + self.int_level |= Interrupt::TX.0; + true + } + #[inline] #[must_use] fn loopback_tx(&mut self, value: registers::Data) -> bool { |