QSPI Driver Bug?

eGuy — Thu, 26 May 2011 01:03:49 GMT

I am uing MQX 3.7

In spi_pol_mcf5xxx_qspi.c function _mcf5xxx_qspi_polled_tx_rx:

    /* Setup queue */
    tmp = qspi_ptr->QWR & (~ (MCF5XXX_QSPI_QWR_NEWQP(0xF) | MCF5XXX_QSPI_QWR_ENDQP(0xF)));
    qspi_ptr->QWR = tmp | MCF5XXX_QSPI_QWR_NEWQP(0) | MCF5XXX_QSPI_QWR_ENDQP(data - 1);

here tmp is defined as uint_8. but QWR is 16 bit register, this will slice off the upper 8 bit. the effect is causing chip-select pins goes low in a very short time (20ns)

Change: define tmp as uinit_16.

Similar problem in spi_int_mcf5xxx_qspi.c _mcf5xxx_qspi_int_isr function:

define reg_tmp as uint_16 instead of uint_8

Regards,

Jeff

Re: QSPI Driver Bug?

PetrM — Thu, 26 May 2011 18:58:41 GMT

Thanks for reporting this. The fix will be available in the next release.

Regards,

PetrM

topic Re: QSPI Driver Bug? in MQX Software Solutions

QSPI Driver Bug?

Re: QSPI Driver Bug?