lopsided98 has quit [Quit: No Ping reply in 180 seconds.]
lopsided98 has joined #linux-rockchip
stikonas has joined #linux-rockchip
warpme_ has joined #linux-rockchip
uis has joined #linux-rockchip
<uis>
What difference between hclk and aclk?
<uis>
And sclk
archetyp has joined #linux-rockchip
wwilly has quit [Quit: Leaving]
<punit>
and pclk
mps has quit [Ping timeout: 264 seconds]
mps has joined #linux-rockchip
mps has quit [Ping timeout: 252 seconds]
mps has joined #linux-rockchip
<mmind00>
don't quote me on actual terms ... but in general hclk/pclk are the ones supplying the circuits of a peripheral, so accessing their registers needs these to be active
stikonas has quit [Remote host closed the connection]
stikonas has joined #linux-rockchip
matthias_bgg has joined #linux-rockchip
wwilly has joined #linux-rockchip
stikonas has quit [Remote host closed the connection]
stikonas has joined #linux-rockchip
<wens>
my guess is a/h/p refer to axi/ahb/apb
<wens>
sclk is special clock or module clock
archetyp has quit [Quit: Leaving]
<mmind00>
what wens said :-)
<mmind00>
mriesch: we're going up to -rc3 now, so I guess for the gmac issue it would definitly be time for some sort of revert
<mmind00>
mriesch: not sure about the pm_runtime details, but can't we just remove the pm_runtime_enable/_disable parts to rely on the stmmac core to do those?
<mmind00>
mriesch: though dwmac_mediatek also does the full enable+get stuff
archetyp has joined #linux-rockchip
matthias_bgg has quit [Ping timeout: 246 seconds]
stikonas has quit [Remote host closed the connection]
stikonas has joined #linux-rockchip
matthias_bgg has joined #linux-rockchip
stikonas has quit [Remote host closed the connection]
stikonas has joined #linux-rockchip
<uis>
If trm says crypto max freq is 150m means max sclk rate is 150?
<diederik>
no idea if it's useful or even relevant, but thought I'd mention it anyway
wwilly has quit [Quit: Leaving]
lurchi_ is now known as lurchi__
sndb has joined #linux-rockchip
<uis>
What purpose of trng_flush?
warpme_ has quit [Quit: Connection closed for inactivity]
yewscion has quit [Remote host closed the connection]
<mriesch>
mmind00: makes sense. due to vacations and the lack of rk3399 hardware (both conditions are planned to change in three weeks or so) i am afraid i cannot be of much help, but let's see...
<mriesch>
mmind00: ...i guess the partial revert you suggested would get rid of the annoying warning, yet should leave the ethernet on the rk3399 intact
<mriesch>
mmind00: if this partial revert works it would be at least a nice intermediate steps. i think we still need to take a look on what is going on here (and maybe compare it to other glue drivers such as the dwmac_mediathek), but the clock is not ticking on that one then
<mriesch>
s/steps/step
<mriesch>
mmind00: btw thanks for the replies to the rfc series. yes, pretty sure that wake-on-lan will break -- at least the internal phy will not be left powered on. not sure what happens to any external phy, though
<mmind00>
uis: I'd guess so, but have never looked too deeply into the crypto module at all
<mriesch>
mmind00: as stated in the cover letter, i could use some pointers here. i'll catch up the work on this cleanup some time in october, if all goes as planned
<mmind00>
mriesch: and yep, that partial revert worked for me on rk3399-puma ... gmac "broken" before and adding the pm_runtime_get_sync + put back in alone (without the enable) did work for me at least
<uis>
Is 100% monobits healthy for TRNG?
mps has quit [Ping timeout: 264 seconds]
mps has joined #linux-rockchip
nullx has joined #linux-rockchip
sndb has quit [Ping timeout: 252 seconds]
warpme_ has joined #linux-rockchip
matthias_bgg has quit [Ping timeout: 264 seconds]
nullx has quit [Ping timeout: 246 seconds]
<uis>
Waait. Is HCLK_CRYPTO_SLV slave AHB for DMA? If so, then why upstream rk3228 crypto enable DMAC1 clk?
sndb has joined #linux-rockchip
<punit>
diederik: That's the commit that broke ethernet on rk3399.
<diederik>
ah ok, thx.
<punit>
mmind00: The partial revert also gets ethernet going on the rockpro64 for me.
<punit>
Looking at my debug prints, the additional reference prevents the device from going through multiple suspend/resume cycles
<punit>
It effectively disables runtime idle
<punit>
... for the gmac
warpme_ has quit [Quit: Connection closed for inactivity]