Re: [Qemu-devel] [PATCH 09/10] sm501: Add some more missing registers

2017-02-25 Thread Peter Maydell
On 24 February 2017 at 21:49, BALATON Zoltan wrote: > On Fri, 24 Feb 2017, BALATON Zoltan wrote: >> >> On Fri, 24 Feb 2017, Peter Maydell wrote: >>> >>> On 19 February 2017 at 16:35, BALATON Zoltan wrote: Write only to allow clients to initialise these without failing Signed-o

Re: [Qemu-devel] [PATCH 09/10] sm501: Add some more missing registers

2017-02-24 Thread BALATON Zoltan
On Fri, 24 Feb 2017, BALATON Zoltan wrote: On Fri, 24 Feb 2017, Peter Maydell wrote: On 19 February 2017 at 16:35, BALATON Zoltan wrote: Write only to allow clients to initialise these without failing Signed-off-by: BALATON Zoltan What's the point in write-only register values? U-boot wr

Re: [Qemu-devel] [PATCH 09/10] sm501: Add some more missing registers

2017-02-24 Thread BALATON Zoltan
On Fri, 24 Feb 2017, Peter Maydell wrote: On 19 February 2017 at 16:35, BALATON Zoltan wrote: Write only to allow clients to initialise these without failing Signed-off-by: BALATON Zoltan What's the point in write-only register values? U-boot writes this register during setting up the dev

Re: [Qemu-devel] [PATCH 09/10] sm501: Add some more missing registers

2017-02-24 Thread Peter Maydell
On 19 February 2017 at 16:35, BALATON Zoltan wrote: > Write only to allow clients to initialise these without failing > > Signed-off-by: BALATON Zoltan What's the point in write-only register values? What does the real hardware do here? If the registers are writes-ignored, there's no need to s

[Qemu-devel] [PATCH 09/10] sm501: Add some more missing registers

2017-02-19 Thread BALATON Zoltan
Write only to allow clients to initialise these without failing Signed-off-by: BALATON Zoltan --- hw/display/sm501.c | 42 ++ 1 file changed, 42 insertions(+) diff --git a/hw/display/sm501.c b/hw/display/sm501.c index 2e1c4b7..16a00cc 100644 --- a/hw/disp