Re: [PATCH v2 2/6] dt-bindings: phy-rockchip-typec: move extcon property to be optional.
On Wed, Mar 7, 2018 at 11:34 AM, Rob Herringwrote: > On Thu, Mar 01, 2018 at 04:25:11PM +0100, Enric Balletbo i Serra wrote: >> The extcon property is used to detect the cable-state but some boards >> just connect the type-c phy to a regular USB-A connector without any >> power-delivery and thus no controller reporting the cable-state. >> So the extcon property is not really a required property, move it to be >> optional instead. >> >> Signed-off-by: Enric Balletbo i Serra >> Reviewed-by: Heiko Stuebner >> --- >> >> Changes in v2: >> - [2/6] Rewrite the justification as suggested by Heiko Stuebner. >> >> Documentation/devicetree/bindings/phy/phy-rockchip-typec.txt | 2 ++ >> 1 file changed, 2 insertions(+) > > And extcon should be deprecated IMO. > > Reviewed-by: Rob Herring Hello Rob, I'm currently learning a bit about USB and extcon. Could you provide some background on why you think extcon should be deprecated? What do you propose we replace it with?
Re: [PATCH v2 2/6] dt-bindings: phy-rockchip-typec: move extcon property to be optional.
On Wed, Mar 7, 2018 at 11:34 AM, Rob Herring wrote: > On Thu, Mar 01, 2018 at 04:25:11PM +0100, Enric Balletbo i Serra wrote: >> The extcon property is used to detect the cable-state but some boards >> just connect the type-c phy to a regular USB-A connector without any >> power-delivery and thus no controller reporting the cable-state. >> So the extcon property is not really a required property, move it to be >> optional instead. >> >> Signed-off-by: Enric Balletbo i Serra >> Reviewed-by: Heiko Stuebner >> --- >> >> Changes in v2: >> - [2/6] Rewrite the justification as suggested by Heiko Stuebner. >> >> Documentation/devicetree/bindings/phy/phy-rockchip-typec.txt | 2 ++ >> 1 file changed, 2 insertions(+) > > And extcon should be deprecated IMO. > > Reviewed-by: Rob Herring Hello Rob, I'm currently learning a bit about USB and extcon. Could you provide some background on why you think extcon should be deprecated? What do you propose we replace it with?
Re: [PATCH v2 2/6] dt-bindings: phy-rockchip-typec: move extcon property to be optional.
On Thu, Mar 01, 2018 at 04:25:11PM +0100, Enric Balletbo i Serra wrote: > The extcon property is used to detect the cable-state but some boards > just connect the type-c phy to a regular USB-A connector without any > power-delivery and thus no controller reporting the cable-state. > So the extcon property is not really a required property, move it to be > optional instead. > > Signed-off-by: Enric Balletbo i Serra> Reviewed-by: Heiko Stuebner > --- > > Changes in v2: > - [2/6] Rewrite the justification as suggested by Heiko Stuebner. > > Documentation/devicetree/bindings/phy/phy-rockchip-typec.txt | 2 ++ > 1 file changed, 2 insertions(+) And extcon should be deprecated IMO. Reviewed-by: Rob Herring
Re: [PATCH v2 2/6] dt-bindings: phy-rockchip-typec: move extcon property to be optional.
On Thu, Mar 01, 2018 at 04:25:11PM +0100, Enric Balletbo i Serra wrote: > The extcon property is used to detect the cable-state but some boards > just connect the type-c phy to a regular USB-A connector without any > power-delivery and thus no controller reporting the cable-state. > So the extcon property is not really a required property, move it to be > optional instead. > > Signed-off-by: Enric Balletbo i Serra > Reviewed-by: Heiko Stuebner > --- > > Changes in v2: > - [2/6] Rewrite the justification as suggested by Heiko Stuebner. > > Documentation/devicetree/bindings/phy/phy-rockchip-typec.txt | 2 ++ > 1 file changed, 2 insertions(+) And extcon should be deprecated IMO. Reviewed-by: Rob Herring