Re: [PATCH v2 2/3] dt-bindings: display: panel: add fannal,c3004

[Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>]

On 06/06/2023 11:53, Paulo Pavacic wrote:
>>> +    #include <dt-bindings/gpio/gpio.h>
>>> +    dsi {
>>> +        #address-cells = <1>;
>>> +        #size-cells = <0>;
>>> +        panel@0 {
>>> +            compatible = "fannal,c3004";
>>> +            reg = <0>;
>>> +            pinctrl-0 = <&pinctrl_mipi_dsi_rst>;
>>> +            pinctrl-names = "default";
>>> +            reset-gpios = <&gpio1 9 GPIO_ACTIVE_LOW>;
>>> +            vdd-supply = <&reg1>;
>>> +            vddio-supply = <&reg2>;
>>> +            width-mm = <93>;
>>> +            height-mm = <56>;
>>> +            panel-timing {
>>> +                clock-frequency = <27000000>;
>>> +                hactive = <480>;
>>> +                vactive = <800>;
>>> +                hfront-porch = <30>;
>>> +                hback-porch = <30>;
>>> +                hsync-len = <8>;
>>> +                vback-porch = <30>;
>>> +                vfront-porch = <30>;
>>> +                vsync-len = <8>;
>>> +            };
>>> +        };
>>> +    };
>>
>>
>> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

So this you decided to ignore?

>>
>>> +...
>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>> index 5c22c828ab46..62374c8424b9 100644
>>> --- a/MAINTAINERS
>>> +++ b/MAINTAINERS
>>> @@ -6427,6 +6427,12 @@ T:     git git://anongit.freedesktop.org/drm/drm-misc
>>>  F:   Documentation/devicetree/bindings/display/panel/ebbg,ft8719.yaml
>>>  F:   drivers/gpu/drm/panel/panel-ebbg-ft8719.c
>>>
>>> +DRM DRIVER FOR FANNAL C3004373132019A
>>> +M:   Paulo Pavacic <pavacic.p@gmail.com>
>>> +S:   Maintained
>>> +C:   matrix:r/mipi-dsi-bringup:matrix.org
>>
>> I don't think we have chat channels for individual, small drivers. Add a
>> channel for entire subsystem.
> Okay I will add matrix:r/linux-drm:matrix.org

I meant, we do not have entries for individual drivers in MAINTAINERS
with mailing lists or with chats. The entries covering entire subsystems
have such links. We are not going to sign up for 100 different channels,
each one per driver.

Best regards,
Krzysztof