On 04/10/2023 20:46, Ayush Singh wrote:
Add DT bindings for Texas Instruments Simplelink CC1352P7 wireless MCU
BeaglePlay has CC1352P7 co-processor connected to the main AM62 (running Linux) over UART. In the BeagleConnect Technology, CC1352 is responsible for handling 6LoWPAN communication with beagleconnect freedom nodes as well as their discovery.
It is used by gb-beagleplay greybus driver.
"Instead drop both sentences"
Signed-off-by: Ayush Singh ayushdevel1325@gmail.com
.../devicetree/bindings/net/ti,cc1352p7.yaml | 51 +++++++++++++++++++ MAINTAINERS | 6 +++ 2 files changed, 57 insertions(+) create mode 100644 Documentation/devicetree/bindings/net/ti,cc1352p7.yaml
diff --git a/Documentation/devicetree/bindings/net/ti,cc1352p7.yaml b/Documentation/devicetree/bindings/net/ti,cc1352p7.yaml new file mode 100644 index 000000000000..291ba34c389b --- /dev/null +++ b/Documentation/devicetree/bindings/net/ti,cc1352p7.yaml @@ -0,0 +1,51 @@ +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/net/ti,cc1352p7.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml#
+title: Texas Instruments Simplelink CC1352P7 wireless MCU
+description:
- The cc1352p7 mcu can be connected via SPI or UART.
+maintainers:
- Ayush Singh ayushdevel1325@gmail.com
+properties:
- compatible:
- const: ti,cc1352p7
- clocks:
- maxItems: 2
- clock-names:
- description:
sclk_hf is the main system (mcu and peripherals) clocksclk_lf is low-frequency system clock
This does no go here, but to clocks. I wrote how it should be done. Don't ignore the feedback.
- items:
- const: sclk_hf- const: sclk_lf- reset-gpios: true
No, really, why do you change correct code into incorrect one? Who asked you to drop maxItems?
- vdds-supply: true
Best regards, Krzysztof