Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:8 as permitted sender) client-ip=2620:137:e000::3:8;
Message-ID: <e51793c4-e58b-8b4c-e936-2938718ee6ce@nvidia.com>
Date:   Wed, 11 Oct 2023 11:04:23 -0700
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0)
 Gecko/20100101 Thunderbird/102.15.1
Subject: Re: [PATCH v1 0/4] hte: Improve GPIO handling and other cleanups
Content-Language: en-US
To:     Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
        Bartosz Golaszewski <brgl@bgdev.pl>
Cc:     Bartosz Golaszewski <bartosz.golaszewski@linaro.org>,
        linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org,
        timestamp@lists.linux.dev, linux-tegra@vger.kernel.org,
        Linus Walleij <linus.walleij@linaro.org>,
        Thierry Reding <thierry.reding@gmail.com>,
        Jonathan Hunter <jonathanh@nvidia.com>
References: <20231010151709.4104747-1-andriy.shevchenko@linux.intel.com>
 <CAMRc=Mdiuth0=XPBsoStXN81ydfpUfqm1BMc2TZOT4Kmx6jNkA@mail.gmail.com>
 <ZSZzsfaunP5yRvZ4@smile.fi.intel.com>
From:   Dipen Patel <dipenp@nvidia.com>
In-Reply-To: <ZSZzsfaunP5yRvZ4@smile.fi.intel.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?YXpJL1FvUkVwdWUxRGNUMmEraGFoUE1vRWU3KzUyN0FSSmlYWGVSbUJjMEcr?=
 =?utf-8?B?WVI3cFJ6c2Q3R1FYcllCSG0zdjhhMFhVTGZWYjZsd3BGWnE4NHByenhhcWxt?=
 =?utf-8?B?K0FmYzN2VzNOeDY2dkRpOXpIUTBVaSt0ZEFCV3cwbzRxRXJjNHhwR0QyUnZr?=
 =?utf-8?B?OXozMUU4MHBSR0Y0REZudnIyL2lSYXV3MmxZeGdSUm1Wb3VZbXlJQ05KVytt?=
 =?utf-8?B?VGlseGpXSExSczdUcStYMFFnZmhydGZrdzA2OWFVRFZOUEhLSnFUM2g1SFZS?=
 =?utf-8?B?UXdueU9GSElOa3JSMkVaaHNBdlQ5eWZnd3JFcm85ZEs1S1M3OEdBaUNZZEh3?=
 =?utf-8?B?VjJnS080dVpvMUVJQ0dJMkgxU1ljZWswNkk2b3NlVGVOYlJydlZ3dDFvK3dF?=
 =?utf-8?B?MTlMOGRiVWNxSi9xNnhsS3JUMk5SRlFCSE9kS3BFTytlSWc0U081Mkl4czI0?=
 =?utf-8?B?L05mK0FveHFOS2w1ak9JVHpUM3NMZ0tQUWc2S29mKzl0Nzl6YWpIQ1BZd1Nr?=
 =?utf-8?B?WCtXWExuSi8yUHdQdnV5d09lL2dteVE3d1BzNy9KcnVvNnBsMzBodTZxQjZO?=
 =?utf-8?B?aWw1bUJBdk9Jc21qUFpia2s5RHRJL0p1WTJMWlVySVM0M1JiY3BJbVppNWtl?=
 =?utf-8?B?NytVK2NWVHo0NE1sOGpoV3p1dFY1QVN2dEFIVGVVeUN3MW1heDBOMzd0TnFS?=
 =?utf-8?B?SDNndmk1dVVOYXRzaUQ4TkdNN2VjWXJacnVJODlEQ0VYdXJxNjVuMUM3YzZr?=
 =?utf-8?B?aXRrRWNDcTFsWUg4UDVjaWVMSCtNaUZVZCs0ekVSb1orWVN4d0lmY0l4bE5N?=
 =?utf-8?B?bVE1RVIrWm9Md2pLVWtaWlMzZ29odDVnb0J1emFDNjN2THFpelY5cmt2N2FI?=
 =?utf-8?B?RElONVFZc25XYmpGelY1bEovaC9WODh4WnBjZDhrbDFYaTFpS2t1VXNNUEZT?=
 =?utf-8?B?UzNPOFlKeFBYYndCQTNDbEQ4TDl5alIrdXRZWW04ODJCS0l0clNtZTlkQ1Rt?=
 =?utf-8?B?QXQ5T21mdHM3NXcvWU4xZ2c3bnpnekV1Q3BUcEEvOUFzQVdBbUtPOGQ4Z1pi?=
 =?utf-8?B?WG80NHJLQUhmejh4QVhWcytnVTdZRVc1MTFGTWV6MEw0b002czByMGNORDZ4?=
 =?utf-8?B?Q2hwSjBES2ZpR2Zvc1RkLzNWelBCNjduRWtNbFFyY0dYZ1c5cmxXVjRBT0xG?=
 =?utf-8?B?YkJDeEFIU0FWcGZ5TG4rM2lsRUw3ZnVxV0ZoU0U3N29BYnpBNWdadklpckdH?=
 =?utf-8?B?UE96RncrcTloQVVZSUV2YklnUkJVc0t4K0lBZ3AxU3J0RVh6RzV4eWJ5ekRS?=
 =?utf-8?B?bjJ6bmo2NEVGcjNKT2xmUmExTjZhT0o5aUFZSlJGak9NcUkvN01odmowOCtV?=
 =?utf-8?B?WHg3TGFGY3J3Y3d1a2FhQSt1cXhTS0NhVUJ1bTBwQStmaEhpRk1hSlljWGMr?=
 =?utf-8?B?OWk4TUJuNWZwR2hZVnF0TVBmVGtuNnQzamZXZEhMREoyZWtkcTVPUnVHS3Nk?=
 =?utf-8?B?dUFxQ1BxR2d4VUpzRGJZUmFlNjR1L2R6bG9LWTBmZk5meWFaNit0dklkSDZm?=
 =?utf-8?B?NXdLa2dzc3dteCtsK1JNRmNHU0Q3TW1mdXAvcUFZTlBkT1lmbVpMLzFtdkRT?=
 =?utf-8?B?ZjZTL3dLUUc1UzQ1S1BESCtwWVRRNERDai9kV0hQZ1FZRkpQVXhEYzA5bVU0?=
 =?utf-8?B?RXl1c1NxYSs1RnFpTHllVnk1S1BsYWZPeitleW8xdXZ2S0FKRldXeG5ORThN?=
 =?utf-8?B?emozWGcwdHVWM0tGVjBmNkRNelh4aVJmaE9BdmNobXFuV09pUlIybm10YkY4?=
 =?utf-8?B?Rys2M2RNTkx1Z1ByNDdHMWpYOEhhZ2F3VFZkMmFqaU5PbzZEdWt1SjM1OWVQ?=
 =?utf-8?B?MzZJcXVYakltRnMyRDNnbUZwa3dPejVJSDVKVDZyYURkdklPbVVyeURlNStM?=
 =?utf-8?B?cDAvSHhNU3ZGdHYwUjJqd01wQ3JFQmd4UHVJQVF5ODAzRUVqaWpBekJDdDlH?=
 =?utf-8?B?eGZrdFFqVkZ5YzIySUFQRmpDVmNHQkdMdlFGQmp6dFUwc01rUG5ubzQzYUpH?=
 =?utf-8?B?ekdQTWlFSlhmSHlFU2xid2FJeXBHOUxsc01NZlRkR00wSEUwZmxMdG1lYlpD?=
 =?utf-8?Q?mplEIbrduu/kD8wkj+fU6/vDc?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 43c212f7-e70c-490c-163a-08dbca848274
