Re: [PATCH 1/4] dp8393x: don't force 32-bit register access

2021-07-08 Thread Mark Cave-Ayland
On 08/07/2021 01:52, Finn Thain wrote: On Wed, 7 Jul 2021, Mark Cave-Ayland wrote: However this conflicts with what you mention above that the SONIC is hard-coded into little-endian mode, in which case we would still need to keep it. If you want to fully implement BMODE in QEMU then you'll

Re: [PATCH 1/4] dp8393x: don't force 32-bit register access

2021-07-07 Thread Finn Thain
On Wed, 7 Jul 2021, Mark Cave-Ayland wrote: > However this conflicts with what you mention above that the SONIC is > hard-coded into little-endian mode, in which case we would still need to > keep it. > If you want to fully implement BMODE in QEMU then you'll need to abandon native endiannes

Re: [PATCH 1/4] dp8393x: don't force 32-bit register access

2021-07-07 Thread Mark Cave-Ayland
On 07/07/2021 00:51, Finn Thain wrote: On Mon, 5 Jul 2021, Mark Cave-Ayland wrote: Commit 3fe9a838ec "dp8393x: Always use 32-bit accesses" set .impl.min_access_size and .impl.max_access_size to 4 to try and fix the Linux jazzsonic driver which uses 32-bit accesses. The problem with forcing

Re: [PATCH 1/4] dp8393x: don't force 32-bit register access

2021-07-06 Thread Finn Thain
On Mon, 5 Jul 2021, Mark Cave-Ayland wrote: > Commit 3fe9a838ec "dp8393x: Always use 32-bit accesses" set > .impl.min_access_size > and .impl.max_access_size to 4 to try and fix the Linux jazzsonic driver > which uses > 32-bit accesses. > > The problem with forcing the register access to

Re: [PATCH 1/4] dp8393x: don't force 32-bit register access

2021-07-06 Thread Philippe Mathieu-Daudé
On 7/6/21 9:22 PM, Mark Cave-Ayland wrote: > On 06/07/2021 18:18, Philippe Mathieu-Daudé wrote: >> On 7/5/21 11:49 PM, Mark Cave-Ayland wrote: >>> Commit 3fe9a838ec "dp8393x: Always use 32-bit accesses" set >>> .impl.min_access_size >>> and .impl.max_access_size to 4 to try and fix the Linux

Re: [PATCH 1/4] dp8393x: don't force 32-bit register access

2021-07-06 Thread Mark Cave-Ayland
On 06/07/2021 18:18, Philippe Mathieu-Daudé wrote: On 7/5/21 11:49 PM, Mark Cave-Ayland wrote: Commit 3fe9a838ec "dp8393x: Always use 32-bit accesses" set .impl.min_access_size and .impl.max_access_size to 4 to try and fix the Linux jazzsonic driver which uses 32-bit accesses. The problem

Re: [PATCH 1/4] dp8393x: don't force 32-bit register access

2021-07-06 Thread Philippe Mathieu-Daudé
On 7/5/21 11:49 PM, Mark Cave-Ayland wrote: > Commit 3fe9a838ec "dp8393x: Always use 32-bit accesses" set > .impl.min_access_size > and .impl.max_access_size to 4 to try and fix the Linux jazzsonic driver > which uses > 32-bit accesses. > > The problem with forcing the register access to 32-bit

[PATCH 1/4] dp8393x: don't force 32-bit register access

2021-07-05 Thread Mark Cave-Ayland
Commit 3fe9a838ec "dp8393x: Always use 32-bit accesses" set .impl.min_access_size and .impl.max_access_size to 4 to try and fix the Linux jazzsonic driver which uses 32-bit accesses. The problem with forcing the register access to 32-bit in this way is that since the dp8393x uses 16-bit