Re: [PATCH v5 5/8] blockdev: Add a new IF type IF_OTHER

2022-08-08 Thread Markus Armbruster
Adding Paolo and Eduardo since we're wandering into QOM-land. Kevin Wolf writes: > Am 08.08.2022 um 08:26 hat Markus Armbruster geschrieben: >> Daniel P. Berrangé writes: >> >> > On Thu, Aug 04, 2022 at 05:30:40PM +0200, Markus Armbruster wrote: >> >> Daniel P. Berrangé writes: >> >> >> >>

Re: [PATCH v5 5/8] blockdev: Add a new IF type IF_OTHER

2022-08-08 Thread Kevin Wolf
Am 08.08.2022 um 08:26 hat Markus Armbruster geschrieben: > Daniel P. Berrangé writes: > > > On Thu, Aug 04, 2022 at 05:30:40PM +0200, Markus Armbruster wrote: > >> Daniel P. Berrangé writes: > >> > >> > On Thu, Aug 04, 2022 at 04:56:15PM +0200, Markus Armbruster wrote: > >> >> Daniel P.

Re: [PATCH v5 5/8] blockdev: Add a new IF type IF_OTHER

2022-08-08 Thread Markus Armbruster
Daniel P. Berrangé writes: > On Thu, Aug 04, 2022 at 05:30:40PM +0200, Markus Armbruster wrote: >> Daniel P. Berrangé writes: >> >> > On Thu, Aug 04, 2022 at 04:56:15PM +0200, Markus Armbruster wrote: >> >> Daniel P. Berrangé writes: >> >> >> >> > On Thu, Jul 28, 2022 at 10:46:35AM +0100,

Re: [PATCH v5 5/8] blockdev: Add a new IF type IF_OTHER

2022-08-04 Thread Daniel P . Berrangé
On Thu, Aug 04, 2022 at 05:30:40PM +0200, Markus Armbruster wrote: > Daniel P. Berrangé writes: > > > On Thu, Aug 04, 2022 at 04:56:15PM +0200, Markus Armbruster wrote: > >> Daniel P. Berrangé writes: > >> > >> > On Thu, Jul 28, 2022 at 10:46:35AM +0100, Peter Maydell wrote: > >> >> On Wed, 27

Re: [PATCH v5 5/8] blockdev: Add a new IF type IF_OTHER

2022-08-04 Thread Markus Armbruster
Daniel P. Berrangé writes: > On Thu, Aug 04, 2022 at 04:56:15PM +0200, Markus Armbruster wrote: >> Daniel P. Berrangé writes: >> >> > On Thu, Jul 28, 2022 at 10:46:35AM +0100, Peter Maydell wrote: >> >> On Wed, 27 Jul 2022 at 20:03, Kevin Wolf wrote: >> >> > >> >> > Am 18.07.2022 um 11:49 hat

Re: [PATCH v5 5/8] blockdev: Add a new IF type IF_OTHER

2022-08-04 Thread Daniel P . Berrangé
On Thu, Aug 04, 2022 at 04:56:15PM +0200, Markus Armbruster wrote: > Daniel P. Berrangé writes: > > > On Thu, Jul 28, 2022 at 10:46:35AM +0100, Peter Maydell wrote: > >> On Wed, 27 Jul 2022 at 20:03, Kevin Wolf wrote: > >> > > >> > Am 18.07.2022 um 11:49 hat Markus Armbruster geschrieben: > >>

Re: [PATCH v5 5/8] blockdev: Add a new IF type IF_OTHER

2022-08-04 Thread Markus Armbruster
Daniel P. Berrangé writes: > On Thu, Jul 28, 2022 at 10:46:35AM +0100, Peter Maydell wrote: >> On Wed, 27 Jul 2022 at 20:03, Kevin Wolf wrote: >> > >> > Am 18.07.2022 um 11:49 hat Markus Armbruster geschrieben: >> > > An OTP device isn't really a parallel flash, and neither are eFuses. >> > >

Re: [PATCH v5 5/8] blockdev: Add a new IF type IF_OTHER

2022-08-04 Thread Daniel P . Berrangé
On Thu, Jul 28, 2022 at 10:46:35AM +0100, Peter Maydell wrote: > On Wed, 27 Jul 2022 at 20:03, Kevin Wolf wrote: > > > > Am 18.07.2022 um 11:49 hat Markus Armbruster geschrieben: > > > An OTP device isn't really a parallel flash, and neither are eFuses. > > > More fast-and-lose use of IF_PFLASH

Re: [PATCH v5 5/8] blockdev: Add a new IF type IF_OTHER

2022-08-04 Thread Markus Armbruster
Peter Maydell writes: > On Thu, 28 Jul 2022 at 15:50, Markus Armbruster wrote: >> Kevin Wolf writes: >> > >> > But if you have more than one device, it becomes hard to predict which >> > device gets which backend - it depends on the initialisation order in >> > the code then, >> >> Really?

Re: [PATCH v5 5/8] blockdev: Add a new IF type IF_OTHER

