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
Crofton has quit [Ping timeout: 260 seconds]
tfiga has quit [Ping timeout: 260 seconds]
netonaut_ has quit [Ping timeout: 244 seconds]
pjw has quit [Ping timeout: 264 seconds]
jamestperk has quit [Ping timeout: 268 seconds]
broonie has quit [Ping timeout: 245 seconds]
sjg1 has quit [Read error: Connection reset by peer]
ccaione has quit [Read error: Connection reset by peer]
unixsmurf has quit [Read error: Connection reset by peer]
khilman has quit [Read error: Connection reset by peer]
narmstrong has quit [Read error: No route to host]
roxell has quit [Read error: Connection reset by peer]
dtor has quit [Read error: Connection reset by peer]
maennich has quit [Read error: Connection reset by peer]
dianders has quit [Read error: Connection reset by peer]
mturquette has quit [Read error: Connection reset by peer]
nathanchance has quit [Read error: Connection reset by peer]
arnd has quit [Read error: Connection reset by peer]
robclark has quit [Read error: Connection reset by peer]
zx2c4 has quit [Ping timeout: 264 seconds]
ndesaulniers has quit [Read error: Connection reset by peer]
NishanthMenon has quit [Write error: Connection reset by peer]
nohit has quit [Ping timeout: 268 seconds]
cengiz_io has quit [Ping timeout: 268 seconds]
steev has quit [Ping timeout: 268 seconds]
olofj has quit [Write error: Connection reset by peer]
robher has quit [Read error: Connection reset by peer]
linusw_ has quit [Ping timeout: 246 seconds]
sboyd has quit [Ping timeout: 246 seconds]
dtor has joined #armlinux
nathanchance has joined #armlinux
narmstrong has joined #armlinux
roxell has joined #armlinux
dianders has joined #armlinux
dtor has quit [Ping timeout: 256 seconds]
nathanchance has quit [Ping timeout: 264 seconds]
narmstrong has quit [Ping timeout: 256 seconds]
roxell has quit [Ping timeout: 264 seconds]
dianders has quit [Ping timeout: 256 seconds]
broonie has joined #armlinux
narmstrong has joined #armlinux
roxell has joined #armlinux
dtor has joined #armlinux
nathanchance has joined #armlinux
Pali has quit [Ping timeout: 256 seconds]
mturquette has joined #armlinux
maennich has joined #armlinux
sjg1 has joined #armlinux
robclark has joined #armlinux
linusw_ has joined #armlinux
ccaione has joined #armlinux
nohit has joined #armlinux
khilman has joined #armlinux
ndesaulniers has joined #armlinux
zx2c4 has joined #armlinux
netonaut_ has joined #armlinux
tfiga has joined #armlinux
pjw has joined #armlinux
jamestperk has joined #armlinux
dianders has joined #armlinux
steev has joined #armlinux
arnd has joined #armlinux
sboyd has joined #armlinux
cengiz_io has joined #armlinux
olofj has joined #armlinux
Crofton has joined #armlinux
unixsmurf has joined #armlinux
robher has joined #armlinux
NishanthMenon has joined #armlinux
apritzel has quit [Ping timeout: 268 seconds]
gdd has joined #armlinux
System_Error has quit [Remote host closed the connection]
System_Error has joined #armlinux
sakman has quit [Quit: Leaving]
luispm has quit [Ping timeout: 246 seconds]
luispm has joined #armlinux
luispm has quit [Ping timeout: 268 seconds]
luispm has joined #armlinux
iivanov has joined #armlinux
roxell has quit [Ping timeout: 256 seconds]
roxell has joined #armlinux
milkylainen has joined #armlinux
bps has joined #armlinux
bps has joined #armlinux
bps has quit [Changing host]
bps has quit [Read error: Connection reset by peer]
CrashTestDummy3 has quit [Read error: No route to host]
CrashTestDummy3 has joined #armlinux
tudorel has joined #armlinux
alexels has joined #armlinux
apritzel_ has joined #armlinux
iivanov__ has joined #armlinux
iivanov__ has quit [Read error: Connection reset by peer]
iivanov__ has joined #armlinux
iivanov has quit [Ping timeout: 250 seconds]
frieder has joined #armlinux
Turingtoast has joined #armlinux
frieder has quit [Remote host closed the connection]
<ukleinek>
mwalle_: nitpick: Put your S-o-b always as last item in the commit log
<mwalle_>
ukleinek: judging by looking at the kernel commits, thats almost never the case, i was just collecting the reviewed-by tags, which i guess would have been in the same order if broonie just picked em up himself, no?
frieder has joined #armlinux
<ukleinek>
mwalle_: yeah, it's a nitpick and there are many offenders.
frieder has quit [Remote host closed the connection]
luispm has quit [Ping timeout: 256 seconds]
prabhakarlad has quit [Quit: Client closed]
prabhakarlad has joined #armlinux
luispm has joined #armlinux
<arnd>
mwalle_: the way I handle it is to leave the existing tags in place, then put the tags I add, and then my own S-o-b last. I think that's what most maintainers do
<arnd>
do if you resend a patch with an added Reviewed-by, that comes before your S-o-b, but if someone else picks it up, your S-o-b comes before the R-b
roxell has quit [Read error: Connection reset by peer]
<mwalle_>
arnd: ukleinek: noted for next time :)
ccaione has quit [Ping timeout: 256 seconds]
linusw_ has quit [Ping timeout: 256 seconds]
jamestperk has quit [Ping timeout: 264 seconds]
tfiga has quit [Ping timeout: 264 seconds]
khilman has quit [Ping timeout: 264 seconds]
<geertu>
arnd: technically the Ab and Rb tags are applied after you signed off on your own patch
netonaut_ has quit [Ping timeout: 268 seconds]
<mwalle_>
arnd: and Cc: tags? like cc: stable
NishanthMenon has quit [Ping timeout: 260 seconds]
Crofton has quit [Ping timeout: 240 seconds]
<geertu>
mwalle_: I don't use them
cengiz_io has quit [Ping timeout: 256 seconds]
unixsmurf has quit [Ping timeout: 256 seconds]
pjw has quit [Ping timeout: 256 seconds]
sboyd has quit [Ping timeout: 268 seconds]
dianders has quit [Ping timeout: 268 seconds]
mturquette has quit [Ping timeout: 268 seconds]
maennich has quit [Ping timeout: 268 seconds]
steev has quit [Ping timeout: 256 seconds]
robclark has quit [Ping timeout: 256 seconds]
narmstrong has quit [Ping timeout: 256 seconds]
sjg1 has quit [Ping timeout: 264 seconds]
nohit_ has joined #armlinux
maennich has joined #armlinux
mturquette has joined #armlinux
sboyd has joined #armlinux
<mwalle_>
geertu: a reviewer put a cc: stable, so i can hardly just skip it ;) anyway, i was under the impression that a Fixes: is enough
robher has quit [Ping timeout: 260 seconds]
olofj has quit [Ping timeout: 256 seconds]
ndesaulniers has quit [Ping timeout: 256 seconds]
nathanchance has quit [Ping timeout: 256 seconds]
sjg1 has joined #armlinux
olofj_ has joined #armlinux
dtor_ has joined #armlinux
netonaut_ has joined #armlinux
dtor has quit [Ping timeout: 264 seconds]
dtor_ is now known as dtor
arnd_ has joined #armlinux
tfiga has joined #armlinux
cengiz_io has joined #armlinux
arnd has quit [Ping timeout: 268 seconds]
zx2c4 has quit [Ping timeout: 268 seconds]
nohit has quit [Ping timeout: 268 seconds]
broonie has quit [Ping timeout: 268 seconds]
nohit_ is now known as nohit
arnd_ is now known as arnd
robher has joined #armlinux
linusw_ has joined #armlinux
ccaione has joined #armlinux
NishanthMenon has joined #armlinux
steev has joined #armlinux
roxell has joined #armlinux
pjw has joined #armlinux
khilman has joined #armlinux
narmstrong has joined #armlinux
dianders has joined #armlinux
jamestperk has joined #armlinux
Crofton has joined #armlinux
unixsmurf has joined #armlinux
broonie has joined #armlinux
ndesaulniers has joined #armlinux
nathanchance has joined #armlinux
robclark has joined #armlinux
zx2c4 has joined #armlinux
zx2c4 has quit [Changing host]
zx2c4 has joined #armlinux
<geertu>
wsYeah, if you have a Fixes tag, the stable machinery will usually be triggered, too
CrashTestDummy3 has quit [Quit: Leaving]
jlinton has joined #armlinux
robher has quit [Ping timeout: 250 seconds]
zx2c4 has quit [Ping timeout: 256 seconds]
Crofton has quit [Ping timeout: 264 seconds]
jamestperk has quit [Ping timeout: 260 seconds]
broonie has quit [Ping timeout: 256 seconds]
dtor has quit [Ping timeout: 240 seconds]
nohit has quit [Ping timeout: 260 seconds]
unixsmurf has quit [Ping timeout: 264 seconds]
netonaut_ has quit [Ping timeout: 250 seconds]
sboyd has quit [Ping timeout: 250 seconds]
arnd has quit [Ping timeout: 240 seconds]
ccaione has quit [Ping timeout: 240 seconds]
olofj_ has quit [Ping timeout: 240 seconds]
sjg1 has quit [Ping timeout: 260 seconds]
steev has quit [Ping timeout: 250 seconds]
nathanchance has quit [Ping timeout: 264 seconds]
pjw has quit [Ping timeout: 250 seconds]
robclark has quit [Ping timeout: 268 seconds]
NishanthMenon has quit [Ping timeout: 240 seconds]
narmstrong has quit [Ping timeout: 240 seconds]
dianders has quit [Ping timeout: 240 seconds]
roxell has quit [Ping timeout: 264 seconds]
linusw_ has quit [Ping timeout: 264 seconds]
cengiz_io has quit [Ping timeout: 264 seconds]
maennich has quit [Ping timeout: 264 seconds]
tfiga has quit [Ping timeout: 264 seconds]
mturquette has quit [Ping timeout: 264 seconds]
ndesaulniers has quit [Ping timeout: 250 seconds]
khilman has quit [Ping timeout: 250 seconds]
nathanchance has joined #armlinux
Crofton has joined #armlinux
netonaut_ has joined #armlinux
narmstrong has joined #armlinux
maennich has joined #armlinux
sboyd has joined #armlinux
pjw has joined #armlinux
ndesaulniers has joined #armlinux
olofj_ has joined #armlinux
dianders has joined #armlinux
NishanthMenon has joined #armlinux
cengiz_io has joined #armlinux
broonie has joined #armlinux
jamestperk has joined #armlinux
mturquette has joined #armlinux
dtor has joined #armlinux
unixsmurf has joined #armlinux
sjg1 has joined #armlinux
robclark has joined #armlinux
robher has joined #armlinux
steev has joined #armlinux
roxell has joined #armlinux
linusw_ has joined #armlinux
nohit has joined #armlinux
khilman has joined #armlinux
tfiga has joined #armlinux
zx2c4 has joined #armlinux
arnd has joined #armlinux
ccaione has joined #armlinux
<arnd>
broonie: I was browsing through some legacy platform_data code and looked at sound/soc/codecs/cs42l56.c, where I noticed that a) nothing defines the platform_data and b) the DT parsing is broken because the pdata gets zeroed in the end using "cs42l56->pdata = *pdata;"
<arnd>
it's been like this since the driver got merged in 2014. Do you prefer to fix or remove such drivers?
<javierm>
geertu: I thought that was the case but just today gregkh pointed me that a Fixes: tag is not enough and you need an Cc: <stable...>
<javierm>
and for tags I usually put in the order that happened. So if I have Reported-by: followed by my S-o-B and then get a Reviewed-by and Acked-by, I put them after my S-o-B in the order that happened
<javierm>
always thought that was the convention
<broonie>
arnd: Seems better to fix the driver, no?
CrashTestDummy has joined #armlinux
<broonie>
It's not doing any harm.
<arnd>
broonie: ok, will do. it just seems unlikely that after seven years of being broken in mainline anyone will miss it
<broonie>
arnd: Well, we didn't force all 32 bit platforms to use device tree.
<arnd>
broonie: I'd like to remove the platform_data probing side from some the drivers that have no remaining board files though, would you take those patches for asoc codecs?
<broonie>
I'm not really enthusiastic about doing that sort of thing just for the sake of it TBH.
<broonie>
If it's getting in someone's way then sure but why bother otherwise?
<arnd>
this is part of looking at the legacy gpio interface users. The problem with leaving the platform data in place is that any possible out-of-tree users are still broken if we change them
<broonie>
Converting to gpiod and invalidating the platform data would be an example of having the platform data getting in the way though.
<broonie>
It's just the patches that delete the platform data with nothing else going on that I'm talking about
<arnd>
Right. I was trying to partition the work a bit by starting with the removal, but I'll stop that for the codecs then and send you just the trivial fix for the broken DT probe
<arnd>
the only other patch I have so far is for the omap2 framebuffer driver, but at least in that case we are sure that nobody is able to use the pdata based probe any more
mag has quit [Remote host closed the connection]
mag has joined #armlinux
elastic_dog has quit [Ping timeout: 268 seconds]
elastic_dog has joined #armlinux
torez has joined #armlinux
System_Error has quit [Remote host closed the connection]
System_Error has joined #armlinux
tudorel has quit [Quit: tudorel]
alexels has quit [Quit: WeeChat 3.3]
apritzel_ has quit [Ping timeout: 264 seconds]
Pali has joined #armlinux
ardb has quit [Ping timeout: 244 seconds]
ardb has joined #armlinux
Amit_T has joined #armlinux
ukleinek has quit [Quit: reboot]
ukleinek has joined #armlinux
torez has quit [Ping timeout: 245 seconds]
torez has joined #armlinux
prabhakarlad has quit [Quit: Client closed]
Amit_T has quit [Quit: Leaving]
jlinton has quit [Quit: Client closed]
jlinton has joined #armlinux
iivanov__ has quit [Quit: Leaving...]
Turingtoast has quit [Quit: My iMac has gone to sleep. ZZZzzz…]
amitk_ has joined #armlinux
amitk has quit [Ping timeout: 240 seconds]
apritzel has joined #armlinux
prabhakarlad has joined #armlinux
jlinton has quit [Quit: Client closed]
torez has quit [Quit: torez]
prabhakarlad has quit [Quit: Client closed]
prabhakarlad has joined #armlinux
djrscally has quit [Quit: Konversation terminated!]