From nobody Wed Oct 8 00:41:56 2025 Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.9]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id B6E612DFA3F; Fri, 4 Jul 2025 07:54:36 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=198.175.65.9 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751615678; cv=none; b=ihQ3vcdPtPbuv3YnAzJjPhApKBU5+Bp3aI2x1lGjAab/kwI0j9YT1pzq4lrQ2gtBCFUQsndox2dcxX7b7w1lNiSj34muHp8pUSr3WwBIxc20vjj+ixYc5e2x9AzEJLa1J1zdTWwvobPqNqq6/M2lRjQthvteVnMte/AjH6L5Lz8= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751615678; c=relaxed/simple; bh=iZ8uhSvPympr52JRHUIloT5pt7V7PTIo8Iu2f3mqroc=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: MIME-Version; b=RsRdOrgYE7YcIYbhf4BY1q4HnoSYdem6MQe1uRGjWfMqlmd1+tQjoj494HHkf4Cxo3VexS/BlZwsxxldX2DLPM42QiZ+H8U5jGfeEU202walF21F+AwJogGItyYni5Zh6AlVC86oQQmLM4GYs2XCu+ei9U5sE2SWd9zSyxtg1bM= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.intel.com; spf=none smtp.mailfrom=linux.intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=U4MChP7r; arc=none smtp.client-ip=198.175.65.9 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.intel.com Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=linux.intel.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="U4MChP7r" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1751615676; x=1783151676; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=iZ8uhSvPympr52JRHUIloT5pt7V7PTIo8Iu2f3mqroc=; b=U4MChP7ryMSj7mfBm1LS09wE196w5SlEnXrgeHaTi8qCcX90q+z3k7zh kTUxNYxjwpJD0ECz2fdLYvcl40KI/FUKLaDGre2tQSndijBYukez1OAUp p44yfRu13XVcdyjOyYhN27i/7SZR9oeWI0Hsr6wPrVt0fKRlfeO1L3uWu r5LaVLbN8m9ui8WmYjHFEurVCzw2jfHf41Q3TED/rQNsy6iS5FwkazWBX GPhsiSgDzLf4ig+41OSDQRF6po0MQxsMCSmQTe3s8H18zUUPO13UE8Wg7 krpn958EYie0nGys2WR2ujeuPUkYC6216QGBxXkTBtQdLR4t5xrNlX7zx A==; X-CSE-ConnectionGUID: fw2XA+ZxQkae+SMAmmvUkw== X-CSE-MsgGUID: YQOFJD05Qmeh2jVFlmG1Xg== X-IronPort-AV: E=McAfee;i="6800,10657,11483"; a="76494740" X-IronPort-AV: E=Sophos;i="6.16,286,1744095600"; d="scan'208";a="76494740" Received: from orviesa003.jf.intel.com ([10.64.159.143]) by orvoesa101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 04 Jul 2025 00:54:32 -0700 X-CSE-ConnectionGUID: Xx2vY2G3RGeFt+YJuPVp5w== X-CSE-MsgGUID: fkSRgt++S62xRvHsdma/qw== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.16,286,1744095600"; d="scan'208";a="158924293" Received: from jkrzyszt-mobl2.ger.corp.intel.com (HELO svinhufvud.fi.intel.com) ([10.245.244.244]) by ORVIESA003-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 04 Jul 2025 00:54:28 -0700 Received: from svinhufvud.lan (localhost [IPv6:::1]) by svinhufvud.fi.intel.com (Postfix) with ESMTP id A6FA044910; Fri, 4 Jul 2025 10:54:26 +0300 (EEST) Organization: Intel Finland Oy - BIC 0357606-4 - c/o Alberga Business Park, 6 krs, Bertel Jungin Aukio 5, 02600 Espoo From: Sakari Ailus To: Jonathan Cameron , David Lechner , =?UTF-8?q?Nuno=20S=C3=A1?= , Andy Shevchenko , Vasileios Amoiridis , Sakari Ailus , Marcelo Schmitt , Jean-Baptiste Maneyrol , Peter Zijlstra , Greg KH , Matti Vaittinen , Al Viro , Javier Carrasco Cc: linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH 36/80] iio: pressure: Remove redundant pm_runtime_mark_last_busy() calls Date: Fri, 4 Jul 2025 10:54:26 +0300 Message-Id: <20250704075426.3219792-1-sakari.ailus@linux.intel.com> X-Mailer: git-send-email 2.39.5 In-Reply-To: <20250704075225.3212486-1-sakari.ailus@linux.intel.com> References: <20250704075225.3212486-1-sakari.ailus@linux.intel.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" pm_runtime_put_autosuspend(), pm_runtime_put_sync_autosuspend(), pm_runtime_autosuspend() and pm_request_autosuspend() now include a call to pm_runtime_mark_last_busy(). Remove the now-reduntant explicit call to pm_runtime_mark_last_busy(). Signed-off-by: Sakari Ailus --- The cover letter of the set can be found here . In brief, this patch depends on PM runtime patches adding marking the last busy timestamp in autosuspend related functions. The patches are here, on rc2: git://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git \ pm-runtime-6.17-rc1 drivers/iio/pressure/bmp280-core.c | 5 ----- drivers/iio/pressure/icp10100.c | 1 - drivers/iio/pressure/mpl115.c | 2 -- drivers/iio/pressure/zpa2326.c | 2 -- 4 files changed, 10 deletions(-) diff --git a/drivers/iio/pressure/bmp280-core.c b/drivers/iio/pressure/bmp2= 80-core.c index 74505c9ec1a0..b66f3b905055 100644 --- a/drivers/iio/pressure/bmp280-core.c +++ b/drivers/iio/pressure/bmp280-core.c @@ -752,7 +752,6 @@ static int bmp280_read_raw(struct iio_dev *indio_dev, =20 pm_runtime_get_sync(data->dev); ret =3D bmp280_read_raw_impl(indio_dev, chan, val, val2, mask); - pm_runtime_mark_last_busy(data->dev); pm_runtime_put_autosuspend(data->dev); =20 return ret; @@ -927,7 +926,6 @@ static int bmp280_write_raw(struct iio_dev *indio_dev, =20 pm_runtime_get_sync(data->dev); ret =3D bmp280_write_raw_impl(indio_dev, chan, val, val2, mask); - pm_runtime_mark_last_busy(data->dev); pm_runtime_put_autosuspend(data->dev); =20 return ret; @@ -2255,7 +2253,6 @@ static int bmp580_nvmem_read(void *priv, unsigned int= offset, void *val, =20 pm_runtime_get_sync(data->dev); ret =3D bmp580_nvmem_read_impl(priv, offset, val, bytes); - pm_runtime_mark_last_busy(data->dev); pm_runtime_put_autosuspend(data->dev); =20 return ret; @@ -2330,7 +2327,6 @@ static int bmp580_nvmem_write(void *priv, unsigned in= t offset, void *val, =20 pm_runtime_get_sync(data->dev); ret =3D bmp580_nvmem_write_impl(priv, offset, val, bytes); - pm_runtime_mark_last_busy(data->dev); pm_runtime_put_autosuspend(data->dev); =20 return ret; @@ -3120,7 +3116,6 @@ static int bmp280_buffer_postdisable(struct iio_dev *= indio_dev) { struct bmp280_data *data =3D iio_priv(indio_dev); =20 - pm_runtime_mark_last_busy(data->dev); pm_runtime_put_autosuspend(data->dev); =20 return 0; diff --git a/drivers/iio/pressure/icp10100.c b/drivers/iio/pressure/icp1010= 0.c index 1951c1cc84cf..3d83d0098a57 100644 --- a/drivers/iio/pressure/icp10100.c +++ b/drivers/iio/pressure/icp10100.c @@ -265,7 +265,6 @@ static int icp10100_get_measures(struct icp10100_state = *st, (be16_to_cpu(measures[1]) >> 8); *temperature =3D be16_to_cpu(measures[2]); =20 - pm_runtime_mark_last_busy(&st->client->dev); error_measure: pm_runtime_put_autosuspend(&st->client->dev); return ret; diff --git a/drivers/iio/pressure/mpl115.c b/drivers/iio/pressure/mpl115.c index 71beb28b7f2c..830a5065c008 100644 --- a/drivers/iio/pressure/mpl115.c +++ b/drivers/iio/pressure/mpl115.c @@ -108,7 +108,6 @@ static int mpl115_read_raw(struct iio_dev *indio_dev, ret =3D mpl115_comp_pressure(data, val, val2); if (ret < 0) return ret; - pm_runtime_mark_last_busy(data->dev); pm_runtime_put_autosuspend(data->dev); =20 return IIO_VAL_INT_PLUS_MICRO; @@ -118,7 +117,6 @@ static int mpl115_read_raw(struct iio_dev *indio_dev, ret =3D mpl115_read_temp(data); if (ret < 0) return ret; - pm_runtime_mark_last_busy(data->dev); pm_runtime_put_autosuspend(data->dev); *val =3D ret >> 6; =20 diff --git a/drivers/iio/pressure/zpa2326.c b/drivers/iio/pressure/zpa2326.c index 6eef37c0952d..4923a558a26a 100644 --- a/drivers/iio/pressure/zpa2326.c +++ b/drivers/iio/pressure/zpa2326.c @@ -697,7 +697,6 @@ static void zpa2326_suspend(struct iio_dev *indio_dev) =20 zpa2326_sleep(indio_dev); =20 - pm_runtime_mark_last_busy(parent); pm_runtime_put_autosuspend(parent); } =20 @@ -708,7 +707,6 @@ static void zpa2326_init_runtime(struct device *parent) pm_runtime_enable(parent); pm_runtime_set_autosuspend_delay(parent, 1000); pm_runtime_use_autosuspend(parent); - pm_runtime_mark_last_busy(parent); pm_runtime_put_autosuspend(parent); } =20 --=20 2.39.5