ChanServ changed the topic of #armlinux to: ARM kernel talk [Upstream kernel, find your vendor forums for questions about their kernels] | https://libera.irclog.whitequark.org/armlinux
djrscally has quit [Ping timeout: 268 seconds]
Pali has quit [Ping timeout: 268 seconds]
Turingtoast has quit [Ping timeout: 264 seconds]
Turingtoast has joined #armlinux
iivanov has joined #armlinux
iivanov has quit [Ping timeout: 268 seconds]
jlinton has quit [Quit: Client closed]
rbutler1728 has quit [Read error: Connection reset by peer]
dianders has quit [Quit: Connection closed for inactivity]
apritzel_ has quit [Ping timeout: 268 seconds]
Nact has joined #armlinux
Nact has quit [Client Quit]
iivanov has joined #armlinux
Turingtoast has quit [Quit: My iMac has gone to sleep. ZZZzzz…]
engest has quit [Ping timeout: 268 seconds]
apritzel_ has joined #armlinux
apritzel_ has quit [Ping timeout: 268 seconds]
iivanov has quit [Quit: Leaving...]
iivanov has joined #armlinux
apritzel_ has joined #armlinux
djrscally has joined #armlinux
Pali has joined #armlinux
apritzel_ has quit [Ping timeout: 264 seconds]
tre has joined #armlinux
sszy has joined #armlinux
matthias_bgg2 has joined #armlinux
tudorel has joined #armlinux
nsaenz has joined #armlinux
apritzel has joined #armlinux
Turingtoast has joined #armlinux
ravan has quit [Remote host closed the connection]
<ardb>
any clue why it claims that HAVE_STATIC_CALL is select by LTO_CLANG_THIN?
<ukleinek>
broonie: I think since 6840615f85f6046039ebc4989870ddb12892b7fc the spidev driver doesn't auto bind to devices that have compatible = "spidev" in the device tree. Was this done on purpose? With that in place keeping the warning message isn't very sensible, is it?
<arnd>
ardb: I think that's the transformation done in Kconfig: CONFIG_ARM64 is unconditional, so "select HAVE_STATIC_CALL if LTO_CLANG_THIN" is the same as having "select HAVE_STATIC_CALL" under CONFIG_LTO_CLANG_THIN
<ardb>
arnd: yeah but it does the opposite
<arnd>
ardb: if you want to break the dependency chain, I would start by making ZSWAP 'select CRYPTO'
<ardb>
it selects HAVE_STATIC_CALL if !LTO_CLANG_THIN
<arnd>
CEC_GPIO should probably "depends on GPIOLIB"
<arnd>
ardb: ah, I see what you mean: that'
<arnd>
that's the same thing for the purpose of loop detection
<ardb>
right
<arnd>
printing it as the "selected by !LTO_CLANG_THIN" would probably be clearer
<arnd>
ardb: the "select DEBUG_FS" also looks wrong, that should again be "depends on DEBUG_FS"
<ardb>
arnd: ok i'll take that one, it's the most obvious one imo
<arnd>
I think we should fix all three: CEC_GPIO, ZSWAP, and GCOV_KERNEL -- otherwise the next person will run into a similar loop
<ardb>
fair enough
<ardb>
but depending on GPIOLIB rather than selecting it is OK?
<arnd>
I suppose this was all triggered by the new SND_SOC_AMD_RV_RT5682_MACH, I've already sent a couple of Kconfig fixes for that one, but as far as I can tell this symbol is not doing anything wrong this time
<arnd>
ardb: yes: the general rule is to use 'depends on' for user-visible symbols, and 'select' for hidden symbols that each user needs to select. the reason is that you get this kind of dependency loop if you start mixing the two
<arnd>
GPIOLIB is now user visible, so normal drivers should use "depends on" if they require it
<arnd>
there are still "select GPIOLIB" statements in arch/*/Kconfig that are reasonable because it
<ardb>
ok because it is already select'ed in many plces
<arnd>
is the platform requiring them, and those platforms don't normally have dependencies themselves
<arnd>
the 'select GPIOLIB' statemetns in drivers/* and sound/soc/* are probably all bugs
<ardb>
right
<arnd>
drivers/pinctrl again is an exception
<arnd>
we should probably change PINCTRL_PISTACHIO to 'select GPIOLIB' here, or possibly make CONFIG_PINCTRL unconditionally select GPIOLIB globally
luispm has quit [Ping timeout: 260 seconds]
<broonie>
ukleinek: It wasn't particularly on purpose.
luispm has joined #armlinux
Amit_T has joined #armlinux
torez has joined #armlinux
engest has joined #armlinux
apritzel has joined #armlinux
milkylainen has quit [Quit: Connection closed]
torez has quit [Ping timeout: 264 seconds]
tre has quit [Remote host closed the connection]
Tokamak has quit [Ping timeout: 264 seconds]
torez has joined #armlinux
Tokamak has joined #armlinux
Tokamak has quit [Read error: Connection reset by peer]