Hello Ahmad, On Tue, Apr 02, 2024 at 04:44:33PM +0200, Ahmad Fatoum wrote: > On 02.04.24 15:09, Uwe Kleine-König wrote: > > Signed-off-by: Uwe Kleine-König <u.kleine-koe...@pengutronix.de> > > Please add the version of the reference manual you took these values from.
Good idea. I'd even add it to the source file, too. > > --- > > src/dtblint-imx-pinmux.c | 1317 ++++++++++++++++++++++++++++++++++++++ > > 1 file changed, 1317 insertions(+) > > > > > + .padname = "SD1_DATA5", > > + .swmux_regoffset = 0x0a8, > > + .swmux_reset_default = 0x00000005, > > + .swmux_writeable_mask = 0x00000017, > > + .swpad_regoffset = 0x308, > > + .swpad_reset_default = 0x00000106, > > + .swpad_writeable_mask = 0x000001f6, > > + .swpad_writeable_mask = 0x000001f6, > > This line is duplicated. Yes, this is fixed in v2, together with two other such duplicates. > > + .padname = "ECSPI2_SS0", > > + .swmux_regoffset = 0x1fc, > > + .swmux_reset_default = 0x00000005, > > + .swmux_writeable_mask = 0x00000017, > > + .swpad_regoffset = 0x45c, > > + .swpad_reset_default = 0x00000106, > > + .swpad_writeable_mask = 0x000001f6, > > + .swpad_writeable_mask = 0x000001f6, > > This is also duplicated. > > > + /* regname="AUDIOMIX_PDM_MIC_PDM_BITSTREAM_SELECT_INPUT_0" > > regoffset=0x4c0 reset_default=0x00000000 */ > > + /* regname="AUDIOMIX_PDM_MIC_PDM_BITSTREAM_SELECT_INPUT_1" > > regoffset=0x4c4 reset_default=0x00000000 */ > > + /* regname="AUDIOMIX_PDM_MIC_PDM_BITSTREAM_SELECT_INPUT_2" > > regoffset=0x4c8 reset_default=0x00000000 */ > > + /* regname="AUDIOMIX_PDM_MIC_PDM_BITSTREAM_SELECT_INPUT_3" > > regoffset=0x4cc reset_default=0x00000000 */ > > [snip] > > > + /* regname="USDHC3_STROBE_SELECT_INPUT" regoffset=0x630 > > reset_default=0x00000000 */ > > + /* regname="USDHC3_WP_ON_SELECT_INPUT" regoffset=0x634 > > reset_default=0x00000000 */ > > What are these comments about? The table of swmux and swpad registers originates from the reference manual. These comments are the registers in the IOMUXC that are not relevant for the linter. I kept them because copying these details from the NXP pdf isn't trivial. I can drop them if they are considered to be useless here. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | https://www.pengutronix.de/ |
signature.asc
Description: PGP signature
