Re: [PATCH 1/4] media: dt-bindings: Add Sony IMX728

Sebastian LaVine
2025-02-26T19:15:38Z

On Wed Feb 12, 2025 at 3:07 PM EST, Krzysztof Kozlowski wrote:
> On 12/02/2025 20:56, Sebastian LaVine wrote:
>> +
>> + reset-gpios:
>> + maxItems: 1
>> + description:
>> + Specifier for the GPIO connected to the XCLR (System Reset) pin.
>
> s/Specifier for the GPIO connected to the//
> But you could say that it is active low, for example.
>
>> +
>> + error0-gpios:
>> + maxItems: 1
>> + description:
>> + Specifier for the GPIO connected to the XWRN pin.
>
> The same.
>

Thanks, I'll make this change in v4.

>>
>> ...
>>
>> +
>> +examples:
>> + - |
>> + #include <dt-bindings/gpio/gpio.h>
>> +
>> + i2c {
>> + clock-frequency = <400000>;
>
> Drop, not really relevant.

Ack, I'll remove in v4.

>>
>> ...
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 575f0e6f0532..50bff3558d7d 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -21885,6 +21885,12 @@ T: git git://linuxtv.org/media.git
>> F: Documentation/devicetree/bindings/media/i2c/sony,imx415.yaml
>> F: drivers/media/i2c/imx415.c
>>
>> +SONY IMX728 SENSOR DRIVER
>> +M: Stuart Burtner <sburtner@d3embedded.com>
>> +L: linux-media@vger.kernel.org
>> +S: Odd Fixes
>
>
> Hm, why only odd fixes? If you don't care about driver, we also kind of
> might not care and remove it soon.
>

Understood -- we've discussed this and Stuart will be able to commit to
"Maintained" status going forward. I'll update this for v4.

>
> ...
>
> Obviously. Please drop it. You can use b4 relay if you need to escape
> corporate junk.
>

Sorry about that. I can look into using `b4 relay` going forward to
prevent that message from being added to patches. Unfortunately I'm not
sure of a way to remove it from my normal mail (like this message).

Thanks,

--
Sebastian