[PATCH 2/2] dt-bindings: media: imx219: Describe sensor address using the reg property

David Heidelberg david at ixit.cz
Tue Dec 17 13:22:29 PST 2024


Oh, the checkpatch script keeps warning me every time I commit—I guess 
it needs to be addressed.

I currently use:

[diff]
   srcprefix = ./
   dstprefix = ./

Originally, my intention was to use:

[diff]
   noprefix = true

This was to make it easier to copy-paste directly into the shell (git 
log -p, etc.). However, it ended up breaking too many applications. 
Setting srcprefix and dstprefix instead keeps everything working properly.

Is there any chance a patch allowing a ./ prefix could make it into 
checkpatch? What do you think, Krzysztof?

Thank you,
David

On 17/12/2024 02:40, Krzysztof Kozlowski wrote:
> On Mon, Dec 16, 2024 at 09:41:51PM -0500, David Heidelberg wrote:
>> Use the reg property instead of text in the description.
>> Drop useless description of reg property.
>>
>> Signed-off-by: David Heidelberg <david at ixit.cz>
>> ---
>>   Documentation/devicetree/bindings/media/i2c/sony,imx219.yaml | 5 ++---
>>   1 file changed, 2 insertions(+), 3 deletions(-)
>>
>> diff --git ./Documentation/devicetree/bindings/media/i2c/sony,imx219.yaml ./Documentation/devicetree/bindings/media/i2c/sony,imx219.yaml
> 
> You are using non-standard tools to generate patches. See checkpatch
> warnings.
> 
> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski at linaro.org>
> 
> Best regards,
> Krzysztof
> 

-- 
David Heidelberg




More information about the linux-arm-kernel mailing list