Re: [PATCH v9 3/6] dt-bindings: reset: Add Delta TN48M

[Robert Marko <robert.marko@xxxxxxxxxx>]

On Fri, Dec 10, 2021 at 8:08 AM Lee Jones <lee.jones@xxxxxxxxxx> wrote:
>
> On Thu, 09 Dec 2021, Robert Marko wrote:
>
> > On Thu, Dec 9, 2021 at 10:40 AM Philipp Zabel <p.zabel@xxxxxxxxxxxxxx> wrote:
> > >
> > > Hi Robert,
> > >
> > > On Wed, 2021-12-01 at 22:28 +0100, Robert Marko wrote:
> > > > On Tue, Nov 9, 2021 at 12:32 PM Robert Marko <robert.marko@xxxxxxxxxx> wrote:
> > > > >
> > > > > Add header for the Delta TN48M CPLD provided
> > > > > resets.
> > > > >
> > > > > Signed-off-by: Robert Marko <robert.marko@xxxxxxxxxx>
> > > > > ---
> > > > >  include/dt-bindings/reset/delta,tn48m-reset.h | 20 +++++++++++++++++++
> > > > >  1 file changed, 20 insertions(+)
> > > > >  create mode 100644 include/dt-bindings/reset/delta,tn48m-reset.h
> > > > >
> > > > > diff --git a/include/dt-bindings/reset/delta,tn48m-reset.h b/include/dt-bindings/reset/delta,tn48m-reset.h
> > > > > new file mode 100644
> > > > > index 000000000000..d4e9ed12de3e
> > > > > --- /dev/null
> > > > > +++ b/include/dt-bindings/reset/delta,tn48m-reset.h
> > > > > @@ -0,0 +1,20 @@
> > > > > +/* SPDX-License-Identifier: GPL-2.0-only */
> > > > > +/*
> > > > > + * Delta TN48M CPLD GPIO driver
> > > > > + *
> > > > > + * Copyright (C) 2021 Sartura Ltd.
> > > > > + *
> > > > > + * Author: Robert Marko <robert.marko@xxxxxxxxxx>
> > > > > + */
> > > > > +
> > > > > +#ifndef _DT_BINDINGS_RESET_TN48M_H
> > > > > +#define _DT_BINDINGS_RESET_TN48M_H
> > > > > +
> > > > > +#define CPU_88F7040_RESET      0
> > > > > +#define CPU_88F6820_RESET      1
> > > > > +#define MAC_98DX3265_RESET     2
> > > > > +#define PHY_88E1680_RESET      3
> > > > > +#define PHY_88E1512_RESET      4
> > > > > +#define POE_RESET              5
> > > > > +
> > > > > +#endif /* _DT_BINDINGS_RESET_TN48M_H */
> > > > >
> > > >
> > > > Does anybody have any comments on the patch as the reset driver got reviewed and
> > > > the bindings have not?
> > >
> > > Not much to review here, I can't tell if the indices are correct.
> > >
> > > Acked-by: Philipp Zabel <p.zabel@xxxxxxxxxxxxxx>
> > >
> > > To be merged with the rest of the series. Or do you want me to pick up
> > > the reset parts individually? In that case you'd have to split out the
> > > reset bindings into a separate patch.
> >
> > Thanks,
> > It has to go with the rest of the series as it all depends on the MFD.
> >
> > We are just waiting for the MFD dt-bindings to be reviewed.
>
> We need Rob to review the set.  Then I'll happily take it.

Hi,

Rob, can you please review the dt-bindings?
Everything else is ready to go.

Regards,
Robert
>
> --
> Lee Jones [李琼斯]
> Senior Technical Lead - Developer Services
> Linaro.org │ Open source software for Arm SoCs
> Follow Linaro: Facebook | Twitter | Blog



-- 
Robert Marko
Staff Embedded Linux Engineer
Sartura Ltd.
Lendavska ulica 16a
10000 Zagreb, Croatia
Email: robert.marko@xxxxxxxxxx
Web: www.sartura.hr