[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH qemu.git v2 7/9] hw/timer/imx_epit: factor out register write
From: |
Peter Maydell |
Subject: |
Re: [PATCH qemu.git v2 7/9] hw/timer/imx_epit: factor out register write handlers |
Date: |
Fri, 18 Nov 2022 16:05:29 +0000 |
On Mon, 7 Nov 2022 at 16:42, ~axelheider <axelheider@git.sr.ht> wrote:
>
> From: Axel Heider <axel.heider@hensoldt.net>
>
> Signed-off-by: Axel Heider <axel.heider@hensoldt.net>
> ---
> hw/timer/imx_epit.c | 211 ++++++++++++++++++++++++--------------------
> 1 file changed, 115 insertions(+), 96 deletions(-)
Good idea (unfortunate that git diff has not shown the change
in a very clear way, but it does that sometimes). I won't
review this one for now because changes I've suggested for
earlier patches are going to affect the detail on this one.
(If you don't already, you might try setting
git config --global diff.algorithm histogram
which I think often produces nicer patches. I don't know
whether it will help in this particular case, though.)
thanks
-- PMM
- [PATCH qemu.git v2 2/9] hw/timer/imx_epit: cleanup CR defines, (continued)
- [PATCH qemu.git v2 2/9] hw/timer/imx_epit: cleanup CR defines, ~axelheider, 2022/11/07
- [PATCH qemu.git v2 9/9] hw/timer/imx_epit: fix compare timer handling, ~axelheider, 2022/11/07
- [PATCH qemu.git v2 8/9] hw/timer/imx_epit: change reset handling, ~axelheider, 2022/11/07
- [PATCH qemu.git v2 6/9] hw/timer/imx_epit: remove explicit fields cnt and freq, ~axelheider, 2022/11/07
- [PATCH qemu.git v2 7/9] hw/timer/imx_epit: factor out register write handlers, ~axelheider, 2022/11/07
- Re: [PATCH qemu.git v2 7/9] hw/timer/imx_epit: factor out register write handlers,
Peter Maydell <=