On 2/15/2016 4:45 AM, Simon Horman wrote:

quoted

quoted

quoted

quoted

quoted

quoted

quoted

Now that Maxim Integrated MAX3355 'extcon' driver and device tree bindings
are upstream along with the 'extcon' hook  in the Renesas USBHS driver, we
can add  MAX3355 device node to  the Porter device tree and properly refer
to it from the USBHS node instead of using  "renesas,enable-gpio" property
which only allowed checking the USB ID pin  at the driver probe time;  now
this  pin's  state can be monitored dynamically!

Signed-off-by: Sergei Shtylyov <redacted>

I tested this patch on Porter, but interrupt of "usb-otg" didn't happened
even if I connected/disconnected a usb convert cable (usb micro-ab to A).
Remarks: I confirmed that the usb convert cable worked correctly on other
board.

     I guess I have the same cable

     No, I have mini-AB connector here and hence my cable is mini-AB to A.

I checked Porter specification and then I found 2 specifications exist...

    I've looked thru my schematics files and found 3: Rev. 1.1 Update Version
2, "PORTER_2" Rev. 3.0 Preliminary, and "PORTER_C" Rev. 3.0 Release version!
I'm not sure which of the last 2 should be considered final...

    Now I've found 5 versions of the Porter schematics... :-)

quoted

quoted

  Porter "B" : mini AB connecter and Max3355 is on the board.
  Porter "C" : micro AB connect and Max3355 is not on the board. (Select
host/peripheral by a jumper.)

    Yeah, seeing JP13 in the PORTER_C schematics.

quoted

I guess that your current target is "B".

    If not earlier. It actually has "Henninger B REV2" printed on it and
"Porter PCB 1 R3" on a pasted sticker.

quoted

If so, I think this patch is acceptable.

    I'm not so sure now...

    My boss told me PORTER_C is a final version, so this patch isn't needed.
But then another patch is needed instead: to remove "renesas,enable-gpio"
prop from the HS-USB device node...

I am marking this patch as "Rejected" in patchwork.
If changes are required in this area please either
resubmit this patch or post a different one.

    Posted a different one on Saturday.

Thanks.

MBR, Sergei