Re: [PATCH 4/6] soc/tegra: pmc: Use X macro to generate IO pad tables

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 11 Jul 2018 09:30:57 +0100
Jon Hunter <jonathanh@xxxxxxxxxx> wrote:

> On 10/07/18 13:47, Aapo Vienamo wrote:
> > Refactor the IO pad tables into macro tables so that they can be reused
> > to generate pinctrl pin descriptors. Also add a name field which is
> > needed by pinctrl.
> > 
> > Signed-off-by: Aapo Vienamo <avienamo@xxxxxxxxxx>
> > ---
> >  drivers/soc/tegra/pmc.c | 231 ++++++++++++++++++++++++++----------------------
> >  1 file changed, 126 insertions(+), 105 deletions(-)
> > 
> > diff --git a/drivers/soc/tegra/pmc.c b/drivers/soc/tegra/pmc.c
> > index 3f5b69fd..b833334 100644
> > --- a/drivers/soc/tegra/pmc.c
> > +++ b/drivers/soc/tegra/pmc.c
> > @@ -137,6 +137,7 @@ struct tegra_io_pad_soc {
> >  	enum tegra_io_pad id;
> >  	unsigned int dpd;
> >  	unsigned int voltage;
> > +	const char *name;
> >  };
> >  
> >  struct tegra_pmc_regs {
> > @@ -1697,37 +1698,49 @@ static const u8 tegra124_cpu_powergates[] = {
> >  	TEGRA_POWERGATE_CPU3,
> >  };
> >  
> > +#define TEGRA_IO_PAD(_id, _dpd, _voltage, _name)	\
> > +	(struct tegra_io_pad_soc) {			\
> > +		.id	= _id,				\
> > +		.dpd	= _dpd,				\
> > +		.voltage = _voltage,			\
> > +		.name	= _name,			\
> > +	},
> > +
> > +#define TEGRA124_IO_PAD_TABLE(_pad)					\
> > +	/* .id                          .dpd    .voltage  .name	*/	\
> > +	_pad(TEGRA_IO_PAD_AUDIO,	17,	UINT_MAX, "audio")	\
> > +	_pad(TEGRA_IO_PAD_BB,		15,	UINT_MAX, "bb")		\
> > +	_pad(TEGRA_IO_PAD_CAM,		36,	UINT_MAX, "cam")	\
> > +	_pad(TEGRA_IO_PAD_COMP,		22,	UINT_MAX, "comp")	\
> > +	_pad(TEGRA_IO_PAD_CSIA,		0,	UINT_MAX, "csia")	\
> > +	_pad(TEGRA_IO_PAD_CSIB,		1,	UINT_MAX, "csb")	\
> > +	_pad(TEGRA_IO_PAD_CSIE,		44,	UINT_MAX, "cse")	\
> > +	_pad(TEGRA_IO_PAD_DSI,		2,	UINT_MAX, "dsi")	\
> > +	_pad(TEGRA_IO_PAD_DSIB,		39,	UINT_MAX, "dsib")	\
> > +	_pad(TEGRA_IO_PAD_DSIC,		40,	UINT_MAX, "dsic")	\
> > +	_pad(TEGRA_IO_PAD_DSID,		41,	UINT_MAX, "dsid")	\
> > +	_pad(TEGRA_IO_PAD_HDMI,		28,	UINT_MAX, "hdmi")	\
> > +	_pad(TEGRA_IO_PAD_HSIC,		19,	UINT_MAX, "hsic")	\
> > +	_pad(TEGRA_IO_PAD_HV,		38,	UINT_MAX, "hv")		\
> > +	_pad(TEGRA_IO_PAD_LVDS,		57,	UINT_MAX, "lvds")	\
> > +	_pad(TEGRA_IO_PAD_MIPI_BIAS,	3,	UINT_MAX, "mipi-bias")	\
> > +	_pad(TEGRA_IO_PAD_NAND,		13,	UINT_MAX, "nand")	\
> > +	_pad(TEGRA_IO_PAD_PEX_BIAS,	4,	UINT_MAX, "pex-bias")	\
> > +	_pad(TEGRA_IO_PAD_PEX_CLK1,	5,	UINT_MAX, "pex-clk1")	\
> > +	_pad(TEGRA_IO_PAD_PEX_CLK2,	6,	UINT_MAX, "pex-clk2")	\
> > +	_pad(TEGRA_IO_PAD_PEX_CNTRL,	32,	UINT_MAX, "pex-cntrl")	\
> > +	_pad(TEGRA_IO_PAD_SDMMC1,	33,	UINT_MAX, "sdmmc1")	\
> > +	_pad(TEGRA_IO_PAD_SDMMC3,	34,	UINT_MAX, "sdmmc3")	\
> > +	_pad(TEGRA_IO_PAD_SDMMC4,	35,	UINT_MAX, "sdmmc4")	\
> > +	_pad(TEGRA_IO_PAD_SYS_DDC,	58,	UINT_MAX, "sys_ddc")	\
> > +	_pad(TEGRA_IO_PAD_UART,		14,	UINT_MAX, "uart")	\
> > +	_pad(TEGRA_IO_PAD_USB0,		9,	UINT_MAX, "usb0")	\
> > +	_pad(TEGRA_IO_PAD_USB1,		10,	UINT_MAX, "usb1")	\
> > +	_pad(TEGRA_IO_PAD_USB2,		11,	UINT_MAX, "usb2")	\
> > +	_pad(TEGRA_IO_PAD_USB_BIAS,	12,	UINT_MAX, "usb_bias")
> > +
> >  static const struct tegra_io_pad_soc tegra124_io_pads[] = {
> > -	{ .id = TEGRA_IO_PAD_AUDIO, .dpd = 17, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_BB, .dpd = 15, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_CAM, .dpd = 36, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_COMP, .dpd = 22, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_CSIA, .dpd = 0, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_CSIB, .dpd = 1, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_CSIE, .dpd = 44, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_DSI, .dpd = 2, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_DSIB, .dpd = 39, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_DSIC, .dpd = 40, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_DSID, .dpd = 41, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_HDMI, .dpd = 28, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_HSIC, .dpd = 19, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_HV, .dpd = 38, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_LVDS, .dpd = 57, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_MIPI_BIAS, .dpd = 3, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_NAND, .dpd = 13, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_PEX_BIAS, .dpd = 4, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_PEX_CLK1, .dpd = 5, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_PEX_CLK2, .dpd = 6, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_PEX_CNTRL, .dpd = 32, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_SDMMC1, .dpd = 33, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_SDMMC3, .dpd = 34, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_SDMMC4, .dpd = 35, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_SYS_DDC, .dpd = 58, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_UART, .dpd = 14, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_USB0, .dpd = 9, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_USB1, .dpd = 10, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_USB2, .dpd = 11, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_USB_BIAS, .dpd = 12, .voltage = UINT_MAX },
> > +	TEGRA124_IO_PAD_TABLE(TEGRA_IO_PAD)
> >  };
> >  
> >  static const struct tegra_pmc_soc tegra124_pmc_soc = {
> > @@ -1779,45 +1792,49 @@ static const u8 tegra210_cpu_powergates[] = {
> >  	TEGRA_POWERGATE_CPU3,
> >  };
> >  
> > +#define TEGRA210_IO_PAD_TABLE(_pad)							\
> > +	/*   .id                        .dpd            .voltage        .name */	\
> > +	_pad(TEGRA_IO_PAD_AUDIO,	17,		5,		"audio")	\
> > +	_pad(TEGRA_IO_PAD_AUDIO_HV,	61,		18,		"audio-hv")	\
> > +	_pad(TEGRA_IO_PAD_CAM,		36,		10,		"cam")		\
> > +	_pad(TEGRA_IO_PAD_CSIA,		0,		UINT_MAX,	"csia")		\
> > +	_pad(TEGRA_IO_PAD_CSIB,		1,		UINT_MAX,	"csib")		\
> > +	_pad(TEGRA_IO_PAD_CSIC,		42,		UINT_MAX,	"csic")		\
> > +	_pad(TEGRA_IO_PAD_CSID,		43,		UINT_MAX,	"csid")		\
> > +	_pad(TEGRA_IO_PAD_CSIE,		44,		UINT_MAX,	"csie")		\
> > +	_pad(TEGRA_IO_PAD_CSIF,		45,		UINT_MAX,	"csif")		\
> > +	_pad(TEGRA_IO_PAD_DBG,		25,		19,		"dbg")		\
> > +	_pad(TEGRA_IO_PAD_DEBUG_NONAO,	26,		UINT_MAX,	"debug-nonao")	\
> > +	_pad(TEGRA_IO_PAD_DMIC,		50,		20,		"dmic")		\
> > +	_pad(TEGRA_IO_PAD_DP,		51,		UINT_MAX,	"dp")		\
> > +	_pad(TEGRA_IO_PAD_DSI,		2,		UINT_MAX,	"dsi")		\
> > +	_pad(TEGRA_IO_PAD_DSIB,		39,		UINT_MAX,	"dsib")		\
> > +	_pad(TEGRA_IO_PAD_DSIC,		40,		UINT_MAX,	"dsic")		\
> > +	_pad(TEGRA_IO_PAD_DSID,		41,		UINT_MAX,	"dsid")		\
> > +	_pad(TEGRA_IO_PAD_EMMC,		35,		UINT_MAX,	"emmc")		\
> > +	_pad(TEGRA_IO_PAD_EMMC2,	37,		UINT_MAX,	"emmc2")	\
> > +	_pad(TEGRA_IO_PAD_GPIO,		27,		21,		"gpio")		\
> > +	_pad(TEGRA_IO_PAD_HDMI,		28,		UINT_MAX,	"hdmi")		\
> > +	_pad(TEGRA_IO_PAD_HSIC,		19,		UINT_MAX,	"hsic")		\
> > +	_pad(TEGRA_IO_PAD_LVDS,		57,		UINT_MAX,	"lvds")		\
> > +	_pad(TEGRA_IO_PAD_MIPI_BIAS,	3,		UINT_MAX,	"mipi-bias")	\
> > +	_pad(TEGRA_IO_PAD_PEX_BIAS,	4,		UINT_MAX,	"pex-bias")	\
> > +	_pad(TEGRA_IO_PAD_PEX_CLK1,	5,		UINT_MAX,	"pex-clk1")	\
> > +	_pad(TEGRA_IO_PAD_PEX_CLK2,	6,		UINT_MAX,	"pex-clk2")	\
> > +	_pad(TEGRA_IO_PAD_PEX_CNTRL,    UINT_MAX,	11,		"pex-cntrl")	\
> > +	_pad(TEGRA_IO_PAD_SDMMC1,	33,		12,		"sdmmc1")	\
> > +	_pad(TEGRA_IO_PAD_SDMMC3,	34,		13,		"sdmmc3")	\
> > +	_pad(TEGRA_IO_PAD_SPI,		46,		22,		"spi")		\
> > +	_pad(TEGRA_IO_PAD_SPI_HV,	47,		23,		"spi-hv")	\
> > +	_pad(TEGRA_IO_PAD_UART,		14,		2,		"uart")		\
> > +	_pad(TEGRA_IO_PAD_USB0,		9,		UINT_MAX,	"usb0")		\
> > +	_pad(TEGRA_IO_PAD_USB1,		10,		UINT_MAX,	"usb1")		\
> > +	_pad(TEGRA_IO_PAD_USB2,		11,		UINT_MAX,	"usb2")		\
> > +	_pad(TEGRA_IO_PAD_USB3,		18,		UINT_MAX,	"usb3")		\
> > +	_pad(TEGRA_IO_PAD_USB_BIAS,	12,		UINT_MAX,	"usb-bias")
> > +
> >  static const struct tegra_io_pad_soc tegra210_io_pads[] = {
> > -	{ .id = TEGRA_IO_PAD_AUDIO, .dpd = 17, .voltage = 5 },
> > -	{ .id = TEGRA_IO_PAD_AUDIO_HV, .dpd = 61, .voltage = 18 },
> > -	{ .id = TEGRA_IO_PAD_CAM, .dpd = 36, .voltage = 10 },
> > -	{ .id = TEGRA_IO_PAD_CSIA, .dpd = 0, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_CSIB, .dpd = 1, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_CSIC, .dpd = 42, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_CSID, .dpd = 43, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_CSIE, .dpd = 44, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_CSIF, .dpd = 45, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_DBG, .dpd = 25, .voltage = 19 },
> > -	{ .id = TEGRA_IO_PAD_DEBUG_NONAO, .dpd = 26, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_DMIC, .dpd = 50, .voltage = 20 },
> > -	{ .id = TEGRA_IO_PAD_DP, .dpd = 51, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_DSI, .dpd = 2, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_DSIB, .dpd = 39, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_DSIC, .dpd = 40, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_DSID, .dpd = 41, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_EMMC, .dpd = 35, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_EMMC2, .dpd = 37, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_GPIO, .dpd = 27, .voltage = 21 },
> > -	{ .id = TEGRA_IO_PAD_HDMI, .dpd = 28, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_HSIC, .dpd = 19, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_LVDS, .dpd = 57, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_MIPI_BIAS, .dpd = 3, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_PEX_BIAS, .dpd = 4, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_PEX_CLK1, .dpd = 5, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_PEX_CLK2, .dpd = 6, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_PEX_CNTRL, .dpd = UINT_MAX, .voltage = 11 },
> > -	{ .id = TEGRA_IO_PAD_SDMMC1, .dpd = 33, .voltage = 12 },
> > -	{ .id = TEGRA_IO_PAD_SDMMC3, .dpd = 34, .voltage = 13 },
> > -	{ .id = TEGRA_IO_PAD_SPI, .dpd = 46, .voltage = 22 },
> > -	{ .id = TEGRA_IO_PAD_SPI_HV, .dpd = 47, .voltage = 23 },
> > -	{ .id = TEGRA_IO_PAD_UART, .dpd = 14, .voltage = 2 },
> > -	{ .id = TEGRA_IO_PAD_USB0, .dpd = 9, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_USB1, .dpd = 10, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_USB2, .dpd = 11, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_USB3, .dpd = 18, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_USB_BIAS, .dpd = 12, .voltage = UINT_MAX },
> > +	TEGRA210_IO_PAD_TABLE(TEGRA_IO_PAD)
> >  };
> >  
> >  static const struct tegra_pmc_soc tegra210_pmc_soc = {
> > @@ -1836,44 +1853,48 @@ static const struct tegra_pmc_soc tegra210_pmc_soc = {
> >  	.setup_irq_polarity = tegra20_pmc_setup_irq_polarity,
> >  };
> >  
> > +#define TEGRA186_IO_PAD_TABLE(_pad)						\
> > +	/*   .id                        .dpd    .voltage        .name */	\
> > +	_pad(TEGRA_IO_PAD_CSIA,		0,	UINT_MAX,	"csia")		\
> > +	_pad(TEGRA_IO_PAD_CSIB,		1,	UINT_MAX,	"csib")		\
> > +	_pad(TEGRA_IO_PAD_DSI,		2,	UINT_MAX,	"dsi")		\
> > +	_pad(TEGRA_IO_PAD_MIPI_BIAS,	3,	UINT_MAX,	"mipi-bias")	\
> > +	_pad(TEGRA_IO_PAD_PEX_CLK_BIAS,	4, 	UINT_MAX,	"pex-clk-bias")	\
> > +	_pad(TEGRA_IO_PAD_PEX_CLK3,	5,	UINT_MAX,	"pex-clk3")	\
> > +	_pad(TEGRA_IO_PAD_PEX_CLK2,	6,	UINT_MAX,	"pex-clk2")	\
> > +	_pad(TEGRA_IO_PAD_PEX_CLK1,	7,	UINT_MAX,	"pex-clk1")	\
> > +	_pad(TEGRA_IO_PAD_USB0,		9,	UINT_MAX,	"usb0")		\
> > +	_pad(TEGRA_IO_PAD_USB1,		10,	UINT_MAX,	"usb1")		\
> > +	_pad(TEGRA_IO_PAD_USB2,		11,	UINT_MAX,	"usb2")		\
> > +	_pad(TEGRA_IO_PAD_USB_BIAS,	12,	UINT_MAX,	"usb-bias")	\
> > +	_pad(TEGRA_IO_PAD_UART,		14,	UINT_MAX,	"uart")		\
> > +	_pad(TEGRA_IO_PAD_AUDIO,	17,	UINT_MAX,	"audio")	\
> > +	_pad(TEGRA_IO_PAD_HSIC,		19,	UINT_MAX,	"hsic")		\
> > +	_pad(TEGRA_IO_PAD_DBG,		25,	UINT_MAX,	"dbg")		\
> > +	_pad(TEGRA_IO_PAD_HDMI_DP0,	28,	UINT_MAX,	"hdmi-dp0")	\
> > +	_pad(TEGRA_IO_PAD_HDMI_DP1,	29,	UINT_MAX,	"hdmi-dp1")	\
> > +	_pad(TEGRA_IO_PAD_PEX_CNTRL,	32,	UINT_MAX,	"pex-cntrl")	\
> > +	_pad(TEGRA_IO_PAD_SDMMC2_HV,	34,	5,		"sdmmc2-hv")	\
> > +	_pad(TEGRA_IO_PAD_SDMMC4,	36,	UINT_MAX,	"sdmmc4")	\
> > +	_pad(TEGRA_IO_PAD_CAM,		38,	UINT_MAX,	"cam")		\
> > +	_pad(TEGRA_IO_PAD_DSIB,		40,	UINT_MAX,	"dsib")		\
> > +	_pad(TEGRA_IO_PAD_DSIC,		41,	UINT_MAX,	"dsic")		\
> > +	_pad(TEGRA_IO_PAD_DSID,		42,	UINT_MAX,	"dsid")		\
> > +	_pad(TEGRA_IO_PAD_CSIC,		43,	UINT_MAX,	"csic")		\
> > +	_pad(TEGRA_IO_PAD_CSID,		44,	UINT_MAX,	"csid")		\
> > +	_pad(TEGRA_IO_PAD_CSIE,		45,	UINT_MAX,	"csie")		\
> > +	_pad(TEGRA_IO_PAD_CSIF,		46,	UINT_MAX,	"csif")		\
> > +	_pad(TEGRA_IO_PAD_SPI,		47,	UINT_MAX,	"spi")		\
> > +	_pad(TEGRA_IO_PAD_UFS,		49,	UINT_MAX,	"ufs")		\
> > +	_pad(TEGRA_IO_PAD_DMIC_HV,	52,	2,		"dmic-hv")	\
> > +	_pad(TEGRA_IO_PAD_EDP,		53,	UINT_MAX,	"edp")		\
> > +	_pad(TEGRA_IO_PAD_SDMMC1_HV,	55,	4,		"sdmmc1-hv")	\
> > +	_pad(TEGRA_IO_PAD_SDMMC3_HV,	56,	6,		"sdmmc3-hv")	\
> > +	_pad(TEGRA_IO_PAD_CONN,		60,	UINT_MAX,	"conn")		\
> > +	_pad(TEGRA_IO_PAD_AUDIO_HV,	61,	1,		"audio-hv")
> > +
> >  static const struct tegra_io_pad_soc tegra186_io_pads[] = {
> > -	{ .id = TEGRA_IO_PAD_CSIA, .dpd = 0, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_CSIB, .dpd = 1, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_DSI, .dpd = 2, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_MIPI_BIAS, .dpd = 3, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_PEX_CLK_BIAS, .dpd = 4, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_PEX_CLK3, .dpd = 5, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_PEX_CLK2, .dpd = 6, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_PEX_CLK1, .dpd = 7, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_USB0, .dpd = 9, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_USB1, .dpd = 10, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_USB2, .dpd = 11, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_USB_BIAS, .dpd = 12, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_UART, .dpd = 14, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_AUDIO, .dpd = 17, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_HSIC, .dpd = 19, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_DBG, .dpd = 25, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_HDMI_DP0, .dpd = 28, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_HDMI_DP1, .dpd = 29, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_PEX_CNTRL, .dpd = 32, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_SDMMC2_HV, .dpd = 34, .voltage = 5 },
> > -	{ .id = TEGRA_IO_PAD_SDMMC4, .dpd = 36, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_CAM, .dpd = 38, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_DSIB, .dpd = 40, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_DSIC, .dpd = 41, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_DSID, .dpd = 42, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_CSIC, .dpd = 43, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_CSID, .dpd = 44, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_CSIE, .dpd = 45, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_CSIF, .dpd = 46, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_SPI, .dpd = 47, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_UFS, .dpd = 49, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_DMIC_HV, .dpd = 52, .voltage = 2 },
> > -	{ .id = TEGRA_IO_PAD_EDP, .dpd = 53, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_SDMMC1_HV, .dpd = 55, .voltage = 4 },
> > -	{ .id = TEGRA_IO_PAD_SDMMC3_HV, .dpd = 56, .voltage = 6 },
> > -	{ .id = TEGRA_IO_PAD_CONN, .dpd = 60, .voltage = UINT_MAX },
> > -	{ .id = TEGRA_IO_PAD_AUDIO_HV, .dpd = 61, .voltage = 1 },
> > +	TEGRA186_IO_PAD_TABLE(TEGRA_IO_PAD)
> >  };
> >  
> >  static const struct tegra_pmc_regs tegra186_pmc_regs = {  
> 
> Can you run checkpatch on this and fix up the warnings?
> 
> Cheers
> Jon
> 

I did run checkpatch on it. The remaining errors and warnings are either
about the table rows being too long or about the macro values used to
generate them not being enclosed in parentheses. I felt that fixing them
could be somewhat detrimental to the overall readability. I probably
could fix the errors about macro parens, but I'd rather not give up the
aligned columns of the tables. I guess I can try to get them squeezed a
bit tighter together by not aligning to tab stops.

 -Aapo

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux