From:   Colin Foster <colin.foster@in-advantage.com>
To:     linux-arm-kernel@lists.infradead.org, linux-gpio@vger.kernel.org,
        linux-kernel@vger.kernel.org
Cc:     Lars Povlsen <lars.povlsen@microchip.com>,
        Steen Hegelund <Steen.Hegelund@microchip.com>,
        UNGLinuxDriver@microchip.com,
        Linus Walleij <linus.walleij@linaro.org>,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        =?UTF-8?q?Cl=C3=A9ment=20L=C3=A9ger?= <clement.leger@bootlin.com>,
        Vladimir Oltean <vladimir.oltean@nxp.com>
Subject: [PATCH v1 net-next 2/4] pinctrl: ocelot: update pinctrl to automatic base address
Date:   Fri, 19 Nov 2021 11:59:26 -0800
Message-Id: <20211119195928.2498441-3-colin.foster@in-advantage.com>
In-Reply-To: <20211119195928.2498441-1-colin.foster@in-advantage.com>
References: <20211119195928.2498441-1-colin.foster@in-advantage.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?cytvUW1TWVVuQ2Uzc3NBWlRrTmNkL0xjNDQ1NlpRazdyK1N2WjVoNmpUcFZt?=
 =?utf-8?B?U2U3UlZZS3pldWNYdFB3WjhZUUtPUWtRU1hwaDM5T0QyS3lBRWFwWU9ZTDRw?=
 =?utf-8?B?SlBONlNOTGJZelJRRHJrUHBwK0F4SGd1ZkpEUVljcFJCL1drMkthbkoxVDBL?=
 =?utf-8?B?Qk5zYjVxRGVJamFjbVJoVzYzQTBpNTl2aGNuc3BsOUJFVzVTK3JVdXA5V0pQ?=
 =?utf-8?B?c3IvY084cGYrdWc1SGNTRjJHQ0dQaXRmNlpwdk9GQTRBRjByTDNMSURFeG5a?=
 =?utf-8?B?U2ZFTE1TUXZKUTF5Tjc1Sy9zcXRzTTRlTExSdEdtQTJGK2Z6WUVLcG9lbHZN?=
 =?utf-8?B?QUNLcGdxZ2Rhc2E4YU9ucjY0QUZaMUF4WEM2STQ0YXVKK3dmTGY4Sk4vUXpj?=
 =?utf-8?B?eFBaTmc1ZG93Nlk5aHloQ2EwRndQNFdZSzUvRG9xbk1WUzN0MW5VOTZ4Vnlm?=
 =?utf-8?B?Ti9sL2c5U1A3RjN0YVU2ZjJJODl1MlB6TEMrRldza0JkY2R0QUJ0cTVGQnRq?=
 =?utf-8?B?NFVEVVczNTlQbCtwUTVpQ3FlWFMyeThaZVpMQmlaM0s1WGcrZ3hnaFZFblJ1?=
 =?utf-8?B?RVJ4Ni92SHhIUHVJTVRHN29IbkhqUlV1Um8xQTJYTVMrbkNrUDFhc1RZOVVM?=
 =?utf-8?B?SmRvUW8xejAzNEdEYU5DQVFFLzB5WExPNUV4WlFGajNUN29OT0ljYk95TzFC?=
 =?utf-8?B?YmRVNHhzM2xBM1NOTkJtR2ZRRE1iNEtnYXE1K3hKYXh6UU95bS9iQVlQQThn?=
 =?utf-8?B?cVVjSjh4YnRHdnZtNkE0U3pYOVZYL0FCOVRWSTVhZzkvMW1ueWZKaFVBdnBo?=
 =?utf-8?B?MEdZOHJGWkp2YjhiMFQzbnZFck05NC9vZ2VtRVNmTG1oZHZaZ0hZeDhNSEdT?=
 =?utf-8?B?dWxLTjZBNWgwYjZ0ZXpQeFd4SUFWWWFzekk4RE92V1lrcXJraGJ5WWpSNmhE?=
 =?utf-8?B?bFhYWUVNbHZENFRMWkZqTmNPL204Z3lPUUJWR3RCQWQ0aVJzeURBT2ttd0RI?=
 =?utf-8?B?TXIrR1k3b094TGRXK0NCcjRDb3Jrakc2VUE0cEwrWDYvSnMrRmlIb2tNOU1n?=
 =?utf-8?B?bDVnWTVLU0RuWTJQRXpBd0NtNGVSczBDdGhrREwvUFFGbWFzWGJTVlhyZk9H?=
 =?utf-8?B?THA2cGNZVldCT1FraEtidy9EU2hxMGp5VUR6c1dFbTdpWTUvdDZONGJIZitQ?=
 =?utf-8?B?Qk5IdnlSUUljWTgzZXppbjdWT2NRcnZzWDR6Z3puTHdsQzF5SThic3N3a1kv?=
 =?utf-8?B?TUlyMzNVODI2SzBHSW9LdGoxeFpEU0JmY29iSFB3emxiTkx1NUpMOWR1U2NU?=
 =?utf-8?B?R05IQ2RpMndJR1hlRFBXcDJCVlNnemt3U20zM2ZrQzlzd0IxTVpWRkVnbVU3?=
 =?utf-8?B?NFRJK0E3ekhmMWVyYlplaHpDOTgyWTFNdEw5bDRRMnlueEtyTDhFcXZabS95?=
 =?utf-8?B?QlVyQlY2RU1LeUZiNS96VmhqU2xjRFRSRUY2NXNzOVZCaVlQQ3pXOEhSWVBB?=
 =?utf-8?B?N3JaNUs3Q0RGZTNVMTNPSmRyakFzWlg0b2F3elU3ei9SSmFxT1U0bWdaaXAv?=
 =?utf-8?B?YWJBWmFna3ZKYlhKdU1oTjdyMGQvdTR2R1NxRENpWXJuc3BESUhuZmVoNnBU?=
 =?utf-8?B?cy9Ecmg2VmlkUFRLSVh5eUhxY29FWDluR2lCbEQ4aFZtSTEzSDhHVEdraFho?=
 =?utf-8?B?NnN1OGR0eEFFaVdKcno1MXdSQmZMM1ZPWDZaVE1wNDJQRm9rbGxhL0VMVUJH?=
 =?utf-8?B?cTU0QzRJbTM2QTZDaUJSczQrQ2M1VE14cW5EZHlBNHFPSnFaL2xNcnBxOVJU?=
 =?utf-8?B?bnhnVWVtSkhpM2Y0OEllRUd2MWN6SDVKL0RNRFRkUjBqditPU0FMNGl0MWlX?=
 =?utf-8?B?WEE1VzkvKzkvRFJZZndXUmVyWG5sdjBWMWVjS2ZCTVZwVGk5U0F4dCtZK1lK?=
 =?utf-8?B?SVhGWkxlUW9tOFVSNVE0OS9ZYzhtbGgzb25WanY4QzBYeHU2Y3QwMFE3Slhm?=
 =?utf-8?B?WEZ6L1pBVndic2VsZDkwQ0dlYW5uazdsNjl6QnYyNXZiMytXMkIxd3Z0S3p1?=
 =?utf-8?B?U3lpQVNaRFFhTlFNbGFDTFFMbWFqcjdnWDlOeWxTQUg0Mk5yVmJIbGhDaUZt?=
 =?utf-8?B?UnZlek5uZ1NlNnVTT1J4ejROYmprVXIrcThDbFJWSlloTTVDMk1lYXRHS3dU?=
 =?utf-8?B?T1YrZnhyQ1RPNkkwVGUzaURpZEJUdnpvZGVWamYrY21haTQ5Y242SlVWWFc2?=
 =?utf-8?B?OGppT1R5MVJMak93VUNEYjI5YU9nPT0=?=
