Received-SPF: pass (google.com: domain of linux-kernel+bounces-200265-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Message-ID:
 <SY4P282MB3063FE1968B72C3187042917C5F82@SY4P282MB3063.AUSP282.PROD.OUTLOOK.COM>
Date: Tue, 4 Jun 2024 18:45:32 +1000
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v4 3/6] hwmon: (spd5118) Add suspend/resume support
To: Guenter Roeck <linux@roeck-us.net>, linux-hwmon@vger.kernel.org
Cc: linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org,
 devicetree@vger.kernel.org, Krzysztof Kozlowski <krzk+dt@kernel.org>,
 Wolfram Sang <wsa+renesas@sang-engineering.com>,
 =?UTF-8?Q?Ren=C3=A9_Rebe?= <rene@exactcode.de>,
 =?UTF-8?Q?Thomas_Wei=C3=9Fschuh?= <linux@weissschuh.net>,
 Armin Wolf <W_Armin@gmx.de>
References: <20240604040237.1064024-1-linux@roeck-us.net>
 <20240604040237.1064024-4-linux@roeck-us.net>
Content-Language: en-AU, en-US, en-GB
From: Stephen Horvath <s.horvath@outlook.com.au>
In-Reply-To: <20240604040237.1064024-4-linux@roeck-us.net>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
X-MS-PublicTrafficType: Email
X-MS-TrafficTypeDiagnostic: SY4P282MB3063:EE_|SY7P282MB4571:EE_
X-MS-Office365-Filtering-Correlation-Id: 8ab923d1-310d-41fb-e2e1-08dc8472b50c
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info:
	YMd5odYv00y4+ciCzzApd19o2BY13azjxdN2miKZso8/nR1DcUK3JFJzF8DXb/7WRZUoaqgJwRs6bMxZqwKbROPbewFvucLA34aatdDQgg0OkDQYmRRsvcchCjxW4VK2XmVdWtjAB6Lp3d5TbbF3wn8TE0S76lEcp79YmHlGWxeq99Xv9yeOtBLtB2lJjp2bd3O5T4qeXzbNjPtUqPMOZsKlOosVvZYZljtuf0702k7G5p6ST6Sh6RRffw+KvqZIWWxIqixD/mzvxzABBojreNYB92uTDH4Ocwe3DHfHs8gfC90Tdpj4OjJHepu9WDKvWDQExmMBamJm2HCC98Ubadyu/amlnivBR1AMK0V8s+oWry5ZVoJr/kuL6R9pUlpTYNFAyH0ZwEUQB+78B4TT+0uivbq2Dt5l/0vCN9jAtFbUyjR7XHXZ4VauwH9SfX5PEgpQO0D0eMAzZHcxSyOHk28AKVn6k4T3ifo/X2Z4DckmgqgPKA84t2JUvqXOAkhrscLhPuc7jmIBg6kAUIwRYYNKEMmdTyAl7sZHkoydqPO8wpdxaato10NHb6O6qfw0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?M3g5U1NpNk5GeG5QTXpaZElzZEphQlVZNEN5QUVxZWlVV0RsRmRBcWF4Vnh4?=
 =?utf-8?B?bjd3LzJxWmR4Y1dYUlJvUy83M2VrWHd5VmlaTGkzcU1mR2ZOSytQd3YraVYx?=
 =?utf-8?B?K25mMzloc2JLdDBESmRjelZNdkhhVHRKcG0wRnQ5Z2Z1MGhJTlMyRmxXYnN6?=
 =?utf-8?B?dmlZcjRWVlFXUDlET01YSEJ5RU9IamthOVFHcVBQODRpVE0vSzFEK3hZbTZW?=
 =?utf-8?B?RjZhVHN4T2lFVFY4Z2F5SXJJNGwzMW81NURkeXgwYjlad200RW51bjB1U1pq?=
 =?utf-8?B?Njk2Y3pnRVZtOEFRRzF0SVlvb2F4cnZQNGlKT21tdDNDYUEwbEJwelJZT2Fs?=
 =?utf-8?B?RmlGQTlOVDhYM2I3RlpMQzZFQy9kRFJlQUhDbWVZOGJBcWVFWDlwYzNoLzMr?=
 =?utf-8?B?MUdmczJqNEkwY1R6TUwxREdyTy9KM2hvSEFsRWY1cXU3QmdBOFlyZytZTW52?=
 =?utf-8?B?OTREb1RrejUwZjVSeFdjeEh4OWEzeVNkbzRWQW1OdnhjSFRUY2lUNEpkTVM3?=
 =?utf-8?B?MGJMT2xLUUlVQmF2Z3RYbTQvb2k5UDFyMnU0ajJKc09hVVBUeW9GMUxTbC9U?=
 =?utf-8?B?S0tCNnpSWnZGWms2TE1reHpuc1Zyck1CMHM0aEsrVUNPdHNYbWpoVlBGY0JS?=
 =?utf-8?B?c2VTVWZCQlR6cnppMHRxWFZSdjh0dzB2NTlBQmpNeEF0YkJQcmQyUktsN1Ju?=
 =?utf-8?B?dlE3OGtLbGdsT0s0Tm11QWV1aW1QVHhsV2FhSHdyQVlSTCtXM250bFN3MUcy?=
 =?utf-8?B?YmFxYkxRby9Ua2xNemhXWEFsSXZveHl0aXhyNUgwd2hqN2VTdWMyNUs4WXVP?=
 =?utf-8?B?cFlzeVozK0syd2dFdVJpQ2lxUVp4VnZzbjBLQVV4YVBPd3JDajdtbG9IWUZm?=
 =?utf-8?B?TVBLYXJ3MDNoZTQ5clF5WThZdHBmb095eENrazhUOTNwdWpWdEJDYlVQV0VW?=
 =?utf-8?B?QVJaTG80VVZNNWFaRDhLSmdzT2U2SVBOTktxd2RZbWZyZnVaNGRqUFdaREVI?=
 =?utf-8?B?RTc5N2dIK0FNdExsWTdTaFJCd3A4ZHJuR0VwWlZVdThWUmxnOUF4VjgxMytQ?=
 =?utf-8?B?Y050bHd0NUhCd05GTmFaekNpOThON29QZE9IQmtNSXZhbWljQXhGRUkzVFF0?=
 =?utf-8?B?OVNjS0g4WFNIWEN4T2FBdk9XVjhsK3ZCcVRNM3NyemhLdzF6Z25IR3d6dllx?=
 =?utf-8?B?Sjd3S2NhcFdZdlJZdVptOUJEb25nOHJqWHd5ZElXS0tmc1llM2VnNEhkT1c0?=
 =?utf-8?B?cUdlVnVOVVN6NEphNHJSMzVJdGtZc2t2YTJtb2RFKzc0cDhKOG1xdnZoZ0tX?=
 =?utf-8?B?Wkp6L0ViZGNDTGNKeUhtZmgvT08vZjB1R2RseXNLQnJKQnp6bDVrclNQVFJl?=
 =?utf-8?B?YUpzbEN4dlBmZ0ZOckZrSUpGNzJTSFNrN1Z5RksyUWdNeXZ4b0xrMmwwZGNI?=
 =?utf-8?B?c1FVNkNrbW5xUG5zS1JYZU8xajg2VVA1K1F5MDZvWUR1Qkx4bFl0bVpKVnJw?=
 =?utf-8?B?UWVucDVZQW1jbnd1YWpPbGwydEFxR0paT2FGenFkY1NEdDE2Wm1neU95RjYy?=
 =?utf-8?B?cnNEQWxCd2NoSFFMSWlWcFZLc0VIS09nR2ZFdFdRelNYQ0xFK0hqS3BuYXpU?=
 =?utf-8?B?Q0VPc0dhVEJSaUxGLzJBVnF6VTlBRTJvZjNoVUQ2ajZrT2h2a21aRVc4c0tY?=
 =?utf-8?B?REw0NGlVZVZtWHhHbEYwa2hyb1ZOWGJVN2EwTlBadmVLZ0lPT1BBclZub0p0?=
 =?utf-8?Q?m3wbFFAWFdB84XHxX6Muy4c1EXFglzk/nUPvqyB?=
X-OriginatorOrg: sct-15-20-4755-11-msonline-outlook-746f3.templateTenant
X-MS-Exchange-CrossTenant-Network-Message-Id: 8ab923d1-310d-41fb-e2e1-08dc8472b50c
X-MS-Exchange-CrossTenant-AuthSource: SY4P282MB3063.AUSP282.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Jun 2024 08:45:40.2921
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa
X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg:
	00000000-0000-0000-0000-000000000000
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SY7P282MB4571

Hi,

On 4/6/24 14:02, Guenter Roeck wrote:
> Add suspend/resume support to ensure that limit and configuration
> registers are updated and synchronized after a suspend/resume cycle.
> 
> Cc: Armin Wolf <W_Armin@gmx.de>
> Cc: Stephen Horvath <s.horvath@outlook.com.au>
> Signed-off-by: Guenter Roeck <linux@roeck-us.net>
> ---
> v4: Fix bug seen if the enable attribute was never read prior
>      to a suspend/resume cycle.

I can confirm this works for my devices, so:

Tested-by: Stephen Horvath <s.horvath@outlook.com.au>

Thanks,
Steve