topic _io_dev_install compile errors in MQX Software Solutions

_io_dev_install compile errors

JKadin — Tue, 26 Apr 2011 22:09:06 GMT

I am trying to use an external SPI flash to store my web page data on a TWRMCF52259.

I have the SPI FLASH working using the BSP's spi0 and MQX fopen ("spi0:", NULL)

I can read, write, and erase the flash using this method.

I am trying now to install an io driver so I can layer on part manager and MFS and I am getting an error when I build with the _io_dev_install

This is my code that gets the error:

_mqx_uint _io_webflash_install(
       char_ptr identifier)
{
_mqx_uint   result;

   result = _io_dev_install(identifier,
       _io_webflash_open,
       _io_webflash_close,
       _io_webflash_read,
       _io_webflash_write,
       _io_webflash_ioctl,
       NULL);

   return result;
}

This is the Error I am getting, and have not been able to solve it with anything I have tried.

>illegal implicit conversion from '__regabi long (struct mqx_file *, unsigned
>char *, long)' to
>'__regabi long (*)(struct mqx_file *, char *, long)'

Does anyony have any ideas?

Thanks for reading

Re: _io_dev_install compile errors

petr_uhlir — Wed, 27 Apr 2011 18:39:34 GMT

Hallo JKadin,

check function prototypes of your _io_webflash_read and _io_webflash_write functions.

It should be

_mqx_int _io_webflash_read(MQX_FILE_PTR, char_ptr, _mqx_int)

Second parameter is char_ptr (char *) and you probably have uchar_ptr (unsigned char *)

Re: _io_dev_install compile errors

JKadin — Wed, 27 Apr 2011 19:17:40 GMT

You were exactly right.

For anyone else looking at this and if I need it again.

memory_write_data was defined as:

uint_32 memory_write_data (MQX_FILE_PTR spifd, uint_32 addr, uint_32 size, uchar_ptr data)

__io_webflash_write was declared as:

_mqx_int __io_webflash_write
(
       FILE_PTR   fd_ptr,
        uchar_ptr   data_ptr,
       _mqx_int   num
                           )
{
IO_DEVICE_STRUCT_PTR io_dev_ptr = fd_ptr->DEV_PTR;
WEB_FLASH_STRUCT_PTR handle_ptr = (WEB_FLASH_STRUCT_PTR)io_dev_ptr->DRIVER_INIT_PTR;
   _mqx_uint   result;
   uint_32       w_addr;

   w_addr = handle_ptr->addr;

   result = memory_write_data (fd_ptr, w_addr, (uint_32)num, data_ptr);

       if(result == num)
           return(MQX_OK);
       else
           return(IO_ERROR);
}

to be clear data_ptr was declared as uchar_ptr because declaring it as a char_ptr caused an error in the call to memory_write_data(). When data_ptr was declared as uchar_ptr the error at memory_write_data went away, and since _io_dev_install was not written yet, that error did not appear ... yet

The way to fix this is to declare data_ptr as a char_ptr to satisfy _io_dev_install() and cast data_ptr to a uchar_ptr in the call to memory_write_data() like so:

memory_write_data (fd_ptr, w_addr, (uint_32)num, (uchar_ptr)data_ptr)