Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Mon, 31 Jan 2022 15:14:29 -0800
From:   Colin Foster <colin.foster@in-advantage.com>
To:     Florian Fainelli <f.fainelli@gmail.com>
Cc:     linux-arm-kernel@lists.infradead.org, linux-gpio@vger.kernel.org,
        netdev@vger.kernel.org, linux-kernel@vger.kernel.org,
        Steen Hegelund <Steen.Hegelund@microchip.com>,
        Lars Povlsen <lars.povlsen@microchip.com>,
        Linus Walleij <linus.walleij@linaro.org>,
        Russell King <linux@armlinux.org.uk>,
        Heiner Kallweit <hkallweit1@gmail.com>,
        Jakub Kicinski <kuba@kernel.org>,
        "David S. Miller" <davem@davemloft.net>,
        Vivien Didelot <vivien.didelot@gmail.com>,
        Andrew Lunn <andrew@lunn.ch>, UNGLinuxDriver@microchip.com,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        Claudiu Manoil <claudiu.manoil@nxp.com>,
        Vladimir Oltean <vladimir.oltean@nxp.com>,
        Lee Jones <lee.jones@linaro.org>, katie.morris@in-advantage.com
Subject: Re: [RFC v6 net-next 2/9] pinctrl: microchip-sgpio: allow sgpio
 driver to be used as a module
