Commit edc72546 authored by Lars Povlsen's avatar Lars Povlsen Committed by Linus Walleij
Browse files

pinctrl: ocelot: Remove instance number from pin functions 



This patch removes the instance number from the "miim", "reco_clk" and
"sfp" pin function.

The change needed is to prepare the driver for adding new platforms
with more of these instances. The instance number is also redundant,
as this is implicit for each pin.

Reviewed-by: default avatarAlexandre Belloni <alexandre.belloni@bootlin.com>
Signed-off-by: default avatarLars Povlsen <lars.povlsen@microchip.com>
Link: https://lore.kernel.org/r/20200513125532.24585-3-lars.povlsen@microchip.com


Signed-off-by: default avatarLinus Walleij <linus.walleij@linaro.org>
parent 550713e3

drivers/pinctrl/pinctrl-ocelot.c

+30 −64
Original line number Diff line number Diff line
@@ -46,32 +46,15 @@ enum { 
	FUNC_IRQ0_OUT,

	FUNC_IRQ1_IN,

	FUNC_IRQ1_OUT,

	FUNC_MIIM1,

	FUNC_MIIM2,

	FUNC_MIIM,

	FUNC_PCI_WAKE,

	FUNC_PTP0,

	FUNC_PTP1,

	FUNC_PTP2,

	FUNC_PTP3,

	FUNC_PWM,

	FUNC_RECO_CLK0,

	FUNC_RECO_CLK1,

	FUNC_SFP0,

	FUNC_SFP1,

	FUNC_SFP2,

	FUNC_SFP3,

	FUNC_SFP4,

	FUNC_SFP5,

	FUNC_SFP6,

	FUNC_SFP7,

	FUNC_SFP8,

	FUNC_SFP9,

	FUNC_SFP10,

	FUNC_SFP11,

	FUNC_SFP12,

	FUNC_SFP13,

	FUNC_SFP14,

	FUNC_SFP15,

	FUNC_RECO_CLK,

	FUNC_SFP,

	FUNC_SG0,

	FUNC_SG1,

	FUNC_SG2,
@@ -92,32 +75,15 @@ static const char *const ocelot_function_names[] = { 
	[FUNC_IRQ0_OUT]		= "irq0_out",

	[FUNC_IRQ1_IN]		= "irq1_in",

	[FUNC_IRQ1_OUT]		= "irq1_out",

	[FUNC_MIIM1]		= "miim1",

	[FUNC_MIIM2]		= "miim2",

	[FUNC_MIIM]		= "miim",

	[FUNC_PCI_WAKE]		= "pci_wake",

	[FUNC_PTP0]		= "ptp0",

	[FUNC_PTP1]		= "ptp1",

	[FUNC_PTP2]		= "ptp2",

	[FUNC_PTP3]		= "ptp3",

	[FUNC_PWM]		= "pwm",

	[FUNC_RECO_CLK0]	= "reco_clk0",

	[FUNC_RECO_CLK1]	= "reco_clk1",

	[FUNC_SFP0]		= "sfp0",

	[FUNC_SFP1]		= "sfp1",

	[FUNC_SFP2]		= "sfp2",

	[FUNC_SFP3]		= "sfp3",

	[FUNC_SFP4]		= "sfp4",

	[FUNC_SFP5]		= "sfp5",

	[FUNC_SFP6]		= "sfp6",

	[FUNC_SFP7]		= "sfp7",

	[FUNC_SFP8]		= "sfp8",

	[FUNC_SFP9]		= "sfp9",

	[FUNC_SFP10]		= "sfp10",

	[FUNC_SFP11]		= "sfp11",

	[FUNC_SFP12]		= "sfp12",

	[FUNC_SFP13]		= "sfp13",

	[FUNC_SFP14]		= "sfp14",

	[FUNC_SFP15]		= "sfp15",

	[FUNC_RECO_CLK]		= "reco_clk",

	[FUNC_SFP]		= "sfp",

	[FUNC_SG0]		= "sg0",

	[FUNC_SG1]		= "sg1",

	[FUNC_SG2]		= "sg2",
@@ -168,18 +134,18 @@ OCELOT_P(6, UART, TWI_SCL_M, NONE); 
OCELOT_P(7,  UART,      TWI_SCL_M, NONE);

OCELOT_P(8,  SI,        TWI_SCL_M, IRQ0_OUT);

OCELOT_P(9,  SI,        TWI_SCL_M, IRQ1_OUT);

OCELOT_P(10, PTP2,      TWI_SCL_M, SFP0);

OCELOT_P(11, PTP3,      TWI_SCL_M, SFP1);

OCELOT_P(12, UART2,     TWI_SCL_M, SFP2);

OCELOT_P(13, UART2,     TWI_SCL_M, SFP3);

OCELOT_P(14, MIIM1,     TWI_SCL_M, SFP4);

OCELOT_P(15, MIIM1,     TWI_SCL_M, SFP5);

OCELOT_P(10, PTP2,      TWI_SCL_M, SFP);

OCELOT_P(11, PTP3,      TWI_SCL_M, SFP);

OCELOT_P(12, UART2,     TWI_SCL_M, SFP);

OCELOT_P(13, UART2,     TWI_SCL_M, SFP);

OCELOT_P(14, MIIM,      TWI_SCL_M, SFP);

OCELOT_P(15, MIIM,      TWI_SCL_M, SFP);

OCELOT_P(16, TWI,       NONE,      SI);

OCELOT_P(17, TWI,       TWI_SCL_M, SI);

OCELOT_P(18, PTP0,      TWI_SCL_M, NONE);

OCELOT_P(19, PTP1,      TWI_SCL_M, NONE);

OCELOT_P(20, RECO_CLK0, TACHO,     NONE);

OCELOT_P(21, RECO_CLK1, PWM,       NONE);

OCELOT_P(20, RECO_CLK,  TACHO,     TWI_SCL_M);

OCELOT_P(21, RECO_CLK,  PWM,       TWI_SCL_M);



#define OCELOT_PIN(n) {						\

	.number = n,						\
@@ -264,22 +230,22 @@ JAGUAR2_P(40, NONE, TWI_SCL_M); 
JAGUAR2_P(41, NONE,      TWI_SCL_M);

JAGUAR2_P(42, NONE,      TWI_SCL_M);

JAGUAR2_P(43, NONE,      TWI_SCL_M);

JAGUAR2_P(44, NONE,      SFP8);

JAGUAR2_P(45, NONE,      SFP9);

JAGUAR2_P(46, NONE,      SFP10);

JAGUAR2_P(47, NONE,      SFP11);

JAGUAR2_P(48, SFP0,      NONE);

JAGUAR2_P(49, SFP1,      SI);

JAGUAR2_P(50, SFP2,      SI);

JAGUAR2_P(51, SFP3,      SI);

JAGUAR2_P(52, SFP4,      NONE);

JAGUAR2_P(53, SFP5,      NONE);

JAGUAR2_P(54, SFP6,      NONE);

JAGUAR2_P(55, SFP7,      NONE);

JAGUAR2_P(56, MIIM1,     SFP12);

JAGUAR2_P(57, MIIM1,     SFP13);

JAGUAR2_P(58, MIIM2,     SFP14);

JAGUAR2_P(59, MIIM2,     SFP15);

JAGUAR2_P(44, NONE,      SFP);

JAGUAR2_P(45, NONE,      SFP);

JAGUAR2_P(46, NONE,      SFP);

JAGUAR2_P(47, NONE,      SFP);

JAGUAR2_P(48, SFP,       NONE);

JAGUAR2_P(49, SFP,       SI);

JAGUAR2_P(50, SFP,       SI);

JAGUAR2_P(51, SFP,       SI);

JAGUAR2_P(52, SFP,       NONE);

JAGUAR2_P(53, SFP,       NONE);

JAGUAR2_P(54, SFP,       NONE);

JAGUAR2_P(55, SFP,       NONE);

JAGUAR2_P(56, MIIM,      SFP);

JAGUAR2_P(57, MIIM,      SFP);

JAGUAR2_P(58, MIIM,      SFP);

JAGUAR2_P(59, MIIM,      SFP);

JAGUAR2_P(60, NONE,      NONE);

JAGUAR2_P(61, NONE,      NONE);

JAGUAR2_P(62, NONE,      NONE);