From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id ED1BEC433F5 for ; Wed, 23 Feb 2022 11:51:01 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230403AbiBWLv1 convert rfc822-to-8bit (ORCPT ); Wed, 23 Feb 2022 06:51:27 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53140 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233286AbiBWLv0 (ORCPT ); Wed, 23 Feb 2022 06:51:26 -0500 Received: from aposti.net (aposti.net [89.234.176.197]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 4DD1A5D183 for ; Wed, 23 Feb 2022 03:50:59 -0800 (PST) Date: Wed, 23 Feb 2022 11:50:48 +0000 From: Paul Cercueil Subject: Re: [PATCH 0/8] IIO: Where dev_pm_ops rework and namespaces meet. To: Jonathan Cameron Cc: linux-iio@vger.kernel.org, "Rafael J . Wysocki" , Lorenzo Bianconi , Tomasz Duszynski , Jonathan Cameron Message-Id: In-Reply-To: <20220220181522.541718-1-jic23@kernel.org> References: <20220220181522.541718-1-jic23@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1; format=flowed Content-Transfer-Encoding: 8BIT Precedence: bulk List-ID: X-Mailing-List: linux-iio@vger.kernel.org Hi Jonathan, Le dim., févr. 20 2022 at 18:15:14 +0000, Jonathan Cameron a écrit : > From: Jonathan Cameron > > Hi All, > > As these two activities have been ongoing across IIO at the same time > it was inevitable that there would be drivers were they both interact. > In particularly when we have EXPORT* of struct dev_pm_ops. > > This series covers those cases by introducing (in patch 2) > EXPORT_NS[_GPL]_SIMPLE_DEV_PM_OPS() and > EXPORT_NS[_GPL]_RUNTIME_DEV_PM_OPS() > that add a namespace as the final parameter. If we now have namespace versions of the macros, I'm starting to wonder if the original macros are still useful. In which case would we need to export a dev_pm_ops outside the scope of a namespace? Cheers, -Paul > In each driver (with the exception of the kxsd9 which was already > using a IIO_KXSD9 namespace) I first move to the new *_PM_OPS() > and pm_[sleep_]_ptr() then in a second patch move to the new > namespaces. Initially I had these two steps done as one patch > per driver but that was harder to follow and broke the rule of > one thing per patch. > > All comments welcome. > > Thanks, > > Jonathan > > Jonathan Cameron (8): > iio: chemical: scd30: Export dev_pm_ops instead of suspend() and > resume() > PM: core: Add NS varients of EXPORT[_GPL]_SIMPLE_DEV_PM_OPS and > runtime pm equiv > iio: chemical: scd30: Move symbol exports into IIO_SCD30 namespace > iio:accel:kxsd9: Switch from CONFIG_PM guards to pm_ptr() etc > iio: humidity: hts221: Use EXPORT_SIMPLE_DEV_PM_OPS() to allow > compiler to remove dead code. > iio: humidity: hts221: Move symbol exports into IIO_HTS221 namespace > iio: imu: lsm6dsx: Use new pm_sleep_ptr() and > EXPORT_SIMPLE_DEV_PM_OPS() > iio: imu: lsm6dsx: Move exported symbols to the IIO_LSM6DSX > namespace > > drivers/iio/accel/kxsd9-i2c.c | 2 +- > drivers/iio/accel/kxsd9-spi.c | 2 +- > drivers/iio/accel/kxsd9.c | 11 ++--------- > drivers/iio/chemical/scd30.h | 5 +---- > drivers/iio/chemical/scd30_core.c | 10 +++++----- > drivers/iio/chemical/scd30_i2c.c | 3 ++- > drivers/iio/chemical/scd30_serial.c | 3 ++- > drivers/iio/humidity/hts221_core.c | 12 +++++------- > drivers/iio/humidity/hts221_i2c.c | 3 ++- > drivers/iio/humidity/hts221_spi.c | 3 ++- > drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c | 12 +++++------- > drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_i2c.c | 3 ++- > drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_i3c.c | 3 ++- > drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_spi.c | 3 ++- > include/linux/pm.h | 14 +++++++++----- > include/linux/pm_runtime.h | 10 ++++++++-- > 16 files changed, 51 insertions(+), 48 deletions(-) > > -- > 2.35.1 >