Received-SPF: pass (google.com: domain of linux-kernel+bounces-127567-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Content-Type: multipart/signed;
 boundary=da0864b69030f7d19a4d86df6475a90ba83b3b0f6fe192d174a168ba9ad7;
 micalg=pgp-sha384; protocol="application/pgp-signature"
Date: Tue, 02 Apr 2024 10:37:24 +0200
Message-Id: <D09I0YEBGBV8.3SOV5AYONO5ZL@kernel.org>
Subject: Re: [PATCH 1/3] nvmem: layouts: store owner from modules with
 nvmem_layout_driver_register()
Cc: <linux-kernel@vger.kernel.org>
From: "Michael Walle" <mwalle@kernel.org>
To: "Krzysztof Kozlowski" <krzysztof.kozlowski@linaro.org>, "Srinivas
 Kandagatla" <srinivas.kandagatla@linaro.org>, "Miquel Raynal"
 <miquel.raynal@bootlin.com>
References: <20240329-module-owner-nvmem-v1-0-d700685f08e1@linaro.org>
 <20240329-module-owner-nvmem-v1-1-d700685f08e1@linaro.org>
In-Reply-To: <20240329-module-owner-nvmem-v1-1-d700685f08e1@linaro.org>
Precedence: bulk

--da0864b69030f7d19a4d86df6475a90ba83b3b0f6fe192d174a168ba9ad7
Mime-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset=UTF-8

On Fri Mar 29, 2024 at 5:13 PM CET, Krzysztof Kozlowski wrote:
> Modules registering driver with nvmem_layout_driver_register() might
> forget to set .owner field.  The field is used by some of other kernel
> parts for reference counting (try_module_get()), so it is expected that
> drivers will set it.
>
> Solve the problem by moving this task away from the drivers to the core
> code, just like we did for platform_driver in
> commit 9447057eaff8 ("platform_device: use a macro instead of
> platform_driver_register").
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Reviewed-by: Michael Walle <mwalle@kernel.org>

-michael

--da0864b69030f7d19a4d86df6475a90ba83b3b0f6fe192d174a168ba9ad7
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iKgEABMJADAWIQTIVZIcOo5wfU/AngkSJzzuPgIf+AUCZgvDxBIcbXdhbGxlQGtl
cm5lbC5vcmcACgkQEic87j4CH/iyWwGApFLJ3/8u7BW1Nu9VLng/g6AXHPywIVXs
y58D+lf5bN/DnIsCHk88i8axaeAc3XiEAX4gazxHDDv8Gg01blW6Jq6aE5pHrSeG
lZpOUdqiPEEOo6WFxWDOn+/oh96hIDPcz9c=
=avti
-----END PGP SIGNATURE-----

--da0864b69030f7d19a4d86df6475a90ba83b3b0f6fe192d174a168ba9ad7--