diff mbox series

[1/2] mfd: pm8008: fix module autoloading

Message ID 20230526091646.17318-2-johan+linaro@kernel.org
State New
Headers
Series mfd: pm8008: fix module autoloading |

Commit Message

Johan Hovold May 26, 2023, 9:16 a.m. UTC 
  Add the missing module device table alias to that the driver can be
autoloaded when built as a module.

Fixes: 6b149f3310a4 ("mfd: pm8008: Add driver for QCOM PM8008 PMIC")
Cc: stable@vger.kernel.org      # 5.14
Signed-off-by: Johan Hovold <johan+linaro@kernel.org>
---
 drivers/mfd/qcom-pm8008.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Johan Hovold May 26, 2023, 9:25 a.m. UTC | #1 
On Fri, May 26, 2023 at 11:22:27AM +0200, Konrad Dybcio wrote:
> On 26.05.2023 11:16, Johan Hovold wrote:
> > Add the missing module device table alias to that the driver can be
> > autoloaded when built as a module.
> > 
> > Fixes: 6b149f3310a4 ("mfd: pm8008: Add driver for QCOM PM8008 PMIC")
> > Cc: stable@vger.kernel.org      # 5.14

> Looks like the commit referenced in Fixes is from 5.13-rc1, perhaps
> 5.13 would be more fitting here?

No, I just double checked. This driver was not present 5.13.

Johan
Lee Jones June 8, 2023, 3:45 p.m. UTC | #2 
On Fri, 26 May 2023, Johan Hovold wrote:

> Add the missing module device table alias to that the driver can be
> autoloaded when built as a module.
> 
> Fixes: 6b149f3310a4 ("mfd: pm8008: Add driver for QCOM PM8008 PMIC")
> Cc: stable@vger.kernel.org      # 5.14
> Signed-off-by: Johan Hovold <johan+linaro@kernel.org>
> ---
>  drivers/mfd/qcom-pm8008.c | 1 +
>  1 file changed, 1 insertion(+)

Applied, thanks
Johan Hovold June 9, 2023, 6:21 a.m. UTC | #3 
On Thu, Jun 08, 2023 at 04:45:03PM +0100, Lee Jones wrote:
> On Fri, 26 May 2023, Johan Hovold wrote:
> 
> > Add the missing module device table alias to that the driver can be
> > autoloaded when built as a module.
> > 
> > Fixes: 6b149f3310a4 ("mfd: pm8008: Add driver for QCOM PM8008 PMIC")
> > Cc: stable@vger.kernel.org      # 5.14
> > Signed-off-by: Johan Hovold <johan+linaro@kernel.org>
> > ---
> >  drivers/mfd/qcom-pm8008.c | 1 +
> >  1 file changed, 1 insertion(+)
> 
> Applied, thanks

Thanks, Lee. Did you forget to push these out? I was gonna see if you
merged them for 6.4 or 6.5, but I can't seem to find them in your mfd
repo.

Johan
Lee Jones June 9, 2023, 6:38 a.m. UTC | #4 
On Fri, 09 Jun 2023, Johan Hovold wrote:

> On Thu, Jun 08, 2023 at 04:45:03PM +0100, Lee Jones wrote:
> > On Fri, 26 May 2023, Johan Hovold wrote:
> > 
> > > Add the missing module device table alias to that the driver can be
> > > autoloaded when built as a module.
> > > 
> > > Fixes: 6b149f3310a4 ("mfd: pm8008: Add driver for QCOM PM8008 PMIC")
> > > Cc: stable@vger.kernel.org      # 5.14
> > > Signed-off-by: Johan Hovold <johan+linaro@kernel.org>
> > > ---
> > >  drivers/mfd/qcom-pm8008.c | 1 +
> > >  1 file changed, 1 insertion(+)
> > 
> > Applied, thanks
> 
> Thanks, Lee. Did you forget to push these out? I was gonna see if you
> merged them for 6.4 or 6.5, but I can't seem to find them in your mfd
> repo.

I don't tend to push daily.  I can push them now if you like.

These were added for v6.5.
Johan Hovold June 9, 2023, 6:41 a.m. UTC | #5 
On Fri, Jun 09, 2023 at 07:38:04AM +0100, Lee Jones wrote:
> On Fri, 09 Jun 2023, Johan Hovold wrote:

> > Thanks, Lee. Did you forget to push these out? I was gonna see if you
> > merged them for 6.4 or 6.5, but I can't seem to find them in your mfd
> > repo.
> 
> I don't tend to push daily.  I can push them now if you like.
> 
> These were added for v6.5.

Perfect, thanks!

Johan
diff mbox series

Patch

diff --git a/drivers/mfd/qcom-pm8008.c b/drivers/mfd/qcom-pm8008.c
index 837246aab4ac..29ec3901564b 100644
--- a/drivers/mfd/qcom-pm8008.c
+++ b/drivers/mfd/qcom-pm8008.c
@@ -199,6 +199,7 @@  static const struct of_device_id pm8008_match[] = {
 	{ .compatible = "qcom,pm8008", },
 	{ },
 };
+MODULE_DEVICE_TABLE(of, pm8008_match);
 
 static struct i2c_driver pm8008_mfd_driver = {
 	.driver = {