nuttx/drivers/mmcsd/sdio.c

548 lines
12 KiB
C
Raw Permalink Normal View History

/****************************************************************************
* drivers/mmcsd/sdio.c
*
* Licensed to the Apache Software Foundation (ASF) under one or more
* contributor license agreements. See the NOTICE file distributed with
* this work for additional information regarding copyright ownership. The
* ASF licenses this file to you under the Apache License, Version 2.0 (the
* "License"); you may not use this file except in compliance with the
* License. You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
* WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
* License for the specific language governing permissions and limitations
* under the License.
*
****************************************************************************/
/****************************************************************************
* Included Files
****************************************************************************/
2017-03-14 21:06:19 +01:00
#include <debug.h>
#include <errno.h>
#include <inttypes.h>
nuttx: Fix the nightly build warning lpc2148_spi1.c:142:24: warning: initialization of 'uint32_t (*)(struct spi_dev_s *, uint32_t)' {aka 'unsigned int (*)(struct spi_dev_s *, unsigned int)'} from incompatible pointer type 'uint16_t (*)(struct spi_dev_s *, uint16_t)' {aka 'short unsigned int (*)(struct spi_dev_s *, short unsigned int)'} [-Wincompatible-pointer-types] 142 | .send = spi_send, | ^~~~~~~~ lpc2148_spi1.c:142:24: note: (near initialization for 'g_spiops.send') In file included from ieee802154/mac802154_bind.c:49: ieee802154/mac802154_internal.h: In function 'mac802154_setdevmode': ieee802154/mac802154_internal.h:788:42: warning: converting a packed 'enum ieee802154_devmode_e' pointer (alignment 1) to a 'const union ieee802154_attr_u' pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 788 | (FAR const union ieee802154_attr_u *)&mode); | ^~~~~~~~~~~~~~~~~ chip/stm32_hciuart.c: In function 'hciuart_read': chip/stm32_hciuart.c:2104:30: warning: statement with no effect [-Wunused-value] 2104 | ntotal == (ssize_t)ret; | ~~~~~~~^~~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c: In function 'bcmf_wl_auth_event_handler': wireless/ieee80211/bcm43xxx/bcmf_driver.c:579:23: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 579 | type = bcmf_getle32(&event->type); | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c:580:25: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 580 | status = bcmf_getle32(&event->status); | ^~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c: In function 'bcmf_wl_scan_event_handler': wireless/ieee80211/bcm43xxx/bcmf_driver.c:619:25: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 619 | status = bcmf_getle32(&event->status); | ^~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c:620:35: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 620 | escan_result_len = bcmf_getle32(&event->len); | ^~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_bdc.c: In function 'bcmf_bdc_process_event_frame': wireless/ieee80211/bcm43xxx/bcmf_bdc.c:166:27: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 166 | event_id = bcmf_getle32(&event_msg->event.type); | ^~~~~~~~~~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c: In function 'sdio_io_rw_direct': wireless/ieee80211/bcm43xxx/mmc_sdio.c:157:3: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 157 | ret = SDIO_RECVR5(dev, SD_ACMD52, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c: In function 'sdio_io_rw_extended': wireless/ieee80211/bcm43xxx/mmc_sdio.c:239:11: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 239 | ret = SDIO_RECVR5(dev, SD_ACMD53, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:244:11: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 244 | ret = SDIO_RECVR5(dev, SD_ACMD53, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:257:7: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 257 | ret = SDIO_RECVR5(dev, SD_ACMD53, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:265:3: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 265 | SDIO_RECVR1(dev, SD_ACMD52ABRT, (uint32_t *)&resp); | ^~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ chip/stm32_adc.c: In function 'adc_reset': chip/stm32_adc.c:2860:7: warning: unused variable 'ret' [-Wunused-variable] 2860 | int ret; | ^~~ chip/stm32_adc.c: In function 'adc_shutdown': chip/stm32_adc.c:3044:7: warning: unused variable 'ret' [-Wunused-variable] 3044 | int ret; | ^~~ chip/stm32_i2c.c:722:12: warning: 'stm32_i2c_sem_wait_noncancelable' defined but not used [-Wunused-function] 722 | static int stm32_i2c_sem_wait_noncancelable(FAR struct i2c_master_s *dev) | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ wireless/gs2200m.c: In function 'gs2200m_read': wireless/gs2200m.c:727:20: warning: passing argument 1 of 'nxsem_wait' from incompatible pointer type [-Wincompatible-pointer-types] 727 | ret = nxsem_wait(dev); | ^~~ | | | struct gs2200m_dev_s * .config:1207:warning: symbol value '' invalid for TESTING_OSTEST_FPUSIZE platform/audio/cxd56_audio_analog.c:69:13: warning: inline function 'cxd56_audio_clock_is_enabled' declared but never defined 69 | inline bool cxd56_audio_clock_is_enabled(void); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ platform/audio/cxd56_audio_analog.c:68:13: warning: inline function 'cxd56_audio_clock_disable' declared but never defined 68 | inline void cxd56_audio_clock_disable(void); | ^~~~~~~~~~~~~~~~~~~~~~~~~ platform/audio/cxd56_audio_analog.c:67:13: warning: inline function 'cxd56_audio_clock_enable' declared but never defined 67 | inline void cxd56_audio_clock_enable(uint32_t clk, uint32_t div); | ^~~~~~~~~~~~~~~~~~~~~~~~ chip/stm32_adc.c: In function 'adc_reset': chip/stm32_adc.c:1348:7: warning: unused variable 'ret' [-Wunused-variable] 1348 | int ret; | ^~~ chip/stm32_adc.c: In function 'adc_shutdown': chip/stm32_adc.c:1496:7: warning: unused variable 'ret' [-Wunused-variable] 1496 | int ret; | ^~~ chip/stm32_i2c.c:729:12: warning: 'stm32_i2c_sem_wait_uninterruptble' defined but not used [-Wunused-function] 729 | static int stm32_i2c_sem_wait_uninterruptble(FAR struct i2c_master_s *dev) | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ wireless/lpwan/sx127x/sx127x.c:147:52: warning: missing terminating ' character 147 | # warning OOK support is not complete, RX+TX doesn't work yet! | ^ str71_spi.c:435:24: warning: initialization of 'uint32_t (*)(struct spi_dev_s *, uint32_t)' {aka 'unsigned int (*)(struct spi_dev_s *, unsigned int)'} from incompatible pointer type 'uint16_t (*)(struct spi_dev_s *, uint16_t)' {aka 'short unsigned int (*)(struct spi_dev_s *, short unsigned int)'} [-Wincompatible-pointer-types] 435 | .send = spi_send, | ^~~~~~~~ str71_spi.c:435:24: note: (near initialization for 'g_spiops.send') chip/pic32mx-lowconsole.c:147:24: warning: 'pic32mx_getreg' defined but not used [-Wunused-function] static inline uint32_t pic32mx_getreg(uintptr_t uart_base, ^ chip/pic32mx-gpio.c:113:20: warning: 'pic32mx_value' defined but not used [-Wunused-function] static inline bool pic32mx_value(uint16_t pinset) ^ chip/pic32mz-gpio.c:124:20: warning: 'pic32mz_value' defined but not used [-Wunused-function] static inline bool pic32mz_value(pinset_t pinset) ^ chip/pic32mx-usbdev.c:3065:1: warning: 'pic32mx_epreserved' defined but not used [-Wunused-function] pic32mx_epreserved(struct pic32mx_usbdev_s *priv, int epno) ^ mmcsd/mmcsd_spi.c: In function 'mmcsd_mediachanged': mmcsd/mmcsd_spi.c:1938:7: warning: 'return' with a value, in function returning void return ret; ^ In file included from partition/fs_partition.c:42:0: partition/partition.h:66:19: warning: 'read_partition_block' defined but not used [-Wunused-function] static inline int read_partition_block(FAR struct partition_state_s *state, ^ local/local_netpoll.c: In function 'local_pollsetup': local/local_netpoll.c:305:1: warning: label 'pollerr' defined but not used [-Wunused-label] pollerr: ^~~~~~~ Signed-off-by: Xiang Xiao <xiaoxiang@xiaomi.com> Change-Id: If3ea8f32b878aa218072130f7c3018f0d3c1aca5
2020-04-12 15:52:28 +02:00
#include <string.h>
2017-03-14 21:06:19 +01:00
#include <nuttx/compiler.h>
2017-03-14 21:06:19 +01:00
#include <nuttx/arch.h>
#include <nuttx/sdio.h>
#include <nuttx/signal.h>
2017-03-14 21:06:19 +01:00
/****************************************************************************
* Pre-processor Definitions
****************************************************************************/
#define SDIO_CMD53_TIMEOUT_MS 1000
2017-03-14 21:06:19 +01:00
#define SDIO_IDLE_DELAY_MS 50
/****************************************************************************
* Private Types
****************************************************************************/
begin_packed_struct struct sdio_cmd52
{
uint32_t write_data : 8;
uint32_t reserved_8 : 1;
uint32_t register_address : 17;
uint32_t reserved_26 : 1;
uint32_t raw_flag : 1;
uint32_t function_number : 3;
uint32_t rw_flag : 1;
} end_packed_struct;
2017-03-14 21:06:19 +01:00
begin_packed_struct struct sdio_cmd53
{
uint32_t byte_block_count : 9;
uint32_t register_address : 17;
uint32_t op_code : 1;
uint32_t block_mode : 1;
uint32_t function_number : 3;
uint32_t rw_flag : 1;
} end_packed_struct;
2017-03-14 21:06:19 +01:00
begin_packed_struct struct sdio_resp_r5
{
uint32_t data : 8;
begin_packed_struct
struct
{
uint32_t out_of_range : 1;
uint32_t function_number : 1;
uint32_t rfu : 1;
uint32_t error : 1;
uint32_t io_current_state : 2;
uint32_t illegal_command : 1;
uint32_t com_crc_error : 1;
}
end_packed_struct flags;
uint32_t reserved_16 : 16;
} end_packed_struct;
2017-03-14 21:06:19 +01:00
union sdio_cmd5x
{
uint32_t value;
struct sdio_cmd52 cmd52;
struct sdio_cmd53 cmd53;
2017-03-14 21:06:19 +01:00
};
/****************************************************************************
* Private Functions
****************************************************************************/
static int sdio_takelock(FAR struct sdio_dev_s *dev)
{
int ret;
/* Take the lock, giving exclusive access to the driver (perhaps
* waiting)
*/
if (!up_interrupt_context() && !sched_idletask())
{
ret = nxmutex_lock(&dev->mutex);
if (ret < 0)
{
return ret;
}
/* Lock the bus if mutually exclusive access to the
* SDIO bus is required on this platform.
*/
#ifdef CONFIG_SDIO_MUXBUS
SDIO_LOCK(dev, true);
#endif
}
else
{
ret = OK;
}
return ret;
}
static void sdio_givelock(FAR struct sdio_dev_s *dev)
{
if (!up_interrupt_context() && !sched_idletask())
{
/* Release the SDIO bus lock, then the MMC/SD driver mutex in the
* opposite order that they were taken to assure that no deadlock
* conditions will arise.
*/
#ifdef CONFIG_SDIO_MUXBUS
SDIO_LOCK(dev, false);
#endif
nxmutex_unlock(&dev->mutex);
}
}
static int sdio_sendcmdpoll(FAR struct sdio_dev_s *dev,
uint32_t cmd, uint32_t arg)
2017-03-14 21:06:19 +01:00
{
int ret;
/* Send the command */
ret = SDIO_SENDCMD(dev, cmd, arg);
if (ret == OK)
{
/* Then poll-wait until the response is available */
ret = SDIO_WAITRESPONSE(dev, cmd);
if (ret != OK)
{
wlerr("ERROR: Wait for response to cmd: %08" PRIx32
" failed: %d\n",
cmd, ret);
2017-03-14 21:06:19 +01:00
}
}
return ret;
}
/****************************************************************************
* Public Functions
****************************************************************************/
2017-03-14 21:06:19 +01:00
int sdio_io_rw_direct(FAR struct sdio_dev_s *dev, bool write,
uint8_t function, uint32_t address,
uint8_t inb, FAR uint8_t *outb)
2017-03-14 21:06:19 +01:00
{
union sdio_cmd5x arg;
struct sdio_resp_r5 resp;
nuttx: Fix the nightly build warning lpc2148_spi1.c:142:24: warning: initialization of 'uint32_t (*)(struct spi_dev_s *, uint32_t)' {aka 'unsigned int (*)(struct spi_dev_s *, unsigned int)'} from incompatible pointer type 'uint16_t (*)(struct spi_dev_s *, uint16_t)' {aka 'short unsigned int (*)(struct spi_dev_s *, short unsigned int)'} [-Wincompatible-pointer-types] 142 | .send = spi_send, | ^~~~~~~~ lpc2148_spi1.c:142:24: note: (near initialization for 'g_spiops.send') In file included from ieee802154/mac802154_bind.c:49: ieee802154/mac802154_internal.h: In function 'mac802154_setdevmode': ieee802154/mac802154_internal.h:788:42: warning: converting a packed 'enum ieee802154_devmode_e' pointer (alignment 1) to a 'const union ieee802154_attr_u' pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 788 | (FAR const union ieee802154_attr_u *)&mode); | ^~~~~~~~~~~~~~~~~ chip/stm32_hciuart.c: In function 'hciuart_read': chip/stm32_hciuart.c:2104:30: warning: statement with no effect [-Wunused-value] 2104 | ntotal == (ssize_t)ret; | ~~~~~~~^~~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c: In function 'bcmf_wl_auth_event_handler': wireless/ieee80211/bcm43xxx/bcmf_driver.c:579:23: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 579 | type = bcmf_getle32(&event->type); | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c:580:25: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 580 | status = bcmf_getle32(&event->status); | ^~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c: In function 'bcmf_wl_scan_event_handler': wireless/ieee80211/bcm43xxx/bcmf_driver.c:619:25: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 619 | status = bcmf_getle32(&event->status); | ^~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c:620:35: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 620 | escan_result_len = bcmf_getle32(&event->len); | ^~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_bdc.c: In function 'bcmf_bdc_process_event_frame': wireless/ieee80211/bcm43xxx/bcmf_bdc.c:166:27: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 166 | event_id = bcmf_getle32(&event_msg->event.type); | ^~~~~~~~~~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c: In function 'sdio_io_rw_direct': wireless/ieee80211/bcm43xxx/mmc_sdio.c:157:3: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 157 | ret = SDIO_RECVR5(dev, SD_ACMD52, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c: In function 'sdio_io_rw_extended': wireless/ieee80211/bcm43xxx/mmc_sdio.c:239:11: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 239 | ret = SDIO_RECVR5(dev, SD_ACMD53, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:244:11: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 244 | ret = SDIO_RECVR5(dev, SD_ACMD53, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:257:7: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 257 | ret = SDIO_RECVR5(dev, SD_ACMD53, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:265:3: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 265 | SDIO_RECVR1(dev, SD_ACMD52ABRT, (uint32_t *)&resp); | ^~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ chip/stm32_adc.c: In function 'adc_reset': chip/stm32_adc.c:2860:7: warning: unused variable 'ret' [-Wunused-variable] 2860 | int ret; | ^~~ chip/stm32_adc.c: In function 'adc_shutdown': chip/stm32_adc.c:3044:7: warning: unused variable 'ret' [-Wunused-variable] 3044 | int ret; | ^~~ chip/stm32_i2c.c:722:12: warning: 'stm32_i2c_sem_wait_noncancelable' defined but not used [-Wunused-function] 722 | static int stm32_i2c_sem_wait_noncancelable(FAR struct i2c_master_s *dev) | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ wireless/gs2200m.c: In function 'gs2200m_read': wireless/gs2200m.c:727:20: warning: passing argument 1 of 'nxsem_wait' from incompatible pointer type [-Wincompatible-pointer-types] 727 | ret = nxsem_wait(dev); | ^~~ | | | struct gs2200m_dev_s * .config:1207:warning: symbol value '' invalid for TESTING_OSTEST_FPUSIZE platform/audio/cxd56_audio_analog.c:69:13: warning: inline function 'cxd56_audio_clock_is_enabled' declared but never defined 69 | inline bool cxd56_audio_clock_is_enabled(void); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ platform/audio/cxd56_audio_analog.c:68:13: warning: inline function 'cxd56_audio_clock_disable' declared but never defined 68 | inline void cxd56_audio_clock_disable(void); | ^~~~~~~~~~~~~~~~~~~~~~~~~ platform/audio/cxd56_audio_analog.c:67:13: warning: inline function 'cxd56_audio_clock_enable' declared but never defined 67 | inline void cxd56_audio_clock_enable(uint32_t clk, uint32_t div); | ^~~~~~~~~~~~~~~~~~~~~~~~ chip/stm32_adc.c: In function 'adc_reset': chip/stm32_adc.c:1348:7: warning: unused variable 'ret' [-Wunused-variable] 1348 | int ret; | ^~~ chip/stm32_adc.c: In function 'adc_shutdown': chip/stm32_adc.c:1496:7: warning: unused variable 'ret' [-Wunused-variable] 1496 | int ret; | ^~~ chip/stm32_i2c.c:729:12: warning: 'stm32_i2c_sem_wait_uninterruptble' defined but not used [-Wunused-function] 729 | static int stm32_i2c_sem_wait_uninterruptble(FAR struct i2c_master_s *dev) | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ wireless/lpwan/sx127x/sx127x.c:147:52: warning: missing terminating ' character 147 | # warning OOK support is not complete, RX+TX doesn't work yet! | ^ str71_spi.c:435:24: warning: initialization of 'uint32_t (*)(struct spi_dev_s *, uint32_t)' {aka 'unsigned int (*)(struct spi_dev_s *, unsigned int)'} from incompatible pointer type 'uint16_t (*)(struct spi_dev_s *, uint16_t)' {aka 'short unsigned int (*)(struct spi_dev_s *, short unsigned int)'} [-Wincompatible-pointer-types] 435 | .send = spi_send, | ^~~~~~~~ str71_spi.c:435:24: note: (near initialization for 'g_spiops.send') chip/pic32mx-lowconsole.c:147:24: warning: 'pic32mx_getreg' defined but not used [-Wunused-function] static inline uint32_t pic32mx_getreg(uintptr_t uart_base, ^ chip/pic32mx-gpio.c:113:20: warning: 'pic32mx_value' defined but not used [-Wunused-function] static inline bool pic32mx_value(uint16_t pinset) ^ chip/pic32mz-gpio.c:124:20: warning: 'pic32mz_value' defined but not used [-Wunused-function] static inline bool pic32mz_value(pinset_t pinset) ^ chip/pic32mx-usbdev.c:3065:1: warning: 'pic32mx_epreserved' defined but not used [-Wunused-function] pic32mx_epreserved(struct pic32mx_usbdev_s *priv, int epno) ^ mmcsd/mmcsd_spi.c: In function 'mmcsd_mediachanged': mmcsd/mmcsd_spi.c:1938:7: warning: 'return' with a value, in function returning void return ret; ^ In file included from partition/fs_partition.c:42:0: partition/partition.h:66:19: warning: 'read_partition_block' defined but not used [-Wunused-function] static inline int read_partition_block(FAR struct partition_state_s *state, ^ local/local_netpoll.c: In function 'local_pollsetup': local/local_netpoll.c:305:1: warning: label 'pollerr' defined but not used [-Wunused-label] pollerr: ^~~~~~~ Signed-off-by: Xiang Xiao <xiaoxiang@xiaomi.com> Change-Id: If3ea8f32b878aa218072130f7c3018f0d3c1aca5
2020-04-12 15:52:28 +02:00
uint32_t data;
int ret;
/* Setup CMD52 argument */
arg.value = 0;
if (write)
{
arg.cmd52.write_data = inb;
}
else
{
arg.cmd52.write_data = 0;
}
arg.cmd52.register_address = address & 0x1ffff;
arg.cmd52.raw_flag = (write && outb);
arg.cmd52.function_number = function & 7;
arg.cmd52.rw_flag = write;
/* Send CMD52 command */
sdio_takelock(dev);
sdio_sendcmdpoll(dev, SD_ACMD52, arg.value);
nuttx: Fix the nightly build warning lpc2148_spi1.c:142:24: warning: initialization of 'uint32_t (*)(struct spi_dev_s *, uint32_t)' {aka 'unsigned int (*)(struct spi_dev_s *, unsigned int)'} from incompatible pointer type 'uint16_t (*)(struct spi_dev_s *, uint16_t)' {aka 'short unsigned int (*)(struct spi_dev_s *, short unsigned int)'} [-Wincompatible-pointer-types] 142 | .send = spi_send, | ^~~~~~~~ lpc2148_spi1.c:142:24: note: (near initialization for 'g_spiops.send') In file included from ieee802154/mac802154_bind.c:49: ieee802154/mac802154_internal.h: In function 'mac802154_setdevmode': ieee802154/mac802154_internal.h:788:42: warning: converting a packed 'enum ieee802154_devmode_e' pointer (alignment 1) to a 'const union ieee802154_attr_u' pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 788 | (FAR const union ieee802154_attr_u *)&mode); | ^~~~~~~~~~~~~~~~~ chip/stm32_hciuart.c: In function 'hciuart_read': chip/stm32_hciuart.c:2104:30: warning: statement with no effect [-Wunused-value] 2104 | ntotal == (ssize_t)ret; | ~~~~~~~^~~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c: In function 'bcmf_wl_auth_event_handler': wireless/ieee80211/bcm43xxx/bcmf_driver.c:579:23: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 579 | type = bcmf_getle32(&event->type); | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c:580:25: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 580 | status = bcmf_getle32(&event->status); | ^~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c: In function 'bcmf_wl_scan_event_handler': wireless/ieee80211/bcm43xxx/bcmf_driver.c:619:25: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 619 | status = bcmf_getle32(&event->status); | ^~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c:620:35: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 620 | escan_result_len = bcmf_getle32(&event->len); | ^~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_bdc.c: In function 'bcmf_bdc_process_event_frame': wireless/ieee80211/bcm43xxx/bcmf_bdc.c:166:27: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 166 | event_id = bcmf_getle32(&event_msg->event.type); | ^~~~~~~~~~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c: In function 'sdio_io_rw_direct': wireless/ieee80211/bcm43xxx/mmc_sdio.c:157:3: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 157 | ret = SDIO_RECVR5(dev, SD_ACMD52, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c: In function 'sdio_io_rw_extended': wireless/ieee80211/bcm43xxx/mmc_sdio.c:239:11: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 239 | ret = SDIO_RECVR5(dev, SD_ACMD53, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:244:11: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 244 | ret = SDIO_RECVR5(dev, SD_ACMD53, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:257:7: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 257 | ret = SDIO_RECVR5(dev, SD_ACMD53, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:265:3: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 265 | SDIO_RECVR1(dev, SD_ACMD52ABRT, (uint32_t *)&resp); | ^~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ chip/stm32_adc.c: In function 'adc_reset': chip/stm32_adc.c:2860:7: warning: unused variable 'ret' [-Wunused-variable] 2860 | int ret; | ^~~ chip/stm32_adc.c: In function 'adc_shutdown': chip/stm32_adc.c:3044:7: warning: unused variable 'ret' [-Wunused-variable] 3044 | int ret; | ^~~ chip/stm32_i2c.c:722:12: warning: 'stm32_i2c_sem_wait_noncancelable' defined but not used [-Wunused-function] 722 | static int stm32_i2c_sem_wait_noncancelable(FAR struct i2c_master_s *dev) | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ wireless/gs2200m.c: In function 'gs2200m_read': wireless/gs2200m.c:727:20: warning: passing argument 1 of 'nxsem_wait' from incompatible pointer type [-Wincompatible-pointer-types] 727 | ret = nxsem_wait(dev); | ^~~ | | | struct gs2200m_dev_s * .config:1207:warning: symbol value '' invalid for TESTING_OSTEST_FPUSIZE platform/audio/cxd56_audio_analog.c:69:13: warning: inline function 'cxd56_audio_clock_is_enabled' declared but never defined 69 | inline bool cxd56_audio_clock_is_enabled(void); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ platform/audio/cxd56_audio_analog.c:68:13: warning: inline function 'cxd56_audio_clock_disable' declared but never defined 68 | inline void cxd56_audio_clock_disable(void); | ^~~~~~~~~~~~~~~~~~~~~~~~~ platform/audio/cxd56_audio_analog.c:67:13: warning: inline function 'cxd56_audio_clock_enable' declared but never defined 67 | inline void cxd56_audio_clock_enable(uint32_t clk, uint32_t div); | ^~~~~~~~~~~~~~~~~~~~~~~~ chip/stm32_adc.c: In function 'adc_reset': chip/stm32_adc.c:1348:7: warning: unused variable 'ret' [-Wunused-variable] 1348 | int ret; | ^~~ chip/stm32_adc.c: In function 'adc_shutdown': chip/stm32_adc.c:1496:7: warning: unused variable 'ret' [-Wunused-variable] 1496 | int ret; | ^~~ chip/stm32_i2c.c:729:12: warning: 'stm32_i2c_sem_wait_uninterruptble' defined but not used [-Wunused-function] 729 | static int stm32_i2c_sem_wait_uninterruptble(FAR struct i2c_master_s *dev) | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ wireless/lpwan/sx127x/sx127x.c:147:52: warning: missing terminating ' character 147 | # warning OOK support is not complete, RX+TX doesn't work yet! | ^ str71_spi.c:435:24: warning: initialization of 'uint32_t (*)(struct spi_dev_s *, uint32_t)' {aka 'unsigned int (*)(struct spi_dev_s *, unsigned int)'} from incompatible pointer type 'uint16_t (*)(struct spi_dev_s *, uint16_t)' {aka 'short unsigned int (*)(struct spi_dev_s *, short unsigned int)'} [-Wincompatible-pointer-types] 435 | .send = spi_send, | ^~~~~~~~ str71_spi.c:435:24: note: (near initialization for 'g_spiops.send') chip/pic32mx-lowconsole.c:147:24: warning: 'pic32mx_getreg' defined but not used [-Wunused-function] static inline uint32_t pic32mx_getreg(uintptr_t uart_base, ^ chip/pic32mx-gpio.c:113:20: warning: 'pic32mx_value' defined but not used [-Wunused-function] static inline bool pic32mx_value(uint16_t pinset) ^ chip/pic32mz-gpio.c:124:20: warning: 'pic32mz_value' defined but not used [-Wunused-function] static inline bool pic32mz_value(pinset_t pinset) ^ chip/pic32mx-usbdev.c:3065:1: warning: 'pic32mx_epreserved' defined but not used [-Wunused-function] pic32mx_epreserved(struct pic32mx_usbdev_s *priv, int epno) ^ mmcsd/mmcsd_spi.c: In function 'mmcsd_mediachanged': mmcsd/mmcsd_spi.c:1938:7: warning: 'return' with a value, in function returning void return ret; ^ In file included from partition/fs_partition.c:42:0: partition/partition.h:66:19: warning: 'read_partition_block' defined but not used [-Wunused-function] static inline int read_partition_block(FAR struct partition_state_s *state, ^ local/local_netpoll.c: In function 'local_pollsetup': local/local_netpoll.c:305:1: warning: label 'pollerr' defined but not used [-Wunused-label] pollerr: ^~~~~~~ Signed-off-by: Xiang Xiao <xiaoxiang@xiaomi.com> Change-Id: If3ea8f32b878aa218072130f7c3018f0d3c1aca5
2020-04-12 15:52:28 +02:00
ret = SDIO_RECVR5(dev, SD_ACMD52, &data);
sdio_givelock(dev);
if (ret != OK)
{
wlerr("ERROR: SDIO_RECVR5 failed %d\n", ret);
return ret;
}
nuttx: Fix the nightly build warning lpc2148_spi1.c:142:24: warning: initialization of 'uint32_t (*)(struct spi_dev_s *, uint32_t)' {aka 'unsigned int (*)(struct spi_dev_s *, unsigned int)'} from incompatible pointer type 'uint16_t (*)(struct spi_dev_s *, uint16_t)' {aka 'short unsigned int (*)(struct spi_dev_s *, short unsigned int)'} [-Wincompatible-pointer-types] 142 | .send = spi_send, | ^~~~~~~~ lpc2148_spi1.c:142:24: note: (near initialization for 'g_spiops.send') In file included from ieee802154/mac802154_bind.c:49: ieee802154/mac802154_internal.h: In function 'mac802154_setdevmode': ieee802154/mac802154_internal.h:788:42: warning: converting a packed 'enum ieee802154_devmode_e' pointer (alignment 1) to a 'const union ieee802154_attr_u' pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 788 | (FAR const union ieee802154_attr_u *)&mode); | ^~~~~~~~~~~~~~~~~ chip/stm32_hciuart.c: In function 'hciuart_read': chip/stm32_hciuart.c:2104:30: warning: statement with no effect [-Wunused-value] 2104 | ntotal == (ssize_t)ret; | ~~~~~~~^~~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c: In function 'bcmf_wl_auth_event_handler': wireless/ieee80211/bcm43xxx/bcmf_driver.c:579:23: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 579 | type = bcmf_getle32(&event->type); | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c:580:25: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 580 | status = bcmf_getle32(&event->status); | ^~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c: In function 'bcmf_wl_scan_event_handler': wireless/ieee80211/bcm43xxx/bcmf_driver.c:619:25: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 619 | status = bcmf_getle32(&event->status); | ^~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c:620:35: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 620 | escan_result_len = bcmf_getle32(&event->len); | ^~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_bdc.c: In function 'bcmf_bdc_process_event_frame': wireless/ieee80211/bcm43xxx/bcmf_bdc.c:166:27: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 166 | event_id = bcmf_getle32(&event_msg->event.type); | ^~~~~~~~~~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c: In function 'sdio_io_rw_direct': wireless/ieee80211/bcm43xxx/mmc_sdio.c:157:3: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 157 | ret = SDIO_RECVR5(dev, SD_ACMD52, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c: In function 'sdio_io_rw_extended': wireless/ieee80211/bcm43xxx/mmc_sdio.c:239:11: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 239 | ret = SDIO_RECVR5(dev, SD_ACMD53, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:244:11: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 244 | ret = SDIO_RECVR5(dev, SD_ACMD53, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:257:7: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 257 | ret = SDIO_RECVR5(dev, SD_ACMD53, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:265:3: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 265 | SDIO_RECVR1(dev, SD_ACMD52ABRT, (uint32_t *)&resp); | ^~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ chip/stm32_adc.c: In function 'adc_reset': chip/stm32_adc.c:2860:7: warning: unused variable 'ret' [-Wunused-variable] 2860 | int ret; | ^~~ chip/stm32_adc.c: In function 'adc_shutdown': chip/stm32_adc.c:3044:7: warning: unused variable 'ret' [-Wunused-variable] 3044 | int ret; | ^~~ chip/stm32_i2c.c:722:12: warning: 'stm32_i2c_sem_wait_noncancelable' defined but not used [-Wunused-function] 722 | static int stm32_i2c_sem_wait_noncancelable(FAR struct i2c_master_s *dev) | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ wireless/gs2200m.c: In function 'gs2200m_read': wireless/gs2200m.c:727:20: warning: passing argument 1 of 'nxsem_wait' from incompatible pointer type [-Wincompatible-pointer-types] 727 | ret = nxsem_wait(dev); | ^~~ | | | struct gs2200m_dev_s * .config:1207:warning: symbol value '' invalid for TESTING_OSTEST_FPUSIZE platform/audio/cxd56_audio_analog.c:69:13: warning: inline function 'cxd56_audio_clock_is_enabled' declared but never defined 69 | inline bool cxd56_audio_clock_is_enabled(void); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ platform/audio/cxd56_audio_analog.c:68:13: warning: inline function 'cxd56_audio_clock_disable' declared but never defined 68 | inline void cxd56_audio_clock_disable(void); | ^~~~~~~~~~~~~~~~~~~~~~~~~ platform/audio/cxd56_audio_analog.c:67:13: warning: inline function 'cxd56_audio_clock_enable' declared but never defined 67 | inline void cxd56_audio_clock_enable(uint32_t clk, uint32_t div); | ^~~~~~~~~~~~~~~~~~~~~~~~ chip/stm32_adc.c: In function 'adc_reset': chip/stm32_adc.c:1348:7: warning: unused variable 'ret' [-Wunused-variable] 1348 | int ret; | ^~~ chip/stm32_adc.c: In function 'adc_shutdown': chip/stm32_adc.c:1496:7: warning: unused variable 'ret' [-Wunused-variable] 1496 | int ret; | ^~~ chip/stm32_i2c.c:729:12: warning: 'stm32_i2c_sem_wait_uninterruptble' defined but not used [-Wunused-function] 729 | static int stm32_i2c_sem_wait_uninterruptble(FAR struct i2c_master_s *dev) | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ wireless/lpwan/sx127x/sx127x.c:147:52: warning: missing terminating ' character 147 | # warning OOK support is not complete, RX+TX doesn't work yet! | ^ str71_spi.c:435:24: warning: initialization of 'uint32_t (*)(struct spi_dev_s *, uint32_t)' {aka 'unsigned int (*)(struct spi_dev_s *, unsigned int)'} from incompatible pointer type 'uint16_t (*)(struct spi_dev_s *, uint16_t)' {aka 'short unsigned int (*)(struct spi_dev_s *, short unsigned int)'} [-Wincompatible-pointer-types] 435 | .send = spi_send, | ^~~~~~~~ str71_spi.c:435:24: note: (near initialization for 'g_spiops.send') chip/pic32mx-lowconsole.c:147:24: warning: 'pic32mx_getreg' defined but not used [-Wunused-function] static inline uint32_t pic32mx_getreg(uintptr_t uart_base, ^ chip/pic32mx-gpio.c:113:20: warning: 'pic32mx_value' defined but not used [-Wunused-function] static inline bool pic32mx_value(uint16_t pinset) ^ chip/pic32mz-gpio.c:124:20: warning: 'pic32mz_value' defined but not used [-Wunused-function] static inline bool pic32mz_value(pinset_t pinset) ^ chip/pic32mx-usbdev.c:3065:1: warning: 'pic32mx_epreserved' defined but not used [-Wunused-function] pic32mx_epreserved(struct pic32mx_usbdev_s *priv, int epno) ^ mmcsd/mmcsd_spi.c: In function 'mmcsd_mediachanged': mmcsd/mmcsd_spi.c:1938:7: warning: 'return' with a value, in function returning void return ret; ^ In file included from partition/fs_partition.c:42:0: partition/partition.h:66:19: warning: 'read_partition_block' defined but not used [-Wunused-function] static inline int read_partition_block(FAR struct partition_state_s *state, ^ local/local_netpoll.c: In function 'local_pollsetup': local/local_netpoll.c:305:1: warning: label 'pollerr' defined but not used [-Wunused-label] pollerr: ^~~~~~~ Signed-off-by: Xiang Xiao <xiaoxiang@xiaomi.com> Change-Id: If3ea8f32b878aa218072130f7c3018f0d3c1aca5
2020-04-12 15:52:28 +02:00
memcpy(&resp, &data, sizeof(resp));
/* Check for errors */
if (resp.flags.error)
{
return -EIO;
}
if (resp.flags.function_number || resp.flags.out_of_range)
{
return -EINVAL;
}
/* Write output byte */
if (outb)
{
*outb = resp.data & 0xff;
}
return OK;
2017-03-14 21:06:19 +01:00
}
2017-03-20 22:40:25 +01:00
int sdio_io_rw_extended(FAR struct sdio_dev_s *dev, bool write,
uint8_t function, uint32_t address,
bool inc_addr, FAR uint8_t *buf,
2017-03-20 22:40:25 +01:00
unsigned int blocklen, unsigned int nblocks)
{
union sdio_cmd5x arg;
struct sdio_resp_r5 resp;
nuttx: Fix the nightly build warning lpc2148_spi1.c:142:24: warning: initialization of 'uint32_t (*)(struct spi_dev_s *, uint32_t)' {aka 'unsigned int (*)(struct spi_dev_s *, unsigned int)'} from incompatible pointer type 'uint16_t (*)(struct spi_dev_s *, uint16_t)' {aka 'short unsigned int (*)(struct spi_dev_s *, short unsigned int)'} [-Wincompatible-pointer-types] 142 | .send = spi_send, | ^~~~~~~~ lpc2148_spi1.c:142:24: note: (near initialization for 'g_spiops.send') In file included from ieee802154/mac802154_bind.c:49: ieee802154/mac802154_internal.h: In function 'mac802154_setdevmode': ieee802154/mac802154_internal.h:788:42: warning: converting a packed 'enum ieee802154_devmode_e' pointer (alignment 1) to a 'const union ieee802154_attr_u' pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 788 | (FAR const union ieee802154_attr_u *)&mode); | ^~~~~~~~~~~~~~~~~ chip/stm32_hciuart.c: In function 'hciuart_read': chip/stm32_hciuart.c:2104:30: warning: statement with no effect [-Wunused-value] 2104 | ntotal == (ssize_t)ret; | ~~~~~~~^~~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c: In function 'bcmf_wl_auth_event_handler': wireless/ieee80211/bcm43xxx/bcmf_driver.c:579:23: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 579 | type = bcmf_getle32(&event->type); | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c:580:25: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 580 | status = bcmf_getle32(&event->status); | ^~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c: In function 'bcmf_wl_scan_event_handler': wireless/ieee80211/bcm43xxx/bcmf_driver.c:619:25: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 619 | status = bcmf_getle32(&event->status); | ^~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c:620:35: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 620 | escan_result_len = bcmf_getle32(&event->len); | ^~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_bdc.c: In function 'bcmf_bdc_process_event_frame': wireless/ieee80211/bcm43xxx/bcmf_bdc.c:166:27: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 166 | event_id = bcmf_getle32(&event_msg->event.type); | ^~~~~~~~~~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c: In function 'sdio_io_rw_direct': wireless/ieee80211/bcm43xxx/mmc_sdio.c:157:3: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 157 | ret = SDIO_RECVR5(dev, SD_ACMD52, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c: In function 'sdio_io_rw_extended': wireless/ieee80211/bcm43xxx/mmc_sdio.c:239:11: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 239 | ret = SDIO_RECVR5(dev, SD_ACMD53, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:244:11: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 244 | ret = SDIO_RECVR5(dev, SD_ACMD53, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:257:7: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 257 | ret = SDIO_RECVR5(dev, SD_ACMD53, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:265:3: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 265 | SDIO_RECVR1(dev, SD_ACMD52ABRT, (uint32_t *)&resp); | ^~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ chip/stm32_adc.c: In function 'adc_reset': chip/stm32_adc.c:2860:7: warning: unused variable 'ret' [-Wunused-variable] 2860 | int ret; | ^~~ chip/stm32_adc.c: In function 'adc_shutdown': chip/stm32_adc.c:3044:7: warning: unused variable 'ret' [-Wunused-variable] 3044 | int ret; | ^~~ chip/stm32_i2c.c:722:12: warning: 'stm32_i2c_sem_wait_noncancelable' defined but not used [-Wunused-function] 722 | static int stm32_i2c_sem_wait_noncancelable(FAR struct i2c_master_s *dev) | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ wireless/gs2200m.c: In function 'gs2200m_read': wireless/gs2200m.c:727:20: warning: passing argument 1 of 'nxsem_wait' from incompatible pointer type [-Wincompatible-pointer-types] 727 | ret = nxsem_wait(dev); | ^~~ | | | struct gs2200m_dev_s * .config:1207:warning: symbol value '' invalid for TESTING_OSTEST_FPUSIZE platform/audio/cxd56_audio_analog.c:69:13: warning: inline function 'cxd56_audio_clock_is_enabled' declared but never defined 69 | inline bool cxd56_audio_clock_is_enabled(void); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ platform/audio/cxd56_audio_analog.c:68:13: warning: inline function 'cxd56_audio_clock_disable' declared but never defined 68 | inline void cxd56_audio_clock_disable(void); | ^~~~~~~~~~~~~~~~~~~~~~~~~ platform/audio/cxd56_audio_analog.c:67:13: warning: inline function 'cxd56_audio_clock_enable' declared but never defined 67 | inline void cxd56_audio_clock_enable(uint32_t clk, uint32_t div); | ^~~~~~~~~~~~~~~~~~~~~~~~ chip/stm32_adc.c: In function 'adc_reset': chip/stm32_adc.c:1348:7: warning: unused variable 'ret' [-Wunused-variable] 1348 | int ret; | ^~~ chip/stm32_adc.c: In function 'adc_shutdown': chip/stm32_adc.c:1496:7: warning: unused variable 'ret' [-Wunused-variable] 1496 | int ret; | ^~~ chip/stm32_i2c.c:729:12: warning: 'stm32_i2c_sem_wait_uninterruptble' defined but not used [-Wunused-function] 729 | static int stm32_i2c_sem_wait_uninterruptble(FAR struct i2c_master_s *dev) | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ wireless/lpwan/sx127x/sx127x.c:147:52: warning: missing terminating ' character 147 | # warning OOK support is not complete, RX+TX doesn't work yet! | ^ str71_spi.c:435:24: warning: initialization of 'uint32_t (*)(struct spi_dev_s *, uint32_t)' {aka 'unsigned int (*)(struct spi_dev_s *, unsigned int)'} from incompatible pointer type 'uint16_t (*)(struct spi_dev_s *, uint16_t)' {aka 'short unsigned int (*)(struct spi_dev_s *, short unsigned int)'} [-Wincompatible-pointer-types] 435 | .send = spi_send, | ^~~~~~~~ str71_spi.c:435:24: note: (near initialization for 'g_spiops.send') chip/pic32mx-lowconsole.c:147:24: warning: 'pic32mx_getreg' defined but not used [-Wunused-function] static inline uint32_t pic32mx_getreg(uintptr_t uart_base, ^ chip/pic32mx-gpio.c:113:20: warning: 'pic32mx_value' defined but not used [-Wunused-function] static inline bool pic32mx_value(uint16_t pinset) ^ chip/pic32mz-gpio.c:124:20: warning: 'pic32mz_value' defined but not used [-Wunused-function] static inline bool pic32mz_value(pinset_t pinset) ^ chip/pic32mx-usbdev.c:3065:1: warning: 'pic32mx_epreserved' defined but not used [-Wunused-function] pic32mx_epreserved(struct pic32mx_usbdev_s *priv, int epno) ^ mmcsd/mmcsd_spi.c: In function 'mmcsd_mediachanged': mmcsd/mmcsd_spi.c:1938:7: warning: 'return' with a value, in function returning void return ret; ^ In file included from partition/fs_partition.c:42:0: partition/partition.h:66:19: warning: 'read_partition_block' defined but not used [-Wunused-function] static inline int read_partition_block(FAR struct partition_state_s *state, ^ local/local_netpoll.c: In function 'local_pollsetup': local/local_netpoll.c:305:1: warning: label 'pollerr' defined but not used [-Wunused-label] pollerr: ^~~~~~~ Signed-off-by: Xiang Xiao <xiaoxiang@xiaomi.com> Change-Id: If3ea8f32b878aa218072130f7c3018f0d3c1aca5
2020-04-12 15:52:28 +02:00
uint32_t data;
int ret;
sdio_eventset_t wkupevent;
/* Setup CMD53 argument */
arg.value = 0;
arg.cmd53.register_address = address & 0x1ffff;
arg.cmd53.op_code = inc_addr;
arg.cmd53.function_number = function & 7;
arg.cmd53.rw_flag = write;
if (nblocks == 0)
{
/* Use byte mode */
arg.cmd53.block_mode = 0;
arg.cmd53.byte_block_count = (blocklen == 512) ? 0 : blocklen;
nblocks = 1;
}
else
{
/* Use block mode */
arg.cmd53.block_mode = 1;
arg.cmd53.byte_block_count = nblocks;
}
sdio_takelock(dev);
/* Send CMD53 command */
SDIO_BLOCKSETUP(dev, blocklen, nblocks);
SDIO_WAITENABLE(dev,
SDIOWAIT_TRANSFERDONE | SDIOWAIT_TIMEOUT | SDIOWAIT_ERROR,
SDIO_CMD53_TIMEOUT_MS);
if (write)
{
wlinfo("prep write %d %d\n", blocklen, nblocks);
/* Get the capabilities of the SDIO hardware */
if ((SDIO_CAPABILITIES(dev) & SDIO_CAPS_DMABEFOREWRITE) != 0)
{
SDIO_DMASENDSETUP(dev, buf, blocklen * nblocks);
SDIO_SENDCMD(dev, SD_ACMD53WR, arg.value);
wkupevent = SDIO_EVENTWAIT(dev);
ret = SDIO_RECVR5(dev, SD_ACMD53WR, &data);
}
else
{
sdio_sendcmdpoll(dev, SD_ACMD53WR, arg.value);
ret = SDIO_RECVR5(dev, SD_ACMD53WR, &data);
SDIO_DMASENDSETUP(dev, buf, blocklen * nblocks);
wkupevent = SDIO_EVENTWAIT(dev);
}
}
else
{
wlinfo("prep read %d\n", blocklen * nblocks);
SDIO_DMARECVSETUP(dev, buf, blocklen * nblocks);
SDIO_SENDCMD(dev, SD_ACMD53RD, arg.value);
wkupevent = SDIO_EVENTWAIT(dev);
ret = SDIO_RECVR5(dev, SD_ACMD53RD, &data);
}
wlinfo("Transaction ends\n");
sdio_sendcmdpoll(dev, SD_ACMD52ABRT, 0);
/* There may not be a response to this, so don't look for one */
nuttx: Fix the nightly build warning lpc2148_spi1.c:142:24: warning: initialization of 'uint32_t (*)(struct spi_dev_s *, uint32_t)' {aka 'unsigned int (*)(struct spi_dev_s *, unsigned int)'} from incompatible pointer type 'uint16_t (*)(struct spi_dev_s *, uint16_t)' {aka 'short unsigned int (*)(struct spi_dev_s *, short unsigned int)'} [-Wincompatible-pointer-types] 142 | .send = spi_send, | ^~~~~~~~ lpc2148_spi1.c:142:24: note: (near initialization for 'g_spiops.send') In file included from ieee802154/mac802154_bind.c:49: ieee802154/mac802154_internal.h: In function 'mac802154_setdevmode': ieee802154/mac802154_internal.h:788:42: warning: converting a packed 'enum ieee802154_devmode_e' pointer (alignment 1) to a 'const union ieee802154_attr_u' pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 788 | (FAR const union ieee802154_attr_u *)&mode); | ^~~~~~~~~~~~~~~~~ chip/stm32_hciuart.c: In function 'hciuart_read': chip/stm32_hciuart.c:2104:30: warning: statement with no effect [-Wunused-value] 2104 | ntotal == (ssize_t)ret; | ~~~~~~~^~~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c: In function 'bcmf_wl_auth_event_handler': wireless/ieee80211/bcm43xxx/bcmf_driver.c:579:23: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 579 | type = bcmf_getle32(&event->type); | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c:580:25: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 580 | status = bcmf_getle32(&event->status); | ^~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c: In function 'bcmf_wl_scan_event_handler': wireless/ieee80211/bcm43xxx/bcmf_driver.c:619:25: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 619 | status = bcmf_getle32(&event->status); | ^~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c:620:35: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 620 | escan_result_len = bcmf_getle32(&event->len); | ^~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_bdc.c: In function 'bcmf_bdc_process_event_frame': wireless/ieee80211/bcm43xxx/bcmf_bdc.c:166:27: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 166 | event_id = bcmf_getle32(&event_msg->event.type); | ^~~~~~~~~~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c: In function 'sdio_io_rw_direct': wireless/ieee80211/bcm43xxx/mmc_sdio.c:157:3: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 157 | ret = SDIO_RECVR5(dev, SD_ACMD52, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c: In function 'sdio_io_rw_extended': wireless/ieee80211/bcm43xxx/mmc_sdio.c:239:11: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 239 | ret = SDIO_RECVR5(dev, SD_ACMD53, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:244:11: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 244 | ret = SDIO_RECVR5(dev, SD_ACMD53, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:257:7: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 257 | ret = SDIO_RECVR5(dev, SD_ACMD53, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:265:3: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 265 | SDIO_RECVR1(dev, SD_ACMD52ABRT, (uint32_t *)&resp); | ^~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ chip/stm32_adc.c: In function 'adc_reset': chip/stm32_adc.c:2860:7: warning: unused variable 'ret' [-Wunused-variable] 2860 | int ret; | ^~~ chip/stm32_adc.c: In function 'adc_shutdown': chip/stm32_adc.c:3044:7: warning: unused variable 'ret' [-Wunused-variable] 3044 | int ret; | ^~~ chip/stm32_i2c.c:722:12: warning: 'stm32_i2c_sem_wait_noncancelable' defined but not used [-Wunused-function] 722 | static int stm32_i2c_sem_wait_noncancelable(FAR struct i2c_master_s *dev) | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ wireless/gs2200m.c: In function 'gs2200m_read': wireless/gs2200m.c:727:20: warning: passing argument 1 of 'nxsem_wait' from incompatible pointer type [-Wincompatible-pointer-types] 727 | ret = nxsem_wait(dev); | ^~~ | | | struct gs2200m_dev_s * .config:1207:warning: symbol value '' invalid for TESTING_OSTEST_FPUSIZE platform/audio/cxd56_audio_analog.c:69:13: warning: inline function 'cxd56_audio_clock_is_enabled' declared but never defined 69 | inline bool cxd56_audio_clock_is_enabled(void); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ platform/audio/cxd56_audio_analog.c:68:13: warning: inline function 'cxd56_audio_clock_disable' declared but never defined 68 | inline void cxd56_audio_clock_disable(void); | ^~~~~~~~~~~~~~~~~~~~~~~~~ platform/audio/cxd56_audio_analog.c:67:13: warning: inline function 'cxd56_audio_clock_enable' declared but never defined 67 | inline void cxd56_audio_clock_enable(uint32_t clk, uint32_t div); | ^~~~~~~~~~~~~~~~~~~~~~~~ chip/stm32_adc.c: In function 'adc_reset': chip/stm32_adc.c:1348:7: warning: unused variable 'ret' [-Wunused-variable] 1348 | int ret; | ^~~ chip/stm32_adc.c: In function 'adc_shutdown': chip/stm32_adc.c:1496:7: warning: unused variable 'ret' [-Wunused-variable] 1496 | int ret; | ^~~ chip/stm32_i2c.c:729:12: warning: 'stm32_i2c_sem_wait_uninterruptble' defined but not used [-Wunused-function] 729 | static int stm32_i2c_sem_wait_uninterruptble(FAR struct i2c_master_s *dev) | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ wireless/lpwan/sx127x/sx127x.c:147:52: warning: missing terminating ' character 147 | # warning OOK support is not complete, RX+TX doesn't work yet! | ^ str71_spi.c:435:24: warning: initialization of 'uint32_t (*)(struct spi_dev_s *, uint32_t)' {aka 'unsigned int (*)(struct spi_dev_s *, unsigned int)'} from incompatible pointer type 'uint16_t (*)(struct spi_dev_s *, uint16_t)' {aka 'short unsigned int (*)(struct spi_dev_s *, short unsigned int)'} [-Wincompatible-pointer-types] 435 | .send = spi_send, | ^~~~~~~~ str71_spi.c:435:24: note: (near initialization for 'g_spiops.send') chip/pic32mx-lowconsole.c:147:24: warning: 'pic32mx_getreg' defined but not used [-Wunused-function] static inline uint32_t pic32mx_getreg(uintptr_t uart_base, ^ chip/pic32mx-gpio.c:113:20: warning: 'pic32mx_value' defined but not used [-Wunused-function] static inline bool pic32mx_value(uint16_t pinset) ^ chip/pic32mz-gpio.c:124:20: warning: 'pic32mz_value' defined but not used [-Wunused-function] static inline bool pic32mz_value(pinset_t pinset) ^ chip/pic32mx-usbdev.c:3065:1: warning: 'pic32mx_epreserved' defined but not used [-Wunused-function] pic32mx_epreserved(struct pic32mx_usbdev_s *priv, int epno) ^ mmcsd/mmcsd_spi.c: In function 'mmcsd_mediachanged': mmcsd/mmcsd_spi.c:1938:7: warning: 'return' with a value, in function returning void return ret; ^ In file included from partition/fs_partition.c:42:0: partition/partition.h:66:19: warning: 'read_partition_block' defined but not used [-Wunused-function] static inline int read_partition_block(FAR struct partition_state_s *state, ^ local/local_netpoll.c: In function 'local_pollsetup': local/local_netpoll.c:305:1: warning: label 'pollerr' defined but not used [-Wunused-label] pollerr: ^~~~~~~ Signed-off-by: Xiang Xiao <xiaoxiang@xiaomi.com> Change-Id: If3ea8f32b878aa218072130f7c3018f0d3c1aca5
2020-04-12 15:52:28 +02:00
SDIO_RECVR1(dev, SD_ACMD52ABRT, &data);
sdio_givelock(dev);
if (ret != OK)
{
wlerr("ERROR: SDIO_RECVR5 failed %d\n", ret);
return ret;
}
nuttx: Fix the nightly build warning lpc2148_spi1.c:142:24: warning: initialization of 'uint32_t (*)(struct spi_dev_s *, uint32_t)' {aka 'unsigned int (*)(struct spi_dev_s *, unsigned int)'} from incompatible pointer type 'uint16_t (*)(struct spi_dev_s *, uint16_t)' {aka 'short unsigned int (*)(struct spi_dev_s *, short unsigned int)'} [-Wincompatible-pointer-types] 142 | .send = spi_send, | ^~~~~~~~ lpc2148_spi1.c:142:24: note: (near initialization for 'g_spiops.send') In file included from ieee802154/mac802154_bind.c:49: ieee802154/mac802154_internal.h: In function 'mac802154_setdevmode': ieee802154/mac802154_internal.h:788:42: warning: converting a packed 'enum ieee802154_devmode_e' pointer (alignment 1) to a 'const union ieee802154_attr_u' pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 788 | (FAR const union ieee802154_attr_u *)&mode); | ^~~~~~~~~~~~~~~~~ chip/stm32_hciuart.c: In function 'hciuart_read': chip/stm32_hciuart.c:2104:30: warning: statement with no effect [-Wunused-value] 2104 | ntotal == (ssize_t)ret; | ~~~~~~~^~~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c: In function 'bcmf_wl_auth_event_handler': wireless/ieee80211/bcm43xxx/bcmf_driver.c:579:23: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 579 | type = bcmf_getle32(&event->type); | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c:580:25: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 580 | status = bcmf_getle32(&event->status); | ^~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c: In function 'bcmf_wl_scan_event_handler': wireless/ieee80211/bcm43xxx/bcmf_driver.c:619:25: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 619 | status = bcmf_getle32(&event->status); | ^~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_driver.c:620:35: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 620 | escan_result_len = bcmf_getle32(&event->len); | ^~~~~~~~~~~ wireless/ieee80211/bcm43xxx/bcmf_bdc.c: In function 'bcmf_bdc_process_event_frame': wireless/ieee80211/bcm43xxx/bcmf_bdc.c:166:27: warning: taking address of packed member of 'struct bcmf_event_s' may result in an unaligned pointer value [-Waddress-of-packed-member] 166 | event_id = bcmf_getle32(&event_msg->event.type); | ^~~~~~~~~~~~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c: In function 'sdio_io_rw_direct': wireless/ieee80211/bcm43xxx/mmc_sdio.c:157:3: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 157 | ret = SDIO_RECVR5(dev, SD_ACMD52, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c: In function 'sdio_io_rw_extended': wireless/ieee80211/bcm43xxx/mmc_sdio.c:239:11: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 239 | ret = SDIO_RECVR5(dev, SD_ACMD53, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:244:11: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 244 | ret = SDIO_RECVR5(dev, SD_ACMD53, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:257:7: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 257 | ret = SDIO_RECVR5(dev, SD_ACMD53, (uint32_t *)&resp); | ^~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:265:3: warning: converting a packed 'struct sdio_resp_R5' pointer (alignment 1) to a 'uint32_t' {aka 'unsigned int'} pointer (alignment 4) may result in an unaligned pointer value [-Waddress-of-packed-member] 265 | SDIO_RECVR1(dev, SD_ACMD52ABRT, (uint32_t *)&resp); | ^~~~~~~~~~~ wireless/ieee80211/bcm43xxx/mmc_sdio.c:79:28: note: defined here 79 | begin_packed_struct struct sdio_resp_R5 | ^~~~~~~~~~~~ chip/stm32_adc.c: In function 'adc_reset': chip/stm32_adc.c:2860:7: warning: unused variable 'ret' [-Wunused-variable] 2860 | int ret; | ^~~ chip/stm32_adc.c: In function 'adc_shutdown': chip/stm32_adc.c:3044:7: warning: unused variable 'ret' [-Wunused-variable] 3044 | int ret; | ^~~ chip/stm32_i2c.c:722:12: warning: 'stm32_i2c_sem_wait_noncancelable' defined but not used [-Wunused-function] 722 | static int stm32_i2c_sem_wait_noncancelable(FAR struct i2c_master_s *dev) | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ wireless/gs2200m.c: In function 'gs2200m_read': wireless/gs2200m.c:727:20: warning: passing argument 1 of 'nxsem_wait' from incompatible pointer type [-Wincompatible-pointer-types] 727 | ret = nxsem_wait(dev); | ^~~ | | | struct gs2200m_dev_s * .config:1207:warning: symbol value '' invalid for TESTING_OSTEST_FPUSIZE platform/audio/cxd56_audio_analog.c:69:13: warning: inline function 'cxd56_audio_clock_is_enabled' declared but never defined 69 | inline bool cxd56_audio_clock_is_enabled(void); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ platform/audio/cxd56_audio_analog.c:68:13: warning: inline function 'cxd56_audio_clock_disable' declared but never defined 68 | inline void cxd56_audio_clock_disable(void); | ^~~~~~~~~~~~~~~~~~~~~~~~~ platform/audio/cxd56_audio_analog.c:67:13: warning: inline function 'cxd56_audio_clock_enable' declared but never defined 67 | inline void cxd56_audio_clock_enable(uint32_t clk, uint32_t div); | ^~~~~~~~~~~~~~~~~~~~~~~~ chip/stm32_adc.c: In function 'adc_reset': chip/stm32_adc.c:1348:7: warning: unused variable 'ret' [-Wunused-variable] 1348 | int ret; | ^~~ chip/stm32_adc.c: In function 'adc_shutdown': chip/stm32_adc.c:1496:7: warning: unused variable 'ret' [-Wunused-variable] 1496 | int ret; | ^~~ chip/stm32_i2c.c:729:12: warning: 'stm32_i2c_sem_wait_uninterruptble' defined but not used [-Wunused-function] 729 | static int stm32_i2c_sem_wait_uninterruptble(FAR struct i2c_master_s *dev) | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ wireless/lpwan/sx127x/sx127x.c:147:52: warning: missing terminating ' character 147 | # warning OOK support is not complete, RX+TX doesn't work yet! | ^ str71_spi.c:435:24: warning: initialization of 'uint32_t (*)(struct spi_dev_s *, uint32_t)' {aka 'unsigned int (*)(struct spi_dev_s *, unsigned int)'} from incompatible pointer type 'uint16_t (*)(struct spi_dev_s *, uint16_t)' {aka 'short unsigned int (*)(struct spi_dev_s *, short unsigned int)'} [-Wincompatible-pointer-types] 435 | .send = spi_send, | ^~~~~~~~ str71_spi.c:435:24: note: (near initialization for 'g_spiops.send') chip/pic32mx-lowconsole.c:147:24: warning: 'pic32mx_getreg' defined but not used [-Wunused-function] static inline uint32_t pic32mx_getreg(uintptr_t uart_base, ^ chip/pic32mx-gpio.c:113:20: warning: 'pic32mx_value' defined but not used [-Wunused-function] static inline bool pic32mx_value(uint16_t pinset) ^ chip/pic32mz-gpio.c:124:20: warning: 'pic32mz_value' defined but not used [-Wunused-function] static inline bool pic32mz_value(pinset_t pinset) ^ chip/pic32mx-usbdev.c:3065:1: warning: 'pic32mx_epreserved' defined but not used [-Wunused-function] pic32mx_epreserved(struct pic32mx_usbdev_s *priv, int epno) ^ mmcsd/mmcsd_spi.c: In function 'mmcsd_mediachanged': mmcsd/mmcsd_spi.c:1938:7: warning: 'return' with a value, in function returning void return ret; ^ In file included from partition/fs_partition.c:42:0: partition/partition.h:66:19: warning: 'read_partition_block' defined but not used [-Wunused-function] static inline int read_partition_block(FAR struct partition_state_s *state, ^ local/local_netpoll.c: In function 'local_pollsetup': local/local_netpoll.c:305:1: warning: label 'pollerr' defined but not used [-Wunused-label] pollerr: ^~~~~~~ Signed-off-by: Xiang Xiao <xiaoxiang@xiaomi.com> Change-Id: If3ea8f32b878aa218072130f7c3018f0d3c1aca5
2020-04-12 15:52:28 +02:00
memcpy(&resp, &data, sizeof(resp));
/* Check for errors */
if (wkupevent & SDIOWAIT_TIMEOUT)
{
wlerr("timeout\n");
return -ETIMEDOUT;
}
if (resp.flags.error || (wkupevent & SDIOWAIT_ERROR))
{
wlerr("error 1\n");
return -EIO;
}
if (resp.flags.function_number || resp.flags.out_of_range)
{
wlerr("error 2\n");
return -EINVAL;
}
return OK;
2017-03-20 22:40:25 +01:00
}
2017-03-14 21:06:19 +01:00
int sdio_set_wide_bus(FAR struct sdio_dev_s *dev)
2017-03-14 21:06:19 +01:00
{
int ret;
uint8_t value;
/* Read Bus Interface Control register */
ret = sdio_io_rw_direct(dev, false, 0, SDIO_CCCR_BUS_IF, 0, &value);
if (ret != OK)
{
return ret;
}
/* Set 4 bits bus width setting */
value &= ~SDIO_CCCR_BUS_IF_WIDTH_MASK;
value |= SDIO_CCCR_BUS_IF_4_BITS;
ret = sdio_io_rw_direct(dev, true, 0, SDIO_CCCR_BUS_IF, value, NULL);
if (ret != OK)
{
return ret;
2017-03-14 21:06:19 +01:00
}
2017-03-14 21:06:19 +01:00
SDIO_WIDEBUS(dev, true);
return OK;
}
int sdio_probe(FAR struct sdio_dev_s *dev)
{
int ret;
uint32_t data = 0;
nxmutex_init(&dev->mutex);
sdio_takelock(dev);
2017-03-14 21:06:19 +01:00
/* Set device state from reset to idle */
ret = sdio_sendcmdpoll(dev, MMCSD_CMD0, 0);
if (ret != OK)
{
goto err;
}
2017-03-14 21:06:19 +01:00
up_mdelay(SDIO_IDLE_DELAY_MS);
/* Device is SDIO card compatible so we can send CMD5 instead of ACMD41 */
ret = sdio_sendcmdpoll(dev, SDIO_CMD5, 0);
if (ret != OK)
{
goto err;
}
2017-03-14 21:06:19 +01:00
/* Receive R4 response */
ret = SDIO_RECVR4(dev, SDIO_CMD5, &data);
if (ret != OK)
{
goto err;
2017-03-14 21:06:19 +01:00
}
/* Device is in Card Identification Mode, request device RCA */
ret = sdio_sendcmdpoll(dev, SD_CMD3, 0);
if (ret != OK)
{
goto err;
}
2017-03-14 21:06:19 +01:00
ret = SDIO_RECVR6(dev, SD_CMD3, &data);
if (ret != OK)
2017-03-14 21:06:19 +01:00
{
wlerr("ERROR: RCA request failed: %d\n", ret);
goto err;
2017-03-14 21:06:19 +01:00
}
wlinfo("rca is %" PRIx32 "\n", data >> 16);
2017-03-14 21:06:19 +01:00
/* Send CMD7 with the argument == RCA in order to select the card
* and put it in Transfer State.
*/
2017-03-14 21:06:19 +01:00
ret = sdio_sendcmdpoll(dev, MMCSD_CMD7S, data & 0xffff0000);
if (ret != OK)
{
wlerr("ERROR: CMD7 request failed: %d\n", ret);
goto err;
}
2017-03-14 21:06:19 +01:00
ret = SDIO_RECVR1(dev, MMCSD_CMD7S, &data);
if (ret != OK)
{
wlerr("ERROR: card selection failed: %d\n", ret);
goto err;
2017-03-14 21:06:19 +01:00
}
/* Configure 4 bits bus width */
2017-03-14 21:06:19 +01:00
sdio_givelock(dev);
return sdio_set_wide_bus(dev);
2017-03-14 21:06:19 +01:00
err:
sdio_givelock(dev);
return ret;
2017-03-14 21:06:19 +01:00
}
int sdio_set_blocksize(FAR struct sdio_dev_s *dev, uint8_t function,
uint16_t blocksize)
{
int ret;
ret = sdio_io_rw_direct(dev, true, 0,
(function << SDIO_FBR_SHIFT) + SDIO_CCCR_FN0_BLKSIZE_0,
blocksize & 0xff, NULL);
if (ret != OK)
{
return ret;
}
ret = sdio_io_rw_direct(dev, true, 0,
(function << SDIO_FBR_SHIFT) + SDIO_CCCR_FN0_BLKSIZE_1,
(blocksize >> 8), NULL);
if (ret != OK)
{
return ret;
}
return OK;
}
int sdio_enable_function(FAR struct sdio_dev_s *dev, uint8_t function)
{
int ret;
uint8_t value;
/* Read current I/O Enable register */
ret = sdio_io_rw_direct(dev, false, 0, SDIO_CCCR_IOEN, 0, &value);
if (ret != OK)
{
return ret;
}
ret = sdio_io_rw_direct(dev, true, 0,
SDIO_CCCR_IOEN, value | (1 << function), NULL);
2017-03-14 21:06:19 +01:00
if (ret != OK)
{
return ret;
}
/* Wait 1s for function to be enabled */
2017-03-14 21:06:19 +01:00
int loops = 100;
2017-03-14 21:06:19 +01:00
while (loops-- > 0)
{
nxsig_usleep(10 * 1000);
2017-03-14 21:06:19 +01:00
ret = sdio_io_rw_direct(dev, false, 0, SDIO_CCCR_IORDY, 0, &value);
2017-03-14 21:06:19 +01:00
if (ret != OK)
{
return ret;
}
if (value & (1 << function))
{
/* Function enabled */
wlinfo("Function %d enabled\n", function);
2017-03-14 21:06:19 +01:00
return OK;
}
}
2017-03-14 21:06:19 +01:00
return -ETIMEDOUT;
}
2017-03-20 22:40:25 +01:00
int sdio_enable_interrupt(FAR struct sdio_dev_s *dev, uint8_t function)
{
int ret;
uint8_t value;
/* Read current Int Enable register */
ret = sdio_io_rw_direct(dev, false, 0, SDIO_CCCR_INTEN, 0, &value);
if (ret != OK)
{
return ret;
}
return sdio_io_rw_direct(dev, true, 0,
SDIO_CCCR_INTEN, value | (1 << function), NULL);
2017-03-20 22:40:25 +01:00
}