Danct12 has quit [resistance.oftc.net coherence.oftc.net]
Danct12 has joined #linux-msm
<aka_[m]>
lumag: regarding your comment on xo for 8976 this is way we do lately and I would love to not pollute board dts but sadly that's way someone made us do it
<aka_[m]>
You can find it same way on sm6115/sm6375/sm8550
<aka_[m]>
I remember it was said xo fixed crystal should not be in soc dts as it's board specific component and not soc part
<lumag>
ah, that part
<lumag>
ack.
* lumag
hates this kind of duplication of standard items
<lumag>
Then the whole xo_board and sleep_clk should go to the board dtsi. Because other boards might have it implemented in a different way, like using an external clock source, etc.
<bamse>
lumag: except that it won't be implemented in different way between different boards...
<bamse>
lumag: the xo_board i mean...
<bamse>
lumag: someone might pick another vendor/part, but there's no much flexibility in the rate (which is the only aspect the dts covers)
<lumag>
bamse, yep. It's that either the whole XO is fixed (including the rate), or the whole xo_board should go to soc-board.dtsi.
<lumag>
Because one can imagine a board, using external source for the XO (especially if it's a networked SoC)
<bamse>
lumag: to my knowledge it's fixed per soc
<lumag>
bamse, to my knowledge too
<minecrell>
lumag: related: https://lore.kernel.org/linux-arm-msm/Y1FTJgloEi5ag2%2Fj@gerhold.net/ I think the end result was that qru1000 has the xo-board & sleep-clk fully in the board DTS now, some SoCs have the clock-frequency in board DTS mess, and the older SoCs have the nodes fully in the SoC dtsi
<minecrell>
so the usual inconsistent mess when conventions are not properly documented and people don't quite agree :S
<lumag>
minecrell, as you probably see, I'd prefer to have xo-board & sleep in SoC.dtsi
<minecrell>
lumag: I'm fine with that as well, but it seems like Krzysztof doesn't/didn't want that
<lumag>
Yep
<lumag>
minecrell, my second preference would be to have both clocks in board.dtsi
<lumag>
but that's my 2c., so it is up to krzk and bamse
<krzk>
The clock is not part of SoC, it is external component, so it should be in the board. To avoid duplication - as it is common input to the SoC - my recommendation was to store almost everything in the SoC and only the frequency outside to indicate, that it has to be intentionally filled in (just like it has to be populated on the board).
<lumag>
krzk, ack, it just seems strange to me to have the xo_board in the dtsi then. But I would not argue here, you definitely know more (and can estimate if it's good or not).
<lumag>
Excuse me for grumbling :D
pespin has quit [Remote host closed the connection]
pespin has joined #linux-msm
pespin has quit [Remote host closed the connection]