[Qemu-devel] [PATCH 0/1] MAINTAINERS: Step in as maintainer for the parallel NOR flash devices

qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed

* [Qemu-devel] [PATCH 0/1] MAINTAINERS: Step in as maintainer for the parallel NOR flash devices
@ 2019-05-05 22:47 Philippe Mathieu-Daudé
  2019-05-05 22:47 ` Philippe Mathieu-Daudé
  2019-05-05 22:47 ` [Qemu-devel] [PATCH 1/1] MAINTAINERS: Add an entry for the Parallel NOR Flash devices Philippe Mathieu-Daudé
  0 siblings, 2 replies; 8+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-05-05 22:47 UTC (permalink / raw)
  To: Thomas Petazzoni, Michael Walle, Jordan Justen, Peter Maydell,
	Edgar E . Iglesias, Paul Burton, Roy Franz, Stephen Checkoway,
	Jan Kiszka, qemu-devel, David Engraf,
	Jean-Christophe PLAGNIOL-VILLARD, Stefan Weil
  Cc: Paolo Bonzini, Laszlo Ersek, Thomas Huth, Alex Bennée,
	Markus Armbruster, Michael S . Tsirkin,
	Philippe Mathieu-Daudé

The parallel NOR flash models don't have a specific maintainer and
default to the 'Block layer core' section.
Step in to maintain them.
The section still get covered by the Block layer team, but the idea
is to offload them.

The two devices are very similar (same technology), the difference
is mostly the protocol to access them.

Amusingly, between the two devices, the 'CFI01' which is used in
enterprise grade products on ARM/X86 archs is the one that received
the less care, while the 'CFI02' used by hobbyist boards is the
more reliable.

To some extent I plan to re-unify the models, and improve testing.

I'm looking for co-maintainers or designated reviewers.
(I asked Stephen Checkoway for help but unfortunately he can't).

Any volunteer?

Regards,

Phil.

PD: I Cc'ed all the people who made sinificant modification in the
    devices the last few years.

Philippe Mathieu-Daudé (1):
  MAINTAINERS: Add an entry for the Parallel NOR Flash devices

 MAINTAINERS | 7 +++++++
 1 file changed, 7 insertions(+)

-- 
2.20.1

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Qemu-devel] [PATCH 0/1] MAINTAINERS: Step in as maintainer for the parallel NOR flash devices
  2019-05-05 22:47 [Qemu-devel] [PATCH 0/1] MAINTAINERS: Step in as maintainer for the parallel NOR flash devices Philippe Mathieu-Daudé
@ 2019-05-05 22:47 ` Philippe Mathieu-Daudé
  2019-05-05 22:47 ` [Qemu-devel] [PATCH 1/1] MAINTAINERS: Add an entry for the Parallel NOR Flash devices Philippe Mathieu-Daudé
  1 sibling, 0 replies; 8+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-05-05 22:47 UTC (permalink / raw)
  To: Thomas Petazzoni, Michael Walle, Jordan Justen, Peter Maydell,
	Edgar E . Iglesias, Paul Burton, Roy Franz, Stephen Checkoway,
	Jan Kiszka, qemu-devel, David Engraf,
	Jean-Christophe PLAGNIOL-VILLARD, Stefan Weil
  Cc: Thomas Huth, Michael S . Tsirkin, Laszlo Ersek, Markus Armbruster,
	Philippe Mathieu-Daudé, Paolo Bonzini, Alex Bennée

The parallel NOR flash models don't have a specific maintainer and
default to the 'Block layer core' section.
Step in to maintain them.
The section still get covered by the Block layer team, but the idea
is to offload them.

The two devices are very similar (same technology), the difference
is mostly the protocol to access them.

Amusingly, between the two devices, the 'CFI01' which is used in
enterprise grade products on ARM/X86 archs is the one that received
the less care, while the 'CFI02' used by hobbyist boards is the
more reliable.

To some extent I plan to re-unify the models, and improve testing.

