On 04/13/2018 04:31 PM, Jun Li wrote:

-----Original Message-----
From: Guenter Roeck [mailto:li...@roeck-us.net]
Sent: 2018年4月14日 0:04
To: Jun Li <jun...@nxp.com>
Cc: gre...@linuxfoundation.org; robh...@kernel.org;
heikki.kroge...@linux.intel.com; hdego...@redhat.com; rmf...@gmail.com;
yueyao....@gmail.com; linux-usb@vger.kernel.org; dl-linux-imx
Subject: Re: [PATCH v4 3/5] dt-bindings: usb: fusb302: remove max-sink-*

On Fri, Apr 13, 2018 at 10:18:43PM +0800, Li Jun wrote:
Remove max-sink-* properties since they are deprecated.

Reviewed-by: Rob Herring <r...@kernel.org>
Reviewed-by: Hans de Goede <hdego...@redhat.com>
Signed-off-by: Li Jun <jun...@nxp.com>
  Documentation/devicetree/bindings/usb/fcs,fusb302.txt | 6 ------
  1 file changed, 6 deletions(-)

diff --git a/Documentation/devicetree/bindings/usb/fcs,fusb302.txt
index 472facf..6087dc7 100644
--- a/Documentation/devicetree/bindings/usb/fcs,fusb302.txt
+++ b/Documentation/devicetree/bindings/usb/fcs,fusb302.txt
@@ -6,12 +6,6 @@ Required properties :
  - interrupts             : Interrupt specifier

  Optional properties :
-- fcs,max-sink-microvolt : Maximum voltage to negotiate when configured as
-- fcs,max-sink-microamp  : Maximum current to negotiate when configured
as sink
-- fcs,max-sink-microwatt : Maximum power to negotiate when configured as
-                          If this is less then max-sink-microvolt *
-                          max-sink-microamp then the configured current will
-                          be clamped.

Is it acceptable to just remove properties ?

What else you suggest I should do? also add document here to explain why
remove them?

I would suggest to either wait for Rob's feedback or check how other bindings
handle deprecated properties.

To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to