Re: [PATCH v2] ARM: dts: samsung: exynos4412-midas: add USB connector and USB OTG

[Krzysztof Kozlowski]

On 25/07/2023 23:22, Henrik Grimler wrote:
>>> MHL works reliably if cable is attached before boot, but if cable is
>>> attached on already running device it sometimes work after detaching
>>> and attaching cable a few times.  On cable attach, when it does not
>>> work, we get:
>>>
>>>   [  252.831734] max77693-muic max77693-muic: external connector is detached(chg_type:0x0, prev_chg_type:0x0)
>>>   [  253.733916] max77693-muic max77693-muic: external connector is attached(chg_type:0x1, prev_chg_type:0x1)
>>>   [  253.735232] max77693-muic max77693-muic: CONTROL1 : 0x09, CONTROL2 : 0x04, state : attached
>>>   [  254.691877] i2c i2c-15: sendbytes: NAK bailout.
>>>   [  254.692150] sii9234 15-0039: writebm:  TPI[0x3d] <- 0x3e
>>>   [  255.984336] sii9234 15-0039: RGND is not 1k
>>>   [  256.068970] sii9234 15-0039: RSEN_HIGH without RGND_1K
>>>   [  256.104457] sii9234 15-0039: discovery failed, no power for MHL?
>>>   [  256.118272] i2c i2c-15: sendbytes: NAK bailout.
>>>   [  256.118539] sii9234 15-0039: writebm:  TPI[0x3d] <- 0x3e
>>>   [  256.375966] sii9234 15-0039: RSEN_HIGH without RGND_1K
>>>   [  256.411561] sii9234 15-0039: discovery failed, no power for MHL?
>>>
>>> while when it works:
>>>
>>>   [  175.348307] max77693-muic max77693-muic: external connector is detached(chg_type:0x1, prev_chg_type:0x0)
>>>   [  175.349576] max77693-muic max77693-muic: CONTROL1 : 0x00, CONTROL2 : 0x01, state : detached
>>>   [  179.304373] dwc2 12480000.usb: new device is full-speed
>>>   [  179.305920] max77693-muic max77693-muic: external connector is detached(chg_type:0x0, prev_chg_type:0x0)
>>>   [  180.205735] max77693-muic max77693-muic: external connector is attached(chg_type:0x1, prev_chg_type:0x1)
>>>   [  180.207211] max77693-muic max77693-muic: CONTROL1 : 0x09, CONTROL2 : 0x04, state : attached
>>>   [  180.309727] i2c i2c-15: sendbytes: NAK bailout.
>>>   [  180.309996] sii9234 15-0039: writebm:  TPI[0x3d] <- 0x3e
>>>
>>> Not sure if issue here is in muic driver or sii9234 driver.
>>>
>>> OTG still does not seem to work, the muic driver detects (only) a
>>> detach when OTG cable is attached:
>>>
>>>   max77693-muic max77693-muic: external connector is detached(chg_type:0x0, prev_chg_type:0x0)
>>>
>>> Same happened when otg cable was attached before these hsotg changes
>>> as well. I suppose issue here is with max77693-muic driver.
>>>
>>
>> Thanks for testing. I miss here one important thing - did anything
>> regressed? Is MHL behaving not worse than before? OTG, as non-working,
>> matters less in such case.
> 
> No regressions as far as I can tell, only improvements.  We want the
> muic node and mhl_to_muic/muic_to_mhl ports so that muic can notify
> the mhl chip of cable attachment/detachment (with requires an
> additional patch to the sii9234 driver that I will send soon [1]).
> The alternative, having the mhl chip always on, has caused issues with
> some GUIs [2].

Awesome, thank you!

Best regards,
Krzysztof