ChanServ changed the topic of #linux-msm to:
<bamse> sboyd: seems like linux-arm-msm didn't reliably deliver emails to me last week :/
<sboyd> bamse: hooray for mailing lists
<bamse> sboyd: feel free to add my Reviewed-by on both patches, that looks good
<sboyd> ok cool I'll do that now
<sboyd> thanks
<bamse> maybe we don't need 32 bits each in struct gdsc?
<sboyd> if it's bits then a u8 will be fine
<sboyd> but it's some count? I guess that's at most 8-bits wide
<bamse> the last two are 4 bits wide, per the shift...checking the upper one
<sboyd> they look like they're all 4
<sboyd> how about sending a followup to combine them? they're all next to eachother so it seems like we'd be better off using a u16 and shifting and packing it
<bamse> yes, they are 4 bits each
<bamse> in the current scheme we can do like downstream and change one of the properties without touching the others
<bamse> i like that
<bamse> on the other hand, that creates the silly dependency on you knowing what the defaults are when you read or write that
<bamse> i'm fine either way...and i'm perfectly fine with us just merging the patch
<sboyd> I'm just going to merge it and it can be cleaned up later
<bamse> leaving an opportunity for someone to come in and save some space in the vmlinux
<sboyd> I'll drop a note on the list
<bamse> sounds good
jn has quit [Read error: Connection reset by peer]
jn has joined #linux-msm
marvin24_ has joined #linux-msm
marvin24 has quit [Ping timeout: 480 seconds]
pevik_ has joined #linux-msm
Marijn[m] has quit [Quit: Bridge terminating on SIGTERM]
aka_[m] has quit [Quit: Bridge terminating on SIGTERM]
wfranken[m] has quit [Quit: Bridge terminating on SIGTERM]
ivoszbg[m] has quit [Quit: Bridge terminating on SIGTERM]
Tooniis[m] has quit []
ungeskriptet[m] has quit [Quit: Bridge terminating on SIGTERM]
maxim[m] has quit [Quit: Bridge terminating on SIGTERM]
vknecht[m] has quit [Quit: Bridge terminating on SIGTERM]
JoelSelvaraj[m] has quit [Quit: Bridge terminating on SIGTERM]
RayyanAnsariOld[m] has quit [Quit: Bridge terminating on SIGTERM]
robbbbbb[m] has quit [Quit: Bridge terminating on SIGTERM]
aedancullen has quit []
fevv8[m] has quit [Quit: Bridge terminating on SIGTERM]
julian[m]1 has quit [Quit: Bridge terminating on SIGTERM]
Newbyte has quit [Quit: Bridge terminating on SIGTERM]
undev[m] has quit [Quit: Bridge terminating on SIGTERM]
DavidHeidelberg[m] has quit []
ajhalaney[m] has quit []
xerpi[m] has quit []
go4godvin has quit [Quit: Bridge terminating on SIGTERM]
travmurav[m] has quit [Quit: Bridge terminating on SIGTERM]
Bazsalanszky[m] has quit [Quit: Bridge terminating on SIGTERM]
x[m]1 has quit []
kholk[m] has quit [Quit: Bridge terminating on SIGTERM]
MatrixTravelerbot[m] has quit [Max SendQ exceeded]
IvanBelokobylskiy[m] has quit [Quit: Bridge terminating on SIGTERM]
MartinBotka[m] has quit [Quit: Bridge terminating on SIGTERM]
Mis012[m] has quit []
z3ntu has quit [Quit: Bridge terminating on SIGTERM]
alikateshethey[m] has quit [Quit: Bridge terminating on SIGTERM]
cmeerw[m] has quit []
konradybcio has quit [Quit: Bridge terminating on SIGTERM]
JulianGoldsmith[m] has quit [Quit: Bridge terminating on SIGTERM]
jojo_autoboy[m] has quit [Quit: Bridge terminating on SIGTERM]
minecrell[m] has quit [Quit: Bridge terminating on SIGTERM]
ichernev[m] has quit [Quit: Bridge terminating on SIGTERM]
aat596 has quit [Quit: Bridge terminating on SIGTERM]
jhovold has joined #linux-msm
AntoniAloyTorrens[m] has joined #linux-msm
AntoniAloyTorrens[m] has quit [Remote host closed the connection]
AntoniAloyTorrens[m] has joined #linux-msm
pespin has joined #linux-msm
aedancullen has joined #linux-msm
ajhalaney[m] has joined #linux-msm
aka_[m] has joined #linux-msm
alikateshethey[m] has joined #linux-msm
Bazsalanszky[m] has joined #linux-msm
cmeerw[m] has joined #linux-msm
fevv8[m] has joined #linux-msm
frytaped1 has joined #linux-msm
ichernev[m] has joined #linux-msm
IvanBelokobylskiy[m] has joined #linux-msm
ivoszbg[m] has joined #linux-msm
JoelSelvaraj[m] has joined #linux-msm
jojo_autoboy[m] has joined #linux-msm
JulianGoldsmith[m] has joined #linux-msm
julian[m]1 has joined #linux-msm
kholk[m] has joined #linux-msm
konradybcio has joined #linux-msm
z3ntu has joined #linux-msm
Marijn[m] has joined #linux-msm
maxim[m] has joined #linux-msm
minecrell[m] has joined #linux-msm
Mis012[m] has joined #linux-msm
Newbyte has joined #linux-msm
DavidHeidelberg[m] has joined #linux-msm
RayyanAnsariOld[m] has joined #linux-msm
MartinBotka[m] has joined #linux-msm
robbbbbb[m] has joined #linux-msm
x[m] has joined #linux-msm
Tooniis[m] has joined #linux-msm
travmurav[m] has joined #linux-msm
undev[m] has joined #linux-msm
MatrixTravelerbot[m] has joined #linux-msm
vknecht[m] has joined #linux-msm
ungeskriptet[m] has joined #linux-msm
wfranken[m] has joined #linux-msm
xerpi[m] has joined #linux-msm
pevik has quit [Server closed connection]
pevik has joined #linux-msm
frytaped1 is now known as go4godvin
<Mis012[m]> huh, this is weird... I have a clock driver, it appears under `/sys/devices/platform/soc/10ac008.ssc_ahb_slave/5e20000.clock-controller/`, nothing in dmesg...
<Mis012[m]> but the clocks don't want to appear under /sys/kernel/debug/clk for some reason?
svarbanov has quit [Remote host closed the connection]
svarbanov has joined #linux-msm
svarbanov has quit [Ping timeout: 480 seconds]
<Mis012[m]> I fail to see how this saves power... 🤨
<Mis012[m]> hi-Z would probably have the least leakage?
<Mis012[m]> / # cat /sys/kernel/debug/devices_deferred
<Mis012[m]> 5eb7000.i2c platform: supplier 5e20000.clock-controller not ready
<Mis012[m]> so it's "not ready"
<Mis012[m]> whatever that means
<ajhalaney[m]> In my experience that means a dependency isn't up yet for whatever reason
<ajhalaney[m]> mostly have seen that when initramfs is missing some driver, may or may not help you out
<Mis012[m]> well, debugfs is able to tell me why the i2c controller got deferred, so if the clock controller was deferred then it should presumably be listed next to it
<Mis012[m]> hmkay, so my probe function isn't getting called...
<Mis012[m]> sounds deferred to me :/
<Mis012[m]> sounds like it should be in the list
<Mis012[m]> uhm, this is embarrassing...
<Mis012[m]> seems I managed to loose the KConfig and Makefile entries when rabasing
svarbanov has joined #linux-msm
mort_ has quit [Server closed connection]
mort_ has joined #linux-msm
deathmist has quit [Server closed connection]
deathmist has joined #linux-msm
pespin has quit [Remote host closed the connection]
jhovold has quit [Ping timeout: 480 seconds]
pevik_ has quit [Remote host closed the connection]
<calebccff> bamse: I read through the replies here https://lore.kernel.org/linux-arm-msm/Yhft4zNcbD3ojN6i@builder.lan/ but there doesn't seem to be a consensus on what format I should use / if I should credit Dan
<calebccff> do you have any preferences?
<bamse> calebccff: i'm not sure what Dan is suggesting...will answer him
<bamse> calebccff: sorry got a littl ebit side tracked on the path there...
<bamse> calebccff: not sure if Lee will pick the patches, with or without the tags, or wait for a conclusion in the discussion first...
<calebccff> bamse: ahaha, yeah I'm happy to wait a while, it's an interesting discussion
<bamse> calebccff: it would be nice if we had a "I-recognize: Jane Dough"...because there's quite some difference between suggesting things that can be improved and putting you reviewed-by stamp on something to document for all eternity that you think it's goot shit<tm>
<calebccff> lol, yeah some kind of tag to acknowledge bug fixes discovered by others would be good
jojo_autoboy[m] has quit [Server closed connection]
jojo_autoboy[m] has joined #linux-msm