Message-ID: <bceedc91-cccd-258a-1cab-57a79f5a294d@nvidia.com>
Date:   Mon, 13 Mar 2023 17:01:55 -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
From:   Dipen Patel <dipenp@nvidia.com>
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>
 <798dd219-f4aa-39b3-2368-7a6d072b6de2@nvidia.com>
In-Reply-To: <798dd219-f4aa-39b3-2368-7a6d072b6de2@nvidia.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?NmZRanU2UFloVGMyZmJ0dDBkUldySmg0TTNjTkJCRDRNMnprVDYraTVxNERM?=
 =?utf-8?B?VEtCYXBlOTFqTndZdlZLb3JLRW9aMHZUQzJmYVVWcXM1QnpWU0VFSDg0Y1U0?=
 =?utf-8?B?SEZOL3l3STBQS2RXMmtTZVhuWEFrSGh2ODIyd3ZLTVhUSmdCR1JDdHBHWURQ?=
 =?utf-8?B?RmM3RzlFT3JoY2VmKytnU1RFMDB6VU4xZ1VWUDE3RFNQdEVYS2dZUHYwZ3RR?=
 =?utf-8?B?TmIvVzZLa2JJaXYzKzAzS0k4WDhuWi9JYTBya2Zyby93SDVMaG1LZmUzbW9O?=
 =?utf-8?B?Mk1tRGsvNjNkdFZFdW9rUXBnUzFzVCtKMlRDZHB0cnovUGtzWTZaTktqK01D?=
 =?utf-8?B?SDFSRFNHNC8zSEVtZENQc09lTHR1RTA5VTFHRytBbm96dXQ2Q1hzenNXMnBk?=
 =?utf-8?B?SWxQUkprd2x5Wk43NEZZa0dPM1BZbW1EV24yS2NZMC91Z1QxV3NXRWlyK2Nr?=
 =?utf-8?B?bTJwU1BGK1NINnBJZ3Q3YkVBUFV6MldQUERBMEFGc3FuaVlkdVo0MG9wWXZi?=
 =?utf-8?B?UnQ0dVZmN3dYRENRN1hLUDN0YlMxNHUwQTdmK1VUNkZ0M0NIbTdxd3VOY3lT?=
 =?utf-8?B?YXZjT2RKc1JWekxFTlA2aDJ1WUVDYzdrYXlDWDVPYW5SZUtrR0JSMDdxR1RE?=
 =?utf-8?B?Uy9wc1hKTnVvZXFXdGJ1a1ZoRWNxNEFsR3lUNGNmOU00SVNCKy9LZDFOUmx1?=
 =?utf-8?B?SjBjSFdjazdBd3Y5TkNyNXNnc0RKYnErMjNYRldlbUdCbHQwaGtmeU9rVjRZ?=
 =?utf-8?B?SldlMW13a1oxYW1VR0t2YkVQRzZKRUtYczJjN0dNWnNyaWlNbUlHSFh5LzE1?=
 =?utf-8?B?RXVZOHlYVXF3My8ySmtPRElDaENOOGozODhiLzRQQmlQVzFBMXlRNWNTekRr?=
 =?utf-8?B?ZHFLUnZibGFMdVhWSU9mOVpTemIwbWExdjdmRnhNRkl0SFdSS3NwOW5YSEFs?=
 =?utf-8?B?V1VENXpFcE5jY2MrRkkvL2pIRDRVUHFYMU9WZmlRSDJYNHg2cS9rUGlqdGNB?=
 =?utf-8?B?aTVTWjhzRjd4aFhqZlo4dU9DU1REcy9zQ2xwcnFsZHVwcm9pNGgwR0lNS1VQ?=
 =?utf-8?B?eUZNNVZocERmUmR3UnVMOGZmdnEyTHh4ZE83WlFSUlU2WTE3UDZOMTd0Y3pv?=
 =?utf-8?B?L3lkWm5lR2dRM05EK0VBU1kxNVY2Sm40KzBQUUJHc2Z3Qm1VNlVSMkpaNEM2?=
 =?utf-8?B?STRrNkVYZGtTcTc1a1VERmVDUHhRNWlFVlJmTlFoUURtbVl5c05aRGFFRnBl?=
 =?utf-8?B?eWtiQ0lBOHJsanhNLzdhZklsUlpEclFZeUlRNERvMVY5MWJGeFZMZHIwM09O?=
 =?utf-8?B?MUhmR3d1ZFRwdGtXNTdIaThVNmlGRjJYbndEcGhvTWFZeWVtNnZ3WTZXeWk4?=
 =?utf-8?B?dU9LQjJsZnR5WG5FTzJqZnh6dmVKMmIxU1hCNC9DbjU0M29RVDFzMEw4UStZ?=
 =?utf-8?B?Z2Jwb014aE14ckhOQ092MU9YNlhNZE54cHp2THFjckp3a0h4eGtsSzNWMW0r?=
 =?utf-8?B?cUdHK1hmbm9hSERWOWJzNko4UGR6elBTVC9GZ0ZSQTRPNVhrYlBPN0sreStt?=
 =?utf-8?B?dThnNzNJRS9LbTNXdTFhS3hia2dOemVOOEdGOXV2MS9WWCs2Y2NHSTduUW9z?=
 =?utf-8?B?dVdMY1NDamRCVlIrOGRIOERjcnlWOVFsRVN4N0JsNE9nRy9pbk40NUQ4cmdp?=
 =?utf-8?B?WUxhVStCdDlSdzRrWnF5a1BnaC9QcXFpUkVkUU5rMVZXSkpvTXUwZ3lNYU5I?=
 =?utf-8?B?WkNFY2t5Q0x6aDhEc3BkcVNRRkQ3UGlZc0czSm5LRTB4WkJkZjVraHE5MUl0?=
 =?utf-8?B?OFZ4SWZwc3BhRDFLY3I5ZmtkSFhtNTM0VGkxWGFqd0hWNlpIM2xuTUZKVHY0?=
 =?utf-8?B?bTI4Sml6WEpaUkZXd2ZhdHBvelJZS1BJWTcvOXRTbUhpYXdSaWQzUmZQTzdE?=
 =?utf-8?B?N0tBam8wanVlYlpZR3duWkUwenN4bWlqUGNQVmlTd1UzRWx3OXlseS9ZR1dk?=
 =?utf-8?B?eVowK1VqUlRYQzNDalVGR3NLMU5PVDFuMy9NSjd6c3Z4TmpoOVJ4UU8zWkpy?=
 =?utf-8?B?NDhGdjJiZGJPbXhwamJQeDNZYngxZE1vajBMc1ViaUwzZ2tOdHIvRVVibDU3?=
 =?utf-8?Q?HLh/7Ml7jc+znJs8a8m3J8LAh?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 3394760c-94e0-44bb-fa58-08db241f5554
X-MS-Exchange-CrossTenant-AuthSource: IA1PR12MB6604.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Mar 2023 00:01:59.9307
 (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: B86fkyr4sgp18dzk9459ZG9js4H89T+dN/8sS72rbSTYsItwzuB6KGRWuuhwbDI5suBnhA9IsQzP3CL+PQE3jQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH8PR12MB6916
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 3/13/23 4:49 PM, 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@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.

However, as I understood, current point of contention/discussion is addition of the
nvidia,gpio-controller property.

>>
>> Yours,
>> Linus Walleij
>