I'm looking for co-maintainers or designated reviewers.
(I asked Stephen Checkoway for help but unfortunately he can't).

Any volunteer?

Regards,

Phil.

PD: I Cc'ed all the people who made sinificant modification in the
    devices the last few years.

Philippe Mathieu-Daudé (1):
  MAINTAINERS: Add an entry for the Parallel NOR Flash devices

 MAINTAINERS | 7 +++++++
 1 file changed, 7 insertions(+)

-- 
2.20.1

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Qemu-devel] [PATCH 1/1] MAINTAINERS: Add an entry for the Parallel NOR Flash devices
  2019-05-05 22:47 [Qemu-devel] [PATCH 0/1] MAINTAINERS: Step in as maintainer for the parallel NOR flash devices Philippe Mathieu-Daudé
  2019-05-05 22:47 ` Philippe Mathieu-Daudé
@ 2019-05-05 22:47 ` Philippe Mathieu-Daudé
  2019-05-05 22:47   ` Philippe Mathieu-Daudé
                     ` (4 more replies)
  1 sibling, 5 replies; 8+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-05-05 22:47 UTC (permalink / raw)
  To: Thomas Petazzoni, Michael Walle, Jordan Justen, Peter Maydell,
	Edgar E . Iglesias, Paul Burton, Roy Franz, Stephen Checkoway,
	Jan Kiszka, qemu-devel, David Engraf,
	Jean-Christophe PLAGNIOL-VILLARD, Stefan Weil
  Cc: Paolo Bonzini, Laszlo Ersek, Thomas Huth, Alex Bennée,
	Markus Armbruster, Michael S . Tsirkin,
	Philippe Mathieu-Daudé

Step in to maintain it, since I have some familiarity with
the technology.

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
 MAINTAINERS | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 66ddbda9c95..633f6315536 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1391,6 +1391,13 @@ F: include/hw/net/
 F: tests/virtio-net-test.c
 T: git https://github.com/jasowang/qemu.git net
 
+Parallel NOR Flash devices
+M: Philippe Mathieu-Daudé <philmd@redhat.com>
+T: git https://gitlab.com/philmd/qemu.git pflash-next
+S: Maintained
+F: hw/block/pflash_cfi*.c
+F: include/hw/block/flash.h
+
 SCSI
 M: Paolo Bonzini <pbonzini@redhat.com>
 R: Fam Zheng <fam@euphon.net>
-- 
2.20.1

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [Qemu-devel] [PATCH 1/1] MAINTAINERS: Add an entry for the Parallel NOR Flash devices
  2019-05-05 22:47 ` [Qemu-devel] [PATCH 1/1] MAINTAINERS: Add an entry for the Parallel NOR Flash devices Philippe Mathieu-Daudé
@ 2019-05-05 22:47   ` Philippe Mathieu-Daudé
  2019-05-06  3:56   ` Thomas Huth
                     ` (3 subsequent siblings)
  4 siblings, 0 replies; 8+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-05-05 22:47 UTC (permalink / raw)
  To: Thomas Petazzoni, Michael Walle, Jordan Justen, Peter Maydell,
	Edgar E . Iglesias, Paul Burton, Roy Franz, Stephen Checkoway,
	Jan Kiszka, qemu-devel, David Engraf,
	Jean-Christophe PLAGNIOL-VILLARD, Stefan Weil
  Cc: Thomas Huth, Michael S . Tsirkin, Laszlo Ersek, Markus Armbruster,
	Philippe Mathieu-Daudé, Paolo Bonzini, Alex Bennée

Step in to maintain it, since I have some familiarity with
the technology.

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
 MAINTAINERS | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 66ddbda9c95..633f6315536 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1391,6 +1391,13 @@ F: include/hw/net/
 F: tests/virtio-net-test.c
 T: git https://github.com/jasowang/qemu.git net
 
+Parallel NOR Flash devices
+M: Philippe Mathieu-Daudé <philmd@redhat.com>
+T: git https://gitlab.com/philmd/qemu.git pflash-next
+S: Maintained
+F: hw/block/pflash_cfi*.c
+F: include/hw/block/flash.h
+
 SCSI
 M: Paolo Bonzini <pbonzini@redhat.com>
 R: Fam Zheng <fam@euphon.net>
-- 
2.20.1



