mirror of
https://github.com/torvalds/linux.git
synced 2024-11-30 16:11:38 +00:00
a62052ba2a
When calling debugfs functions, there is no need to ever check the return value. The function can work or not, but the code logic should never do something different based on this. This cleans up a lot of unneeded code and logic around the debugfs wimax files, making all of this much simpler and easier to understand. Cc: Inaky Perez-Gonzalez <inaky.perez-gonzalez@intel.com> Cc: linux-wimax@intel.com Cc: netdev@vger.kernel.org Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Signed-off-by: David S. Miller <davem@davemloft.net>
86 lines
2.5 KiB
C
86 lines
2.5 KiB
C
/* SPDX-License-Identifier: GPL-2.0-only */
|
|
/*
|
|
* Linux WiMAX
|
|
* Internal API for kernel space WiMAX stack
|
|
*
|
|
* Copyright (C) 2007 Intel Corporation <linux-wimax@intel.com>
|
|
* Inaky Perez-Gonzalez <inaky.perez-gonzalez@intel.com>
|
|
*
|
|
* This header file is for declarations and definitions internal to
|
|
* the WiMAX stack. For public APIs and documentation, see
|
|
* include/net/wimax.h and include/linux/wimax.h.
|
|
*/
|
|
|
|
#ifndef __WIMAX_INTERNAL_H__
|
|
#define __WIMAX_INTERNAL_H__
|
|
#ifdef __KERNEL__
|
|
|
|
#ifdef pr_fmt
|
|
#undef pr_fmt
|
|
#endif
|
|
|
|
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
|
|
|
|
#include <linux/device.h>
|
|
#include <net/wimax.h>
|
|
|
|
|
|
/*
|
|
* Decide if a (locked) device is ready for use
|
|
*
|
|
* Before using the device structure, it must be locked
|
|
* (wimax_dev->mutex). As well, most operations need to call this
|
|
* function to check if the state is the right one.
|
|
*
|
|
* An error value will be returned if the state is not the right
|
|
* one. In that case, the caller should not attempt to use the device
|
|
* and just unlock it.
|
|
*/
|
|
static inline __must_check
|
|
int wimax_dev_is_ready(struct wimax_dev *wimax_dev)
|
|
{
|
|
if (wimax_dev->state == __WIMAX_ST_NULL)
|
|
return -EINVAL; /* Device is not even registered! */
|
|
if (wimax_dev->state == WIMAX_ST_DOWN)
|
|
return -ENOMEDIUM;
|
|
if (wimax_dev->state == __WIMAX_ST_QUIESCING)
|
|
return -ESHUTDOWN;
|
|
return 0;
|
|
}
|
|
|
|
|
|
static inline
|
|
void __wimax_state_set(struct wimax_dev *wimax_dev, enum wimax_st state)
|
|
{
|
|
wimax_dev->state = state;
|
|
}
|
|
void __wimax_state_change(struct wimax_dev *, enum wimax_st);
|
|
|
|
#ifdef CONFIG_DEBUG_FS
|
|
void wimax_debugfs_add(struct wimax_dev *);
|
|
void wimax_debugfs_rm(struct wimax_dev *);
|
|
#else
|
|
static inline void wimax_debugfs_add(struct wimax_dev *wimax_dev) {}
|
|
static inline void wimax_debugfs_rm(struct wimax_dev *wimax_dev) {}
|
|
#endif
|
|
|
|
void wimax_id_table_add(struct wimax_dev *);
|
|
struct wimax_dev *wimax_dev_get_by_genl_info(struct genl_info *, int);
|
|
void wimax_id_table_rm(struct wimax_dev *);
|
|
void wimax_id_table_release(void);
|
|
|
|
int wimax_rfkill_add(struct wimax_dev *);
|
|
void wimax_rfkill_rm(struct wimax_dev *);
|
|
|
|
/* generic netlink */
|
|
extern struct genl_family wimax_gnl_family;
|
|
|
|
/* ops */
|
|
int wimax_gnl_doit_msg_from_user(struct sk_buff *skb, struct genl_info *info);
|
|
int wimax_gnl_doit_reset(struct sk_buff *skb, struct genl_info *info);
|
|
int wimax_gnl_doit_rfkill(struct sk_buff *skb, struct genl_info *info);
|
|
int wimax_gnl_doit_state_get(struct sk_buff *skb, struct genl_info *info);
|
|
|
|
#endif /* #ifdef __KERNEL__ */
|
|
#endif /* #ifndef __WIMAX_INTERNAL_H__ */
|