From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id B04DFC6FD1F for ; Tue, 2 Apr 2024 14:09:41 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=DnoOF4MmIpi5wBLApOitTEkLWZOYnjS7bc0EFY1r8d8=; b=oosndkp2BKCz6Z 9Bl/tmTKfTpl84Xee/So4F5rz6jhunkdZk6IIC9XNvjm7VBCxNRgf01BaN0eX6jZ5Nz3LgdduZsNy wQOjvbxlYHcYXwkmoy5waLiaxAByESnosGV78JM8MR9CWokharZfZrERZ2KSPFUeShOtMVSva82rW TQtg2qgZWSmaIHIacDAw95tqmGSdAvgYWww9Du7rgMTPKb/ypOeyGVLS/7k+sTkalccqDZyKWxIuD zxD7dC5XwB0RyX6TrQkHguOmq+hwZ2rS4ZBcJx7407nBJeWW33fXxLkkaJYXNEIbx3dJbmG1zoCi8 GxeNrQ+E/YuEUXWS63fg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rrepK-0000000BVr1-45xA; Tue, 02 Apr 2024 14:09:26 +0000 Received: from mgamail.intel.com ([198.175.65.12]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1rrepI-0000000BVq7-03RW for linux-arm-kernel@lists.infradead.org; Tue, 02 Apr 2024 14:09:25 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1712066964; x=1743602964; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=uEC3a2yUsQ6IMUZDu++fBR8R8oM+dRgeV4GJKC1kyJ4=; b=MMupnFhb3XCs6dLvyp0XcP/G1ag++noF/AYxfGUXTZpoH+iIbzbEZQRt MJgWC5u/Tcw/uClJAUaswWKoL7fhziqRJGwExJ/ylwzj1qfXwkMNbhNcN 7H2W6tIPxPdkGiblZb7WEjqKJPZa6SpzazuxXlU0yDX/TALs+0w7Du5gg 04Tn0Cz13XVmDy7jVfSLmfbGgzLPjqOgG+Zz6FHfgpF0yLm2D6kRJlrRE XILk6uToUBy+h6YNGd1gXKg6DXds+wtGxI1CG/a6oGwes5R9hY4NysAmQ FHxfY0uQv/atDQaUwpTlTnAbyOsWA5KkJkeWbCI4Z/g2Eh+KqywonNnjZ A==; X-CSE-ConnectionGUID: GGoaUkoHT1mxLh4RWkT5aA== X-CSE-MsgGUID: O68OQxdXQpO9vmP2pu16vg== X-IronPort-AV: E=McAfee;i="6600,9927,11032"; a="18689673" X-IronPort-AV: E=Sophos;i="6.07,175,1708416000"; d="scan'208";a="18689673" Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by orvoesa104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 02 Apr 2024 07:09:24 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,11032"; a="915142247" X-IronPort-AV: E=Sophos;i="6.07,175,1708416000"; d="scan'208";a="915142247" Received: from smile.fi.intel.com (HELO smile) ([10.237.72.54]) by fmsmga002.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 02 Apr 2024 07:09:19 -0700 Received: from andy by smile with local (Exim 4.97) (envelope-from ) id 1rrepB-00000000pQY-1kBR; Tue, 02 Apr 2024 17:09:17 +0300 Date: Tue, 2 Apr 2024 17:09:17 +0300 From: Andy Shevchenko To: Peng Fan Cc: "Peng Fan (OSS)" , Sudeep Holla , Cristian Marussi , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Linus Walleij , Dan Carpenter , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" , "linux-gpio@vger.kernel.org" , Oleksii Moisieiev Subject: Re: [PATCH v7 4/4] pinctrl: Implementation of the generic scmi-pinctrl driver Message-ID: References: <20240402-pinctrl-scmi-v7-0-3ea519d12cf7@nxp.com> <20240402-pinctrl-scmi-v7-4-3ea519d12cf7@nxp.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240402_070924_167114_DB517FC4 X-CRM114-Status: GOOD ( 16.29 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue, Apr 02, 2024 at 01:59:19PM +0000, Peng Fan wrote: > > On Tue, Apr 02, 2024 at 10:22:24AM +0800, Peng Fan (OSS) wrote: ... > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > > Missing headers. > > Not sure there is an easy way to filter out what is missed. And?.. You are the author, not me. You know your code much better and it will be quite easy to perform. I may miss things, but reading briefly the 1000 lines and get what headers are required takes no more than half an hour. (Tools that help me, in case I don't remember by heart, are `cscope` and `git grep ...`.) ... > > > + ret = pinctrl_ops->name_get(pmx->ph, i, PIN_TYPE, > > &pins[i].name); > > > + if (ret) > > > > How does the cleanup work for the previously assigned pin names? Is it > > needed? > > No need. The "name" memory region is allocated in firmware pinctrl > Protocol init phase. > > > Maybe a comment? > > ok. As below. > /* > * The region for name is handled by the scmi firmware driver, > * no need free here > */ LGTM. > > > + return dev_err_probe(pmx->dev, ret, > > > + "Can't get name for pin %d", i); -- With Best Regards, Andy Shevchenko _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel