Message-ID: <5159fc03-e39d-b784-e852-8a4057ec5354@nvidia.com>
Date:   Tue, 21 Mar 2023 07:46:50 +0000
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.8.0
Subject: Re: [PATCH v2 2/2] mfd: tps6586x: register restart handler
Content-Language: en-US
To:     Benjamin Bara <bbara93@gmail.com>, lee@kernel.org
Cc:     dmitry.osipenko@collabora.com, treding@nvidia.com,
        richard.leitner@linux.dev, benjamin.bara@skidata.com,
        linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org
References: <20230320220345.1463687-1-bbara93@gmail.com>
 <20230320220345.1463687-3-bbara93@gmail.com>
From:   Jon Hunter <jonathanh@nvidia.com>
In-Reply-To: <20230320220345.1463687-3-bbara93@gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?bXBXR3Y4ZVQzUVFRN3RpZkgwaDhqVGZyWlo5VCt3d0ViRy80Wm9JcFZOWjFq?=
 =?utf-8?B?NmJ3T3pDZlliTk5rSlk0NW5FaElkMkZMNVNkaUlmdFhOSUhLTGVkSFdhUk5W?=
 =?utf-8?B?eVBJeGtDSXJVcm0vNnlwc3UxS0EwMExuQVhuaFM0SW0xb0FocTNyOW1hcTho?=
 =?utf-8?B?Yk1PQk1vL1ZFc2pCMi9VRkZhL0pSR00vcGtLWG15dHk3UDB5VjRZekRxdmFp?=
 =?utf-8?B?ZkF1TE9IbmRHVDQycGxocGhwTFArVmNDcmNuK3FaV2tKYjFvN2JtcmppQjY1?=
 =?utf-8?B?L3ZLb3c0UGFka3pSaDJYV0lzSGRXWEZ3NzlUeW9idHZPK1NvcXAvT1p3U1o1?=
 =?utf-8?B?MHVPS2l5Y0J6QVd6THRETFFtNzFhY0tybjRaejNMaUtQc1RXK0xVZno3ajQy?=
 =?utf-8?B?dXFmbXUzOVlOcTZmZWpPL1k3eFRFTzFMWGM5QUlsQ2ZZMVFXSEx6Z1hxUXQz?=
 =?utf-8?B?M3JwNElpZ0JMOHlORFU0a0huNFpKSVlNWFc3MERCNktJRTJVRTdRU3pnOVF2?=
 =?utf-8?B?c0FLbmRyVWZEOTdaajN4cjhPaTQ5UWVTZmZoaWlndVJRZ2tVU2Jka0lpaStI?=
 =?utf-8?B?M3JaWTVTM3lEMkg2K1pUYUttdGU1WjNtNG5xZTFpd1JPRCtacWZmVGx6azNK?=
 =?utf-8?B?bHpZdllxVG1HYUdhRGg5Q3hFUk5hUXdWeExmRDh0TXUvTXEyRzlTd21ITzhK?=
 =?utf-8?B?b3poUVVKUGFCQ0k0VFhiTkFWaVVBbHBYNGNnZ2VPVmxhVlJGMDNCaVBpYWR6?=
 =?utf-8?B?Z3loS0gvWnVkOVZzbmpVNTR1ekpXK1dQL1VXQjZSTFBTVFJZVVNSMXhMNVZk?=
 =?utf-8?B?TWVEdDZTRVRGZUZTcGliTUQybmN6Y3JzQ24va2k2OHhnQWZqVEVlSHpDZG03?=
 =?utf-8?B?N2FaRXNpZ3NIeEJkWFpiSXArMlFiM1pWcWJzdzJYcDEwYTRlMHlBTHJza2Zo?=
 =?utf-8?B?N1N2eXF1WXdoSUxmRmp5V2VEVHZuTHRQRXB5Rmg2ZlZLb3pEdGRhWlhrUUNN?=
 =?utf-8?B?OU93Q3pOZmhMZzVPbEx4SWlVdUZuMlhHV3dUTHhWajBySEh3WDJyNlRYTmZO?=
 =?utf-8?B?TGVpTFJxQ1hTM2VBVDJTUGNKYXNkNHU0OEJyZm5LcUw4RWl2SExibnJYRXRZ?=
 =?utf-8?B?aEd6R3BrVnlmQ0krK2VadDBnaWNOcmZCTk04VmRQV1l2QWRqMEJuM3o4SHNZ?=
 =?utf-8?B?MzlaN3hPM0RpdnltbmhFVkhxQjNqajR3SkJQeC9pSU11WEhIemFzOHRoZ204?=
 =?utf-8?B?SE9NUWtoM2xUNUtEY1FHNzBLMndrWFlkNEVNY1liYlc0ZG5LUVM2WXJ2VldQ?=
 =?utf-8?B?UU5UeGMyS1pKakt4UXJzb2JPbVowQjRDUTVDdVV2TC9Ham5hdXZ1bDJkVElR?=
 =?utf-8?B?OVMyM3p3bStuZW5rU1RuUHg1Rzdqc0VOVkJoMjFuYjVLUDdKT2MzQzZicTln?=
 =?utf-8?B?Vk1nTXJibmZzRnV3OVVaR3AzZEQwNW85TEtsdXhCbHBUMFJIVllzSjNBZ0xE?=
 =?utf-8?B?VkdEK3BYY1c5SVczWW9PeXMva05jS2JjODBmVlZ6bnY3MXFDKzFPTUhEbVFk?=
 =?utf-8?B?L21UeGpmUXh1b0dLNlhFSU9LY25QQjVadTRkenJpYW1HNktmelR4L1ZSd3p6?=
 =?utf-8?B?UzlLaVN4WWhYRVk0MVVhNlhhUURRcHBRRzRkQ25ES3BhZE13OTlja3kxa1Y2?=
 =?utf-8?B?aXJXVjM3WEtRWHYwRjM1T2JZRzVtTkVJVE9SSUNpTDE0SUZhdkpVTkNrb0lk?=
 =?utf-8?B?YXd1cS9CM3lpMk5VYjVTdjF6dWtXRlNKUmM0RVptMUYvUWxtTnNFbEZRTkZE?=
 =?utf-8?B?QnhVaWtxKytyQTFWV1N6R1loR2sxVFZLK2FqK01Md2VVanBFQXJUam1Bd3ZK?=
 =?utf-8?B?SndBbHpqU1BoOUgweGxZWi9GOTdlRUY3c0huOEtDR1hIeFZVMkhiYmRTL0N4?=
 =?utf-8?B?WEZ0dmZwQXJab2RxSzZKTi9zekdKQ0xaL0tDakdIS2NEaGVlVEhvYUVOZ1Jr?=
 =?utf-8?B?SEhsU3BDL00rY2FFVEkrWC83N05waXZDV2w0aFZWbXBIRjNBa3ozMTZqWmQw?=
 =?utf-8?B?MnlVS1RRTGFhd1FDOXAvYTlLT056Q082MitJa1lpcVdySDB6bEpmZSt3SndM?=
 =?utf-8?B?SjdJUVFIYzFQMVE3K2RmM1Y3RkNXU08vNTU1eXZvSlB4SkFHSWZjYnJDVWpC?=
 =?utf-8?B?RDJPRnlISXFyTWdWRlVYZmh6Qnc0eXd2K3pSOTd5ZzQ5TVlUNTNVWGZSR2xm?=
 =?utf-8?B?b0h6R3J6cTdMYW9NcDdFZEhVVHR3PT0=?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b538def6-422d-47f5-2c55-08db29e07285
X-MS-Exchange-CrossTenant-AuthSource: CO6PR12MB5444.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 Mar 2023 07:46:57.6769
 (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: sqxpm3xSp7OQaZF3WOLMul+mnk2Gv3b5GolMltfoy6W4J+THqqg23+Sf79Ic6Qhq3GDHpuLP7fz5eTx11bk0jA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA0PR12MB4400
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 20/03/2023 22:03, Benjamin Bara wrote:
> From: Benjamin Bara <benjamin.bara@skidata.com>
> 
> The TPS658629-Q1 (unfortunately the only TPS6586x with public data sheet)
> provides a SOFT RST bit in the SUPPLYENE reg to request a (cold) reboot.
> 
> Use it to implement and register a restart handler.


This does not explain why this is needed and/or why we are adding this.

Jon

-- 
nvpublic