Re: [PATCH V3 2/6] dt-bindings: timestamp: Add Tegra234 support

[Jon Hunter]

On 13/03/2023 23:49, Dipen Patel wrote:
> On 3/13/23 2:57 PM, Linus Walleij wrote:
> > Hi Dipen,
> >
> > thanks for maintaining HTE!
> >
> > On Fri, Mar 10, 2023 at 8:06 PM Dipen Patel <dipenp@xxxxxxxxxx> wrote:
> >
> > > -  nvidia,slices:
> > > -     $ref: /schemas/types.yaml#/definitions/uint32
> >
> > I would not delete this, just mark it deprecated.
> >
> > nvidia,slices:
> >      $ref: /schemas/types.yaml#/definitions/uint32
> >      deprecated: true
> >
> > (And remove it from required, of course)
> >
> > This way you do not need to explain about why it was
> > deleted, it's just deprecated, which is fine.
>
> Great suggestion, thanks, will make changes in the next patch.


When you deprecate it, please make this a separate patch because it is 
separate from adding Tegra234 support. Similarly you will want to have a 
separate patch to remove support of the property from the driver as well.

Jon

--
nvpublic