mbox series

[0/2] !fixup target/avr merger-rc4

Message ID 20200131010941.10636-1-philmd@redhat.com (mailing list archive)
Headers show
Series !fixup target/avr merger-rc4 | expand

Message

Philippe Mathieu-Daudé Jan. 31, 2020, 1:09 a.m. UTC 
Aleksandar, I addressed Alex Bennée comment as fixup, so you
can squash directly. See:
https://www.mail-archive.com/qemu-devel@nongnu.org/msg673846.html

- convert DB_PRINT() to trace-events
- fix style/indentation

Based-on: <1580428993-4767-1-git-send-email-aleksandar.markovic@rt-rk.com>

Philippe Mathieu-Daudé (2):
  !fixup "hw/misc: Add limited support for AVR power device"
  !fixup "hw/timer: Add limited support for AVR 16-bit timer peripheral"

 hw/misc/avr_power.c    | 17 +++++++++--------
 hw/timer/avr_timer16.c | 25 +++++++++++++++----------
 hw/misc/trace-events   |  4 ++++
 hw/timer/trace-events  | 12 ++++++++++++
 4 files changed, 40 insertions(+), 18 deletions(-)

Comments

Aleksandar Markovic Jan. 31, 2020, 1:12 a.m. UTC | #1 
On Fri, Jan 31, 2020 at 2:09 AM Philippe Mathieu-Daudé <philmd@redhat.com>
wrote:
>
> Aleksandar, I addressed Alex Bennée comment as fixup, so you
> can squash directly. See:
> https://www.mail-archive.com/qemu-devel@nongnu.org/msg673846.html
>
> - convert DB_PRINT() to trace-events
> - fix style/indentation
>
> Based-on: <1580428993-4767-1-git-send-email-aleksandar.markovic@rt-rk.com>
>

That is great!

I am going to squash it into corresponding patches, and mention that in
commit messages.
Then I guess some r-b (by Alex) can be added.

Cool!

> Philippe Mathieu-Daudé (2):
>   !fixup "hw/misc: Add limited support for AVR power device"
>   !fixup "hw/timer: Add limited support for AVR 16-bit timer peripheral"
>
>  hw/misc/avr_power.c    | 17 +++++++++--------
>  hw/timer/avr_timer16.c | 25 +++++++++++++++----------
>  hw/misc/trace-events   |  4 ++++
>  hw/timer/trace-events  | 12 ++++++++++++
>  4 files changed, 40 insertions(+), 18 deletions(-)
>
> --
> 2.21.1
>