Re: [PATCH 1/3] media: qcom: camss: vfe: Stop spamming logs with version

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

On 20/05/2025 10:58, Bryan O'Donoghue wrote:
> On 20/05/2025 09:51, Krzysztof Kozlowski wrote:
>>           qcom-camss ac5a000.camss: VFE:0 HW Version = 1.2.2
>>           qcom-camss ac5a000.camss: VFE:1 HW Version = 1.2.2
>>           qcom-camss ac5a000.camss: VFE:2 HW Version = 1.2.2
>>           qcom-camss ac5a000.camss: VFE:3 HW Version = 1.2.2
>>           qcom-camss ac5a000.camss: VFE:4 HW Version = 1.3.0
>>           qcom-camss ac5a000.camss: VFE:5 HW Version = 1.3.0
>>           qcom-camss ac5a000.camss: VFE:6 HW Version = 1.3.0
>>           qcom-camss ac5a000.camss: VFE:7 HW Version = 1.3.0
> 
> This prints the hardware version of eight distinct hardware blocks VFE 
> index increases.
> 
> TBH I still find this useful when debugging hardware.

How? Can you respond to actual arguments repeated 5 times that this is
fixed and always known?

If this is always known, in what way this is useful?

> 
> My personal preference is to print it once on boot and skip subsequent. 
> Which I think is perfectly reasonable for DEBUG scenario.
What are you responding to? Before you said you find it useful for
knowing each block power up and down?



Best regards,
Krzysztof