Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <a3d7c16f-fe87-9c04-5f92-42f10597df6d@nvidia.com>
Date:   Tue, 23 May 2023 14:42:52 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.11.0
Subject: Re: [PATCH v3] gpio: tegra186: Check GPIO pin permission before
 access.
Content-Language: en-US
To:     Bartosz Golaszewski <brgl@bgdev.pl>
Cc:     Linus Walleij <linus.walleij@linaro.org>,
        Prathamesh Shete <pshete@nvidia.com>, thierry.reding@gmail.com,
        bgolaszewski@baylibre.com, linux-gpio@vger.kernel.org,
        linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org,
        smangipudi@nvidia.com, kyarlagadda@nvidia.com,
        Manish Bhardwaj <mbhardwaj@nvidia.com>
References: <Yz62XmiH8YG3Dtsf@orome> <20221007055936.5446-1-pshete@nvidia.com>
 <CACRpkdYkJdZ67kyTnDg3xFzO8MJhC0nHK98O+KJwCLBqV_5f-Q@mail.gmail.com>
 <1de5b7fb-a39e-183a-1407-7d6489f706b4@nvidia.com>
 <CAMRc=MdCiieaYR3BeATm+2o8UDz+9D5vs=dTNDAavi19BmKZ8w@mail.gmail.com>
From:   Jon Hunter <jonathanh@nvidia.com>
In-Reply-To: <CAMRc=MdCiieaYR3BeATm+2o8UDz+9D5vs=dTNDAavi19BmKZ8w@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?R09FNGZiTTZqRzlwaHh4cStOZDRuOTVyQlM5SE84b1RuWEpEQklIQTI2R2tR?=
 =?utf-8?B?ai9HMlpoU3JXQlJ3UkNqbnBFYUdncytEanVWYlhhbjNnclFYUWE5TlltYUVX?=
 =?utf-8?B?NjVkQ1hDNmRndjMxeXk1YldMNUpZNUhnNGsxakQrdW1WczNjTzFDZmxtazM3?=
 =?utf-8?B?Wmc1MERFOXIrT2tQbzZUZ0o4aVJ1N25FU3NCUXJBVHdGWXJDZHNEVzd3dHYz?=
 =?utf-8?B?bmU5SWFUYmk5bmNKOE9CY01URWdPRUNNcEMwMzJ1UmRZK1U5aHk3YnJiNGl5?=
 =?utf-8?B?NkpIeTdlcnEyU3BWZ29KTStmalcvU2RVOFliMXJJcnZVZmo3WmdqTE9qbXN0?=
 =?utf-8?B?a3R1cS8zUVd4SXJta2hTc3FGNElrSWxhbXNJcW41aGxqSksvK2llR3VCQ3hr?=
 =?utf-8?B?elg3TGFBZmxnUm1NN0E3RnlCcG5xY29pMVpidUZVdUdRVVhwN1NEbHZicGcv?=
 =?utf-8?B?KzJJYU96Y2Vjdm5IMWR3NjRKYURKOFZ5aitta09BNE5wQUNCaVJHU1R6VjI0?=
 =?utf-8?B?N1lKSXZjRC9rU3ZhcGNhYjBmSXpLcndYL1krNzVPNmZWUnErRTZDb2NqSk53?=
 =?utf-8?B?VVlHdU5wYm9vQVVwemxQRFNFb3prTmt1WVpIMThKZ1JUd2EvcGViOUhIYW84?=
 =?utf-8?B?SGpVWlg2MndST1RqeW9pNWp2SUl0SXE2TEx4RG1sdFpXeTRHWHd1MS9oRksy?=
 =?utf-8?B?aVJabnYrSkY4anFWTm1seG90VmZkd0xUMTZ6Z0xWK2VBcWRueG56eDlBWTVq?=
 =?utf-8?B?QU0zUW9RVHEzT0d6alMyVlNlbjY2TUpSY3B0THNrbFczVVhUdDkybGwvVzZF?=
 =?utf-8?B?OFRNTjA0MmJiZm43UnBEWlRqRW1qNXBTWUVSS1plTW90RkVKT2tEMFMxSGJ0?=
 =?utf-8?B?UG90WFVOSjNMUFZnVFJrL3o0UWVoZ2JiNldXRHNMME9RSzdtWFFHdDdVZ25u?=
 =?utf-8?B?Nm1MVzc2NHN0Mkh0OUZpS3NRUFJZcjJXWjMwc2dCNG84YmtLRGo2MXVpVjV6?=
 =?utf-8?B?bGhDWEV4VnFlYzdaS3NVZUI3WFluOXFFYnAvTkw1aUMySzFXMHNsNHhQS3lD?=
 =?utf-8?B?dzBkZTlMQzBaMXE2TXhPWEtPOTNpS28rZ0txNDlmcm8zZ090UHQrK0pmUmdJ?=
 =?utf-8?B?WFFyWXRFc2tuNnkxUlB4Wm0xUDVsMHA3QkxBT0dsQWcwbmlLR1ZYaWhncTNB?=
 =?utf-8?B?cHNxdGZkWE1YTXg4RXFPMStKT1dNL1dRT01yNUxPZ0hhTENhTXh3N21aUmM5?=
 =?utf-8?B?aE9YM3JsNDJvd3IvcUJoN1FTODhmcU1aUUkxajczVW8wSXg2S2hDMzFQMjVR?=
 =?utf-8?B?MUVMNzlseStJQlhGTUREVFpKUVMwWXhObjVCS1dTbEdYS29BSVFWT3lwWU1Z?=
 =?utf-8?B?aUxqZkxaK0ZrTjJqMjBTb2hmNldvaHpoUDNvbS9XZ1E5V3NoTEY1K0c5REww?=
 =?utf-8?B?dG9ZQlZxUEZqVnRJZVM4VnNMYWhJVXZXbWtMWG8xYXBCWWgvWWVwMnd3cjZX?=
 =?utf-8?B?M1o1WDd4NzFXTElWQ2ZpK29XcHVjdDkwZW9udVlKb3ZxejFjTEI1S2NSb0JX?=
 =?utf-8?B?djBLaWdVRlhYV3poRUF2cy8yYUsxVndWSUJ1REVQTWYyYkhoQlFqUURCYzdB?=
 =?utf-8?B?ZTNkMUVBdVdDWEZTN2N2OVRCaXcyb2kwKytNZkh0SFYxT3FKWmVsbFp2SDJI?=
 =?utf-8?B?SWx1SFN2ZnVJUGtxaWhMVVR2dkFCb2VhVlRyQk81am5LcVU4RmpTNEhRY2JQ?=
 =?utf-8?B?SkkrZjNVcjlqUlpQM0VpdjV3eEpGMExNUFFmcVZVdG1mOUhGNGZyVEF0MXFI?=
 =?utf-8?B?blFuYm1UUVhsRCs3YXBOaVFURCt6M1V6cVNObWFrRHBXVzNtc2kyUXFBcWRL?=
 =?utf-8?B?R2R6cVJScS9HWlAwTU94UkpaQXozS3V2dGd3d1dNcitQOFhOV05tWXFHY1Qr?=
 =?utf-8?B?ZHhWU0ttL1R4aURRMjZHRmkzUktxY2p2NzZ1cUtkdjFjY3UvZWUvRnNwUCtQ?=
 =?utf-8?B?Vlg2bXdNTGczdFhSdFZ1NkF6cFZTcWZWTyt1b3FuNzlNK29CL1BVVGFUYjUw?=
 =?utf-8?B?YXdidmtQdFY2bzRjOXBBODNsR0o4dmh6eXdkT1Zkb3NnNFJQeUsxVlE2Nzc2?=
 =?utf-8?B?d2VadlB4czZtVE9vbis3VlJjMm42Q2hKUkxsdGxDdThuZHJRNXkxT2VvMTdK?=
 =?utf-8?B?SDRpcFdBdGVyREZqMnZWL1l0UEtodkJRRnN6YnlEKzRCKzJHSXhwQ0U1MHg5?=
 =?utf-8?B?QU44NEJVNkI1cTJEdys4U3ZEa2FBPT0=?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: fba45945-7a39-4e09-ba5c-08db5b939fb3
