<palmer>
yep, I'd guess that Rob says to keep that all together
<conchuod>
Thanks :)
<palmer>
looks like Rob and Krzyszotf are on the To, so they should see it
<palmer>
they're usually pretty fast about this stuff and pretty easy to sync up with about what tree things target
<conchuod>
Yup, was just wondering if you were okay with it before even asking them
<palmer>
I'm OK if they are ;)
<palmer>
it's generally easier to keep the bindings with the DT files, as splitting them up just leads to a cylce of warnings in the various sub-trees
<palmer>
and I don't see any reason to change the status quo here
<conchuod>
Warnings make my heart sink at this point!
ghee has joined #riscv
<palmer>
yep, and when it's just due to splitting the patch sets it's just a headache
<palmer>
best to check, though, as sometimes there's other reasons for splitting stuff up that is more important than the warnings
<palmer>
what I do is keep around a tree (called "not-for-next") that contains all the stuff that's necessary to make my trees build cleanly, and then I have that merged in locally for testing
<palmer>
it's a bit of a headache, but I have at least one patch every merge window so I guess I'm used to it
<conchuod>
I figured it is bindings + drivers (and maybe the dts unless there's potential conflicts)
<palmer>
so I guess maybe Arnd knows better than I do here, I'd generally err towards keeping bindings+dts+drivers all together -- that can be a bit of a headache, though, as it starts to touch a few trees
<palmer>
IIUC that's one of the things the SOC tree is aimed at fixing, but then there's more trees involved ;)
<arnd>
palmer: in the soc tree, I keep the dts files separate from everything else, mainly to avoid merge conflicts
<conchuod>
^ ye, that's what's "making" me take my dts stuff myself.
<palmer>
OK, so you take the DTs on that branch without the bindings yet?
<conchuod>
I am confused now.. I don't take bindings at all as they go to the driver trees.
<palmer>
sorry, I thought you were talking about taking bindings that your DTs depend on?
<conchuod>
I was asking specificaly about taking the riscv bindings like the board and soc definitions in Documentation/devicetree/bindings/riscv/microchip.yaml
<palmer>
yes, and the dependency there is the DT, right?
<conchuod>
Ye, the dts needs them to not generate warnings
<palmer>
ya, so I think that makes sense to take together
<conchuod>
Right. Sorry if I asked that all confusingly
<palmer>
but for drivers there's three trees: the bindings, the DTS (which depend on the bindings), and the drivers (which depend on the bindings too)
<conchuod>
Bindings go to the driver trees in that case
<palmer>
ya, I think that's a reasonable way to go -- but then the DTS needs to go somewhere too, so it's either conflicts (likely if there's multiple drivers in flight) or warnings (if I take the DTS before the bindings) or some sort of shared tag
<conchuod>
I think having the warnings there is unavoidable. I think the warnings there is normal
<palmer>
ya, I think most folks take the warnings
<conchuod>
Shared tags for dts would just be a drag on people
<conchuod>
I think the reasonable thing to do when checking for new warnings in that situation is to check against -next
<conchuod>
At least, that is what I've been doing.
<palmer>
ya, it's just sort of a manual process at that point
<palmer>
no big deal, just one more thing to trip folks up
<conchuod>
On the subject of warnings, 1 left in -next...
<conchuod>
soonTM
ghee has quit [Quit: EOF]
matt__ has joined #riscv
frkzoid has quit [Ping timeout: 244 seconds]
frkzoid has joined #riscv
matt__ has quit [Ping timeout: 255 seconds]
davidlt has quit [Ping timeout: 260 seconds]
dor has joined #riscv
matt__ has joined #riscv
KombuchaKip has quit [Quit: Leaving.]
frkzoid has quit [Ping timeout: 260 seconds]
frkzoid has joined #riscv
matt__ has quit [Ping timeout: 248 seconds]
GenTooMan has quit [Ping timeout: 255 seconds]
EchelonX has quit [Quit: Leaving]
GenTooMan has joined #riscv
dramforever_ has joined #riscv
aerkiaga has quit [Remote host closed the connection]
dramforever__ has quit [Ping timeout: 252 seconds]
dramforever_ has quit [Remote host closed the connection]
dramforever_ has joined #riscv
dor has quit [Remote host closed the connection]
matt__ has joined #riscv
frkzoid has quit [Ping timeout: 255 seconds]
brazuca has joined #riscv
dramforever__ has joined #riscv
dramforever_ has quit [Read error: Connection reset by peer]
zjason has quit [Read error: Connection reset by peer]
zjason has joined #riscv
Andre_H has quit [Quit: Leaving.]
crabbedhaloablut has quit [Ping timeout: 268 seconds]