X-OriginatorOrg: in-advantage.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 7b462b8d-3ea6-442c-751b-08d9ab971f18
X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2351.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Nov 2021 19:59:40.1682
 (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: U/N/MdwmArpI8bWxynBszLXut/UcUiYE1u5Z1snXnN3cuEzWqs1P+e1ezwsTPKSKPCiqPBeRKjwD7A1diWJnymPD4lCt2ySiDzb+moo3FTo=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CO6PR10MB5651
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

struct gpio_chip recommends passing -1 as base to gpiolib. Doing so avoids
conflicts when the chip is external and gpiochip0 already exists.

Signed-off-by: Colin Foster <colin.foster@in-advantage.com>
Tested-by: Clément Léger <clement.leger@bootlin.com>
---
 drivers/pinctrl/pinctrl-ocelot.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/pinctrl/pinctrl-ocelot.c b/drivers/pinctrl/pinctrl-ocelot.c
index 306293314069..6941c1b45b00 100644
--- a/drivers/pinctrl/pinctrl-ocelot.c
+++ b/drivers/pinctrl/pinctrl-ocelot.c
@@ -1308,7 +1308,7 @@ static int ocelot_gpiochip_register(struct platform_device *pdev,
 	gc = &info->gpio_chip;
 	gc->ngpio = info->desc->npins;
 	gc->parent = &pdev->dev;
-	gc->base = 0;
+	gc->base = -1;
 	gc->of_node = info->dev->of_node;
 	gc->label = "ocelot-gpio";
 
-- 
2.25.1