On Fri, Feb 7, 2025 at 9:40 AM Paolo Bonzini wrote:
>
> Being the first crate added to QEMU, pl011 has a rather restrictive
> Clippy setup. This can be sometimes a bit too heavy on its suggestions,
> for example
>
> error: this could be a `const fn`
>--> hw/char/pl011/src/device.rs:382:5
> |
> 382 | / fn set_read_trigger(&mut self) {
> 383 | | self.read_trigger = 1;
> 384 | | }
> | |_^
>
> Just use the standard set that is present in rust/Cargo.toml, with
> just a small adjustment to allow upper case acronyms which are used
> for register names.
>
> Reported-by: Stefan Hajnoczi
> Signed-off-by: Paolo Bonzini
> ---
> rust/hw/char/pl011/src/lib.rs | 9 -
> 1 file changed, 9 deletions(-)
(I'm late, don't worry if my R-b gets lost.)
Reviewed-by: Stefan Hajnoczi
>
> diff --git a/rust/hw/char/pl011/src/lib.rs b/rust/hw/char/pl011/src/lib.rs
> index 88452dc888c..1bf46c65af2 100644
> --- a/rust/hw/char/pl011/src/lib.rs
> +++ b/rust/hw/char/pl011/src/lib.rs
> @@ -12,16 +12,7 @@
> //! See [`PL011State`](crate::device::PL011State) for the device model type
> and
> //! the [`registers`] module for register types.
>
> -#![deny(
> -clippy::correctness,
> -clippy::suspicious,
> -clippy::complexity,
> -clippy::perf,
> -clippy::nursery,
> -clippy::style
> -)]
> #![allow(clippy::upper_case_acronyms)]
> -#![allow(clippy::result_unit_err)]
>
> use qemu_api::c_str;
>
> --
> 2.48.1
>
>
