RE: [PATCH v4 3/5] dt-bindings: usb: fusb302: remove max-sink-* properties

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

 



> -----Original Message-----
> From: Guenter Roeck [mailto:linux@xxxxxxxxxxxx]
> Sent: 2018年4月14日 0:04
> To: Jun Li <jun.li@xxxxxxx>
> Cc: gregkh@xxxxxxxxxxxxxxxxxxx; robh+dt@xxxxxxxxxx;
> heikki.krogerus@xxxxxxxxxxxxxxx; hdegoede@xxxxxxxxxx; rmfrfs@xxxxxxxxx;
> yueyao.zhu@xxxxxxxxx; linux-usb@xxxxxxxxxxxxxxx; dl-linux-imx
> <linux-imx@xxxxxxx>
> 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 <robh@xxxxxxxxxx>
> > Reviewed-by: Hans de Goede <hdegoede@xxxxxxxxxx>
> > Signed-off-by: Li Jun <jun.li@xxxxxxx>
> > ---
> >  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?

Thanks
Jun
> 
> >  - fcs,operating-sink-microwatt :
> >  			   Minimum amount of power accepted from a sink
> >  			   when negotiating
> > --
> > 2.7.4
> >
?韬{.n?????%??檩??w?{.n???{炳???骅w*jg????????G??⒏⒎?:+v????????????"??????



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux