Checkpatch warning of qcom ath12k wifi node
Dmitry Baryshkov
dmitry.baryshkov at oss.qualcomm.com
Tue Dec 9 22:11:32 PST 2025
On 10/12/2025 05:51, Yingying Tang wrote:
> ./scripts/checkpatch.pl arch/arm64/boot/dts/qcom/sm8750-mtp.dts
> WARNING: DT compatible string vendor "pci17cb" appears un-documented --
> check ./Documentation/devicetree/bindings/vendor-prefixes.yaml#1026:
> FILE: arch/arm64/boot/dts/qcom/sm8750-mtp.dts:1026:
> + compatible = "pci17cb,1107";
>
> total: 0 errors, 1 warnings, 1275 lines checked
>
> NOTE: For some of the reported defects, checkpatch may be able to
> mechanically convert to the typical style using --fix or --fix-
> inplace.
>
> arch/arm64/boot/dts/qcom/sm8750-mtp.dts has style problems, please review.
>
> NOTE: If any of the errors are false positives, please report
> them to the maintainer, see CHECKPATCH in MAINTAINERS.
>
>
> All dts files which contain (compatible = "pci17cb,1107"; ) met this
> warning. But in fact this DT compatible string is already defined in
> devicetree/bindings/net/wireless/qcom,ath12k.yaml
Have you read the message carefully? It thinks that "pci17cb" "vendor"
string is not documented, but it is not a real "vendor" string.
>
> I raised a new change review https://lore.kernel.org/
> all/20251210032817.1264850-1-yingying.tang at oss.qualcomm.com, it also met
> this issue.
>
>
> Could please help to resolve this issue, thanks very much
Please send a patch, making checkpatch.pl stop from threating pciNNNN as
a vendor prefix.
--
With best wishes
Dmitry
More information about the ath12k
mailing list