| Message ID | 20241028-topic-mcan-wakeup-source-v6-12-v5-2-33edc0aba629@baylibre.com |
|---|---|
| State | Not Applicable |
| Headers | show |
| Series | can: m_can: Add am62 wakeup support | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success | |
| robh/patch-applied | success | |
| robh/dtbs-check | warning | build log |
| robh/dt-meta-schema | success |
On Oct 28, 2024 at 18:38:08 +0100, Markus Schneider-Pargmann wrote: > The m_can unit can be integrated in different ways. For AM62 the unit is > integrated in different parts of the system (MCU or Main domain) and can > be powered by different external power sources. For example on am62-lp-sk > mcu_mcan0 and mcu_mcan1 are powered through VDDSHV_CANUART by an > external regulator. To be able to describe these relationships, add a > vio-supply property to this binding. > > Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > Signed-off-by: Markus Schneider-Pargmann <msp@baylibre.com> > --- > Documentation/devicetree/bindings/net/can/bosch,m_can.yaml | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml b/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml > index 0c1f9fa7371897d45539ead49c9d290fb4966f30..aac2add319e240f4f561b755f41bf267b807ebcd 100644 > --- a/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml > +++ b/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml > @@ -140,6 +140,10 @@ properties: > > wakeup-source: true > > + vio-supply: > + description: > + Reference to the main power supply of the unit. > + Reviewed-by: Dhruva Gole <d-gole@ti.com>
On 12:28-20241029, Dhruva Gole wrote: > On Oct 28, 2024 at 18:38:08 +0100, Markus Schneider-Pargmann wrote: > > The m_can unit can be integrated in different ways. For AM62 the unit is > > integrated in different parts of the system (MCU or Main domain) and can > > be powered by different external power sources. For example on am62-lp-sk > > mcu_mcan0 and mcu_mcan1 are powered through VDDSHV_CANUART by an > > external regulator. To be able to describe these relationships, add a > > vio-supply property to this binding. > > > > Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > Signed-off-by: Markus Schneider-Pargmann <msp@baylibre.com> > > --- > > Documentation/devicetree/bindings/net/can/bosch,m_can.yaml | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml b/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml > > index 0c1f9fa7371897d45539ead49c9d290fb4966f30..aac2add319e240f4f561b755f41bf267b807ebcd 100644 > > --- a/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml > > +++ b/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml > > @@ -140,6 +140,10 @@ properties: > > > > wakeup-source: true > > > > + vio-supply: > > + description: > > + Reference to the main power supply of the unit. > > + > > Reviewed-by: Dhruva Gole <d-gole@ti.com> Might want to conclude on the usage model discussed in [1] [1] https://lore.kernel.org/all/20241029120302.3twkliytrn5hjufi@sleek/
diff --git a/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml b/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml index 0c1f9fa7371897d45539ead49c9d290fb4966f30..aac2add319e240f4f561b755f41bf267b807ebcd 100644 --- a/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml +++ b/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml @@ -140,6 +140,10 @@ properties: wakeup-source: true + vio-supply: + description: + Reference to the main power supply of the unit. + required: - compatible - reg