^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [Qemu-devel] [PATCH 1/1] MAINTAINERS: Add an entry for the Parallel NOR Flash devices
  2019-05-05 22:47 ` [Qemu-devel] [PATCH 1/1] MAINTAINERS: Add an entry for the Parallel NOR Flash devices Philippe Mathieu-Daudé
  2019-05-05 22:47   ` Philippe Mathieu-Daudé
@ 2019-05-06  3:56   ` Thomas Huth
  2019-05-06 12:59   ` Michael S. Tsirkin
                     ` (2 subsequent siblings)
  4 siblings, 0 replies; 8+ messages in thread
From: Thomas Huth @ 2019-05-06  3:56 UTC (permalink / raw)
  To: Philippe Mathieu-Daudé, Thomas Petazzoni, Michael Walle,
	Jordan Justen, Peter Maydell, Edgar E . Iglesias, Paul Burton,
	Roy Franz, Stephen Checkoway, Jan Kiszka, qemu-devel,
	David Engraf, Jean-Christophe PLAGNIOL-VILLARD, Stefan Weil
  Cc: Alex Bennée, Paolo Bonzini, Laszlo Ersek, Markus Armbruster,
	Michael S . Tsirkin

On 06/05/2019 00.47, Philippe Mathieu-Daudé wrote:
> Step in to maintain it, since I have some familiarity with
> the technology.
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
>  MAINTAINERS | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 66ddbda9c95..633f6315536 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1391,6 +1391,13 @@ F: include/hw/net/
>  F: tests/virtio-net-test.c
>  T: git https://github.com/jasowang/qemu.git net
>  
> +Parallel NOR Flash devices
> +M: Philippe Mathieu-Daudé <philmd@redhat.com>
> +T: git https://gitlab.com/philmd/qemu.git pflash-next
> +S: Maintained
> +F: hw/block/pflash_cfi*.c
> +F: include/hw/block/flash.h

FWIW:

Reviewed-by: Thomas Huth <thuth@redhat.com>


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [Qemu-devel] [PATCH 1/1] MAINTAINERS: Add an entry for the Parallel NOR Flash devices
  2019-05-05 22:47 ` [Qemu-devel] [PATCH 1/1] MAINTAINERS: Add an entry for the Parallel NOR Flash devices Philippe Mathieu-Daudé
  2019-05-05 22:47   ` Philippe Mathieu-Daudé
  2019-05-06  3:56   ` Thomas Huth
@ 2019-05-06 12:59   ` Michael S. Tsirkin
  2019-05-07 10:04   ` Laszlo Ersek
  2019-05-07 11:36   ` Kevin Wolf
  4 siblings, 0 replies; 8+ messages in thread
From: Michael S. Tsirkin @ 2019-05-06 12:59 UTC (permalink / raw)
  To: Philippe Mathieu-Daudé
  Cc: Peter Maydell, Thomas Huth, Stephen Checkoway, Paul Burton,
	Alex Bennée, Jordan Justen, qemu-devel, Laszlo Ersek,
	Roy Franz, Michael Walle, Paolo Bonzini, Thomas Petazzoni,
	Jan Kiszka, Edgar E . Iglesias, Stefan Weil,
	Jean-Christophe PLAGNIOL-VILLARD, David Engraf, Markus Armbruster

On Mon, May 06, 2019 at 12:47:37AM +0200, Philippe Mathieu-Daudé wrote:
> Step in to maintain it, since I have some familiarity with
> the technology.
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Acked-by: Michael S. Tsirkin <mst@redhat.com>

> ---
>  MAINTAINERS | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 66ddbda9c95..633f6315536 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1391,6 +1391,13 @@ F: include/hw/net/
>  F: tests/virtio-net-test.c
>  T: git https://github.com/jasowang/qemu.git net
>  
> +Parallel NOR Flash devices
> +M: Philippe Mathieu-Daudé <philmd@redhat.com>
> +T: git https://gitlab.com/philmd/qemu.git pflash-next
> +S: Maintained
> +F: hw/block/pflash_cfi*.c
> +F: include/hw/block/flash.h
> +
>  SCSI
>  M: Paolo Bonzini <pbonzini@redhat.com>
>  R: Fam Zheng <fam@euphon.net>
> -- 
> 2.20.1


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [Qemu-devel] [PATCH 1/1] MAINTAINERS: Add an entry for the Parallel NOR Flash devices
  2019-05-05 22:47 ` [Qemu-devel] [PATCH 1/1] MAINTAINERS: Add an entry for the Parallel NOR Flash devices Philippe Mathieu-Daudé
                     ` (2 preceding siblings ...)
  2019-05-06 12:59   ` Michael S. Tsirkin
@ 2019-05-07 10:04   ` Laszlo Ersek
  2019-05-07 11:36   ` Kevin Wolf
  4 siblings, 0 replies; 8+ messages in thread
From: Laszlo Ersek @ 2019-05-07 10:04 UTC (permalink / raw)
  To: Philippe Mathieu-Daudé, Thomas Petazzoni, Michael Walle,
	Jordan Justen, Peter Maydell, Edgar E . Iglesias, Paul Burton,
	Roy Franz, Stephen Checkoway, Jan Kiszka, qemu-devel,
	David Engraf, Jean-Christophe PLAGNIOL-VILLARD, Stefan Weil
  Cc: Paolo Bonzini, Thomas Huth, Alex Bennée, Markus Armbruster,
	Michael S . Tsirkin

On 05/06/19 00:47, Philippe Mathieu-Daudé wrote:
> Step in to maintain it, since I have some familiarity with
> the technology.
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
>  MAINTAINERS | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 66ddbda9c95..633f6315536 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1391,6 +1391,13 @@ F: include/hw/net/
>  F: tests/virtio-net-test.c
>  T: git https://github.com/jasowang/qemu.git net
>  
> +Parallel NOR Flash devices
> +M: Philippe Mathieu-Daudé <philmd@redhat.com>
> +T: git https://gitlab.com/philmd/qemu.git pflash-next
> +S: Maintained
> +F: hw/block/pflash_cfi*.c
> +F: include/hw/block/flash.h
> +
>  SCSI
>  M: Paolo Bonzini <pbonzini@redhat.com>
>  R: Fam Zheng <fam@euphon.net>
> 

Acked-by: Laszlo Ersek <lersek@redhat.com>


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [Qemu-devel] [PATCH 1/1] MAINTAINERS: Add an entry for the Parallel NOR Flash devices
  2019-05-05 22:47 ` [Qemu-devel] [PATCH 1/1] MAINTAINERS: Add an entry for the Parallel NOR Flash devices Philippe Mathieu-Daudé
                     ` (3 preceding siblings ...)
  2019-05-07 10:04   ` Laszlo Ersek
@ 2019-05-07 11:36   ` Kevin Wolf
  4 siblings, 0 replies; 8+ messages in thread
From: Kevin Wolf @ 2019-05-07 11:36 UTC (permalink / raw)
  To: Philippe Mathieu-Daudé
  Cc: Peter Maydell, Thomas Huth, Stephen Checkoway, Paul Burton,
	Michael S . Tsirkin, Jordan Justen, qemu-devel, Laszlo Ersek,
	Roy Franz, Michael Walle, Paolo Bonzini, Alex Bennée,
	Thomas Petazzoni, Jan Kiszka, Edgar E . Iglesias, Stefan Weil,
	Jean-Christophe PLAGNIOL-VILLARD, David Engraf, Markus Armbruster

Am 06.05.2019 um 00:47 hat Philippe Mathieu-Daudé geschrieben:
> Step in to maintain it, since I have some familiarity with
> the technology.
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Thanks, applied to the block branch.

Kevin


^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2019-05-07 11:37 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-05-05 22:47 [Qemu-devel] [PATCH 0/1] MAINTAINERS: Step in as maintainer for the parallel NOR flash devices Philippe Mathieu-Daudé
2019-05-05 22:47 ` Philippe Mathieu-Daudé
2019-05-05 22:47 ` [Qemu-devel] [PATCH 1/1] MAINTAINERS: Add an entry for the Parallel NOR Flash devices Philippe Mathieu-Daudé
2019-05-05 22:47   ` Philippe Mathieu-Daudé
2019-05-06  3:56   ` Thomas Huth
2019-05-06 12:59   ` Michael S. Tsirkin
2019-05-07 10:04   ` Laszlo Ersek
2019-05-07 11:36   ` Kevin Wolf

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).