2022-07-28 Thread Kevin Wolf
Am 28.07.2022 um 16:50 hat Markus Armbruster geschrieben: > Kevin Wolf writes: > > > Am 28.07.2022 um 11:46 hat Peter Maydell geschrieben: > >> On Wed, 27 Jul 2022 at 20:03, Kevin Wolf wrote: > >> > > >> > Am 18.07.2022 um 11:49 hat Markus Armbruster geschrieben: > >> > > An OTP device isn't

Re: [PATCH v5 5/8] blockdev: Add a new IF type IF_OTHER

2022-07-28 Thread Markus Armbruster
Kevin Wolf writes: > Am 28.07.2022 um 11:46 hat Peter Maydell geschrieben: >> On Wed, 27 Jul 2022 at 20:03, Kevin Wolf wrote: >> > >> > Am 18.07.2022 um 11:49 hat Markus Armbruster geschrieben: >> > > An OTP device isn't really a parallel flash, and neither are eFuses. >> > > More fast-and-lose

Re: [PATCH v5 5/8] blockdev: Add a new IF type IF_OTHER

2022-07-28 Thread Peter Maydell
On Thu, 28 Jul 2022 at 15:50, Markus Armbruster wrote: > Kevin Wolf writes: > > > > But if you have more than one device, it becomes hard to predict which > > device gets which backend - it depends on the initialisation order in > > the code then, > > Really? Board code should use IF_OTHER

Re: [PATCH v5 5/8] blockdev: Add a new IF type IF_OTHER

2022-07-28 Thread Cédric Le Goater
On 7/28/22 15:29, Kevin Wolf wrote: Am 28.07.2022 um 11:46 hat Peter Maydell geschrieben: On Wed, 27 Jul 2022 at 20:03, Kevin Wolf wrote: Am 18.07.2022 um 11:49 hat Markus Armbruster geschrieben: An OTP device isn't really a parallel flash, and neither are eFuses. More fast-and-lose use of

Re: [PATCH v5 5/8] blockdev: Add a new IF type IF_OTHER

2022-07-28 Thread Peter Maydell
On Thu, 28 Jul 2022 at 14:30, Kevin Wolf wrote: > > Am 28.07.2022 um 11:46 hat Peter Maydell geschrieben: > > On Wed, 27 Jul 2022 at 20:03, Kevin Wolf wrote: > > > If the existing types aren't good enough (I don't have an opinion on > > > whether IF_PFLASH is a good match), let's add a new one.

Re: [PATCH v5 5/8] blockdev: Add a new IF type IF_OTHER

2022-07-28 Thread Kevin Wolf
Am 28.07.2022 um 11:46 hat Peter Maydell geschrieben: > On Wed, 27 Jul 2022 at 20:03, Kevin Wolf wrote: > > > > Am 18.07.2022 um 11:49 hat Markus Armbruster geschrieben: > > > An OTP device isn't really a parallel flash, and neither are eFuses. > > > More fast-and-lose use of IF_PFLASH may exist

Re: [PATCH v5 5/8] blockdev: Add a new IF type IF_OTHER

2022-07-28 Thread Peter Maydell
On Wed, 27 Jul 2022 at 20:03, Kevin Wolf wrote: > > Am 18.07.2022 um 11:49 hat Markus Armbruster geschrieben: > > An OTP device isn't really a parallel flash, and neither are eFuses. > > More fast-and-lose use of IF_PFLASH may exist in the tree, and maybe of > > other interface types, too. > > >

Re: [PATCH v5 5/8] blockdev: Add a new IF type IF_OTHER

2022-07-27 Thread Kevin Wolf
Am 18.07.2022 um 11:49 hat Markus Armbruster geschrieben: > Hao Wu writes: > > > This type is used to represent block devs that are not suitable to > > be represented by other existing types. > > > > A sample use is to represent an at24c eeprom device defined in > > hw/nvram/eeprom_at24c.c. The

Re: [PATCH v5 5/8] blockdev: Add a new IF type IF_OTHER

2022-07-18 Thread Thomas Huth
On 18/07/2022 11.49, Markus Armbruster wrote: [...] An OTP device isn't really a parallel flash, and neither are eFuses. More fast-and-lose use of IF_PFLASH may exist in the tree, and maybe of other interface types, too. This patch introduces IF_OTHER. The patch after next uses it for an

Re: [PATCH v5 5/8] blockdev: Add a new IF type IF_OTHER

2022-07-18 Thread Markus Armbruster
Hao Wu writes: > This type is used to represent block devs that are not suitable to > be represented by other existing types. > > A sample use is to represent an at24c eeprom device defined in > hw/nvram/eeprom_at24c.c. The block device can be used to contain the > content of the said eeprom

[PATCH v5 5/8] blockdev: Add a new IF type IF_OTHER

2022-07-14 Thread Hao Wu
This type is used to represent block devs that are not suitable to be represented by other existing types. A sample use is to represent an at24c eeprom device defined in hw/nvram/eeprom_at24c.c. The block device can be used to contain the content of the said eeprom device. Signed-off-by: Hao Wu