Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <8a8dc05b-f173-bea8-a59a-137b95007be6@intel.com>
Date:   Wed, 31 May 2023 10:20:09 +0200
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.11.2
Subject: Re: [PATCH v1 5/5] ACPI: thermal: Eliminate struct
 acpi_thermal_state_flags
Content-Language: en-US
To:     "Rafael J. Wysocki" <rjw@rjwysocki.net>,
        Linux ACPI <linux-acpi@vger.kernel.org>
CC:     LKML <linux-kernel@vger.kernel.org>,
        Zhang Rui <rui.zhang@intel.com>,
        Daniel Lezcano <daniel.lezcano@linaro.org>
References: <5675481.DvuYhMxLoT@kreacher> <3737693.kQq0lBPeGt@kreacher>
From:   "Wilczynski, Michal" <michal.wilczynski@intel.com>
In-Reply-To: <3737693.kQq0lBPeGt@kreacher>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?YTNRSERWWEpjanNSV3FSR3g4ZjhYUnhGcy9nZVpLbUZwMFozdHd2UHJyamRz?=
 =?utf-8?B?R0NYRG4rWE81bjJid0pmd25iTWtoSmZYWXZYdVBHWEU2VWRJZjNoRkZDOTR4?=
 =?utf-8?B?bjl3ck1CUXg5MXNqaEI2ZTl2M29pdEhwbGJYUDN5UjQ2UzNKS1lvZ1c5dkcx?=
 =?utf-8?B?bDFIdU1ZL0F4YTBBSEhtWkNBdnZQSmJuWU5PaTdYcGdjeEc0bTlWTUZxRkRv?=
 =?utf-8?B?bTZUQ3B0RDRlWEd4MEptc2RXWWhvZjgxOWZCWVFuSG11b29MZGJ3MVhnMisv?=
 =?utf-8?B?SHFUNWVqN2k4eHllandvMkdOTzVxekJib0l0c2VlK3JnYlNsNGl4c0ZZaWZl?=
 =?utf-8?B?ZnNJajB2QUdFellyakl1SUo5eldrUk5YRU9tSFZVblFaOEhxSlhJL251eTFk?=
 =?utf-8?B?ZEloN0dqUkU0eFNGek1mclNBamI3dC9kMU90U21yejNOR2xvZGpJdHlVdzZU?=
 =?utf-8?B?dVhlTVQvRjBQRUxiTkI3YXpIWFpMSDEwSWEwc3ZNTTR3SFZ0UzVjV1NPTHhT?=
 =?utf-8?B?TzV1R216Ykt4dkJDYkR2bUE5SEtFTEQ2QXJHbDBvVUdPVVNRZTRoVGduOWMy?=
 =?utf-8?B?N20zQkZrR2xYVlJCS1FYU09uc04zVFRhNWNvTkJCWmw3ejVXdmYwbHVxVEZL?=
 =?utf-8?B?UUxmSm5hL2JtNmQ3ZGlXTEJZNGE5emhmamdJc200cGtveXU4WWZRS1VtNzkz?=
 =?utf-8?B?S2VjcXVDUEl5ajR1MW93V0xHUWsveml6NDROam0wYzJBZG8rUERiejlnZHlE?=
 =?utf-8?B?dHpiVEpXaHhKQUJXcTVTdGowUlE5a0VEYkJxOXU1RklqZ3o0Nlg3ZENQeTdu?=
 =?utf-8?B?N290MVpWdUpBOENEZEhqTmpvRkk3L0MvU2RBUUpFeVBmMEJBWlF0R3g3R1Iy?=
 =?utf-8?B?SU02bmc5elZreWQ0YzlYdkt4bE9GTVdOUXBBVHN1K1ZGS3hranRQdW9zdGYw?=
 =?utf-8?B?V0FoOExYL3lTdHluK051cGRUZEhmNlRmQ2R6cTdiRlpIR3h3UVVzbHVLbTN2?=
 =?utf-8?B?MTQwSUVFU3BRajMrZUZNMUkwUWduQ0Mra3RQNmFtUWhhYU5rWmFralB1emZN?=
 =?utf-8?B?TWpBV3NrdEFEL1BwOHYrME1OVFNmdUE5U1JnRDB2TEtqSElDQnZvUTV3bXNX?=
 =?utf-8?B?U2Zkc0VlTkZPaDAyZEJVakd6WHFDb0ltOUZmU29DSHZnUVRPK0I2ZkFVY0lk?=
 =?utf-8?B?ekJwditGQ1J6dC9TSzA2b1lWNHkzYlJTSjhsbmx2VXZKMW5ma2NiL3BVT3lk?=
 =?utf-8?B?Tk4zYkdxbHNweENhMGYzZW1VQnJ1cWU3SGFrZCtaZEVRN1VNdkczVHdlNXNq?=
 =?utf-8?B?cTBOclpodHJWUXozVDYwbkpucUZYS0d6ZnNXSWo2d3ZNK09qTnYwdnRueXM2?=
 =?utf-8?B?NndFMWRLc1B2TkN2eDRjQ1FzQVdZUlBkNDEzM0JRWUNxOHBLMVZzdlpUVUly?=
 =?utf-8?B?UGFlckwxSzVZMVNMR0Fvd1dVUm1MVnZld1l4VkxBVFZzaEdNcmVqaFJFcGhR?=
 =?utf-8?B?WHkxazFlbW5Ia3E2ZDFuaktpK0Y3dmZtZjNsMkdadlA0bGExRjV2Lyt4VlQy?=
 =?utf-8?B?bkV6YzJLbDFlVGw4dzEvdlVoYThIVmNxZFFLMUxJL3M1VHNOQ3g4MG9QWUNo?=
 =?utf-8?B?LythUEx5UHo0K2wydUlKNnR6QjZLQmh6VThBWUhud1hBb0ZVdVc2NDVIRmdx?=
 =?utf-8?B?Y0F6MDFlODRmcUdaRHdIRW1PNEN3U0d4eDZrWVk0R1RFTlNVa0NTNHJ3ak1y?=
 =?utf-8?B?QmJzcWtkK3J5MU0vMklhUkY5aDZHSUJxZU9hVUVjWWhKc0RTamI5Y3c5dmZ4?=
 =?utf-8?B?UnRwVDk4WU40cGgzdHFTd01aU2xJVVR6aDdnZlRLU3N0aGRZSUs1RmNlOEth?=
 =?utf-8?B?K0I4NUZlTzZFOE56bWxGcHlyL0NpMTA5UDI1MHAzS2dVcUp3Y3psOHcwT3Rp?=
 =?utf-8?B?T2EzMWFIWFlkUHFDMURkZHlZeWp3SUJJSVB3SDh6Tkl2dlZwNmNmVkdURzJC?=
 =?utf-8?B?dVI0My8wVmV0QkJISDhsa0ZieDRBUnM2NTVBM0wrQ3BHVi9TdWhPZnJ1Wkk5?=
 =?utf-8?B?NVhia1MzRVhITXBWZW9wd2tQdHRKTVYxeHdxUUI5NCtGdUd0dXBvS1Zlb0U3?=
 =?utf-8?B?YlNneUVZYk5QWCtldXRtZmZxU08weEg1ZTZ2ZFRJWTVNRFlmdmR5b3dtMjhJ?=
 =?utf-8?B?dVE9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: d8f37e51-6fa2-4c23-9039-08db61afdca4
