Message-ID: <798dd219-f4aa-39b3-2368-7a6d072b6de2@nvidia.com>
Date:   Mon, 13 Mar 2023 16:49:48 -0700
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0)
 Gecko/20100101 Thunderbird/102.8.0
Subject: Re: [PATCH V3 2/6] dt-bindings: timestamp: Add Tegra234 support
Content-Language: en-US
To:     Linus Walleij <linus.walleij@linaro.org>
Cc:     thierry.reding@gmail.com, jonathanh@nvidia.com,
        linux-kernel@vger.kernel.org, linux-tegra@vger.kernel.org,
        linux-gpio@vger.kernel.org, devicetree@vger.kernel.org,
        linux-doc@vger.kernel.org, robh+dt@kernel.org,
        timestamp@lists.linux.dev, krzysztof.kozlowski+dt@linaro.org,
        brgl@bgdev.pl, corbet@lwn.net, gregkh@linuxfoundation.org
References: <20230310190634.5053-1-dipenp@nvidia.com>
 <20230310190634.5053-3-dipenp@nvidia.com>
 <CACRpkdYmC=eLiWPPPRG7DYBmQfPNCN4pcDhzTCO=hkQDx2HoGw@mail.gmail.com>
From:   Dipen Patel <dipenp@nvidia.com>
In-Reply-To: <CACRpkdYmC=eLiWPPPRG7DYBmQfPNCN4pcDhzTCO=hkQDx2HoGw@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?U25qN0ZnS1ZPdmRRMjdFbEtWUGVpdXVrT2k3TUJkRDdNV2lFczRVcjRLWmdv?=
 =?utf-8?B?Vmt4SHp3T1FseTVrRm50ajJwRTJBMStZbHczMmk1SDYxUkJNUHNnZXhDZFpw?=
 =?utf-8?B?ZHNkSzlmaUV0S3luanQ4dDBTWSsvOVN6WkEwaTNQUkNZMmw2VzFJV2F3RGlO?=
 =?utf-8?B?S3oxUEhremljeExRcEp3SnIrRE5ZbE90UW1rZlFITDQxZytmWEVEdUdSSDly?=
 =?utf-8?B?eVZlWmhGTHhxTWJDR3JsZGw3NTVRaUlYZktxTVQ3aFhsR2ZqZS9zMXBIYnBW?=
 =?utf-8?B?Vk1Hd1FiY0Zpc0k1NUJZMnM3YWFzRGp4Mk1zaEtaZFc1Zi81bHF6dXhIVFpR?=
 =?utf-8?B?YXByd0s2eDJOUE9teWNuR282ZnVHeWQ1a0dGTjB2a09SdjJrdmdXVVAwQWIw?=
 =?utf-8?B?RmozU2VDZGQrU1pJYXdXUmpza0JKNjVJNW5UYlcrL3o3N2xXOG56akNSemRW?=
 =?utf-8?B?Y282NXZ5ajVvbzJ3TTNqWldNMUw2eUhsdS96cXJPdVRRZFc3bEx4aG9pWnc5?=
 =?utf-8?B?b1FNV2w0SmtQZXJrL29MTnlocXcvZFhLRG9VR1NHb0RLQVZwdjh0c21uQVo5?=
 =?utf-8?B?MlpYVXJ0NUtVZkhzK0thcE5PQkNNZGt4bnJjVjBMcGQzTnk2NDgrSEMwWitD?=
 =?utf-8?B?cjZRZEZ3cXVXVFo4Z08rMzdGYkt5UE1YUTRuck1GeWNiZkR5UWpsamh1ZWRz?=
 =?utf-8?B?SThobXVYUWQ2MDNzL1RMa3V5UFFXekF4S0kxbVNOL1FQbDE0OG9SRllyc1ds?=
 =?utf-8?B?aElJQ2xwSHUvUmFaVHgyY09CRXpzWnJyYmpZU2lJYTFYYWhWcDhVS3BYQmRI?=
 =?utf-8?B?UFFHQTZYL01tM3p5MWhjc2JNeXRWSHZPRm91SDdqSnV3cGx5b2dyYWN3RUxV?=
 =?utf-8?B?ZVBRcy9EejE0T3Zta1hOVzZIUnlLVU5tSlhhazh5VzQ0QVRGZFBteXIrMVV0?=
 =?utf-8?B?bkV1Q2krL1N6cU9RMkN2YmRuUTY5U3BSQjQvNWtqTUJkZ1pKVUdudmlMREkw?=
 =?utf-8?B?RlFKZGsvTG52MXJOR05HVkNaZGtZTGZwODJ0U0krUUpDZ05IMWdOVUdzZGRU?=
 =?utf-8?B?cmF6b1pnUjFYWW5wR2JyS0hvQWlOTTNxV3laTWRZb2EwdHQ3b1BQUitEOHlI?=
 =?utf-8?B?NTB6Vk82YzE2S1MrSXdNdENxL1Frcm52UHkwRUx1SEJsMWhmdDAyeXNrMVJU?=
 =?utf-8?B?R0s3TG1zNmM4U2IxU0NmUjlKTm5jOXg5UzdFakhicGZqSHNIUVdSbkM2L3I0?=
 =?utf-8?B?U2hmT1Bpcy91SGg3UjJvZXBSR2R6aDV4MXovMVpZRUNLcFg2WmRxQWxhNTVM?=
 =?utf-8?B?RXpyVzEwZUIvdDNmUzRpZTFwR1g5KzArRlNxdGhtemppeHhmVmxDSDZwMTgy?=
 =?utf-8?B?ZlZGNEptRjZjODVsbG5wcTNRbE51U1pZRmtWaGRUbVpsRmNzWTFiOEprVVE4?=
 =?utf-8?B?dnhzWXRnUHg3VFlCQjVOaUFlYWhaZTZERmplUmloc2w0UDRmT2l4OHFVU2g3?=
 =?utf-8?B?K3A5S1Q3V2JBZStieVlEOFVHVE9rTEZIUkg0SnFuTU9PVFBjM3lneUx0Skt4?=
 =?utf-8?B?WnBEdkJ4MW5taEtSb0xwaHFJUDBLYVZTYkVXTm5wN2NMTXJXVFYxZ3RUS1VE?=
 =?utf-8?B?TnZGOWo3c2RCOHdHaHpzdS9hOXFlYW1wYzdCLzRFdEM3QWJuSzlBK3pBRGRq?=
 =?utf-8?B?OXI5aVV1ZmpVdE5xamxMc2U5S0MrdEFwaXdLL1FYNmI2bFhVVFRNbkR6eS9S?=
 =?utf-8?B?MFhZb2psNG92c1Jzcmo4em5GdFdmc2tKbVh3cjZZLytNejg3L3R0SHhFZW9N?=
 =?utf-8?B?LzNNbGtPeWt5Rnh0QUNqZWxLRm9NUXl2RGF3MU11TThIUTJPclZXV1FVNTRZ?=
 =?utf-8?B?VmZGM01oR1R6dDRQdTFUSGdwa2Z2QStWTDczOXQ2TldRUCswcTU5NUN6bkZR?=
 =?utf-8?B?WXhNZWJqL3h2MHR0RHlXR3Faa0x4Mzl0Y2phdSs2Vyt2M2NHWEVNNlpSaEUx?=
 =?utf-8?B?REkvUXNhbVhDVDg0UW5TTHRVVko4TXJSUzBtbVZBT3RUbVpqMEhVZUhaaStZ?=
 =?utf-8?B?WU54TFNQTTJSWmJ0TFhoRWNaenhzakdmT0VoSFphcFRBSVNjZFNJeVRuOGdw?=
 =?utf-8?Q?qSj8JZEV7Oeh9+ydb/w4N7c53?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 90c6a1df-ff2a-498e-9aae-08db241da3d3
X-MS-Exchange-CrossTenant-AuthSource: IA1PR12MB6604.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Mar 2023 23:49:52.8949
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: Zsqv0oYHytIWJoLRG8GQp/PwjrlXdOiiSnp99jzoP3f1X6NRukKhX6ExdlOKnIUVKLFgREISWlSrz+nmxNAuUg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR12MB6496
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

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@nvidia.com> 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.
> 
> Yours,
> Linus Walleij