| Message ID | 20230718-pm8916-mclk-v1-2-4b4a58b4240a@gerhold.net |
|---|---|
| State | Accepted |
| Commit | dfc491e55255a96b2d43cdb74db10d4222890769 |
| Headers | show |
| Series | ASoC: codecs: msm8916-wcd-analog: Cleanup DT bindings | expand |
On Tue, 18 Jul 2023 13:40:14 +0200, Stephan Gerhold wrote: > pm8916-wcd-analog-codec has just a single reg region, so having a name > for it provides no additional value. > > Drop it completely from the schema and example. There is not really any > point in keeping it (even as deprecated) because it was never used. In > old DTBs it will just be ignored as before. > > Signed-off-by: Stephan Gerhold <stephan@gerhold.net> > --- > .../devicetree/bindings/sound/qcom,pm8916-wcd-analog-codec.yaml | 5 ----- > 1 file changed, 5 deletions(-) > Reviewed-by: Rob Herring <robh@kernel.org>
diff --git a/Documentation/devicetree/bindings/sound/qcom,pm8916-wcd-analog-codec.yaml b/Documentation/devicetree/bindings/sound/qcom,pm8916-wcd-analog-codec.yaml index 77e3cfba4746..5053799c88b5 100644 --- a/Documentation/devicetree/bindings/sound/qcom,pm8916-wcd-analog-codec.yaml +++ b/Documentation/devicetree/bindings/sound/qcom,pm8916-wcd-analog-codec.yaml @@ -19,10 +19,6 @@ properties: reg: maxItems: 1 - reg-names: - items: - - const: pmic-codec-core - clocks: maxItems: 1 @@ -126,7 +122,6 @@ examples: audio-codec@f000 { compatible = "qcom,pm8916-wcd-analog-codec"; reg = <0xf000>; - reg-names = "pmic-codec-core"; clocks = <&gcc GCC_CODEC_DIGCODEC_CLK>; clock-names = "mclk"; qcom,mbhc-vthreshold-low = <75 150 237 450 500>;
pm8916-wcd-analog-codec has just a single reg region, so having a name for it provides no additional value. Drop it completely from the schema and example. There is not really any point in keeping it (even as deprecated) because it was never used. In old DTBs it will just be ignored as before. Signed-off-by: Stephan Gerhold <stephan@gerhold.net> --- .../devicetree/bindings/sound/qcom,pm8916-wcd-analog-codec.yaml | 5 ----- 1 file changed, 5 deletions(-)