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
heat_ is now known as heat
apritzel_ has quit [Ping timeout: 265 seconds]
mraynal has quit [Read error: Connection reset by peer]
mraynal has joined #armlinux
prabhakarlad has quit [Ping timeout: 260 seconds]
markkc has joined #armlinux
markkc has quit [Client Quit]
Nact has joined #armlinux
xv8 has joined #armlinux
Guest319 has quit [Ping timeout: 265 seconds]
nsc has joined #armlinux
nsc is now known as Guest2808
jclsn has quit [Ping timeout: 248 seconds]
jclsn has joined #armlinux
rvalue has quit [Read error: Connection reset by peer]
rvalue has joined #armlinux
amitk has joined #armlinux
iivanov has joined #armlinux
heat has quit [Ping timeout: 246 seconds]
amitk_ has joined #armlinux
iivanov has quit [Quit: https://quassel-irc.org - Chat comfortably. Anywhere.]
hanetzer has quit [Ping timeout: 265 seconds]
hanetzer has joined #armlinux
frieder has joined #armlinux
guillaume_g has joined #armlinux
iivanov has joined #armlinux
cleger has joined #armlinux
headless has joined #armlinux
matthias_bgg has joined #armlinux
monstr has joined #armlinux
prabhakarlad has joined #armlinux
sszy has joined #armlinux
mcoquelin has quit [Ping timeout: 250 seconds]
mcoquelin has joined #armlinux
viorel_suman has joined #armlinux
Nact has quit [Quit: Konversation terminated!]
luispm has joined #armlinux
apritzel_ has joined #armlinux
apritzel_ has quit [Ping timeout: 246 seconds]
apritzel has joined #armlinux
Guest2808 is now known as nsc
Amit_T has joined #armlinux
elastic_dog has quit [Ping timeout: 246 seconds]
elastic_dog has joined #armlinux
cbeznea has joined #armlinux
cbeznea has quit [Quit: Leaving.]
cbeznea has joined #armlinux
amitk_ has quit [Ping timeout: 268 seconds]
Nact has joined #armlinux
cleger has quit [Quit: Leaving]
mrutland- is now known as mrutland
mrutland is now known as Guest5389
Guest5389 is now known as mrutland
cbeznea has quit [Ping timeout: 276 seconds]
<krzk> ukleinek: I saw you got interesting response to aardvark patch... so now you see, you should have checked all possible drivers in kernel (just hundreds? thousands?) for pending patches and do not submit anything less important than current queue :)
<krzk> or better... Collect all pending patches (not applied) for these hundreds/thousands drivers and rebase on top of them. Should also work.
headless has quit [Quit: Konversation terminated!]
elastic_dog has quit [Ping timeout: 240 seconds]
elastic_1 has joined #armlinux
elastic_1 is now known as elastic_dog
prabhakarlad has quit [Quit: Client closed]
elastic_dog has quit [Ping timeout: 246 seconds]
elastic_dog has joined #armlinux
prabhakarlad has joined #armlinux
sudeepholla_ has quit [Ping timeout: 246 seconds]
bryanb has quit [Remote host closed the connection]
tom5760 has quit [Remote host closed the connection]
bryanb has joined #armlinux
tom5760 has joined #armlinux
frieder has quit [Ping timeout: 240 seconds]
sudeepholla_ has joined #armlinux
krzk has quit [Remote host closed the connection]
frieder has joined #armlinux
krzk has joined #armlinux
tom5760 has quit [Write error: Broken pipe]
bryanb has quit [Remote host closed the connection]
prabhakarlad has quit [Quit: Client closed]
bryanb has joined #armlinux
tom5760 has joined #armlinux
Nact has quit [Quit: Konversation terminated!]
monstr has quit [Remote host closed the connection]
headless has joined #armlinux
prabhakarlad has joined #armlinux
guillaume_g has quit [Quit: Konversation terminated!]
cbeznea has joined #armlinux
viorel_suman has quit [Quit: WeeChat 3.6]
sszy has quit [Quit: http://quassel-irc.org - Chat comfortably. Anywhere.]
cbeznea has quit [Quit: Leaving.]
<ukleinek> krzk: I was surprised by that reply and considered contacting conduct@kernel.org. In the end I just replied to it.
<krzk> ukleinek: I think it was quite good reply :)
heat has joined #armlinux
<ukleinek> krzk: thanks for that feedback, I already wondered what others think about it.
<ukleinek> krzk: while I have you attention. Do you consider to address my feedback in https://patchwork.ozlabs.org/project/linux-pwm/patch/20230312135120.357713-1-krzysztof.kozlowski@linaro.org/ and send another still more improved commit log?
<ukleinek> krzk: if not, I'd pick up this patch and add a note that not only !CONFIG_OF is required but also !CONFIG_MODULES and W=1
<ukleinek> (cc: tagr)
<krzk> ukleinek: uh, apologies, I totally missed and marked as read.
<ukleinek> krzk: no problem
<krzk> Or I probably thought that at the end of email you reproduced it actually?
<javierm> krzk, ukleinek: I wonder if a better strategy wouldn't be to modify the ifdef guards for the of_match_ptr() macro instead?
<ukleinek> krzk: yeah, I can reproduce, but that means the commit log isn't as specific as it {c,sh}ould
<ukleinek> javierm: dropping of_match_ptr is fine IMHO
<krzk> javierm: ukleinek: I got already one more comment about this - for of_match_ptr() to convert to static inline
<krzk> and one more - to make it depending on ACPI as well, so ACPI setups have of_device_id for PRP001
<javierm> ukleinek: yeah I know. But then it requires to audit each driver to determine if is using the macro correctly or not
<javierm> I would prefer if it would always safe to use it and never lead to an unused variable warning
apritzel has quit [Ping timeout: 240 seconds]
frieder has quit [Remote host closed the connection]
cbeznea has joined #armlinux
prabhakarlad has quit [Quit: Client closed]
apritzel_ has joined #armlinux
cbeznea1 has joined #armlinux
cbeznea has quit [Read error: Connection reset by peer]
cbeznea1 has quit [Read error: Connection reset by peer]
cbeznea has joined #armlinux
amitk has quit [Ping timeout: 255 seconds]
balrog has quit [Quit: Bye]
chip_x has joined #armlinux
balrog has joined #armlinux
chip_x has quit [Quit: Leaving]
Pali has joined #armlinux
Amit_T has quit [Quit: Leaving]
<ukleinek> javierm: let's do: #define of_match_ptr(x) x
<ukleinek> javierm: then you don't get unused warnings :-)
<javierm> ukleinek: that's how the macro works, isn't?
<javierm> what I'm arguing is to change the condition when #define of_match_ptr(_ptr)NULL
heat_ has joined #armlinux
heat has quit [Ping timeout: 256 seconds]
prabhakarlad has joined #armlinux
<javierm> ukleinek: because if the device can only be registered via OF, then probably the driver should depend on OF and the macro could be of_matr_ptr(_ptr) (_ptr) also when COMPILE_TEST or smt like that
headless has quit [Quit: Konversation terminated!]
<ukleinek> javierm: my suggestion was to define it as x unconditionally :-)
<javierm> ukleinek: I always wondered the value of that macro (and the acpi one) tbh
* ukleinek would expect it was introduced to please someone who didn't use OF and didn't want the additional table in their modules.
mal``` has quit [Quit: Leaving]
DynamiteDan has quit [Excess Flood]
DynamiteDan has joined #armlinux
DynamiteDan has quit [Excess Flood]
DynamiteDan has joined #armlinux
<krzk> This should be static inline rather, not macro, so the variable will be always referenced (but both dropped by compiler)
mal`` has joined #armlinux
<javierm> krzk: that makes sense indeed
matthias_bgg has quit [Read error: Connection reset by peer]
prabhakarlad has quit [Quit: Client closed]
matthias_bgg has joined #armlinux
iivanov has quit [Ping timeout: 268 seconds]
heat_ has quit [Remote host closed the connection]
heat_ has joined #armlinux
heat_ is now known as heat
Pali has quit [Quit: Pali]
cbeznea has quit [Quit: Leaving.]
matthias_bgg has quit [Read error: Connection reset by peer]
matthias_bgg has joined #armlinux