Re: [PATCH 2/2] MAINTAINERS: Mark "vhost-user-scmi" orphaned

[Markus Armbruster <armbru@redhat.com>]

Philippe Mathieu-Daudé <philmd@oss.qualcomm.com> writes:

> On 13/7/26 08:52, Michael S. Tsirkin wrote:
>> On Mon, Jul 13, 2026 at 08:13:38AM +0200, Philippe Mathieu-Daudé wrote:
>>> On 11/7/26 10:00, Michael S. Tsirkin wrote:
>>>> On Fri, Jul 10, 2026 at 01:43:05PM +0200, Markus Armbruster wrote:
>>>>> Commit a5dab090e1 (hw/virtio: Add boilerplate for vhost-user-scmi
>>>>> device, 2023-06-28) added the section with S: Supported, but no M:.
>>>>> This makes no sense.  Downgrade to S: Orphan.
>>>>>
>>>>> Signed-off-by: Markus Armbruster <armbru@redhat.com>
>>>>
>>>>
>>>> No, this makes sense. The reason this section was added is so that Milan can
>>>> get copied on vhost-user-scmi but not other virtio stuff:
>>>>
>>>> ./scripts/get_maintainer.pl -f hw/virtio/vhost-user-scmi.c
>>>> Milan Zamazal <mzamazal@redhat.com> (reviewer:vhost-user-scmi)
>>>> "Michael S. Tsirkin" <mst@redhat.com> (supporter:vhost)
>>>> Stefano Garzarella <sgarzare@redhat.com> (reviewer:vhost)
>>>> qemu-devel@nongnu.org (open list:All patches CC here)
>>>>
>>>> Pls drop this patch.
>>>>
>>>>> ---
>>>>>    MAINTAINERS | 2 +-
>>>>>    1 file changed, 1 insertion(+), 1 deletion(-)
>>>>>
>>>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>>>> index aa2df9cbe3..4d558ddee9 100644
>>>>> --- a/MAINTAINERS
>>>>> +++ b/MAINTAINERS
>>>>> @@ -2622,7 +2622,7 @@ F: include/hw/virtio/vhost-user-snd.h
>>>>>    vhost-user-scmi
>>>
>>> So what we want is to add here:
>>>
>>>    M: Michael S. Tsirkin <mst@redhat.com>
>>>    R: Stefano Garzarella <sgarzare@redhat.com>
>>
>> But these files already get M from the match against hw/virtio/ -
>> what am I missing?
>
> Yeah, just trying to make Markus script happy ;)

Appreciate the kindness!

In decreasing importance:

1. Correct get_maintainers.pl output

2. Legible MAINTAINERS

99. My script is happy.

[...]