Re: [PATCH V1 1/2] dt-bindings: phy: Add QMP UFS PHY comptible for SC7280

[Nitin Rawat]

Hi Dmitry and Vinod,

Thanks for the review. I taken care of all the comments in latest 
patchset I posted.

Regards,
Nitin


On 8/17/2023 12:19 PM, Vinod Koul wrote:
> On 16-08-23, 23:31, Dmitry Baryshkov wrote:
> > On Wed, 16 Aug 2023 at 18:50, Nitin Rawat <quic_nitirawa@xxxxxxxxxxx> wrote:
> > > Document the QMP UFS PHY compatible for SC7280.
> > >
> > > Signed-off-by: Nitin Rawat <quic_nitirawa@xxxxxxxxxxx>
> > > ---
> > >   .../devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml      | 2 ++
> > >   1 file changed, 2 insertions(+)
> > >
> > > diff --git a/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml b/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml
> > > index d981d77e82e4..ad78da9c2c1a 100644
> > > --- a/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml
> > > +++ b/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml
> > > @@ -31,6 +31,7 @@ properties:
> > >         - qcom,sm8350-qmp-ufs-phy
> > >         - qcom,sm8450-qmp-ufs-phy
> > >         - qcom,sm8550-qmp-ufs-phy
> > > +      - qcom,sc7280-qmp-ufs-phy
> > >
> > >     reg:
> > >       maxItems: 1
> > > @@ -110,6 +111,7 @@ allOf:
> > >                 - qcom,sm8250-qmp-ufs-phy
> > >                 - qcom,sm8350-qmp-ufs-phy
> > >                 - qcom,sm8550-qmp-ufs-phy
> > > +              - qcom,sc7280-qmp-ufs-phy
> >
> > Please keep both lists sorted.
>
> Yes please