* Re: [PATCH] media: i2c: imx290: fix conditional function defintions
[not found] ` <Y+J+7lsf083k4x80@pendragon.ideasonboard.com>
@ 2023-03-20 9:18 ` Linux regression tracking (Thorsten Leemhuis)
2023-03-20 9:32 ` Sakari Ailus
0 siblings, 1 reply; 5+ messages in thread
From: Linux regression tracking (Thorsten Leemhuis) @ 2023-03-20 9:18 UTC (permalink / raw)
To: Sakari Ailus, Manivannan Sadhasivam
Cc: Mauro Carvalho Chehab, Alexander Stein, Arnd Bergmann,
Dave Stevenson, linux-media, linux-kernel, Laurent Pinchart,
Arnd Bergmann, Linux kernel regressions list
On 07.02.23 17:40, Laurent Pinchart wrote:
> On Tue, Feb 07, 2023 at 05:13:12PM +0100, Arnd Bergmann wrote:
>> From: Arnd Bergmann <arnd@arndb.de>
>>
>> The runtime suspend/resume functions are only referenced from the
>> dev_pm_ops, but they use the old SET_RUNTIME_PM_OPS() helper
>> that requires a __maybe_unused annotation to avoid a warning:
>>
>> drivers/media/i2c/imx290.c:1082:12: error: unused function 'imx290_runtime_resume' [-Werror,-Wunused-function]
>> static int imx290_runtime_resume(struct device *dev)
>> ^
>> drivers/media/i2c/imx290.c:1090:12: error: unused function 'imx290_runtime_suspend' [-Werror,-Wunused-function]
>> static int imx290_runtime_suspend(struct device *dev)
>> ^
>>
I might be missing something (if so, please tell me), but to me it looks
this fix for a build issue in 6.3-rc (which shows up in Guenters weekly
reports to Linus) didn't make any progress in the past few weeks. Is
there a reason why? Who actually needs to pick it up and send it towards
mainline? Manivannan Sadhasivam? Sakari Ailus?
Ciao, Thorsten (wearing his 'the Linux kernel's regression tracker' hat)
--
Everything you wanna know about Linux kernel regression tracking:
https://linux-regtracking.leemhuis.info/about/#tldr
If I did something stupid, please tell me, as explained on that page.
#regzbot poke
>> Convert this to the new RUNTIME_PM_OPS() helper that so this
>> is not required. To improve this further, also use the pm_ptr()
>> helper that lets the dev_pm_ops get dropped entirely when
>> CONFIG_PM is disabled.
>>
>> A related mistake happened in the of_match_ptr() macro here, which
>> like SET_RUNTIME_PM_OPS() requires the match table to be marked
>> as __maybe_unused, though I could not reproduce building this without
>> CONFIG_OF. Remove the of_match_ptr() here as there is no point in
>> dropping the match table in configurations without CONFIG_OF.
>>
>> Fixes: 02852c01f654 ("media: i2c: imx290: Initialize runtime PM before subdev")
>> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
>
> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
>
>> ---
>> drivers/media/i2c/imx290.c | 6 +++---
>> 1 file changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/drivers/media/i2c/imx290.c b/drivers/media/i2c/imx290.c
>> index 49d6c8bdec41..48ae2e0adf9e 100644
>> --- a/drivers/media/i2c/imx290.c
>> +++ b/drivers/media/i2c/imx290.c
>> @@ -1098,7 +1098,7 @@ static int imx290_runtime_suspend(struct device *dev)
>> }
>>
>> static const struct dev_pm_ops imx290_pm_ops = {
>> - SET_RUNTIME_PM_OPS(imx290_runtime_suspend, imx290_runtime_resume, NULL)
>> + RUNTIME_PM_OPS(imx290_runtime_suspend, imx290_runtime_resume, NULL)
>> };
>>
>> /* ----------------------------------------------------------------------------
>> @@ -1362,8 +1362,8 @@ static struct i2c_driver imx290_i2c_driver = {
>> .remove = imx290_remove,
>> .driver = {
>> .name = "imx290",
>> - .pm = &imx290_pm_ops,
>> - .of_match_table = of_match_ptr(imx290_of_match),
>> + .pm = pm_ptr(&imx290_pm_ops),
>> + .of_match_table = imx290_of_match,
>> },
>> };
>>
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] media: i2c: imx290: fix conditional function defintions
2023-03-20 9:18 ` [PATCH] media: i2c: imx290: fix conditional function defintions Linux regression tracking (Thorsten Leemhuis)
@ 2023-03-20 9:32 ` Sakari Ailus
2023-03-20 9:42 ` Linux regression tracking (Thorsten Leemhuis)
0 siblings, 1 reply; 5+ messages in thread
From: Sakari Ailus @ 2023-03-20 9:32 UTC (permalink / raw)
To: Linux regressions mailing list
Cc: Manivannan Sadhasivam, Mauro Carvalho Chehab, Alexander Stein,
Arnd Bergmann, Dave Stevenson, linux-media, linux-kernel,
Laurent Pinchart, Arnd Bergmann
Hello,
On Mon, Mar 20, 2023 at 10:18:23AM +0100, Linux regression tracking (Thorsten Leemhuis) wrote:
> On 07.02.23 17:40, Laurent Pinchart wrote:
> > On Tue, Feb 07, 2023 at 05:13:12PM +0100, Arnd Bergmann wrote:
> >> From: Arnd Bergmann <arnd@arndb.de>
> >>
> >> The runtime suspend/resume functions are only referenced from the
> >> dev_pm_ops, but they use the old SET_RUNTIME_PM_OPS() helper
> >> that requires a __maybe_unused annotation to avoid a warning:
> >>
> >> drivers/media/i2c/imx290.c:1082:12: error: unused function 'imx290_runtime_resume' [-Werror,-Wunused-function]
> >> static int imx290_runtime_resume(struct device *dev)
> >> ^
> >> drivers/media/i2c/imx290.c:1090:12: error: unused function 'imx290_runtime_suspend' [-Werror,-Wunused-function]
> >> static int imx290_runtime_suspend(struct device *dev)
> >> ^
> >>
>
> I might be missing something (if so, please tell me), but to me it looks
> this fix for a build issue in 6.3-rc (which shows up in Guenters weekly
> reports to Linus) didn't make any progress in the past few weeks. Is
> there a reason why? Who actually needs to pick it up and send it towards
> mainline? Manivannan Sadhasivam? Sakari Ailus?
Mauro has recently pulled my PR including this and I understand it's in
Mauro's tree now.
--
Kind regards,
Sakari Ailus
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] media: i2c: imx290: fix conditional function defintions
2023-03-20 9:32 ` Sakari Ailus
@ 2023-03-20 9:42 ` Linux regression tracking (Thorsten Leemhuis)
2023-03-20 9:46 ` Thorsten Leemhuis
0 siblings, 1 reply; 5+ messages in thread
From: Linux regression tracking (Thorsten Leemhuis) @ 2023-03-20 9:42 UTC (permalink / raw)
To: Sakari Ailus, Linux regressions mailing list
Cc: Manivannan Sadhasivam, Mauro Carvalho Chehab, Alexander Stein,
Arnd Bergmann, Dave Stevenson, linux-media, linux-kernel,
Laurent Pinchart, Arnd Bergmann
On 20.03.23 10:32, Sakari Ailus wrote:
> On Mon, Mar 20, 2023 at 10:18:23AM +0100, Linux regression tracking (Thorsten Leemhuis) wrote:
>> On 07.02.23 17:40, Laurent Pinchart wrote:
>>> On Tue, Feb 07, 2023 at 05:13:12PM +0100, Arnd Bergmann wrote:
>>>> From: Arnd Bergmann <arnd@arndb.de>
>>>>
>>>> The runtime suspend/resume functions are only referenced from the
>>>> dev_pm_ops, but they use the old SET_RUNTIME_PM_OPS() helper
>>>> that requires a __maybe_unused annotation to avoid a warning:
>>>>
>>>> drivers/media/i2c/imx290.c:1082:12: error: unused function 'imx290_runtime_resume' [-Werror,-Wunused-function]
>>>> static int imx290_runtime_resume(struct device *dev)
>>>> ^
>>>> drivers/media/i2c/imx290.c:1090:12: error: unused function 'imx290_runtime_suspend' [-Werror,-Wunused-function]
>>>> static int imx290_runtime_suspend(struct device *dev)
>>>> ^
>>>>
>>
>> I might be missing something (if so, please tell me), but to me it looks
>> this fix for a build issue in 6.3-rc (which shows up in Guenters weekly
>> reports to Linus) didn't make any progress in the past few weeks. Is
>> there a reason why? Who actually needs to pick it up and send it towards
>> mainline? Manivannan Sadhasivam? Sakari Ailus?
>
> Mauro has recently pulled my PR including this and I understand it's in
> Mauro's tree now.
Ahh, great, many thx.
I checked Linux-next before writing my mail, but didn't sport it there. :-/
Ciao, Thorsten
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] media: i2c: imx290: fix conditional function defintions
2023-03-20 9:42 ` Linux regression tracking (Thorsten Leemhuis)
@ 2023-03-20 9:46 ` Thorsten Leemhuis
2023-03-20 9:51 ` Sakari Ailus
0 siblings, 1 reply; 5+ messages in thread
From: Thorsten Leemhuis @ 2023-03-20 9:46 UTC (permalink / raw)
To: Sakari Ailus, Linux regressions mailing list
Cc: Manivannan Sadhasivam, Mauro Carvalho Chehab, Alexander Stein,
Arnd Bergmann, Dave Stevenson, linux-media, linux-kernel,
Laurent Pinchart, Arnd Bergmann
On 20.03.23 10:42, Linux regression tracking (Thorsten Leemhuis) wrote:
> On 20.03.23 10:32, Sakari Ailus wrote:
>> On Mon, Mar 20, 2023 at 10:18:23AM +0100, Linux regression tracking (Thorsten Leemhuis) wrote:
>>> On 07.02.23 17:40, Laurent Pinchart wrote:
>>>> On Tue, Feb 07, 2023 at 05:13:12PM +0100, Arnd Bergmann wrote:
>>>>> From: Arnd Bergmann <arnd@arndb.de>
>>>>>
>>>>> The runtime suspend/resume functions are only referenced from the
>>>>> dev_pm_ops, but they use the old SET_RUNTIME_PM_OPS() helper
>>>>> that requires a __maybe_unused annotation to avoid a warning:
>>>>>
>>>>> drivers/media/i2c/imx290.c:1082:12: error: unused function 'imx290_runtime_resume' [-Werror,-Wunused-function]
>>>>> static int imx290_runtime_resume(struct device *dev)
>>>>> ^
>>>>> drivers/media/i2c/imx290.c:1090:12: error: unused function 'imx290_runtime_suspend' [-Werror,-Wunused-function]
>>>>> static int imx290_runtime_suspend(struct device *dev)
>>>>> ^
>>>>>
>>>
>>> I might be missing something (if so, please tell me), but to me it looks
>>> this fix for a build issue in 6.3-rc (which shows up in Guenters weekly
>>> reports to Linus) didn't make any progress in the past few weeks. Is
>>> there a reason why? Who actually needs to pick it up and send it towards
>>> mainline? Manivannan Sadhasivam? Sakari Ailus?
>>
>> Mauro has recently pulled my PR including this and I understand it's in
>> Mauro's tree now.
>
> Ahh, great, many thx.
>
> I checked Linux-next before writing my mail, but didn't sport it there. :-/
Argh, ohh no, it's there. Sorry for the noise. Second time something
like this happens today. Seems today is not my best day. :-/
Ciao, Thorsten
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] media: i2c: imx290: fix conditional function defintions
2023-03-20 9:46 ` Thorsten Leemhuis
@ 2023-03-20 9:51 ` Sakari Ailus
0 siblings, 0 replies; 5+ messages in thread
From: Sakari Ailus @ 2023-03-20 9:51 UTC (permalink / raw)
To: Linux regressions mailing list
Cc: Manivannan Sadhasivam, Mauro Carvalho Chehab, Alexander Stein,
Arnd Bergmann, Dave Stevenson, linux-media, linux-kernel,
Laurent Pinchart, Arnd Bergmann
On Mon, Mar 20, 2023 at 10:46:45AM +0100, Thorsten Leemhuis wrote:
> Argh, ohh no, it's there. Sorry for the noise. Second time something
> like this happens today. Seems today is not my best day. :-/
No worries!
--
Sakari Ailus
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-03-20 9:51 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20230207161316.293923-1-arnd@kernel.org>
[not found] ` <Y+J+7lsf083k4x80@pendragon.ideasonboard.com>
2023-03-20 9:18 ` [PATCH] media: i2c: imx290: fix conditional function defintions Linux regression tracking (Thorsten Leemhuis)
2023-03-20 9:32 ` Sakari Ailus
2023-03-20 9:42 ` Linux regression tracking (Thorsten Leemhuis)
2023-03-20 9:46 ` Thorsten Leemhuis
2023-03-20 9:51 ` Sakari Ailus
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).