nxp-mcuxpresso / mcux-sdk

MCUXpresso SDK
BSD 3-Clause "New" or "Revised" License
301 stars 136 forks source link

[BUG] SEMC_ConfigureDBI not setting REH2/REL2 #108

Closed marcoosi closed 1 year ago

marcoosi commented 1 year ago

Describe the bug According to i.MX RT1060X Processor Reference Manual, Rev. 1, 05/2022, the DBICR1 register has 6 bits for REH and REL configuration. The SEMC_ConfigureDBI function in fsl_semc.c only writes the lower 4 bits.

To Reproduce

Expected behavior Given that the IMXRT106x have 6 bits to configure the RDX Low and High Times, these should be taken into account.

Screenshots and console output

Additional context

mcuxcc commented 1 year ago

@marcoosi thank you for reporting the issue.

i have forward the issue to internal team, i will update to you if any feedback

zejiang0jason commented 1 year ago

Hi @marcoosi , please help to check the pull request for this issue, thanks: https://github.com/nxp-mcuxpresso/mcux-sdk/pull/111

marcoosi commented 1 year ago

@zejiang0jason the fix in #111 works for me.