On Wed, 13 Mar 2019, Hans de Goede <hdego...@redhat.com> wrote:
> Hi,
>
> On 3/13/19 9:26 AM, Jani Nikula wrote:
>> On Mon, 11 Mar 2019, Hans de Goede <hdego...@redhat.com> wrote:
>>> Add a MAINTAINERS entry for the vboxvideo driver, now that it has been
>>> moved out of staging.
>>>
>>> Acked-by: Daniel Vetter <daniel.vet...@ffwll.ch>
>>> Signed-off-by: Hans de Goede <hdego...@redhat.com>
>>> ---
>>>   MAINTAINERS | 7 +++++++
>>>   1 file changed, 7 insertions(+)
>>>
>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>> index 81d484891b53..36577f5db872 100644
>>> --- a/MAINTAINERS
>>> +++ b/MAINTAINERS
>>> @@ -4932,6 +4932,13 @@ S:   Odd Fixes
>>>   F:        drivers/gpu/drm/udl/
>>>   T:        git git://anongit.freedesktop.org/drm/drm-misc
>>>   
>>> +DRM DRIVER FOR VIRTUALBOX VIRTUAL GPU
>>> +M: Hans de Goede <hdego...@redhat.com>
>>> +L: dri-devel@lists.freedesktop.org
>>> +S: Maintained
>>> +F: drivers/gpu/drm/vboxvideo/
>>> +T: git git://anongit.freedesktop.org/drm/drm-misc
>> 
>> Please consider adding B: and C: entries, and perhaps W: if applicable.
>
> Should the B: point to bugzilla.kernel.org or bugzilla.freedesktop.org?

This is perhaps a point of contention. drm/i915 has completely switched
to fdo bugzilla, and we close any kernel.org bugs on sight with
instructions to file the bug at fdo. IMHO all of drm would benefit from
switching over to fdo bugzilla (or gitlab issues going forward).

> also most B: entries just point to a bugzilla, without
> specifying which component to use, which is not really helpful.
> The B: for i915 seems good, but setting up a similar webpage for
> each driver seems like a bad idea.

The i915 page needs an overhaul. But you could also use something like:

https://bugs.freedesktop.org/enter_bug.cgi?product=DRI&component=DRM/Intel

> I think that we may need some
> generic howto report drm driver bugs page somewhere ?
>
> As for the C: I am more or less the main point of contact and I'm
> more often not on irc then that I'm on irc since I find that irc
> distracts me too much. So I think it is better to not have a C:
> for the vboxvideo driver.

I think that's fine.

Also, I have nothing against merging the patch at hand as-is, and
updating the rest later.


BR,
Jani.


-- 
Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to