Hi Guenter, > -----Original Message----- > From: Guenter Roeck [mailto:[email protected]] On Behalf Of Guenter Roeck > Sent: 2018年4月14日 11:03 > To: Jun Li <[email protected]> > Cc: [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; dl-linux-imx > <[email protected]> > Subject: Re: [PATCH v4 3/5] dt-bindings: usb: fusb302: remove max-sink-* > properties > > On 04/13/2018 04:31 PM, Jun Li wrote: > > > >> -----Original Message----- > >> From: Guenter Roeck [mailto:[email protected]] > >> Sent: 2018年4月14日 0:04 > >> To: Jun Li <[email protected]> > >> Cc: [email protected]; [email protected]; > >> [email protected]; [email protected]; > >> [email protected]; [email protected]; [email protected]; > >> dl-linux-imx <[email protected]> > >> Subject: Re: [PATCH v4 3/5] dt-bindings: usb: fusb302: remove > >> max-sink-* properties > >> > >> 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 <[email protected]> > >>> Reviewed-by: Hans de Goede <[email protected]> > >>> Signed-off-by: Li Jun <[email protected]> > >>> --- > >>> Documentation/devicetree/bindings/usb/fcs,fusb302.txt | 6 ------ > >>> 1 file changed, 6 deletions(-) > >>> > >>> diff --git a/Documentation/devicetree/bindings/usb/fcs,fusb302.txt > >> b/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 > >> sink > >>> -- fcs,max-sink-microamp : Maximum current to negotiate when > >>> configured > >> as sink > >>> -- fcs,max-sink-microwatt : Maximum power to negotiate when > >>> configured as > >> sink > >>> - 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.
Rob already reviewed this patch and gave his reviewed-by[1], you can see it in the patch tags. [1]https://www.spinics.net/lists/devicetree/msg221601.html Thanks Jun > > Guenter
