| Message ID | 20240614151233.398463-1-biju.das.jz@bp.renesas.com |
|---|---|
| State | Accepted |
| Headers | show |
| Series | pwm: Fix namespace error | expand |
On Fri, Jun 14, 2024 at 04:12:33PM +0100, Biju Das wrote: > Fix namespace error due to typo by replacing USB_STORAGE->PWM. > > Fixes: aeb9cedc87a2 ("pwm: Make use of a symbol namespace for the core") > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com> I found that issue locally, too, but obviously failed to squash this change into the commit. How embarrassing. Thanks for catching, Uwe
Hi Uwe, > -----Original Message----- > From: Uwe Kleine-König <u.kleine-koenig@baylibre.com> > Subject: Re: [PATCH] pwm: Fix namespace error > > On Fri, Jun 14, 2024 at 04:12:33PM +0100, Biju Das wrote: > > Fix namespace error due to typo by replacing USB_STORAGE->PWM. > > > > Fixes: aeb9cedc87a2 ("pwm: Make use of a symbol namespace for the > > core") > > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com> > > I found that issue locally, too, but obviously failed to squash this change into the commit. How > embarrassing. Yes, It make sense to squash the commit before it lands to linus tree. Cheers, Biju
Hello, On Fri, Jun 14, 2024 at 03:44:27PM +0000, Biju Das wrote: > > -----Original Message----- > > From: Uwe Kleine-König <u.kleine-koenig@baylibre.com> > > Subject: Re: [PATCH] pwm: Fix namespace error > > > > On Fri, Jun 14, 2024 at 04:12:33PM +0100, Biju Das wrote: > > > Fix namespace error due to typo by replacing USB_STORAGE->PWM. > > > > > > Fixes: aeb9cedc87a2 ("pwm: Make use of a symbol namespace for the > > > core") > > > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com> > > > > I found that issue locally, too, but obviously failed to squash this change into the commit. How > > embarrassing. > > Yes, It make sense to squash the commit before it lands to linus tree. I squashed your patch into mine now and added: Thanks to Biju Das for pointing out a cut'n'paste failure in my initial patch. to the commit log. Is this ok for you? If not, keeping the fix separate would be ok for me, too. Best regards Uwe
Hi Uwe, > -----Original Message----- > From: Uwe Kleine-König <u.kleine-koenig@baylibre.com> > Sent: Friday, June 14, 2024 5:03 PM > Subject: Re: [PATCH] pwm: Fix namespace error > > Hello, > > On Fri, Jun 14, 2024 at 03:44:27PM +0000, Biju Das wrote: > > > -----Original Message----- > > > From: Uwe Kleine-König <u.kleine-koenig@baylibre.com> > > > Subject: Re: [PATCH] pwm: Fix namespace error > > > > > > On Fri, Jun 14, 2024 at 04:12:33PM +0100, Biju Das wrote: > > > > Fix namespace error due to typo by replacing USB_STORAGE->PWM. > > > > > > > > Fixes: aeb9cedc87a2 ("pwm: Make use of a symbol namespace for the > > > > core") > > > > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com> > > > > > > I found that issue locally, too, but obviously failed to squash this > > > change into the commit. How embarrassing. > > > > Yes, It make sense to squash the commit before it lands to linus tree. > > I squashed your patch into mine now and added: > > Thanks to Biju Das for pointing out a cut'n'paste failure in my > initial patch. > > to the commit log. > > Is this ok for you? If not, keeping the fix separate would be ok for me, too. Yes, It is ok, if you just squash the patch. I don't mind, if you remove "Thanks to Biju Das for pointing out a cut'n'paste failure in my initial patch." as you already found the issue locally. Cheers, biju
diff --git a/include/linux/pwm.h b/include/linux/pwm.h index c68576067eca..75ad0d2fd949 100644 --- a/include/linux/pwm.h +++ b/include/linux/pwm.h @@ -8,7 +8,7 @@ #include <linux/mutex.h> #include <linux/of.h> -MODULE_IMPORT_NS(USB_STORAGE); +MODULE_IMPORT_NS(PWM); struct pwm_chip;
Fix namespace error due to typo by replacing USB_STORAGE->PWM. Fixes: aeb9cedc87a2 ("pwm: Make use of a symbol namespace for the core") Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com> --- include/linux/pwm.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)