Commit c10bf392 authored by Rui Miguel Silva's avatar Rui Miguel Silva Committed by Greg Kroah-Hartman

staging: greybus: add missing includes

Before moving greybus core out of staging and moving header files to
include/linux some greybus header files were missing the necessary
includes. This would trigger compilation faillures with some example
errors logged bellow for with CONFIG_KERNEL_HEADER_TEST=y.

So, add the necessary headers to compile clean before relocating the
header files.

./include/linux/greybus/hd.h:23:50: error: unknown type name 'u16'
  int (*cport_disable)(struct gb_host_device *hd, u16 cport_id); ^~~
./include/linux/greybus/greybus_protocols.h:1314:2: error: unknown type name '__u8'
  __u8 data[0];
  ^~~~
./include/linux/greybus/hd.h:24:52: error: unknown type name 'u16'
  int (*cport_connected)(struct gb_host_device *hd, u16 cport_id); ^~~
./include/linux/greybus/hd.h:25:48: error: unknown type name 'u16'
  int (*cport_flush)(struct gb_host_device *hd, u16 cport_id); ^~~
./include/linux/greybus/hd.h:26:51: error: unknown type name 'u16'
  int (*cport_shutdown)(struct gb_host_device *hd, u16 cport_id, ^~~
./include/linux/greybus/hd.h:27:5: error: unknown type name 'u8'
u8 phase, unsigned int timeout);
     ^~
./include/linux/greybus/hd.h:28:50: error: unknown type name 'u16'
  int (*cport_quiesce)(struct gb_host_device *hd, u16 cport_id, ^~~
./include/linux/greybus/hd.h:29:5: error: unknown type name 'size_t'
     size_t peer_space, unsigned int timeout);
     ^~~~~~
./include/linux/greybus/hd.h:29:5: note: 'size_t' is defined in header '<stddef.h>'; did you forget to '#include <stddef.h>'?
./include/linux/greybus/hd.h:1:1:
+#include <stddef.h>
 /* SPDX-License-Identifier: GPL-2.0 */
./include/linux/greybus/hd.h:29:5:
     size_t peer_space, unsigned int timeout);
     ^~~~~~
./include/linux/greybus/hd.h:30:48: error: unknown type name 'u16'
  int (*cport_clear)(struct gb_host_device *hd, u16 cport_id); ^~~
./include/linux/greybus/hd.h:32:49: error: unknown type name 'u16'
  int (*message_send)(struct gb_host_device *hd, u16 dest_cport_id, ^~~
./include/linux/greybus/hd.h:33:32: error: unknown type name 'gfp_t'
struct gb_message *message, gfp_t gfp_mask); ^~~~~
./include/linux/greybus/hd.h:35:55: error: unknown type name 'u16'
  int (*latency_tag_enable)(struct gb_host_device *hd, u16 cport_id);
