From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.8]) (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 538453BD649; Wed, 25 Mar 2026 12:28:11 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=192.198.163.8 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774441692; cv=none; b=XTTbxrpqdIojqqn9nao320t/fY8cRX4XfnMJ5gzXcG4dcM+K/aV3snqneB4h903F9HsnyX+/MRLhSXOvDdAsdps1sjonWF01vUmruTZGKWIkDexftECUZ3IB5EEj5udksPXu1RLVJCtGycSZ0m4HOUWap/HJINIe+pEuuovnzlw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774441692; c=relaxed/simple; bh=HKBn5uysag1AqcbJn18u4AYp5pUb3WuGKnYvEnUulkE=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=ieZ8B+7t6ANre+SXiXEz+AiktBHcqLkhqWmqfjdCsUZJpqQAIXAmm4uFSis42lMXiBOMl3q3KL8ORm6vhEs9mzKKNn+HWd+TC/sDg9fxi4xpfbJsVp8tWbkKL6B4WtmBQjIxamMD8Eix2C3TWKt33m0K+zY8poCJxQVQfyMtTnk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.intel.com; spf=pass smtp.mailfrom=linux.intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=myg16ehK; arc=none smtp.client-ip=192.198.163.8 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.intel.com Authentication-Results: smtp.subspace.kernel.org; spf=pass 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="myg16ehK" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1774441691; x=1805977691; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=HKBn5uysag1AqcbJn18u4AYp5pUb3WuGKnYvEnUulkE=; b=myg16ehKmA7xEsii7u94I/IRj1NYUh3VGaOEtyT6qCLxsd57kpM63J8D hl8RfSlm5Bbs8D7Wa12ZepHb9tT+AONaBQ8u4Tgj3scQfoPOn2iJnXxkD rdAattLzBEsFY/a85fNiGHcyh8h6JndRBmYrOEwq0Ris8Qt32cJfJObaW q2B+8YVf9P+eYrwwOSgyxEjwvJLJyllfQpFTcV5+P/j7Uzb/GvBX1feFN aqYe9ZM9KFASCXTyV0dBlrMWdvwRvhGeL6TiTaXPD40aTc1I8zMcxobwg ju4ZicRUtJx417b0+kYSwoSNjPvKacjRtVZdV5F3cfkCcZ3JheeuIvGZU w==; X-CSE-ConnectionGUID: oCBm2pqQT2G3o64MS6pIwA== X-CSE-MsgGUID: dkQ3CnT6QtKVPjI1wwjEzw== X-IronPort-AV: E=McAfee;i="6800,10657,11739"; a="93052995" X-IronPort-AV: E=Sophos;i="6.23,140,1770624000"; d="scan'208";a="93052995" Received: from orviesa007.jf.intel.com ([10.64.159.147]) by fmvoesa102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 25 Mar 2026 05:28:11 -0700 X-CSE-ConnectionGUID: WXmCbLvHSYWw9oqjto/1YQ== X-CSE-MsgGUID: IjuF3ATRRquJzxN6WE8fcQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.23,140,1770624000"; d="scan'208";a="224929354" Received: from dalessan-mobl3.ger.corp.intel.com (HELO kekkonen.fi.intel.com) ([10.245.245.64]) by orviesa007-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 25 Mar 2026 05:28:09 -0700 Received: from kekkonen.localdomain (localhost [IPv6:::1]) by kekkonen.fi.intel.com (Postfix) with SMTP id EB56C121CF9; Wed, 25 Mar 2026 14:28:12 +0200 (EET) Date: Wed, 25 Mar 2026 14:28:12 +0200 Organization: Intel Finland Oy - BIC 0357606-4 - c/o Alberga Business Park, 6 krs, Bertel Jungin Aukio 5, 02600 Espoo From: Sakari Ailus To: Michael Riesch Cc: Elgin Perumbilly , tarang.raval@siliconsignals.io, Mauro Carvalho Chehab , linux-media@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] media: i2c: imx415: Drop redundant runtime PM callbacks Message-ID: References: <20260324124524.135278-1-elgin.perumbilly@siliconsignals.io> <1378ed13-6744-422a-b0b8-02621117238f@collabora.com> Precedence: bulk X-Mailing-List: linux-media@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1378ed13-6744-422a-b0b8-02621117238f@collabora.com> Hi Michael, On Tue, Mar 24, 2026 at 08:42:31PM +0100, Michael Riesch wrote: > Hi Elgin, > > Thanks for the patch but... > > On 3/24/26 13:45, Elgin Perumbilly wrote: > > Replace runtime_suspend/resume wrappers by using power helpers > > directly with DEFINE_RUNTIME_DEV_PM_OPS(). > > ...why? What advantage does this refactoring bring? It looks like patch removes two redundant functions, doesn't it? :-) -- Regards, Sakari Ailus