[PATCH] MAINTAINERS: Add Fan Ni as Compute eXpress Link QEMU reviewer

Jonathan Cameron via posted 1 patch 2 years, 11 months ago
Failed in applying to current master (apply log)
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
[PATCH] MAINTAINERS: Add Fan Ni as Compute eXpress Link QEMU reviewer
Posted by Jonathan Cameron via 2 years, 11 months ago
Fan Ni has offered to help out with QEMU CXL emulation reviewing.
Add him as a designated reviewer.

Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

--
Thanks to Fan for stepping up after I requested help following Ben
stepping down as co-maintainer. Fan base been active in testing
and review recently so great to have Fan on board.

Based on patch [PATCH] MAINTAINERS: Remove CXL maintainer Ben Widawsky

Based-of: Message-id: 20230220212437.1462314-1-armbru@redhat.com
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 7ea56c6ccc..838e1a91a4 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2633,6 +2633,7 @@ T: git https://gitlab.com/vsementsov/qemu.git block
 
 Compute Express Link
 M: Jonathan Cameron <jonathan.cameron@huawei.com>
+R: Fan Ni <fan.ni@samsung.com>
 S: Supported
 F: hw/cxl/
 F: hw/mem/cxl_type3.c
-- 
2.37.2
Re: [PATCH] MAINTAINERS: Add Fan Ni as Compute eXpress Link QEMU reviewer
Posted by Peter Maydell 3 months ago
On Tue, 28 Feb 2023 at 11:39, Jonathan Cameron via
<qemu-devel@nongnu.org> wrote:
>
> Fan Ni has offered to help out with QEMU CXL emulation reviewing.
> Add him as a designated reviewer.
>
> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
>
> --
> Thanks to Fan for stepping up after I requested help following Ben
> stepping down as co-maintainer. Fan base been active in testing
> and review recently so great to have Fan on board.
>
> Based on patch [PATCH] MAINTAINERS: Remove CXL maintainer Ben Widawsky
>
> Based-of: Message-id: 20230220212437.1462314-1-armbru@redhat.com
> ---
>  MAINTAINERS | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 7ea56c6ccc..838e1a91a4 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -2633,6 +2633,7 @@ T: git https://gitlab.com/vsementsov/qemu.git block
>
>  Compute Express Link
>  M: Jonathan Cameron <jonathan.cameron@huawei.com>
> +R: Fan Ni <fan.ni@samsung.com>
>  S: Supported
>  F: hw/cxl/
>  F: hw/mem/cxl_type3.c

Hi; I noticed today that Fan's samsung email is now bouncing
with "550 5.1.1 Recipient address rejected: User unknown".
Do we have a altenate email for him, or should we remove
his entry from the QEMU MAINTAINERS file; and if so is there
anybody else we should add as a reviewer for CXL ?

thanks
-- PMM
Re: [PATCH] MAINTAINERS: Add Fan Ni as Compute eXpress Link QEMU reviewer
Posted by Jonathan Cameron via 3 months ago
On Tue, 4 Nov 2025 16:17:07 +0000
Peter Maydell <peter.maydell@linaro.org> wrote:

> On Tue, 28 Feb 2023 at 11:39, Jonathan Cameron via
> <qemu-devel@nongnu.org> wrote:
> >
> > Fan Ni has offered to help out with QEMU CXL emulation reviewing.
> > Add him as a designated reviewer.
> >
> > Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> >
> > --
> > Thanks to Fan for stepping up after I requested help following Ben
> > stepping down as co-maintainer. Fan base been active in testing
> > and review recently so great to have Fan on board.
> >
> > Based on patch [PATCH] MAINTAINERS: Remove CXL maintainer Ben Widawsky
> >
> > Based-of: Message-id: 20230220212437.1462314-1-armbru@redhat.com
> > ---
> >  MAINTAINERS | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 7ea56c6ccc..838e1a91a4 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -2633,6 +2633,7 @@ T: git https://gitlab.com/vsementsov/qemu.git block
> >
> >  Compute Express Link
> >  M: Jonathan Cameron <jonathan.cameron@huawei.com>
> > +R: Fan Ni <fan.ni@samsung.com>
> >  S: Supported
> >  F: hw/cxl/
> >  F: hw/mem/cxl_type3.c  
> 
> Hi; I noticed today that Fan's samsung email is now bouncing
> with "550 5.1.1 Recipient address rejected: User unknown".
> Do we have a altenate email for him, or should we remove
> his entry from the QEMU MAINTAINERS file; and if so is there
> anybody else we should add as a reviewer for CXL ?

My bad. I've had this on my todo list since Fan moved.

nifan.cxl@gmail.com
is the address Fan asked me to use.

I do plan to add a second reviewer (more welcome!) as Fan's time is
limited (not the day job any more), but wanted to let them ramp up
and review informally for a bit first.

Thanks,

Jonathan

> 
> thanks
> -- PMM
Re: [PATCH] MAINTAINERS: Add Fan Ni as Compute eXpress Link QEMU reviewer
Posted by Fan Ni 2 years, 11 months ago
On Tue, Feb 28, 2023 at 11:39:26AM +0000, Jonathan Cameron wrote:
> Fan Ni has offered to help out with QEMU CXL emulation reviewing.
> Add him as a designated reviewer.
> 
> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> 

Acked-by: Fan Ni <fan.ni@samsung.com>

> --
> Thanks to Fan for stepping up after I requested help following Ben
> stepping down as co-maintainer. Fan base been active in testing
> and review recently so great to have Fan on board.
> 
> Based on patch [PATCH] MAINTAINERS: Remove CXL maintainer Ben Widawsky
> 
> Based-of: Message-id: 20230220212437.1462314-1-armbru@redhat.com
> ---
>  MAINTAINERS | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 7ea56c6ccc..838e1a91a4 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -2633,6 +2633,7 @@ T: git https://urldefense.com/v3/__https://gitlab.com/vsementsov/qemu.git__;!!EwVzqGoTKBqv-0DWAJBm!UcJW-DGImFKtSoRVFI1IzD3TFOpkBbeSwsmRlUdUKPdhiOMXbTFOABJvxDRBLgLAEt0RvW0cdBG19HgqiR_VceRIA91N$  block
>  
>  Compute Express Link
>  M: Jonathan Cameron <jonathan.cameron@huawei.com>
> +R: Fan Ni <fan.ni@samsung.com>
>  S: Supported
>  F: hw/cxl/
>  F: hw/mem/cxl_type3.c
> -- 
> 2.37.2
> 
Re: [PATCH] MAINTAINERS: Add Fan Ni as Compute eXpress Link QEMU reviewer
Posted by Markus Armbruster 2 years, 11 months ago
Jonathan Cameron <Jonathan.Cameron@huawei.com> writes:

> Fan Ni has offered to help out with QEMU CXL emulation reviewing.
> Add him as a designated reviewer.
>
> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
>
> --
> Thanks to Fan for stepping up after I requested help following Ben
> stepping down as co-maintainer. Fan base been active in testing
> and review recently so great to have Fan on board.
>
> Based on patch [PATCH] MAINTAINERS: Remove CXL maintainer Ben Widawsky
>
> Based-of: Message-id: 20230220212437.1462314-1-armbru@redhat.com

Glad you found a co-maintainer!

Acked-by: Markus Armbruster <armbru@redhat.com>
Re: [PATCH] MAINTAINERS: Add Fan Ni as Compute eXpress Link QEMU reviewer
Posted by Philippe Mathieu-Daudé 2 years, 11 months ago
On 28/2/23 12:39, Jonathan Cameron via wrote:
> Fan Ni has offered to help out with QEMU CXL emulation reviewing.
> Add him as a designated reviewer.
> 
> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> 
> --
> Thanks to Fan for stepping up after I requested help following Ben
> stepping down as co-maintainer. Fan base been active in testing
> and review recently so great to have Fan on board.
> 
> Based on patch [PATCH] MAINTAINERS: Remove CXL maintainer Ben Widawsky
> 
> Based-of: Message-id: 20230220212437.1462314-1-armbru@redhat.com
> ---
>   MAINTAINERS | 1 +
>   1 file changed, 1 insertion(+)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>