There seems something wrong with pinmux's define in file mux33xx.c

long qin

Hi Vidin

After detailed compared,I think there also prolem. in the file mux33xx.c,

such as

_AM33XX_MUXENTRY(UART0_CTSN, 0, "uart0_ctsn", NULL, "d_can1_tx", "i2c1_sda",
"spi1_d0", NULL, NULL, "gpio1_8"),

in the above statement,mode1 is NULL,but in pinmux, mode 1 is uart4_rxd,

please refer to the attatched image.

also there are many difference between mux33xx.c with pinmux tool.

so i want to know if i can correct the static struct omap_mux am33xx_muxmodes[]?

for example,

_AM33XX_MUXENTRY(UART0_CTSN, 0, "uart0_ctsn", NULL, "d_can1_tx", "i2c1_sda",
"spi1_d0", NULL, NULL, "gpio1_8"),

will be changed to

_AM33XX_MUXENTRY(UART0_CTSN, 0, "uart0_ctsn", "uart4_rxd", "d_can1_tx", "i2c1_sda",
"spi1_d0", "timer7", "pr1_edc_sync0_out", "gpio1_8"),

please confirm it.

Best Regard

Qin Long

over 11 years ago

0 Biser Gatchev-XID over 11 years ago

TI__Guru**** 393215 points

Yes, you can do that. What Linux build/version are you using?

0 Vidin Tsonev-XID over 11 years ago

TI__Intellectual 1340 points

Hi Qin,

The original structure in the TI SDK static struct omap_mux am33xx_muxmodes[] is intended to be used with AM335x EVM boards, so not all options of the pinmux are listed.

However you can modify it as you need for your custom board to work, keeping in mind the pinmux tool valid IO sets.

Please note that pinmux tool will output the needed valid configuration of pinmux settings.

Best regards,

Vidin

0 long qin over 11 years ago in reply to Vidin Tsonev-XID

Intellectual 915 points

Hi Vidin, Biser

I have modified the content of static struct omap_mux am33xx_muxmodes[] to be fit for our custom board,

and it have been run correctly.

but another problem have not still been solved,

my custom board still stopped when ran to "Waiting for root device /dev/mmcblk0p2..."

it means that it is booting rootfs,but not work correctly.

How can i solve the problem?

0 Biser Gatchev-XID over 11 years ago in reply to long qin

TI__Guru**** 393215 points

Please don't double-post. You have another thread opened for that.

Processors

Processors forum

There seems something wrong with pinmux's define in file mux33xx.c