[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/2] goldfish_rtc: change MemoryRegionOps endianness to DEVIC
From: |
Laurent Vivier |
Subject: |
Re: [PATCH 2/2] goldfish_rtc: change MemoryRegionOps endianness to DEVICE_NATIVE_ENDIAN |
Date: |
Mon, 12 Oct 2020 16:40:21 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.11.0 |
Le 09/10/2020 à 13:38, Laurent Vivier a écrit :
> The doc [1] doesn't define the endianness, but the kernel driver
> uses readl() to access the registers, so we can guess it depends
> on the architecture endianness.
>
> As riscv architecture endianness is little it might not change anything
> for it.
>
> Moreover, android implementation uses DEVICE_NATIVE_ENDIAN [2]
>
> [1]
> https://android.googlesource.com/platform/external/qemu/+/master/docs/GOLDFISH-VIRTUAL-HARDWARE.TXT
> [2]
> https://android.googlesource.com/platform/external/qemu/+/refs/heads/emu-master-dev/hw/timer/goldfish_timer.c#177
>
> Fixes: 9a5b40b84279 ("hw: rtc: Add Goldfish RTC device")
> Cc: Anup.Patel@wdc.com
> Signed-off-by: Laurent Vivier <lvivier@redhat.com>
> ---
> hw/rtc/goldfish_rtc.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/rtc/goldfish_rtc.c b/hw/rtc/goldfish_rtc.c
> index 6ddd45cce039..0f4e8185a796 100644
> --- a/hw/rtc/goldfish_rtc.c
> +++ b/hw/rtc/goldfish_rtc.c
> @@ -217,7 +217,7 @@ static int goldfish_rtc_post_load(void *opaque, int
> version_id)
> static const MemoryRegionOps goldfish_rtc_ops = {
> .read = goldfish_rtc_read,
> .write = goldfish_rtc_write,
> - .endianness = DEVICE_LITTLE_ENDIAN,
> + .endianness = DEVICE_NATIVE_ENDIAN,
> .valid = {
> .min_access_size = 4,
> .max_access_size = 4
>
Applied to my trivial-patches branch.
Thanks,
Laurent
- [PATCH 1/2] hw/char/serial: remove duplicate .class_init in serial_mm_info, Laurent Vivier, 2020/10/09
- [PATCH 2/2] goldfish_rtc: change MemoryRegionOps endianness to DEVICE_NATIVE_ENDIAN, Laurent Vivier, 2020/10/09
- Re: [PATCH 1/2] hw/char/serial: remove duplicate .class_init in serial_mm_info, Philippe Mathieu-Daudé, 2020/10/09
- Re: [PATCH 1/2] hw/char/serial: remove duplicate .class_init in serial_mm_info, Li Qiang, 2020/10/09
- Re: [PATCH 1/2] hw/char/serial: remove duplicate .class_init in serial_mm_info, Alistair Francis, 2020/10/09
- Re: [PATCH 1/2] hw/char/serial: remove duplicate .class_init in serial_mm_info, Laurent Vivier, 2020/10/12