Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
From:   Peter Chen <peter.chen@nxp.com>
To:     Jisheng Zhang <Jisheng.Zhang@synaptics.com>,
        "yossim@codeaurora.org" <yossim@codeaurora.org>,
        Stephen Boyd <sboyd@codeaurora.org>
CC:     Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        "linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        linux-arm-kernel <linux-arm-kernel@lists.infradead.org>
Subject: RE: [PATCH resend] usb: chipidea: Don't select EXTCON
Thread-Topic: [PATCH resend] usb: chipidea: Don't select EXTCON
Thread-Index: AQHT17S23GjAgRoHOUOzcNsdlHwgEKQI32DwgAB9iQCAAAaj0IAEl7MAgALC9JA=
Date:   Wed, 25 Apr 2018 01:47:42 +0000
Message-ID: <HE1PR04MB14509924E386B58D5A95C4D38B8F0@HE1PR04MB1450.eurprd04.prod.outlook.com>
References: <20180419160144.0412a510@xhacker.debian>
        <HE1PR04MB1450E11255E0828180B3E8188BB40@HE1PR04MB1450.eurprd04.prod.outlook.com>
        <20180420170126.732e093b@xhacker.debian>
        <VI1PR04MB14558BCFB82A06C92F7AA0CA8BB40@VI1PR04MB1455.eurprd04.prod.outlook.com>
 <20180423153310.47a20e6d@xhacker.debian>
In-Reply-To: <20180423153310.47a20e6d@xhacker.debian>
Accept-Language: en-US
Content-Language: en-US
received-spf: None (protection.outlook.com: nxp.com does not designate
 permitted sender hosts)
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 6157e891-ed81-45eb-a6f3-08d5aa4e88df
X-MS-Exchange-CrossTenant-originalarrivaltime: 25 Apr 2018 01:47:42.1603
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR04MB3018
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

=20
> > >
> > > The patch doesn't remove extcon support from chipidea driver.
> > > I just want to not select EXTCON unconditionally, but let the users
> > > choose. If the users need extcon, they could enable EXTCON
> > > themselves
> > >
> > > I just searched all the dts in arch/arm/boot/dts and
> > > arch/arm64/boot/dts only the four dts give extcon phandle to chipidea=
 host, other
> users don't make use of it:
> > >
> > > arch/arm64/boot/dts/qcom/apq8016-sbc.dtsi
> > >
> > > arch/arm/boot/dts/qcom-apq8074-dragonboard.dts
> > >
> > > arch/arm/boot/dts/qcom-msm8974-fairphone-fp2.dts
> > >
> > > arch/arm/boot/dts/qcom-msm8974-sony-xperia-castor.dts
> > >
> >
> > I see, but I do not want to break msm platforms. You may try to create
> > Glue driver Kconfig entry for chipidea like dwc3, and let msm depends o=
n EXTCON.
>=20
> Got your points. Since multi_v7_defconfig has selected EXTCON, and
> EXTCON_USB_GPIO(which depends on EXTCON) is enabled in arm64 defconfig,
> so what about:
>=20
> enable EXTCON explicitly in arm64 defconfig?
> then add this patch?
>=20

I am not sure if Qualcomm platforms use these, add Qualcomm guys to confirm=
 it.

Peter