[PATCH v1 1/2] rename tas2764 to tas27xx

[PATCH v1 1/2] rename tas2764 to tas27xx

[Raphael-Xu]

rename tas2764.yaml to tas27xx.yaml

Signed-off-by: Raphael-Xu <13691752556@139.com>
---
 .../devicetree/bindings/sound/{tas2764.yaml => tas27xx.yaml}      | 0
 1 file changed, 0 insertions(+), 0 deletions(-)
 rename Documentation/devicetree/bindings/sound/{tas2764.yaml => tas27xx.yaml} (100%)

diff --git a/Documentation/devicetree/bindings/sound/tas2764.yaml b/Documentation/devicetree/bindings/sound/tas27xx.yaml
similarity index 100%
rename from Documentation/devicetree/bindings/sound/tas2764.yaml
rename to Documentation/devicetree/bindings/sound/tas27xx.yaml
-- 
2.35.1

[PATCH v1 2/2] update to support either TAS2764 or TAS2780

[Raphael-Xu]

update tas27xx.yaml to support either TAS2764 or TAS2780

Signed-off-by: Raphael-Xu <13691752556@139.com>
---
 .../devicetree/bindings/sound/tas27xx.yaml     | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

diff --git a/Documentation/devicetree/bindings/sound/tas27xx.yaml b/Documentation/devicetree/bindings/sound/tas27xx.yaml
index 5bf8c76ecda1..191f4bceb4ee 100644
--- a/Documentation/devicetree/bindings/sound/tas27xx.yaml
+++ b/Documentation/devicetree/bindings/sound/tas27xx.yaml
@@ -1,25 +1,27 @@
 # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
-# Copyright (C) 2020 Texas Instruments Incorporated
+# Copyright (C) 2020-2022 Texas Instruments Incorporated
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/sound/tas2764.yaml#"
+$id: "http://devicetree.org/schemas/sound/tas27xx.yaml#"
 $schema: "http://devicetree.org/meta-schemas/core.yaml#"
 
-title: Texas Instruments TAS2764 Smart PA
+title: Texas Instruments TAS2764/TAS2780 Smart PA
 
 maintainers:
-  - Dan Murphy <dmurphy@ti.com>
+  - Shenghao Ding <shenghao-ding@ti.com>
 
 description: |
-  The TAS2764 is a mono, digital input Class-D audio amplifier optimized for
-  efficiently driving high peak power into small loudspeakers.
-  Integrated speaker voltage and current sense provides for
-  real time monitoring of loudspeaker behavior.
+  The TAS2764/TAS2780 is a mono, digital input Class-D audio amplifier
+  optimized for efficiently driving high peak power into small
+  loudspeakers. Integrated speaker voltage and current sense provides
+  for real time monitoring of loudspeaker behavior.
 
 properties:
   compatible:
     enum:
       - ti,tas2764
+      or
+      - ti,tas2780
 
   reg:
     maxItems: 1
-- 
2.35.1

Re: [PATCH v1 1/2] rename tas2764 to tas27xx

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 206 bytes --]

On Fri, Apr 08, 2022 at 10:11:18PM +0800, Raphael-Xu wrote:
> rename tas2764.yaml to tas27xx.yaml
> 
> Signed-off-by: Raphael-Xu <13691752556@139.com>

This is fine but it's just the binding updates?

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: [PATCH v1 1/2] rename tas2764 to tas27xx

[Mark Brown]

On Fri, 8 Apr 2022 22:11:18 +0800, Raphael-Xu wrote:
> rename tas2764.yaml to tas27xx.yaml
> 
> 

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next

Thanks!

[1/2] rename tas2764 to tas27xx
      commit: aecf03614d5ded3f930143cda2ba3d76d09b107d
[2/2] update to support either TAS2764 or TAS2780
      commit: 239556a34385706562a481f452ce0963fb569cb8

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark

Re: [PATCH v1 2/2] update to support either TAS2764 or TAS2780

[Rob Herring]

On Fri, Apr 08, 2022 at 10:11:19PM +0800, Raphael-Xu wrote:
> update tas27xx.yaml to support either TAS2764 or TAS2780

DT patches must go to DT list. 

Now that this is in linux-next, it's breaking 'make dt_binding_check':


