<mmind00>
ayaka: it depends ... for a lot of boards it is enough to bring the reset pin into a known state (which those pinctrl entries do)
<mmind00>
ayaka: on all boards I have (firefly, popmetal, miqi) it is enough to at least make the the usb host enumerate devices connected on boot
cnxsoft has quit [Quit: cnxsoft]
dlezcano has quit [Ping timeout: 258 seconds]
Aussie_matt has joined #linux-rockchip
erc has quit []
aalm has quit [Ping timeout: 276 seconds]
libv_ has joined #linux-rockchip
libv has quit [Ping timeout: 240 seconds]
cnxsoft has joined #linux-rockchip
libv has joined #linux-rockchip
cnxsoft has quit [Quit: cnxsoft]
libv_ has quit [Ping timeout: 276 seconds]
Aussie_matt has quit [Remote host closed the connection]
dlezcano has joined #linux-rockchip
libv has quit [Ping timeout: 264 seconds]
libv has joined #linux-rockchip
dlezcano has quit [Ping timeout: 250 seconds]
<ayaka>
mmind00, ok I need to read the code again, I just miss where it is implement
<ayaka>
Oh, maybe I mistake what it does, it won't actually does a reset, but just configure that pin to the state we want
<ayaka>
so I don't see something like request gpio in dwc2 driver, am I?
<mmind00>
ayaka: correct ... right now we only have the pinctrl that puts the reset gpio into a known state
<ayaka>
mmind00, so it is necessary to put that into &usb_host1, the work would be done in &pinctrl?
<ayaka>
is it necessary
<ayaka>
anyway, the pwrseq-simple is really a good idea, so just fix what I want, I want to move the idea from rk3288 to a exynos 4412 board of mine
<mmind00>
the position is arguable :-) ... in the usbhost you at least have that connection that the pin is usb-related and the pin is only set on probe of the usb host
<mmind00>
ayaka: if you need the pin to be set _before_ probe, just change the state from "default" to "init"
<mmind00>
ayaka: see the rk3288 tsadc for reference
<ayaka>
mmind00, wow, I don't know there is a such tip, I think it must be implement in the of driver. Thank you for telling me that
<mmind00>
ayaka: I think it was actually done, because the tsadc needed it ... if you want to look it up, it's in drivers/base/dd.c in the function really_probe()