Re: [PATCH] ARM: dts: porter: add MAX3355 support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

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 <sergei.shtylyov@xxxxxxxxxxxxxxxxxx>

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... :-)

  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.

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.

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




[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux