On 1/7/21 10:00 PM, Fabio Estevam wrote:

On Thu, Jan 7, 2021 at 2:38 PM Fabio Estevam [off-list ref] wrote:

quoted

Hi,

The imx6ul-evk uses a MC34901 CAN transceiver and the STB pin
(standby_ needs to be controlled (1 = standby, 0 = normal mode)

As far as I can see, there is no support for the STB pin currently in
he can framework.

What would be the correct way to describe it in the devicetree?

Would it be acceptable to use something like this?
https://pastebin.com/raw/SsY6USdT

I made a simpler approach and submitted patches for review.

There is at least one board (IIRC the mx28evk) which has a dual transceiver with
one standby pin for both channels. This is why we modeled the standby, enable or
listen-only pin as a regulator. It's already supported by the flexcan driver.

See xceiver-supply in the imx dst, e.g.:

https://elixir.bootlin.com/linux/v5.10/source/arch/arm/boot/dts/imx6qdl-tx6.dtsi#L240

Marc

-- 
Pengutronix e.K.                 | Marc Kleine-Budde           |
Embedded Linux                   | https://www.pengutronix.de  |
Vertretung West/Dortmund         | Phone: +49-231-2826-924     |
Amtsgericht Hildesheim, HRA 2686 | Fax:   +49-5121-206917-5555 |