<drewfustini>
*are the patches that touch arch/riscv files in a series like...
Stat_headcrabbed has joined #riscv
<alexghiti>
I think we should, I used to not do it when it was very straightforward but now, I'll do it systematically since I find that it actually helps the submitter *and* the maintainer that will merge it :)
<drewfustini>
Okay, thanks.
<drewfustini>
I've been curating the thead stuff in patchwork for myself since I got an account. I've been checking patchwork daily, and a few days ago I asked palmer if it should set patches in series that only touch files in other arch directories to 'handled elsewhere'.
<drewfustini>
Then I started getting curious about what actually needs to be done for these cross-tree / cross-arch large patch series. It seems mostly to be a matter of palmer or you providing a ach/nack for the patches in a big series that actually touch arch/riscv files
JRepin has quit [Read error: Connection reset by peer]
JRepin has joined #riscv
itrsea has quit [Remote host closed the connection]
<conchuod>
I stopped bothering to set the exact status stuff fairly quickly to be honest, and just went to either marking series that I knew palmer wouldn't care about to handled elsewhere
<conchuod>
including all soc related things
<conchuod>
iow, if it doesnt go in palmer's pr or he doesnt need to ack it -> handled elsewhere
<palmer>
Ya, for me it’s basically just binary in patchwork — if it’s on the list it’s still on my TODO list, otherwise it’s not
<palmer>
I don’t even really know how to find stuff that’s not on the default list ;)
Stat_headcrabbed has quit [Quit: Stat_headcrabbed]
<conchuod>
and tbh, I'd likely have marked the series you're talking about handled elsewhere Drew, cos it appears to be entirely mechanical changes
<palmer>
Ya, that’s probably the way to go with the big stuff.
<conchuod>
Also anything entirely KVM I also mark handled elsewhere
<palmer>
Unless for some reason the guy wants an Ack, but the last one Drew pointed me to looked like someone who wouldn’t want an ack
<palmer>
Ya, Anup does KVM somewhere else
<conchuod>
Some riscv-specific drivers I leave in, like the timer, where the maintainers are not super responsive and it's likely as not to go in Palmer's tree
<conchuod>
But anything soc-specific I get rid of whenever I look through patchwork
prpplague has joined #riscv
Stat_headcrabbed has joined #riscv
sm2n has quit [Server closed connection]
sm2n has joined #riscv
Stat_headcrabbed has quit [Quit: Stat_headcrabbed]
<drewfustini>
conchuod: thanks, do you mean leave "riscv: Use GCR.U timer device as clocksource" as "New"?
<drewfustini>
interesting that drivers/clocksource/timer-clint.c seems to just have Daniel and tglx
<conchuod>
Ye, I stopped bothering with any detailed status stuff because it went out of sync real fast
<conchuod>
I ended up only either applying handled elsewhere or changes requested, depending.
<drewfustini>
ah okay, so: New, Handled Elsewhere, or Changes Requested
<drewfustini>
which tree would you expect devicetree/bindings/timer/mips,p8700-gcru.yaml to go through? Daniel?
<conchuod>
ye, there's a clocksource tree that's daniel -> tglx -> linus
<conchuod>
fwiw, I think the other statuses are still useful but if everyone isn't keeping them up to date, they become incorrect fairly quickly
xypron has quit [Server closed connection]
xypron has joined #riscv
xypron has quit [Changing host]
xypron has joined #riscv
xbjfk has quit [Server closed connection]
xbjfk has joined #riscv
<drewfustini>
palmer: By TODO list, do you mean the 'Action required' filter? Do you want me to change Delegate to you when it looks like it needs your Ack or Review?
<conchuod>
Pretty much everything needs his ack or review, since otherwise it can get changed requested or handled elsewhere
ttydes has quit [Ping timeout: 240 seconds]
<drewfustini>
Okay, and those needing ack or review should just stay as 'New' with no Delegate?
itrsea has quit [Remote host closed the connection]
ttydes has joined #riscv
itrsea has joined #riscv
<conchuod>
You can assign it if you want, I did for a while and stopped bothering
Finde has quit [Read error: Connection reset by peer]
Finde has joined #riscv
<drewfustini>
Okay, I guess it doesn't matter if palmer is just looking at the default view which doesn't filter based on delegate, it only filters based on 'action required'
coldfeet has quit [Quit: Lost terminal]
wingsorc has joined #riscv
Andre_Z has joined #riscv
Andre_Z has quit [Quit: Leaving.]
jobol has quit [Quit: Leaving]
cousteau has joined #riscv
cousteau has quit [Quit: Client closed]
cousteau has joined #riscv
ZipCPU has quit [Read error: Connection reset by peer]
<palmer>
So I’m happy to look at any list, I just want one list. Right now that’s just the default list, but for GCC we have a non-default list and that works fine — just as long as everyone knows what the list is
cousteau has quit [Quit: Client closed]
agentcasey has joined #riscv
vagrantc has quit [Quit: leaving]
ZipCPU has joined #riscv
<drewfustini>
I think the current one which is just "Action required" makes sense. If we filter by delegate then we won't have a unified view