| Message ID | 20240729074135.3850634-2-msp@baylibre.com |
|---|---|
| State | Changes Requested |
| Headers | show |
| Series | can: m_can: Add am62 wakeup support | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success | |
| robh/patch-applied | fail | build log |
On Mon, Jul 29, 2024 at 09:41:29AM +0200, Markus Schneider-Pargmann wrote: > m_can can be a wakeup source on some devices. Especially on some of the > am62* SoCs pins, connected to m_can in the mcu, can be used to wakeup > the SoC. > > The wakeup-source property defines on which devices m_can can be used > for wakeup. > > The pins associated with m_can have to have a special configuration to > be able to wakeup the SoC. This configuration is described in the wakeup > pinctrl state while the default state describes the default > configuration. > > Signed-off-by: Markus Schneider-Pargmann <msp@baylibre.com> > --- > .../bindings/net/can/bosch,m_can.yaml | 20 +++++++++++++++++++ > 1 file changed, 20 insertions(+) > > diff --git a/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml b/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml > index c4887522e8fe..ef63f6b8455d 100644 > --- a/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml > +++ b/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml > @@ -106,6 +106,22 @@ properties: > maximum: 32 > minItems: 1 > > + pinctrl-0: > + description: Default pinctrl state > + > + pinctrl-1: > + description: Wakeup pinctrl state > + > + pinctrl-names: > + description: > + When present should contain at least "default" describing the default pin > + states. The second state called "wakeup" describes the pins in their > + wakeup configuration required to exit sleep states. > + minItems: 1 > + items: > + - const: default > + - const: wakeup > + > power-domains: > description: > Power domain provider node and an args specifier containing > @@ -122,6 +138,10 @@ properties: > minItems: 1 > maxItems: 2 > > + wakeup-source: > + $ref: /schemas/types.yaml#/definitions/flag I thought we had a common schema defining the type, but we don't. I'm going to add it now. So just need: wakeup-source: true > + description: This device is capable to wakeup the SoC. > + > required: > - compatible > - reg > -- > 2.45.2 >
diff --git a/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml b/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml index c4887522e8fe..ef63f6b8455d 100644 --- a/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml +++ b/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml @@ -106,6 +106,22 @@ properties: maximum: 32 minItems: 1 + pinctrl-0: + description: Default pinctrl state + + pinctrl-1: + description: Wakeup pinctrl state + + pinctrl-names: + description: + When present should contain at least "default" describing the default pin + states. The second state called "wakeup" describes the pins in their + wakeup configuration required to exit sleep states. + minItems: 1 + items: + - const: default + - const: wakeup + power-domains: description: Power domain provider node and an args specifier containing @@ -122,6 +138,10 @@ properties: minItems: 1 maxItems: 2 + wakeup-source: + $ref: /schemas/types.yaml#/definitions/flag + description: This device is capable to wakeup the SoC. + required: - compatible - reg
m_can can be a wakeup source on some devices. Especially on some of the am62* SoCs pins, connected to m_can in the mcu, can be used to wakeup the SoC. The wakeup-source property defines on which devices m_can can be used for wakeup. The pins associated with m_can have to have a special configuration to be able to wakeup the SoC. This configuration is described in the wakeup pinctrl state while the default state describes the default configuration. Signed-off-by: Markus Schneider-Pargmann <msp@baylibre.com> --- .../bindings/net/can/bosch,m_can.yaml | 20 +++++++++++++++++++ 1 file changed, 20 insertions(+)