mbox series

[RESEND,0/2] Resolve some redundant property accessors

Message ID 20220301225220.239065-1-shentey@gmail.com (mailing list archive)
Headers show
Series Resolve some redundant property accessors | expand

Message

Bernhard Beschow March 1, 2022, 10:52 p.m. UTC
No changes. Just also CC'ed to qemu-trivial.

The QOM API already provides appropriate accessors, so reuse them.

Testing done:

  :$ make check
  Ok:                 570
  Expected Fail:      0
  Fail:               0
  Unexpected Pass:    0
  Skipped:            178
  Timeout:            0


Bernhard Beschow (2):
  hw/vfio/pci-quirks: Resolve redundant property getters
  hw/riscv/sifive_u: Resolve redundant property accessors

 hw/riscv/sifive_u.c  | 24 ++++--------------------
 hw/vfio/pci-quirks.c | 34 +++++++++-------------------------
 2 files changed, 13 insertions(+), 45 deletions(-)

Comments

Bernhard Beschow May 5, 2022, 4:52 p.m. UTC | #1
Am 1. März 2022 22:52:18 UTC schrieb Bernhard Beschow <shentey@gmail.com>:
>No changes. Just also CC'ed to qemu-trivial.
>
>The QOM API already provides appropriate accessors, so reuse them.
>
>Testing done:
>
>  :$ make check
>  Ok:                 570
>  Expected Fail:      0
>  Fail:               0
>  Unexpected Pass:    0
>  Skipped:            178
>  Timeout:            0
>
>
>Bernhard Beschow (2):
>  hw/vfio/pci-quirks: Resolve redundant property getters
>  hw/riscv/sifive_u: Resolve redundant property accessors
>
> hw/riscv/sifive_u.c  | 24 ++++--------------------
> hw/vfio/pci-quirks.c | 34 +++++++++-------------------------
> 2 files changed, 13 insertions(+), 45 deletions(-)
>

Ping

First round of trivial patches went already in for 7.1, hence a reminder.

Best regards,
Bernhard
Bernhard Beschow May 13, 2022, 10:39 a.m. UTC | #2
Am 5. Mai 2022 16:52:13 UTC schrieb Bernhard Beschow <shentey@gmail.com>:
>Am 1. März 2022 22:52:18 UTC schrieb Bernhard Beschow <shentey@gmail.com>:
>>No changes. Just also CC'ed to qemu-trivial.
>>
>>The QOM API already provides appropriate accessors, so reuse them.
>>
>>Testing done:
>>
>>  :$ make check
>>  Ok:                 570
>>  Expected Fail:      0
>>  Fail:               0
>>  Unexpected Pass:    0
>>  Skipped:            178
>>  Timeout:            0
>>
>>
>>Bernhard Beschow (2):
>>  hw/vfio/pci-quirks: Resolve redundant property getters
>>  hw/riscv/sifive_u: Resolve redundant property accessors
>>
>> hw/riscv/sifive_u.c  | 24 ++++--------------------
>> hw/vfio/pci-quirks.c | 34 +++++++++-------------------------
>> 2 files changed, 13 insertions(+), 45 deletions(-)
>>
>
>Ping
>
>First round of trivial patches went already in for 7.1, hence a reminder.
>
>Best regards,
>Bernhard
Ping 2
Alistair Francis May 17, 2022, 3:41 a.m. UTC | #3
On Wed, Mar 2, 2022 at 8:54 AM Bernhard Beschow <shentey@gmail.com> wrote:
>
> No changes. Just also CC'ed to qemu-trivial.
>
> The QOM API already provides appropriate accessors, so reuse them.
>
> Testing done:
>
>   :$ make check
>   Ok:                 570
>   Expected Fail:      0
>   Fail:               0
>   Unexpected Pass:    0
>   Skipped:            178
>   Timeout:            0
>
>
> Bernhard Beschow (2):
>   hw/vfio/pci-quirks: Resolve redundant property getters
>   hw/riscv/sifive_u: Resolve redundant property accessors

Thanks!

Applied to riscv-to-apply.next

Alistair

>
>  hw/riscv/sifive_u.c  | 24 ++++--------------------
>  hw/vfio/pci-quirks.c | 34 +++++++++-------------------------
>  2 files changed, 13 insertions(+), 45 deletions(-)
>
> --
> 2.35.1
>
>
Bernhard Beschow May 17, 2022, 11:11 a.m. UTC | #4
Am 17. Mai 2022 03:41:51 UTC schrieb Alistair Francis <alistair23@gmail.com>:
>On Wed, Mar 2, 2022 at 8:54 AM Bernhard Beschow <shentey@gmail.com> wrote:
>>
>> No changes. Just also CC'ed to qemu-trivial.
>>
>> The QOM API already provides appropriate accessors, so reuse them.
>>
>> Testing done:
>>
>>   :$ make check
>>   Ok:                 570
>>   Expected Fail:      0
>>   Fail:               0
>>   Unexpected Pass:    0
>>   Skipped:            178
>>   Timeout:            0
>>
>>
>> Bernhard Beschow (2):
>>   hw/vfio/pci-quirks: Resolve redundant property getters
>>   hw/riscv/sifive_u: Resolve redundant property accessors
>
>Thanks!
>
>Applied to riscv-to-apply.next

Finally :)

Thanks,
Bernhard

>
>Alistair
>
>>
>>  hw/riscv/sifive_u.c  | 24 ++++--------------------
>>  hw/vfio/pci-quirks.c | 34 +++++++++-------------------------
>>  2 files changed, 13 insertions(+), 45 deletions(-)
>>
>> --
>> 2.35.1
>>
>>