* [PATCH 1/1] ipu3-cio2: Build bridge only if ACPI is enabled
@ 2021-02-02 20:14 Sakari Ailus
2021-02-02 20:24 ` Randy Dunlap
0 siblings, 1 reply; 5+ messages in thread
From: Sakari Ailus @ 2021-02-02 20:14 UTC (permalink / raw)
To: Randy Dunlap
Cc: Stephen Rothwell, Linux Next Mailing List,
Linux Kernel Mailing List, linux-media, Dan Scally, Bingbu Cao,
Yong Zhi
ipu3-cio2-bridge uses several features of the ACPI framework that have no
meaningful replacement when ACPI is disabled. Instead of adding #ifdefs to
the affected places, only build the bridge code if CONFIG_ACPI is enabled.
Fixes: 803abec64ef9 ("media: ipu3-cio2: Add cio2-bridge to ipu3-cio2 driver")
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
---
Hi Randy,
Thanks for reporting this.
This patch should address the problem.
drivers/media/pci/intel/ipu3/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/pci/intel/ipu3/Kconfig b/drivers/media/pci/intel/ipu3/Kconfig
index 24f4e79fe0cb..dce8274c81e6 100644
--- a/drivers/media/pci/intel/ipu3/Kconfig
+++ b/drivers/media/pci/intel/ipu3/Kconfig
@@ -20,7 +20,7 @@ config VIDEO_IPU3_CIO2
config CIO2_BRIDGE
bool "IPU3 CIO2 Sensors Bridge"
- depends on VIDEO_IPU3_CIO2
+ depends on VIDEO_IPU3_CIO2 && ACPI
help
This extension provides an API for the ipu3-cio2 driver to create
connections to cameras that are hidden in the SSDB buffer in ACPI.
--
2.29.2
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 1/1] ipu3-cio2: Build bridge only if ACPI is enabled
2021-02-02 20:14 [PATCH 1/1] ipu3-cio2: Build bridge only if ACPI is enabled Sakari Ailus
@ 2021-02-02 20:24 ` Randy Dunlap
2021-02-02 20:30 ` Sakari Ailus
0 siblings, 1 reply; 5+ messages in thread
From: Randy Dunlap @ 2021-02-02 20:24 UTC (permalink / raw)
To: Sakari Ailus
Cc: Stephen Rothwell, Linux Next Mailing List,
Linux Kernel Mailing List, linux-media, Dan Scally, Bingbu Cao,
Yong Zhi
On 2/2/21 12:14 PM, Sakari Ailus wrote:
> ipu3-cio2-bridge uses several features of the ACPI framework that have no
> meaningful replacement when ACPI is disabled. Instead of adding #ifdefs to
> the affected places, only build the bridge code if CONFIG_ACPI is enabled.
>
> Fixes: 803abec64ef9 ("media: ipu3-cio2: Add cio2-bridge to ipu3-cio2 driver")
> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Reported-by: Randy Dunlap <rdunlap@infradead.org>
Acked-by: Randy Dunlap <rdunlap@infradead.org> # build-tested
Thanks.
> ---
> Hi Randy,
>
> Thanks for reporting this.
>
> This patch should address the problem.
>
> drivers/media/pci/intel/ipu3/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/media/pci/intel/ipu3/Kconfig b/drivers/media/pci/intel/ipu3/Kconfig
> index 24f4e79fe0cb..dce8274c81e6 100644
> --- a/drivers/media/pci/intel/ipu3/Kconfig
> +++ b/drivers/media/pci/intel/ipu3/Kconfig
> @@ -20,7 +20,7 @@ config VIDEO_IPU3_CIO2
>
> config CIO2_BRIDGE
> bool "IPU3 CIO2 Sensors Bridge"
> - depends on VIDEO_IPU3_CIO2
> + depends on VIDEO_IPU3_CIO2 && ACPI
> help
> This extension provides an API for the ipu3-cio2 driver to create
> connections to cameras that are hidden in the SSDB buffer in ACPI.
>
--
~Randy
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/1] ipu3-cio2: Build bridge only if ACPI is enabled
2021-02-02 20:24 ` Randy Dunlap
@ 2021-02-02 20:30 ` Sakari Ailus
2021-02-02 21:37 ` Daniel Scally
0 siblings, 1 reply; 5+ messages in thread
From: Sakari Ailus @ 2021-02-02 20:30 UTC (permalink / raw)
To: Randy Dunlap
Cc: Stephen Rothwell, Linux Next Mailing List,
Linux Kernel Mailing List, linux-media, Dan Scally, Bingbu Cao,
Yong Zhi
On Tue, Feb 02, 2021 at 12:24:54PM -0800, Randy Dunlap wrote:
> On 2/2/21 12:14 PM, Sakari Ailus wrote:
> > ipu3-cio2-bridge uses several features of the ACPI framework that have no
> > meaningful replacement when ACPI is disabled. Instead of adding #ifdefs to
> > the affected places, only build the bridge code if CONFIG_ACPI is enabled.
> >
> > Fixes: 803abec64ef9 ("media: ipu3-cio2: Add cio2-bridge to ipu3-cio2 driver")
> > Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
>
> Reported-by: Randy Dunlap <rdunlap@infradead.org>
> Acked-by: Randy Dunlap <rdunlap@infradead.org> # build-tested
Thanks! I'll include this in a pull request to Mauro shortly.
--
Sakari Ailus
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/1] ipu3-cio2: Build bridge only if ACPI is enabled
2021-02-02 20:30 ` Sakari Ailus
@ 2021-02-02 21:37 ` Daniel Scally
2021-02-02 22:02 ` Sakari Ailus
0 siblings, 1 reply; 5+ messages in thread
From: Daniel Scally @ 2021-02-02 21:37 UTC (permalink / raw)
To: Sakari Ailus, Randy Dunlap
Cc: Stephen Rothwell, Linux Next Mailing List,
Linux Kernel Mailing List, linux-media, Bingbu Cao, Yong Zhi
On 02/02/2021 20:30, Sakari Ailus wrote:
> On Tue, Feb 02, 2021 at 12:24:54PM -0800, Randy Dunlap wrote:
>> On 2/2/21 12:14 PM, Sakari Ailus wrote:
>>> ipu3-cio2-bridge uses several features of the ACPI framework that have no
>>> meaningful replacement when ACPI is disabled. Instead of adding #ifdefs to
>>> the affected places, only build the bridge code if CONFIG_ACPI is enabled.
>>>
>>> Fixes: 803abec64ef9 ("media: ipu3-cio2: Add cio2-bridge to ipu3-cio2 driver")
>>> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
>> Reported-by: Randy Dunlap <rdunlap@infradead.org>
>> Acked-by: Randy Dunlap <rdunlap@infradead.org> # build-tested
> Thanks! I'll include this in a pull request to Mauro shortly.
>
Ah - thank you both; sorry to have missed that.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/1] ipu3-cio2: Build bridge only if ACPI is enabled
2021-02-02 21:37 ` Daniel Scally
@ 2021-02-02 22:02 ` Sakari Ailus
0 siblings, 0 replies; 5+ messages in thread
From: Sakari Ailus @ 2021-02-02 22:02 UTC (permalink / raw)
To: Daniel Scally
Cc: Randy Dunlap, Stephen Rothwell, Linux Next Mailing List,
Linux Kernel Mailing List, linux-media, Bingbu Cao, Yong Zhi
On Tue, Feb 02, 2021 at 09:37:47PM +0000, Daniel Scally wrote:
> On 02/02/2021 20:30, Sakari Ailus wrote:
> > On Tue, Feb 02, 2021 at 12:24:54PM -0800, Randy Dunlap wrote:
> >> On 2/2/21 12:14 PM, Sakari Ailus wrote:
> >>> ipu3-cio2-bridge uses several features of the ACPI framework that have no
> >>> meaningful replacement when ACPI is disabled. Instead of adding #ifdefs to
> >>> the affected places, only build the bridge code if CONFIG_ACPI is enabled.
> >>>
> >>> Fixes: 803abec64ef9 ("media: ipu3-cio2: Add cio2-bridge to ipu3-cio2 driver")
> >>> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
> >> Reported-by: Randy Dunlap <rdunlap@infradead.org>
> >> Acked-by: Randy Dunlap <rdunlap@infradead.org> # build-tested
> > Thanks! I'll include this in a pull request to Mauro shortly.
> >
> Ah - thank you both; sorry to have missed that.
No worries; this was missed in review, too. There are just combinations you
don't always end up testing before merging the patches.
--
Regards,
Sakari Ailus
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-02-02 22:05 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-02 20:14 [PATCH 1/1] ipu3-cio2: Build bridge only if ACPI is enabled Sakari Ailus
2021-02-02 20:24 ` Randy Dunlap
2021-02-02 20:30 ` Sakari Ailus
2021-02-02 21:37 ` Daniel Scally
2021-02-02 22:02 ` 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).