./Documentation/devicetree/bindings/sound/tas27xx.yaml:24:7: [error] syntax error: could not find expected ':' (syntax)
./Documentation/devicetree/bindings/sound/tas27xx.yaml:  while scanning a simple key
  in "<unicode string>", line 23, column 7
could not find expected ':'
  in "<unicode string>", line 24, column 7
make[1]: *** Deleting file 'Documentation/devicetree/bindings/sound/tas27xx.example.dts'
Traceback (most recent call last):
  File "/usr/local/bin/dt-extract-example", line 52, in <module>
    binding = yaml.load(open(args.yamlfile, encoding='utf-8').read())
  File "/usr/local/lib/python3.8/dist-packages/ruamel/yaml/main.py", line 434, in load
    return constructor.get_single_data()
  File "/usr/local/lib/python3.8/dist-packages/ruamel/yaml/constructor.py", line 119, in get_single_data
    node = self.composer.get_single_node()
  File "_ruamel_yaml.pyx", line 706, in _ruamel_yaml.CParser.get_single_node
  File "_ruamel_yaml.pyx", line 724, in _ruamel_yaml.CParser._compose_document
  File "_ruamel_yaml.pyx", line 775, in _ruamel_yaml.CParser._compose_node
  File "_ruamel_yaml.pyx", line 889, in _ruamel_yaml.CParser._compose_mapping_node
  File "_ruamel_yaml.pyx", line 775, in _ruamel_yaml.CParser._compose_node
  File "_ruamel_yaml.pyx", line 889, in _ruamel_yaml.CParser._compose_mapping_node
  File "_ruamel_yaml.pyx", line 775, in _ruamel_yaml.CParser._compose_node
  File "_ruamel_yaml.pyx", line 889, in _ruamel_yaml.CParser._compose_mapping_node
  File "_ruamel_yaml.pyx", line 773, in _ruamel_yaml.CParser._compose_node
  File "_ruamel_yaml.pyx", line 852, in _ruamel_yaml.CParser._compose_sequence_node
  File "_ruamel_yaml.pyx", line 904, in _ruamel_yaml.CParser._parse_next_event
ruamel.yaml.scanner.ScannerError: while scanning a simple key
  in "<unicode string>", line 23, column 7
could not find expected ':'
  in "<unicode string>", line 24, column 7


> 
> Signed-off-by: Raphael-Xu <13691752556@139.com>
> ---
>  .../devicetree/bindings/sound/tas27xx.yaml     | 18 ++++++++++--------
>  1 file changed, 10 insertions(+), 8 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/sound/tas27xx.yaml b/Documentation/devicetree/bindings/sound/tas27xx.yaml
> index 5bf8c76ecda1..191f4bceb4ee 100644
> --- a/Documentation/devicetree/bindings/sound/tas27xx.yaml
> +++ b/Documentation/devicetree/bindings/sound/tas27xx.yaml
> @@ -1,25 +1,27 @@
>  # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> -# Copyright (C) 2020 Texas Instruments Incorporated
> +# Copyright (C) 2020-2022 Texas Instruments Incorporated
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/sound/tas2764.yaml#"
> +$id: "http://devicetree.org/schemas/sound/tas27xx.yaml#"

This belonged in patch 1.

>  $schema: "http://devicetree.org/meta-schemas/core.yaml#"
>  
> -title: Texas Instruments TAS2764 Smart PA
> +title: Texas Instruments TAS2764/TAS2780 Smart PA
>  
>  maintainers:
> -  - Dan Murphy <dmurphy@ti.com>
> +  - Shenghao Ding <shenghao-ding@ti.com>
>  
>  description: |
> -  The TAS2764 is a mono, digital input Class-D audio amplifier optimized for
> -  efficiently driving high peak power into small loudspeakers.
> -  Integrated speaker voltage and current sense provides for
> -  real time monitoring of loudspeaker behavior.
> +  The TAS2764/TAS2780 is a mono, digital input Class-D audio amplifier
> +  optimized for efficiently driving high peak power into small
> +  loudspeakers. Integrated speaker voltage and current sense provides
> +  for real time monitoring of loudspeaker behavior.
>  
>  properties:
>    compatible:
>      enum:
>        - ti,tas2764
> +      or
> +      - ti,tas2780

'or'  in not valid.

Please submit a fix ASAP.

Rob