Thread (22 messages) 22 messages, 3 authors, 2018-01-05

Re: [PATCH v6 3/6] can: m_can: Add PM Runtime

From: Marc Kleine-Budde <mkl@pengutronix.de>
Date: 2018-01-03 15:17:35
Also in: linux-can, linux-devicetree, lkml

On 01/03/2018 04:06 PM, Faiz Abbas wrote:
Hi,

On Wednesday 03 January 2018 07:55 PM, Marc Kleine-Budde wrote:
quoted
On 01/03/2018 01:39 PM, Faiz Abbas wrote:
quoted
On Tuesday 02 January 2018 09:37 PM, Marc Kleine-Budde wrote:
quoted
On 12/22/2017 02:31 PM, Faiz Abbas wrote:
quoted
From: Franklin S Cooper Jr <redacted>

Add support for PM Runtime which is the new way to handle managing clocks.
However, to avoid breaking SoCs not using PM_RUNTIME leave the old clk
management approach in place.
There is no PM_RUNTIME anymore since 464ed18ebdb6 ("PM: Eliminate
CONFIG_PM_RUNTIME")
Ok. Will change the commit message.
quoted
Have a look at the discussion: https://patchwork.kernel.org/patch/9436507/ :
quoted
quoted
Well, I admit it would be nicer if drivers didn't have to worry about 
whether or not CONFIG_PM was enabled.  A slightly cleaner approach 
from the one outlined above would have the probe routine do this:

	my_power_up(dev);
	pm_runtime_set_active(dev);
	pm_runtime_get_noresume(dev);
	pm_runtime_enable(dev);
This discussion seems to be about cases in which CONFIG_PM is not
enabled. CONFIG_PM is always selected in the case of omap devices.
Yes, but in the commit message you state that you need to support
systems that don't have PM_RUNTIME enabled. The only mainline SoCs I see
is "arch/arm/boot/dts/sama5d2.dtsi" so far. Please check if they select
CONFIG_PM, then we can make the driver much simpler.
Actually the old clock management (for hclk which is the interface
clock) is still required as mentioned in the cover letter. Will change
the rather misleading description.
Ok. So you can use the code as discussed on
https://patchwork.kernel.org/patch/9436507/ ?

Marc

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

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help