Hi, On Mon, Feb 24, 2025 at 12:14 AM Damon Ding <damon.ding@xxxxxxxxxxxxxx> wrote: > > Remove the check related to CONFIG_PM in order to make the code more > concise, as the CONFIG_PM should be a required option for many drivers. > > In addition, it is preferable to use devm_pm_runtime_enable() instead of > manually invoking pm_runtime_enable() followed by pm_runtime_disable(). > > Signed-off-by: Damon Ding <damon.ding@xxxxxxxxxxxxxx> > --- > .../drm/bridge/analogix/analogix_dp_core.c | 30 ++++--------------- > 1 file changed, 6 insertions(+), 24 deletions(-) If someone really has a reason to run this driver without CONFIG_PM then happy to hear about it. Until then, I'd say this is fine and matches what most drivers assume. Suggested-by: Douglas Anderson <dianders@xxxxxxxxxxxx> Reviewed-by: Douglas Anderson <dianders@xxxxxxxxxxxx>