X-MS-Exchange-CrossTenant-AuthSource: IA1PR12MB6604.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Oct 2023 18:04:27.8967
 (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: vQKDJ/Qeov6v1XYERSEpygQ2QTi+Sv6bnNZ0aLANELECsgZWZjdjz0dyrZTcRastW8o3+Cu4nMJZcO3V5vquTg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS0PR12MB6631
X-Spam-Status: No, score=-0.6 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,NICE_REPLY_A,RCVD_IN_SBL_CSS,SPF_HELO_NONE,SPF_PASS,
	URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on fry.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (fry.vger.email [0.0.0.0]); Wed, 11 Oct 2023 11:04:49 -0700 (PDT)

On 10/11/23 3:06 AM, Andy Shevchenko wrote:
> On Wed, Oct 11, 2023 at 11:33:51AM +0200, Bartosz Golaszewski wrote:
>> On Tue, Oct 10, 2023 at 5:18 PM Andy Shevchenko
>> <andriy.shevchenko@linux.intel.com> wrote:
>>>
>>> This is a series provides a new API to GPIO library (so far only
>>> available in the GPIO tree), and respective update to the Tegra
>>> HTE driver. On top a couple of other cleaups (patches 3 & 4, they
>>> can be applied separately).
>>>
>>> Patch 2 inherited tags from its respective discussion thread [1],
>>> but I believe the Tested-by needs to be confirmed again.
>>>
>>> Due to dependencies this either should be applied to the GPIO tree,
>>> or to the HTE when GPIO updates land the upstream (optionally with
>>> the first patch be applied even now to the GPIO tree independently).
>>>
>>> Another option is to have an immutable branch or tag, but I assume
>>> that was discussed and rejected (?) in [1].
>>
>> The series looks good to me. I'd like to take patches 1 and 2 through
>> the GPIO tree once v2 is out. This way we could potentially remove
>> gpiochip_find() for v6.7 already.
> 
> It would be nice to see it being removed sooner than later!
> I'm waiting for the test results by Dipen, I'll send the v2
> ASAP if tests pass.
> 
Reviewed-by: Dipen Patel <dipenp@nvidia.com>
After correcting typo manually
Tested-by: Dipen Patel <dipenp@nvidia.com>