Thread (37 messages) 37 messages, 5 authors, 2020-03-17

RE: [PATCH V3 1/7] firmware: imx: Add stubs for !CONFIG_IMX_SCU case

From: Anson Huang <hidden>
Date: 2020-03-16 08:44:28
Also in: linux-input, linux-pm, linux-rtc, linux-watchdog, lkml

Subject: Re: [PATCH V3 1/7] firmware: imx: Add stubs for !CONFIG_IMX_SCU
case

On 16/03/2020 08:04:17+0000, Anson Huang wrote:
quoted
Hi, Shawn
quoted
Subject: Re: [PATCH V3 1/7] firmware: imx: Add stubs for
!CONFIG_IMX_SCU case

On Mon, Mar 16, 2020 at 03:18:43AM +0000, Peng Fan wrote:
quoted
Just sent out. One more thing, I think all drivers depends on
IMX_SCU should not have COMPILE_TEST if we plan not to add dummy
functions. I see you picked up Anson's patch in imx/drivers branch,
please check more.
quoted
quoted
Ha, yes. COMPILE_TEST should be dropped for IMX_SCU_PD in Anson's
patch.
quoted
quoted
Thanks for reminding.
I still NOT quite understand why we won't support COMPILE_TEST for SCU
drivers, with whose stubs, the build should be OK, if there is any
build error, we should try to fix it, NOT just remove the COMPILE_TEST
support, any special reason?
quoted
COMPILE_TEST is supported as long as IMX_SCU is selected like is it for any
driver depending on any bus.
But without having " || COMPILE_TEST " in kconfig, COMPILE_TEST will NOT be supported,
I think as long as we have stubs for those SCU APIs, all drivers depending on IMX_SCU can
support COMPILE_TEST independently.

Anson  
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help