diff options
| author | Daniel Hellstrom <daniel@gaisler.com> | 2019-04-12 12:17:57 +0200 |
|---|---|---|
| committer | Daniel Hellstrom <daniel@gaisler.com> | 2021-03-07 16:08:23 +0100 |
| commit | 84fb340ac464c6aba2c5b4f3352c1c8f60466d47 (patch) | |
| tree | 136cf7fb537391d2e5cd0dd0ef975236ec6de0b0 | |
| parent | leon,gr1553b: Only align allocated memory. Verify alignment of memory. (diff) | |
| download | rtems-84fb340ac464c6aba2c5b4f3352c1c8f60466d47.tar.bz2 | |
leon,grspw: fix for SET_PACKET_SIZE
When the DMA table has been allocated dynamically, the IOCTL_SET_PACKETSIZE
will trigger an issue where pDev->rx and pDev->tx are not updated with
the new DMA tables base address. Instead the old pointers are used.
There is no point in reallocting the DMA tables because there is no
configuration option to it. Therefore the DMA tables allocation is
moved to a separate function never called from SET_PACKETSIZE.
Update #4304.
| -rw-r--r-- | bsps/shared/grlib/spw/grspw.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/bsps/shared/grlib/spw/grspw.c b/bsps/shared/grlib/spw/grspw.c index 2e1e8e90e9..dd61b7a472 100644 --- a/bsps/shared/grlib/spw/grspw.c +++ b/bsps/shared/grlib/spw/grspw.c @@ -261,6 +261,7 @@ static void check_rx_errors(GRSPW_DEV *pDev, int ctrl); static void grspw_rxnext(GRSPW_DEV *pDev); static void grspw_interrupt(void *arg); static int grspw_buffer_alloc(GRSPW_DEV *pDev); +static int grspw_dmatables_alloc(GRSPW_DEV *pDev); static rtems_device_driver grspw_initialize( rtems_device_major_number major, @@ -553,6 +554,8 @@ int grspw_device_init(GRSPW_DEV *pDev) if (grspw_buffer_alloc(pDev)) return RTEMS_NO_MEMORY; + if (grspw_dmatables_alloc(pDev)) + return RTEMS_NO_MEMORY; /* Create semaphores */ rtems_semaphore_create( @@ -678,7 +681,11 @@ static int grspw_buffer_alloc(GRSPW_DEV *pDev) (void **)&pDev->ptr_txhbuf0_remote, pDev->txhbufsize * pDev->txbufcnt); } + return 0; +} +static int grspw_dmatables_alloc(GRSPW_DEV *pDev) +{ /* DMA DESCRIPTOR TABLES */ if (pDev->bd_dma_area & 1) { /* Address given in remote address */ |