Date:   Mon, 20 Feb 2023 10:22:25 +0200
From:   Ido Schimmel <idosch@nvidia.com>
To:     Daniel Lezcano <daniel.lezcano@linaro.org>
Cc:     rafael@kernel.org, linux-pm@vger.kernel.org,
        linux-kernel@vger.kernel.org, Zhang Rui <rui.zhang@intel.com>,
        Len Brown <lenb@kernel.org>, Petr Machata <petrm@nvidia.com>,
        "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>,
        Amit Kucheria <amitk@kernel.org>,
        Eduardo Valentin <edubezval@gmail.com>,
        Keerthy <j-keerthy@ti.com>,
        Matthias Brugger <matthias.bgg@gmail.com>,
        AngeloGioacchino Del Regno 
        <angelogioacchino.delregno@collabora.com>,
        Balsam CHIHI <bchihi@baylibre.com>,
        "open list:ACPI THERMAL DRIVER" <linux-acpi@vger.kernel.org>,
        "open list:MELLANOX ETHERNET SWITCH DRIVERS" <netdev@vger.kernel.org>,
        "open list:TI BANDGAP AND THERMAL DRIVER" 
        <linux-omap@vger.kernel.org>,
        "moderated list:ARM/Mediatek SoC support" 
        <linux-arm-kernel@lists.infradead.org>,
        "moderated list:ARM/Mediatek SoC support" 
        <linux-mediatek@lists.infradead.org>
Subject: Re: [PATCH v1 10/17] thermal: Do not access 'type' field, use the tz
 id instead
Message-ID: <Y/MtwTIuVDrt8PaP@shredder>
References: <20230219143657.241542-1-daniel.lezcano@linaro.org>
 <20230219143657.241542-11-daniel.lezcano@linaro.org>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20230219143657.241542-11-daniel.lezcano@linaro.org>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?brvGlaw9nvCsgfmPzQqdGW20NRoT03nuITeV0eGYV2UxcDALvFktdJX22eJh?=
 =?us-ascii?Q?8WYl3CXZP+0tSVe7VOx4plqwXLdUd7DKOwPf+zoJNZVOc9QIUjDCbziAX/u+?=
 =?us-ascii?Q?9kAoBXkTI7/9c87/0vfgTxy06u/QqqSAHousylPWwwgUYpadw1+r6EJwmJ8N?=
 =?us-ascii?Q?CIeIXNvnV+sGdka4LpMyYJIgaEDEPvbRDl1X0Lb/BcsM/u/yThpLl11XA3Me?=
 =?us-ascii?Q?36itsX2tA68Hxyg8BA9gsrgIlMzXJ4H1QxSsRj/oGHoTHqNy69ORa/azJbNC?=
 =?us-ascii?Q?Ya+S5aoovDMyqfPwM/yQoZg6s2H8e7GbWiD1IjnzrkczYPqyRR+w1abeVSWB?=
 =?us-ascii?Q?J9iD3nHvGwCU9Jsud18/5YL+bPF71hbao9BPf4b4OOgS+jXOdGG3IvsgMm2I?=
 =?us-ascii?Q?Tua5j3zZcMJ94SX48DSG6BgJ+i+3hnVYEL/tbqLiZDho6mGvS5xJI+TfkKjH?=
 =?us-ascii?Q?j+d8cQRKPshLQ25HBchv97M3E7jt/rOEjhRxNqDAr+akk+6y5DYU0a0lCd3E?=
 =?us-ascii?Q?+mWtSixyA/4hQbbu6N9/AI8mDdXVwf/S5XwBzj3yDcJbp0FNtia3ZrWwjqVB?=
 =?us-ascii?Q?8AGrwRfgpYyVRicPknezjL0H13iJycPn9qIe0fI4v5DyQd3aOs0V9vY3Isvr?=
 =?us-ascii?Q?0QI4Lj7rSy6J/C6tQMrUcuzlOuKOW+Oeqj4CBtIeTOOzd7Pc2k3reD3+deBz?=
 =?us-ascii?Q?bCDUbRgL0RP8CgAGmh2W37iRI9NS5CRKdX7eAzEPuBjVkY5N2ErdA4YkBIsX?=
 =?us-ascii?Q?wt7PcqbZAYXUKJtcH8LWfoivC0dmp8unzQx1IcHxsJQ23Cy+5/mQ+8Dra/fs?=
 =?us-ascii?Q?J/5BHohDY/XnN4Emg+frTqAzN9/msQtrkWo74zDIGm+YBiMeVzqloF3KVWpE?=
 =?us-ascii?Q?WHXSCEiKolcSl3GK8pA4z4OysJa1+quvUUE2gvxqj/PjMwsvwqcO0Hz3G72U?=
 =?us-ascii?Q?v47oiNBg9a5gGE8RXR5z8SrNpHfTjfxkplN7O4dcEZ9C3yMnEL3+bBIs2nwE?=
 =?us-ascii?Q?1KcPzgYkSq/QH0k1AnrlTObttYjLBPMLyrs7oX1k0EjqhdGGCxSdUWOdLoVc?=
 =?us-ascii?Q?aUF7LzwTQogxmxQ/pPIcAkkcHjbsuS1l7m5ibM1iY2WLJ9I1YQrMUsqTDTty?=
 =?us-ascii?Q?tNOsXJCOXVkvxpiMWbaGWskReGShegz63aEqIQKK+QGP8HWr+K1Aw+ky2viS?=
 =?us-ascii?Q?4sCVPXwJfeVO+bbrtX35El0PtbDiGpc6bBwQcEb4KuciDdDhEazWnx1fh5hi?=
 =?us-ascii?Q?bB8QIyxPJKP5TcAegec77TByyuzv9vDaICbJ8BNldE+1QDm6jw74nKThvWJg?=
 =?us-ascii?Q?LQRHMOpLhs3WakJJ9BIrYRpuqmMu4TKEX/jPKeCejU6rHNdnYWpyxk1ektEJ?=
 =?us-ascii?Q?b4LsP/NdJMpa1ONL1pZiGnJSOoldvwRdy9/Ij2fTkMS/MI1LcT5KrtNPaRUo?=
 =?us-ascii?Q?4n/B5q+sO3OTNM0xIfDQ+p4v1TF5BzaK0kc2PqiithZMpGbg3Jr0XmfJxVz4?=
 =?us-ascii?Q?S9M5czDhP+b41AKli1yoUu21vvTz7TpFW2kGD3spBj76TbtHr6SHW8UhZJHi?=
 =?us-ascii?Q?MLtYmI7hSi2dbF7e/Unck2i7OtOU9pQVWj6t3PvM?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 88fdae44-0087-4777-b125-08db131b9be2
X-MS-Exchange-CrossTenant-AuthSource: CY5PR12MB6179.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Feb 2023 08:22:30.5313
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: pssYh0A1vBG/2EBCb8vFtsq54jlid6iQ8akX9knHzVmCCf+RYba9MrCn17NmTxAWKL6WH0BSIlP98c8Xj5ffJQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR12MB5040
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Sun, Feb 19, 2023 at 03:36:50PM +0100, Daniel Lezcano wrote:
> The 'type' field is used as a name in the message. However we can have
> multiple thermal zone with the same type. The information is not
> accurate.
> 
> Moreover, the thermal zone device structure is directly accessed while
> we want to improve the self-encapsulation of the code.
> 
> Replace the 'type' in the message by the thermal zone id.
> 
> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>

For mlxsw:

Reviewed-by: Ido Schimmel <idosch@nvidia.com>