Received-SPF: pass (google.com: domain of linux-kernel+bounces-44130-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
Message-ID: <6568893d-13c7-ef1f-9c3f-88de0701c7aa@axentia.se>
Date: Tue, 30 Jan 2024 09:25:21 +0100
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.6.0
Subject: Re: [PATCH v2 04/15] mux: add mux_chip_resume() function
Content-Language: sv-SE
To: Thomas Richard <thomas.richard@bootlin.com>,
 Linus Walleij <linus.walleij@linaro.org>, Bartosz Golaszewski
 <brgl@bgdev.pl>, Andy Shevchenko <andy@kernel.org>,
 Tony Lindgren <tony@atomide.com>, Haojian Zhuang
 <haojian.zhuang@linaro.org>, Vignesh R <vigneshr@ti.com>,
 Aaro Koskinen <aaro.koskinen@iki.fi>,
 Janusz Krzysztofik <jmkrzyszt@gmail.com>, Andi Shyti
 <andi.shyti@kernel.org>, Vinod Koul <vkoul@kernel.org>,
 Kishon Vijay Abraham I <kishon@kernel.org>,
 Philipp Zabel <p.zabel@pengutronix.de>, Tom Joseph <tjoseph@cadence.com>,
 Lorenzo Pieralisi <lpieralisi@kernel.org>,
 =?UTF-8?Q?Krzysztof_Wilczy=c5=84ski?= <kw@linux.com>,
 Rob Herring <robh@kernel.org>, Bjorn Helgaas <bhelgaas@google.com>
Cc: linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org,
 linux-arm-kernel@lists.infradead.org, linux-omap@vger.kernel.org,
 linux-i2c@vger.kernel.org, linux-phy@lists.infradead.org,
 linux-pci@vger.kernel.org, gregory.clement@bootlin.com,
 theo.lebrun@bootlin.com, thomas.petazzoni@bootlin.com, u-kumar1@ti.com
References: <20240102-j7200-pcie-s2r-v2-0-8e4f7d228ec2@bootlin.com>
 <20240102-j7200-pcie-s2r-v2-4-8e4f7d228ec2@bootlin.com>
From: Peter Rosin <peda@axentia.se>
In-Reply-To: <20240102-j7200-pcie-s2r-v2-4-8e4f7d228ec2@bootlin.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?TGtwU2xKZUh3dHc1Tm5YZGdPUEphMEVqL2xkbmwweE9nYWhwaWNPcEVMTDBp?=
 =?utf-8?B?VTVwcnkwbG5zMG9zaXVNUVd6c0NKV3I2YjV4TVZlNU5zV0E4TWQvMTN2NWdw?=
 =?utf-8?B?bzA3WGFuRTRqMlozdzBUa3owNGMyeGxPOFFqeWN2MlVLaTNwZFZLdmNwZUNv?=
 =?utf-8?B?YnNnK1dQOGhpdmJnTkJ1QjRnNmNqRzk3U21PeUxnc0VEVjBBWkhodkljSnl1?=
 =?utf-8?B?OHFpOXgyeDNsM29Lb1p3YUd3aDJ0SElVYnJONndsK1BUM3ljLy9TendoYlF6?=
 =?utf-8?B?bSt5SVNFNC9FbjdWK1FId1hLN3dTMWljcWNNNGh1T2wwUGorSkFQSEpXNVV6?=
 =?utf-8?B?RGpRZFNQNWlCTExPNEhUWmxKazFFOCtNUUhSWFZXTmhodmtRK0ZOVTZOYlJ3?=
 =?utf-8?B?ZnRNZXF6bVZrSHo2Uk9IVVpLYzJRa1ZYMUVSRWFWYkhldjU1NXcwU0ozQlBv?=
 =?utf-8?B?YzZQT29idEVZZFFUWXUzbzNMdjdTcXR2WXVPdURXdzJndGN6SHZHZGlEV1FQ?=
 =?utf-8?B?UkE3cS9WYWJpUXRmb1I0Sko2eUpNMWRmY3lSMmlHb3ZpMVNMNGZWVS9ZNFBv?=
 =?utf-8?B?Y05xY2FIbTcrUHFTbUQrRFduRkV3cE5uVlZaSWZaSnBxNzZ4VitsS0lhTGky?=
 =?utf-8?B?VXpvOGUyWk1nanVEMXdPdVFFcWh1dnFDcU0yWEgxRjdwYUQ5RmlJRkcxZFNz?=
 =?utf-8?B?R3ZQS3hCckE3Q1AwT2IwZ091RHo4akJBbUlHTkxMcXJNb1BSdkQ4MEpZaUN4?=
 =?utf-8?B?emxtekExc2kzeVE0cnEzSVZEeXpkandjcVNsTmhHZ2E4RVF2VGM5cURTSUhH?=
 =?utf-8?B?eGo0V3gzN2o3U1Q0d3hEa2RVWVRpUkJkUHlKT3czbmlDVmRWdVJ4aTBkdUZC?=
 =?utf-8?B?RFJHb0RtL2pCRzBvL1gzNFJ6MmtWSjJ5R0lPOG1GeTBmQzVxUy9Fd1BrVnJI?=
 =?utf-8?B?cXEyMDV1RVZBYTljVUNIZFlqNjc3ME9MZVR5NXFIUm01MVNrSXBSTTM1OHBF?=
 =?utf-8?B?YzdmdnhmS050SURlc0NOa1plZGwyb1l6RzZ5Q29EamNxOExheWpZNnZMSkRK?=
 =?utf-8?B?YnkzWTNsSjZ0Sk82MXBOYTk5WThieWdwc2lWcVVOL3B5VlNGOXJtR0tSVW5Z?=
 =?utf-8?B?cFVEblEyYTRic1RFbVhDK2ZzMHQ4VTRyVzYzU2VESjhyMnU4bzBnZ0JJNWtz?=
 =?utf-8?B?ZStjUTlEOE94NEtvWUhJdlFqZGxEMThETk5YbDRxZUVpZHFRRlBUamFOcVRh?=
 =?utf-8?B?N2E4WW9HWHNrd3AvKzZhb1U1T25JeHhiWFZBMnk0NGF0bWl2ejE3bWRDT3Nm?=
 =?utf-8?B?eWdMK2VOcCthdlFlaWh4UGUrZCtzcVd3VkhmUmxXNkdPbVFZSUNGZ2pHZGpv?=
 =?utf-8?B?aHo2WllLMTBzWS9GOHQ1Q3B3VzJkTkhLUjRJb0J0Vy85WXN6cForelRqQngx?=
 =?utf-8?B?bENsNlU0R05iMHZHNC83anQwMXBDQ01HYTlPaUlPanFJYlZlSUE5YVhvdkpq?=
 =?utf-8?B?MUFPSHFvMGdJL3E2bFEybXdCRUVhNTdmMTBMOExFSWhPd3o1Q1VRUDg0VFlh?=
 =?utf-8?B?dGxZSCtUMUhWSUdQaVhqRmRDNDkvM2VWbVB2TGdVOG1MUDBWWFpsQnhYeVZn?=
 =?utf-8?B?SHJJT2FqS29HUitYWjg0ajVRaW9qejNBODZrUzVrZFJVYW40anh4UTRXM005?=
 =?utf-8?B?WEFTcHp1ZjFOeEtxUDFxUFByak5VNlIyanRUbWFGMHNlTHNBbjJvQkJUT0tv?=
 =?utf-8?B?SnFGY3M4dkpDT04rM2RKY0IwN2ZIWkI4VU5lNmQ2M1JHV2RRblAvUE8zTjM4?=
 =?utf-8?B?OFljcUtqWmg1RDZmU0RyZSt1cTU5Wm9ENTl1SWNKdE00OVp0RVZucGttMGNk?=
 =?utf-8?B?OUtXbUZhYnVOYUxyaUdLM1VicUxySlR2TlNBcnkxYkZEWnc3ZUswcmdOcXFJ?=
 =?utf-8?B?ejFGZDB6YzZodUJySWtPUzdGa0hzSVhkMlZWZnlyM3dRU1BlWkErZGFDSmVW?=
 =?utf-8?B?V0d1cmcxbXpweXFCNlEyTmtOVFZGOWY5NWQxOXh4SGhteFFmRHNDMmJLL1NG?=
 =?utf-8?B?ZFV4OGFZNGt0VkttV3NOVnBMNytkbXNick5HM1BmZ1B1aE5SK2p6aG0vV0gr?=
 =?utf-8?Q?b0xcwN9G1pzXA915Z9s6xwAvd?=
X-OriginatorOrg: axentia.se
X-MS-Exchange-CrossTenant-Network-Message-Id: 2c71b344-6f10-4607-3003-08dc216d0214
X-MS-Exchange-CrossTenant-AuthSource: DU0PR02MB8500.eurprd02.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 Jan 2024 08:25:25.1781
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4ee68585-03e1-4785-942a-df9c1871a234
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: eqJVlZjfmz6iQUQ/qKs1u/d22La4q9Mh0Zjb/23Vwct2dVeZz5S1jSDF4fUq0h6s
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR02MB5936

Hi!

2024-01-26 at 15:36, Thomas Richard wrote:
> The mux_chip_resume() function restores a mux_chip using the cached state
> of each mux.
> 
> Signed-off-by: Thomas Richard <thomas.richard@bootlin.com>
> ---
>  drivers/mux/core.c         | 27 +++++++++++++++++++++++++++
>  include/linux/mux/driver.h |  1 +
>  2 files changed, 28 insertions(+)
> 
> diff --git a/drivers/mux/core.c b/drivers/mux/core.c
> index 775816112932..896f74b34eb8 100644
> --- a/drivers/mux/core.c
> +++ b/drivers/mux/core.c
> @@ -215,6 +215,33 @@ void mux_chip_free(struct mux_chip *mux_chip)
>  }
>  EXPORT_SYMBOL_GPL(mux_chip_free);
>  
> +/**
> + * mux_chip_resume() - restores the mux-chip state
> + * @mux_chip: The mux-chip to resume.
> + *
> + * Restores the mux-chip state.
> + *
> + * Return: Zero on success or a negative errno on error.
> + */
> +int mux_chip_resume(struct mux_chip *mux_chip)
> +{
> +	int ret, i;
> +
> +	for (i = 0; i < mux_chip->controllers; ++i) {
> +		struct mux_control *mux = &mux_chip->mux[i];
> +
> +		if (mux->cached_state != MUX_CACHE_UNKNOWN) {
> +			ret = mux_control_set(mux, mux->cached_state);
> +			if (ret < 0) {
> +				dev_err(&mux_chip->dev, "unable to restore state\n");
> +				return ret;

I'm don't know what is expected of the core resume code on error,
but is it ok to return on first failure? Is it not better to try
to restore all muxes and return zero if all is well or the first
failure when something is up?

But maybe the resume is completely dead anyway if there is any
failure? In that case the above early return is fine, I guess...

Cheers,
Peter

> +			}
> +		}
> +	}
> +	return 0;
> +}
> +EXPORT_SYMBOL_GPL(mux_chip_resume);
> +
>  static void devm_mux_chip_release(struct device *dev, void *res)
>  {
>  	struct mux_chip *mux_chip = *(struct mux_chip **)res;
> diff --git a/include/linux/mux/driver.h b/include/linux/mux/driver.h
> index 18824064f8c0..2a7e5ec5d540 100644
> --- a/include/linux/mux/driver.h
> +++ b/include/linux/mux/driver.h
> @@ -88,6 +88,7 @@ struct mux_chip *mux_chip_alloc(struct device *dev,
>  int mux_chip_register(struct mux_chip *mux_chip);
>  void mux_chip_unregister(struct mux_chip *mux_chip);
>  void mux_chip_free(struct mux_chip *mux_chip);
> +int mux_chip_resume(struct mux_chip *mux_chip);
>  
>  struct mux_chip *devm_mux_chip_alloc(struct device *dev,
>  				     unsigned int controllers,
>