Re: [PATCH] can: tcan4x5x: put tcan into sleep when removing driver
Marc Kleine-Budde <[email protected]> Mon, 27 Jul 2026 12:06:36 +0200
| Newsgroups | org.kernel.vger.linux-can,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <[email protected]> |
On 27.07.2026 11:48:59, Sean Nyekjaer wrote: > Put the tcan4x5x transceiver into sleep mode when the driver is Ah, the device has an internal transceiver. > removed, instead of leaving it in its current operating mode. > This reduces power consumption(3mA@12V) once the driver is > no longer bound to the device. > > Note that the reset pin is needed to bring the device out of sleep > and back into standby mode again. Please add a comment, that the reset pin is needed and maybe refer to "8.4.3 Sleep Mode" of https://www.ti.com/lit/gpn/tcan4550 Does it still work, if you move the regmap_update_bits() to tcan4x5x_power_enable()? > Signed-off-by: Sean Nyekjaer <[email protected]> > --- > I havn't come across any drivers that leaves the devices in the lowest > power mode when unbinded. But we need this to enter our lowest power > mode in suspend. > If it doesn't fit in, I'm happy to carry this out of tree... The mcp251xfd driver puts the device into sleep mode. regards, Marc -- Pengutronix e.K. | Marc Kleine-Budde | Embedded Linux | https://www.pengutronix.de | Vertretung Nürnberg | Phone: +49-5121-206917-129 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-9 |
signature.asc
(application/pgp-signature, 228 B)
-----BEGIN PGP SIGNATURE----- iHUEABYKAB0WIQSl+MghEFFAdY3pYJLMOmT6rpmt0gUCamctqQAKCRDMOmT6rpmt 0jaWAQDBYRFintTueqsVK/UF4xNRS11PDNKNfJJWBGTYFIpxJAD/Q4wh9VAaXgk2 nRE14j0VSB022raMfgGMoklVhbUX9ww= =AiFj -----END PGP SIGNATURE-----