Message-ID: <20220131231429.GA10212@COLIN-DESKTOP1.localdomain>
References: <20220129220221.2823127-1-colin.foster@in-advantage.com>
 <20220129220221.2823127-3-colin.foster@in-advantage.com>
 <f9e160d8-09e9-46af-df7d-8f128b5eab4b@gmail.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <f9e160d8-09e9-46af-df7d-8f128b5eab4b@gmail.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?aUJlTTrSofQb2yvtS3b8DA86zsOoFXXymF8z6uNI1thxOZ8Gkx5CruN0XL3+?=
 =?us-ascii?Q?Vj/NM/nlwMSNqWrXYNtYu+Tp10MXopRCn2aWWQLuXehSwwd848xKuGX7fpke?=
 =?us-ascii?Q?M4LS9p/rqGrjO8EisK68XN42dcOgRc32CXyFOGEf8JzrubPQ58YtrRedWe40?=
 =?us-ascii?Q?f+u26blsr7x/PcxDagMOGwhjgfwisGoE1fPaNY4TjQfVtjGqTAf1OabHm3AU?=
 =?us-ascii?Q?PPXi/NTE7NgbRWHbulf4ijsArloZ2DhDRUwtm1tTVBvSwgaT700w7CN1okmX?=
 =?us-ascii?Q?rquo2e7v+D35dPyTGCoFKwK3PcpQfeRiDlO4X3AWN6+TrvOy/FXkP5fdfFYR?=
 =?us-ascii?Q?wDOjzzGjvW8wIbiPj7oD++wWcAkoXzXX6t3CJTgMj7wILsCb3KoV3DS9nPXQ?=
 =?us-ascii?Q?nBogyDV2M7o5kMKq/TBmzE8I/iwoTcztcuOzF491Te4+O6sOHDmVs0oNeH+T?=
 =?us-ascii?Q?GuX1vaqGh2VoERD6yHB9IrUjCk98Sjf+A9yVUk69E2OskznkirU8hxHa6hfI?=
 =?us-ascii?Q?Dm19+I32tIscz7924cVRf9OXqWxqS4k8kGOWIoW5/5PKRwAi1XDVwgx6c+c9?=
 =?us-ascii?Q?aqMDKmRTHpd6dvX3hgzSu3GyTss08Sws39Z2Rzbq4IUnP+TI9tPx2/D1VGwe?=
 =?us-ascii?Q?/WsWBeM+20W8rWRAlSU9QlhEjRaPVBcfnd+BpnESR8MaG0a2Z8k1LfQRzSHq?=
 =?us-ascii?Q?VaQETIwFaFUaKQHV11a/rA3HONRgPsS4l6bBhFldby/UplKgBd2qzzZAPXuY?=
 =?us-ascii?Q?v8jvJIRfh6vNjtz2sTwdpTojT41bvS0oVA2qwuZjoxE8qvkvY6ZC+iadaoUf?=
 =?us-ascii?Q?Z/mGOhiY36z16GmEtdIhcYOq8zQUvQ/U3RJNlrELSG6aW6U8+mFhen+EQVGq?=
 =?us-ascii?Q?/ULFZizeL4EDiKCjIr0ETYhe0T0tF4xDN2l278HJksrDziOUUZTBQwKhqvfU?=
 =?us-ascii?Q?826FzQwXLIEVQ4FihRDM2DKg6KaXK0dO4JHw36wADbP6PnFZNZKtufaAiZyc?=
 =?us-ascii?Q?BMIUT8GtgIt6n36beTwel1oMp7zKLLPiqT9dGjhDir2sT9ZKmchdqRkcP4gl?=
 =?us-ascii?Q?ukgP5SbRLoIVZnsW72MohSRRp7vtgmWVQYhSHlasuYSZlNUj0aH19ofM1UcJ?=
 =?us-ascii?Q?Nmadwn7p/DmtwWN4PmyOD76wy1WV8Lv3boPIIQLqslq1bGAvXIzn5vEokQQ9?=
 =?us-ascii?Q?yr4XvD2r4v1MepVwPv3AwO2qfCtg6msr6SeK9+Yd4AeppNLBgu2EEzdzLs+m?=
 =?us-ascii?Q?TwEtOYC+71MkOKmNA0pD7njnKpClbnic8hYIHkUo+nERxbI3Oouc0+wtkoHB?=
 =?us-ascii?Q?fROLuZutW6bU86MP5NtOLU2ujQDSCau6XZmn0iCLEuvBHT4eQMvTfwwVYOmx?=
 =?us-ascii?Q?kYAfopn2H0s0kivuSvTjm1HoedUp2k32yUNp/w+c0t8gLHRituZRCLINI+j/?=
 =?us-ascii?Q?FRKoWOSr8IWZUuulj1+YcYg/ygBxHYi019+rAAOQtY+gw17SXyhL3uXa1lMc?=
 =?us-ascii?Q?xrYZ7VvDVIpKyDKw1coL5+ZS/FwYpRuyKl1+vwZwjmDKh/BIUvL0msjWAUf4?=
 =?us-ascii?Q?awGY17z/AyPP4xYTG8NZ/iNSKRLG+D3NmbT4lrdN6g9r1rDqtw+YkLQQeHaa?=
 =?us-ascii?Q?4mIVkpAN2dmidnj1afVtXHZwyVIXy3GBzxoMs8qVGNoBmcxhevWM8+mTK3NU?=
 =?us-ascii?Q?HFT6oA=3D=3D?=
X-OriginatorOrg: in-advantage.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 0bdd83f7-db5f-4cab-b475-08d9e50f70b7
X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2351.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 31 Jan 2022 23:14:33.0649
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 48e842ca-fbd8-4633-a79d-0c955a7d3aae
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: o6yMmmN9bNzgep0ft/8VAhYQyDQVpRqWklO5u6PcovwqA/qC5WzfbcumoyVeURjZkpqSf55qZuvkq9W0Ucj+ZsnL2O2M9WjjhlWyRANHGhM=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY5PR10MB4274
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Mon, Jan 31, 2022 at 03:11:55PM -0800, Florian Fainelli wrote:
> 
> 
> On 1/29/2022 2:02 PM, Colin Foster wrote:
> > As the commit message suggests, this simply adds the ability to select
> > SGPIO pinctrl as a module. This becomes more practical when the SGPIO
> > hardware exists on an external chip, controlled indirectly by I2C or SPI.
> > This commit enables that level of control.
> > 
> > Signed-off-by: Colin Foster <colin.foster@in-advantage.com>
> 
> Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>

Thanks Florian

> -- 
> Florian