michaelh/zephyr
1
0
Fork 0
Code Releases Activity

drivers: udc: use explicit casts from void* to the target type

Browse source 
This header file provides the public API of the UDC driver,
however it also contains inline function definitions.
Unfortunately these definitions contain implicit pointer conversions,
which are one of the cases where valid C syntax is invalid C++ syntax,
meaning that any C++ source code that intends to use the UDC API
cannot be compiled error-free (or warning-free, if using -fpermissive).

Signed-off-by: Benedek Kupper <kupper.benedek@gmail.com>
This commit is contained in:
Benedek Kupper 2024-02-26 21:41:38 +01:00 • committed by Benjamin Cabé
commit 332115cdf6
1 changed files with 8 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

16
include/zephyr/drivers/usb/udc.h
View file

@ -311,7 +311,7 @@ struct udc_data {
*/
static inline bool udc_is_initialized(const struct device *dev)
{
struct udc_data *data = dev->data;
struct udc_data *data = (struct udc_data *)dev->data;
return atomic_test_bit(&data->status, UDC_STATUS_INITIALIZED);
}
@ -325,7 +325,7 @@ static inline bool udc_is_initialized(const struct device *dev)
*/
static inline bool udc_is_enabled(const struct device *dev)
{
struct udc_data *data = dev->data;
struct udc_data *data = (struct udc_data *)dev->data;
return atomic_test_bit(&data->status, UDC_STATUS_ENABLED);
}
@ -339,7 +339,7 @@ static inline bool udc_is_enabled(const struct device *dev)
*/
static inline bool udc_is_suspended(const struct device *dev)
{
struct udc_data *data = dev->data;
struct udc_data *data = (struct udc_data *)dev->data;
return atomic_test_bit(&data->status, UDC_STATUS_SUSPENDED);
}
@ -415,7 +415,7 @@ int udc_shutdown(const struct device *dev);
*/
static inline struct udc_device_caps udc_caps(const struct device *dev)
{
struct udc_data *data = dev->data;
struct udc_data *data = (struct udc_data *)dev->data;
return data->caps;
}
@ -445,7 +445,7 @@ enum udc_bus_speed udc_device_speed(const struct device *dev);
*/
static inline int udc_set_address(const struct device *dev, const uint8_t addr)
{
const struct udc_api *api = dev->api;
const struct udc_api *api = (const struct udc_api *)dev->api;
int ret;
if (!udc_is_enabled(dev)) {
@ -477,7 +477,7 @@ static inline int udc_set_address(const struct device *dev, const uint8_t addr)
static inline int udc_test_mode(const struct device *dev,
const uint8_t mode, const bool dryrun)
{
const struct udc_api *api = dev->api;
const struct udc_api *api = (const struct udc_api *)dev->api;
int ret;
if (!udc_is_enabled(dev)) {
@ -507,7 +507,7 @@ static inline int udc_test_mode(const struct device *dev,
*/
static inline int udc_host_wakeup(const struct device *dev)
{
const struct udc_api *api = dev->api;
const struct udc_api *api = (const struct udc_api *)dev->api;
int ret;
if (!udc_is_enabled(dev)) {
@ -724,7 +724,7 @@ static inline struct udc_buf_info *udc_get_buf_info(const struct net_buf *const
*/
static inline const void *udc_get_event_ctx(const struct device *dev)
{
struct udc_data *data = dev->data;
struct udc_data *data = (struct udc_data *)dev->data;
return data->event_ctx;
}