X-MS-Exchange-CrossTenant-AuthSource: CO6PR12MB5444.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 May 2023 13:43:00.3694
 (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: qk0a2dDSn9JV69gWhE1KESIa+B8WE8tDkWGJCHycWcuNo78VAeIONO+Nuc+0+xszwNiLm56YGMAjllu5G9q3Zg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR12MB5422
X-Spam-Status: No, score=-1.2 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,
        NICE_REPLY_A,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,
        SPF_NONE,T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=no
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 23/05/2023 10:17, Bartosz Golaszewski wrote:
> On Tue, May 23, 2023 at 8:22 AM Jon Hunter <jonathanh@nvidia.com> wrote:
>>
>>
>> On 17/10/2022 10:31, Linus Walleij wrote:
>>> On Fri, Oct 7, 2022 at 7:59 AM Prathamesh Shete <pshete@nvidia.com> wrote:
>>>
>>>> This change checks if we have the necessary permission to
>>>> access the GPIO. For devices that have support for virtualisation
>>>> we need to check both the TEGRA186_GPIO_VM_REG and the
>>>> TEGRA186_GPIO_SCR_REG registers. For device that do not have
>>>> virtualisation support for GPIOs we only need to check the
>>>> TEGRA186_GPIO_SCR_REG register.
>>>>
>>>> Signed-off-by: Manish Bhardwaj <mbhardwaj@nvidia.com>
>>>> Signed-off-by: Prathamesh Shete <pshete@nvidia.com>
>>>
>>> Very nice patch!
>>> Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
>>
>>
>> I did not see this anywhere in the mainline/next. However, I also
>> noticed that we don't have the correct email address for Bartosz (again).
>>
> 
> I have only ever changed my address in MAINTAINERS once, so "again" is
> not really the right term. And scripts/get_maintainer.pl should be
> used anyway every time when submitting patches.

Sorry I meant that WE did not send to the correct email AGAIN and not 
that you updated your email address :-)

Thanks
Jon

-- 
nvpublic