[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <mknxgesog6aghc6cjzm63g63zqbqvysxf6ktmnbrbtafervveg@uoiohk3yclso>
Date: Tue, 16 Sep 2025 17:11:08 +0200
From: Uwe Kleine-König <u.kleine-koenig@...libre.com>
To: Andy Shevchenko <andriy.shevchenko@...el.com>
Cc: AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>,
sboyd@...nel.org, jic23@...nel.org, dlechner@...libre.com, nuno.sa@...log.com,
andy@...nel.org, arnd@...db.de, gregkh@...uxfoundation.org, srini@...nel.org,
vkoul@...nel.org, kishon@...nel.org, sre@...nel.org,
krzysztof.kozlowski@...aro.org, linux-arm-msm@...r.kernel.org, linux-iio@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-phy@...ts.infradead.org, linux-pm@...r.kernel.org,
kernel@...labora.com, wenst@...omium.org, casey.connolly@...aro.org,
Konrad Dybcio <konrad.dybcio@....qualcomm.com>, Neil Armstrong <neil.armstrong@...aro.org>
Subject: Re: [PATCH v4 2/7] nvmem: qcom-spmi-sdam: Migrate to
devm_spmi_subdevice_alloc_and_add()
On Tue, Sep 16, 2025 at 04:35:35PM +0300, Andy Shevchenko wrote:
> On Tue, Sep 16, 2025 at 03:24:56PM +0200, Uwe Kleine-König wrote:
> > On Tue, Sep 16, 2025 at 10:44:40AM +0200, AngeloGioacchino Del Regno wrote:
>
> ...
>
> > > +MODULE_IMPORT_NS("SPMI");
> >
> > If it's exactly the files that #include <linux/spmi.h> should have that
> > namespace import, you can put the MODULE_IMPORT_NS into that header.
>
> Which makes anyone to import namespace even if they just want to use some types
> out of the header.
Notice that I carefully formulated my suggestion to cope for this case.
> This is not good solution generally speaking. Also this will
> diminish one of the purposes of _NS variants of MODULE*/EXPORT*, i.e. make it
> invisible that some of the code may become an abuser of the API just by someone
> include the header (for a reason or by a mistake).
Yeah, opinions differ. In my eyes it's quite elegant.
Best regards
Uwe
Download attachment "signature.asc" of type "application/pgp-signature" (489 bytes)
Powered by blists - more mailing lists