A recent commit refactored the printing of the VFE hardware version, but
(without it being mentioned) also changed the log level from debug to
info.
This results in several hundred lines of repeated log spam during boot
and use, for example, on the Lenovo ThinkPad X13s:
qcom-camss
ac5a000.camss: VFE:1 HW Version = 1.2.2
qcom-camss
ac5a000.camss: VFE:0 HW Version = 1.2.2
qcom-camss
ac5a000.camss: VFE:2 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:5 HW Version = 1.3.0
qcom-camss
ac5a000.camss: VFE:6 HW Version = 1.3.0
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
qcom-camss
ac5a000.camss: VFE:7 HW Version = 1.3.0
qcom-camss
ac5a000.camss: VFE:7 HW Version = 1.3.0
...
Suppress the version logging by demoting to debug level again.
Fixes: 10693fed125d ("media: qcom: camss: vfe: Move common code into vfe core")
Cc: stable@vger.kernel.org
Cc: Depeng Shao <quic_depengs@quicinc.com>
Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Signed-off-by: Johan Hovold <johan+linaro@kernel.org>
Signed-off-by: Bryan O'Donoghue <bod@kernel.org>
Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
u32 rev = (hw_version >> HW_VERSION_REVISION) & 0xFFF;
u32 step = (hw_version >> HW_VERSION_STEPPING) & 0xFFFF;
- dev_info(vfe->camss->dev, "VFE:%d HW Version = %u.%u.%u\n",
- vfe->id, gen, rev, step);
+ dev_dbg(vfe->camss->dev, "VFE:%d HW Version = %u.%u.%u\n",
+ vfe->id, gen, rev, step);
return hw_version;
}