Reported-by: default avatarkbuild test robot <lkp@intel.com>
Reported-by: default avatarGao Xiang <hsiangkao@aol.com>
Signed-off-by: default avatarRui Miguel Silva <rmfrfs@gmail.com>
Signed-off-by: default avatarRui Miguel Silva <rui.silva@linaro.org>
Link: https://lore.kernel.org/r/20190827155302.25704-1-rui.silva@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent b81beec9
...@@ -9,7 +9,10 @@ ...@@ -9,7 +9,10 @@
#ifndef __BUNDLE_H #ifndef __BUNDLE_H
#define __BUNDLE_H #define __BUNDLE_H
#include <linux/types.h>
#include <linux/list.h> #include <linux/list.h>
#include <linux/pm_runtime.h>
#include <linux/device.h>
#define BUNDLE_ID_NONE U8_MAX #define BUNDLE_ID_NONE U8_MAX
......
...@@ -9,8 +9,11 @@ ...@@ -9,8 +9,11 @@
#ifndef __CONNECTION_H #ifndef __CONNECTION_H
#define __CONNECTION_H #define __CONNECTION_H
#include <linux/bits.h>
#include <linux/list.h> #include <linux/list.h>
#include <linux/kfifo.h> #include <linux/kfifo.h>
#include <linux/kref.h>
#include <linux/workqueue.h>
#define GB_CONNECTION_FLAG_CSD BIT(0) #define GB_CONNECTION_FLAG_CSD BIT(0)
#define GB_CONNECTION_FLAG_NO_FLOWCTRL BIT(1) #define GB_CONNECTION_FLAG_NO_FLOWCTRL BIT(1)
......
...@@ -9,6 +9,9 @@ ...@@ -9,6 +9,9 @@
#ifndef __CONTROL_H #ifndef __CONTROL_H
#define __CONTROL_H #define __CONTROL_H
#include <linux/types.h>
#include <linux/device.h>
struct gb_control { struct gb_control {
struct device dev; struct device dev;
struct gb_interface *intf; struct gb_interface *intf;
......
...@@ -14,6 +14,9 @@ ...@@ -14,6 +14,9 @@
#ifndef __GREYBUS_MANIFEST_H #ifndef __GREYBUS_MANIFEST_H
#define __GREYBUS_MANIFEST_H #define __GREYBUS_MANIFEST_H
#include <linux/bits.h>
#include <linux/types.h>
enum greybus_descriptor_type { enum greybus_descriptor_type {
GREYBUS_TYPE_INVALID = 0x00, GREYBUS_TYPE_INVALID = 0x00,
GREYBUS_TYPE_INTERFACE = 0x01, GREYBUS_TYPE_INTERFACE = 0x01,
......
...@@ -7,6 +7,8 @@ ...@@ -7,6 +7,8 @@
#ifndef __GREYBUS_PROTOCOLS_H #ifndef __GREYBUS_PROTOCOLS_H
#define __GREYBUS_PROTOCOLS_H #define __GREYBUS_PROTOCOLS_H
#include <linux/types.h>
/* Fixed IDs for control/svc protocols */ /* Fixed IDs for control/svc protocols */
/* SVC switch-port device ids */ /* SVC switch-port device ids */
......
...@@ -9,6 +9,9 @@ ...@@ -9,6 +9,9 @@
#ifndef __HD_H #ifndef __HD_H
#define __HD_H #define __HD_H
#include <linux/types.h>
#include <linux/device.h>
struct gb_host_device; struct gb_host_device;
struct gb_message; struct gb_message;
......
...@@ -9,6 +9,9 @@ ...@@ -9,6 +9,9 @@
#ifndef __INTERFACE_H #ifndef __INTERFACE_H
#define __INTERFACE_H #define __INTERFACE_H
#include <linux/types.h>
#include <linux/device.h>
enum gb_interface_type { enum gb_interface_type {
GB_INTERFACE_TYPE_INVALID = 0, GB_INTERFACE_TYPE_INVALID = 0,
GB_INTERFACE_TYPE_UNKNOWN, GB_INTERFACE_TYPE_UNKNOWN,
......
...@@ -9,6 +9,8 @@ ...@@ -9,6 +9,8 @@
#ifndef __MANIFEST_H #ifndef __MANIFEST_H
#define __MANIFEST_H #define __MANIFEST_H
#include <linux/types.h>
struct gb_interface; struct gb_interface;
bool gb_manifest_parse(struct gb_interface *intf, void *data, size_t size); bool gb_manifest_parse(struct gb_interface *intf, void *data, size_t size);
......
...@@ -9,6 +9,9 @@ ...@@ -9,6 +9,9 @@
#ifndef __MODULE_H #ifndef __MODULE_H
#define __MODULE_H #define __MODULE_H
#include <linux/types.h>
#include <linux/device.h>
struct gb_module { struct gb_module {
struct device dev; struct device dev;
struct gb_host_device *hd; struct gb_host_device *hd;
......
...@@ -10,6 +10,11 @@ ...@@ -10,6 +10,11 @@
#define __OPERATION_H #define __OPERATION_H
#include <linux/completion.h> #include <linux/completion.h>
#include <linux/kref.h>
#include <linux/timer.h>
#include <linux/types.h>
#include <linux/workqueue.h>
struct gb_operation; struct gb_operation;
......
...@@ -9,6 +9,9 @@ ...@@ -9,6 +9,9 @@
#ifndef __SVC_H #ifndef __SVC_H
#define __SVC_H #define __SVC_H
#include <linux/types.h>
#include <linux/device.h>
#define GB_SVC_CPORT_FLAG_E2EFC BIT(0) #define GB_SVC_CPORT_FLAG_E2EFC BIT(0)
#define GB_SVC_CPORT_FLAG_CSD_N BIT(1) #define GB_SVC_CPORT_FLAG_CSD_N BIT(1)
#define GB_SVC_CPORT_FLAG_CSV_N BIT(2) #define GB_SVC_CPORT_FLAG_CSV_N BIT(2)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment