Re: [PATCH v5 11/16] hw/microblaze: Support various endianness for s3adsp1800 machines

On 6/2/25 14:20, Daniel P. Berrangé wrote: On Thu, Feb 06, 2025 at 02:10:47PM +0100, Philippe Mathieu-Daudé wrote: Introduce an abstract machine parent class which defines the 'little_endian' property. Duplicate the current machine, which endian is tied to the binary endianness, to one big endia

Re: [PATCH v5 11/16] hw/microblaze: Support various endianness for s3adsp1800 machines

On 6/2/25 19:29, Daniel P. Berrangé wrote: On Thu, Feb 06, 2025 at 07:24:55PM +0100, Philippe Mathieu-Daudé wrote: +Michal On 6/2/25 19:06, Daniel P. Berrangé wrote: On Thu, Feb 06, 2025 at 06:49:38PM +0100, Philippe Mathieu-Daudé wrote: On 6/2/25 18:12, Daniel P. Berrangé wrote: On Thu, Feb

Re: [PATCH v5 11/16] hw/microblaze: Support various endianness for s3adsp1800 machines

(sorry, posted too quick) On 6/2/25 19:24, Philippe Mathieu-Daudé wrote: +Michal On 6/2/25 19:06, Daniel P. Berrangé wrote: On Thu, Feb 06, 2025 at 06:49:38PM +0100, Philippe Mathieu-Daudé wrote: On 6/2/25 18:12, Daniel P. Berrangé wrote: On Thu, Feb 06, 2025 at 04:04:20PM +0100, Philippe Ma

Re: [PATCH v5 11/16] hw/microblaze: Support various endianness for s3adsp1800 machines

On Thu, Feb 06, 2025 at 07:24:55PM +0100, Philippe Mathieu-Daudé wrote: > +Michal > > On 6/2/25 19:06, Daniel P. Berrangé wrote: > > On Thu, Feb 06, 2025 at 06:49:38PM +0100, Philippe Mathieu-Daudé wrote: > > > On 6/2/25 18:12, Daniel P. Berrangé wrote: > > > > On Thu, Feb 06, 2025 at 04:04:20PM +

Re: [PATCH v5 11/16] hw/microblaze: Support various endianness for s3adsp1800 machines

+Michal On 6/2/25 19:06, Daniel P. Berrangé wrote: On Thu, Feb 06, 2025 at 06:49:38PM +0100, Philippe Mathieu-Daudé wrote: On 6/2/25 18:12, Daniel P. Berrangé wrote: On Thu, Feb 06, 2025 at 04:04:20PM +0100, Philippe Mathieu-Daudé wrote: On 6/2/25 15:31, Daniel P. Berrangé wrote: On Thu, Feb

Re: [PATCH v5 11/16] hw/microblaze: Support various endianness for s3adsp1800 machines

On Thu, Feb 06, 2025 at 06:49:38PM +0100, Philippe Mathieu-Daudé wrote: > On 6/2/25 18:12, Daniel P. Berrangé wrote: > > On Thu, Feb 06, 2025 at 04:04:20PM +0100, Philippe Mathieu-Daudé wrote: > > > On 6/2/25 15:31, Daniel P. Berrangé wrote: > > > > On Thu, Feb 06, 2025 at 02:53:58PM +0100, Philipp

Re: [PATCH v5 11/16] hw/microblaze: Support various endianness for s3adsp1800 machines

On 6/2/25 18:12, Daniel P. Berrangé wrote: On Thu, Feb 06, 2025 at 04:04:20PM +0100, Philippe Mathieu-Daudé wrote: On 6/2/25 15:31, Daniel P. Berrangé wrote: On Thu, Feb 06, 2025 at 02:53:58PM +0100, Philippe Mathieu-Daudé wrote: Hi Daniel, On 6/2/25 14:20, Daniel P. Berrangé wrote: On Thu,

Re: [PATCH v5 11/16] hw/microblaze: Support various endianness for s3adsp1800 machines

On 6/2/25 18:34, Max Filippov wrote: On Thu, Feb 6, 2025 at 7:04 AM Philippe Mathieu-Daudé wrote: On 6/2/25 15:31, Daniel P. Berrangé wrote: We would pick an arbitrary endianness of our choosing I guess. How does this work in physical machines ? Is the choice of endianess a firmware setting, o

Re: [PATCH v5 11/16] hw/microblaze: Support various endianness for s3adsp1800 machines

On Thu, Feb 6, 2025 at 7:04 AM Philippe Mathieu-Daudé wrote: > On 6/2/25 15:31, Daniel P. Berrangé wrote: > > We would pick an arbitrary endianness of our choosing > > I guess. How does this work in physical machines ? Is > > the choice of endianess a firmware setting, or a choice > > by the vendo

Re: [PATCH v5 11/16] hw/microblaze: Support various endianness for s3adsp1800 machines

On 06/02/2025 16.04, Philippe Mathieu-Daudé wrote: On 6/2/25 15:31, Daniel P. Berrangé wrote: On Thu, Feb 06, 2025 at 02:53:58PM +0100, Philippe Mathieu-Daudé wrote: Hi Daniel, On 6/2/25 14:20, Daniel P. Berrangé wrote: On Thu, Feb 06, 2025 at 02:10:47PM +0100, Philippe Mathieu-Daudé wrote:

Re: [PATCH v5 11/16] hw/microblaze: Support various endianness for s3adsp1800 machines

On Thu, Feb 06, 2025 at 04:04:20PM +0100, Philippe Mathieu-Daudé wrote: > On 6/2/25 15:31, Daniel P. Berrangé wrote: > > On Thu, Feb 06, 2025 at 02:53:58PM +0100, Philippe Mathieu-Daudé wrote: > > > Hi Daniel, > > > > > > On 6/2/25 14:20, Daniel P. Berrangé wrote: > > > > On Thu, Feb 06, 2025 at 0

Re: [PATCH v5 11/16] hw/microblaze: Support various endianness for s3adsp1800 machines

On 6/2/25 15:31, Daniel P. Berrangé wrote: On Thu, Feb 06, 2025 at 02:53:58PM +0100, Philippe Mathieu-Daudé wrote: Hi Daniel, On 6/2/25 14:20, Daniel P. Berrangé wrote: On Thu, Feb 06, 2025 at 02:10:47PM +0100, Philippe Mathieu-Daudé wrote: Introduce an abstract machine parent class which def

Re: [PATCH v5 11/16] hw/microblaze: Support various endianness for s3adsp1800 machines

On Thu, Feb 06, 2025 at 02:53:58PM +0100, Philippe Mathieu-Daudé wrote: > Hi Daniel, > > On 6/2/25 14:20, Daniel P. Berrangé wrote: > > On Thu, Feb 06, 2025 at 02:10:47PM +0100, Philippe Mathieu-Daudé wrote: > > > Introduce an abstract machine parent class which defines > > > the 'little_endian' p

Re: [PATCH v5 11/16] hw/microblaze: Support various endianness for s3adsp1800 machines

Hi Daniel, On 6/2/25 14:20, Daniel P. Berrangé wrote: On Thu, Feb 06, 2025 at 02:10:47PM +0100, Philippe Mathieu-Daudé wrote: Introduce an abstract machine parent class which defines the 'little_endian' property. Duplicate the current machine, which endian is tied to the binary endianness, to o

Re: [PATCH v5 11/16] hw/microblaze: Support various endianness for s3adsp1800 machines

On Thu, Feb 06, 2025 at 02:10:47PM +0100, Philippe Mathieu-Daudé wrote: > Introduce an abstract machine parent class which defines > the 'little_endian' property. Duplicate the current machine, > which endian is tied to the binary endianness, to one big > endian and a little endian machine; updatin

[PATCH v5 11/16] hw/microblaze: Support various endianness for s3adsp1800 machines

Introduce an abstract machine parent class which defines the 'little_endian' property. Duplicate the current machine, which endian is tied to the binary endianness, to one big endian and a little endian machine; updating the machine description. Keep the current default machine for each binary. 'p