On 08/03/2023 14:10, Stephan Gerhold wrote: > Since the SoC compatibles must be followed by the IP version compatible > (e.g. compatible = "qcom,msm8916-tsens", "qcom,tsens-v0_1";) it is > redundant to list all the SoC compatibles again in the if statement. > It will already match the IP-version compatible. > > The list has already become inconsistent since for example > "qcom,msm8939-tsens" is covered by the if statement but is not listed > there explicitly like the other SoCs. > > Simplify this by dropping the redundant SoC compatibles. ipq8064 and > msm8960 are still needed because they do not have an IP-version > compatible. > > Signed-off-by: Stephan Gerhold <stephan.gerhold@xxxxxxxxxxxxxxx> > --- > Changes in v2: > - Drop new redundant qcom,tsens-v2 compatibles as well (see discussion > in https://lore.kernel.org/linux-arm-msm/Y3ZFDRI6ypg18S27@xxxxxxxxxxx/) Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> Best regards, Krzysztof