X-MS-Exchange-CrossTenant-AuthSource: CO6PR11MB5603.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 31 May 2023 08:20:15.5252
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: nKc5niuI0+FnYvvcLgzGN+3fYpXMETPZ2gpCL3oKV8Ykb9c7dFLzze0t8SWTVUma0+HirWcLEDy7r65S6tnqMSGniSZPMa7JMw7ZvsauofY=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN7PR11MB7041
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-4.7 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,
        RCVD_IN_DNSWL_MED,SPF_HELO_PASS,SPF_NONE,T_SCC_BODY_TEXT_LINE,
        URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 5/30/2023 5:45 PM, Rafael J. Wysocki wrote:
> |From: Rafael J. Wysocki <rafael.j.wysocki@intel.com> Notice that the enabled flag is only needed for active trip points, so drop struct acpi_thermal_state_flags, add a simple "bool valid" field to the definitions of all trip point structures instead of flags and add a "bool enabled" field to struct acpi_thermal_active. Adjust the code using the modified structures accordingly. No intentional functional impact. Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> --- drivers/acpi/thermal.c | 132 +++++++++++++++++++++++-------------------------- 1 file changed, 64 insertions(+), 68 deletions(-)|

Reviewed-by: Michal Wilczynski <michal.wilczynski@intel.com>


> ||