From:   Christophe Leroy <christophe.leroy@csgroup.eu>
To:     Benjamin Herrenschmidt <benh@kernel.crashing.org>,
        Paul Mackerras <paulus@samba.org>,
        Michael Ellerman <mpe@ellerman.id.au>
CC:     Christophe Leroy <christophe.leroy@csgroup.eu>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linuxppc-dev@lists.ozlabs.org" <linuxppc-dev@lists.ozlabs.org>,
        Erhard Furtner <erhard_f@mailbox.org>
Subject: [PATCH] powerpc/powermac: Add additional missing
 lockdep_register_key()
Thread-Topic: [PATCH] powerpc/powermac: Add additional missing
 lockdep_register_key()
Thread-Index: AQHX7Fow0VFka0nYfkWuyKwlY1DdIg==
Date:   Wed, 8 Dec 2021 17:36:52 +0000
Message-ID: <2c7e421874e21b2fb87813d768cf662f630c2ad4.1638984999.git.christophe.leroy@csgroup.eu>
Accept-Language: fr-FR, en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?iso-8859-1?Q?m7ft4xYMpYF1WfkJzx12fyuxwZCez8rSnr/FQvag9UWblYLkzgHbSfJmre?=
 =?iso-8859-1?Q?mvqBVNYT9bQaAviZxs72lq/fu67SWbGiIfsatUrBrUVm43rDXadaZY0cOS?=
 =?iso-8859-1?Q?tQPicbN8Lf6AiqG7x9vvWE2lA301Tq1X/p+NBe0w6lsf+ZQoPQjxqNeBIC?=
 =?iso-8859-1?Q?K3f/AiEDUHaQl+yfU52xXx33MfOd18Yz3nWEJYx03hGPZR/Cd0j6cBACcg?=
 =?iso-8859-1?Q?PodJ+DQGsS6/3qxFc7TX4jJqyA1Tn7fkQRhCY/aAUlptOzDeIWcLQ1ay/x?=
 =?iso-8859-1?Q?jWU1S1zvIq4DTRCl5YgoWtuUTDidIkwyegk7N7N9EbIWmFhiM4sOb2AzvX?=
 =?iso-8859-1?Q?0BcCBe0Gv4yhrOOJDsfbijRFXPoYbfrEnd8LxjdaaJf+/QpYBuGf3sZ0Fw?=
 =?iso-8859-1?Q?viCdx4db7mQzwYRmtkFm5NKVHBaAQZ/FXh3y5rGYm4uB0WRAnalyfmhoJn?=
 =?iso-8859-1?Q?ebbJxeZSurpzjwYITQyRBlFbA6r2y2+RasabWg3bFVUJ75r8fSd/hl1jHe?=
 =?iso-8859-1?Q?cqeYV7U0wLNcjSNNj34iCVkWr8rL2/vgv3XGPrPCqU6+MJLsxtGZcg/JM4?=
 =?iso-8859-1?Q?QYZdy1npUGNJ9/UwcoQHkcp4mxnIjAXXITeEA3uEOVleIcG42PcYq/SH5T?=
 =?iso-8859-1?Q?R/vNuLg7H6QYaP3MbYRKpnpQL4gQeKlwo4LuFYJi/Xs3G2FEgyaw6cYcLY?=
 =?iso-8859-1?Q?KPLQU23DBs+iZn0E4iECuVYM0P/YNT/Jr0TNw9Rshch0zNj98eNq/6uxqg?=
 =?iso-8859-1?Q?Dwbyi2v5PFh4OfrZpS8Kfo8O4vDp8aYsOU8eb7x3lHXUYrGK/J3a6kxzxd?=
 =?iso-8859-1?Q?TANFv3BLwzzeQxAzvRZ5nyEPCmmadYjRJUy9u3kQXCQLJuUvvjl/i3fw9q?=
 =?iso-8859-1?Q?az2B+7LLkWwW6oQCNUD2LSyhz4jat6SAsQPrUKU7/sdKbeIf48DZc6ywiw?=
 =?iso-8859-1?Q?SQPEy3gquhU5jGNrKVhtcOAlirNgmHsWFcl8oxVb+lC8zOt8ez8lMNP2al?=
 =?iso-8859-1?Q?1m/HtbD9OYvTd8tvyGrP6t9dZ4ELNkXB2lgctmoNj4sQqeToiRyZA03WIe?=
 =?iso-8859-1?Q?dxR7dcuOWgs3tEuBXQ4RR68r11MBUBgZarPuskSwzcrrAuPOONJV3alKIJ?=
 =?iso-8859-1?Q?zRBPaLtNNaNq1ibntofH8VUfFT3dmsZvDh50C1y0I7R02Y+M1x5ASdN66E?=
 =?iso-8859-1?Q?jEQ9dOv0ZbZ2aLR7RSCX0ixmFfqKma0yL6SLThbzOs81uFepMAnNBGzZH3?=
 =?iso-8859-1?Q?TUw33dYjP7RNurOcIVQMpGivYLjVkuxkSzgheS4OrEz3D6sBI9Ulwi2kRl?=
 =?iso-8859-1?Q?xiMFUo4I5ohvqfFxqhKa0TqIgKTabocvwmrNSCenSGZQxFej7z/UEOkWxM?=
 =?iso-8859-1?Q?vVGH8ABAIpwrDG4/8NaniPbxbhU0B8ETl/8VnIjm7iPKVXmLMx6pRDPl8E?=
 =?iso-8859-1?Q?Ohx0qLesgvPZ2FxsZKpZ5NW2/afxdJUm9jrkrUKbEceBnkryqfkxJ2sCnk?=
 =?iso-8859-1?Q?R4tVpKN97ZEZDaE3ZavdOWyqYHAvQzzb99BAi3fZhejpqHEo45Ht7Y1epN?=
 =?iso-8859-1?Q?v45ruRgQvE1wfJ7sdDvLdaE6EF2ysEMeQW5rGuINiXw44nkavM/xKSRboS?=
 =?iso-8859-1?Q?qeJLeV129i+8NFf7yzy0IHD3tBeKnEnOtH+xwf8PBVZvwdFYHXj1GAYznb?=
 =?iso-8859-1?Q?TBuTjF//wpjvMeSZJCtI7Qx/ZhOctMJUtx3ZFQHHnUSHIZOjfPCVvgVL4l?=
 =?iso-8859-1?Q?zq7T/FsjLlQmHHZfNAJ+DZsaA=3D?=
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: 8BIT
MIME-Version: 1.0
X-OriginatorOrg: csgroup.eu
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: MRZP264MB2988.FRAP264.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-Network-Message-Id: 2bf15e22-9bbb-4335-620a-08d9ba7152ad
X-MS-Exchange-CrossTenant-originalarrivaltime: 08 Dec 2021 17:36:52.9855
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 9914def7-b676-4fda-8815-5d49fb3b45c8
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: 4DjIUU2mo6HV6El4aUuBpHVgRBO1eEPwIAoNG/hruqTsK2B04b4FT1BHjY/6FU8pyc8ZRmqf7U4NodjjjXZx/BHmOy66i1XcNHGerSPS3mo=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MR2P264MB0419
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Commit df1f679d19ed ("powerpc/powermac: Add missing
lockdep_register_key()") fixed a problem that was causing a WARNING.

There are two other places in the same file with the same problem
originating from commit 9e607f72748d ("i2c_powermac: shut up lockdep
warning").

Add missing lockdep_register_key()

Reported-by: Erhard Furtner <erhard_f@mailbox.org>
Link: https://bugzilla.kernel.org/show_bug.cgi?id=200055
Fixes: 9e607f72748d ("i2c_powermac: shut up lockdep warning")
Depends-on: df1f679d19ed ("powerpc/powermac: Add missing lockdep_register_key()")
Signed-off-by: Christophe Leroy <christophe.leroy@csgroup.eu>
---
 arch/powerpc/platforms/powermac/low_i2c.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/powerpc/platforms/powermac/low_i2c.c b/arch/powerpc/platforms/powermac/low_i2c.c
index de34fa34c42d..df89d916236d 100644
--- a/arch/powerpc/platforms/powermac/low_i2c.c
+++ b/arch/powerpc/platforms/powermac/low_i2c.c
@@ -811,6 +811,7 @@ static void __init pmu_i2c_probe(void)
 		bus->hostdata = bus + 1;
 		bus->xfer = pmu_i2c_xfer;
 		mutex_init(&bus->mutex);
+		lockdep_register_key(&bus->lock_key);
 		lockdep_set_class(&bus->mutex, &bus->lock_key);
 		bus->flags = pmac_i2c_multibus;
 		list_add(&bus->link, &pmac_i2c_busses);
@@ -934,6 +935,7 @@ static void __init smu_i2c_probe(void)
 		bus->hostdata = bus + 1;
 		bus->xfer = smu_i2c_xfer;
 		mutex_init(&bus->mutex);
+		lockdep_register_key(&bus->lock_key);
 		lockdep_set_class(&bus->mutex, &bus->lock_key);
 		bus->flags = 0;
 		list_add(&bus->link, &pmac_i2c_busses);
-- 
2.33.1