platform: goldfish: pipe: Separate the host interface to a separate header
authorRoman Kiryanov <rkir@google.com>
Mon, 27 Aug 2018 18:22:59 +0000 (11:22 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 14 Sep 2018 13:31:30 +0000 (15:31 +0200)
These are several enums that must kept in sync with the host side.
This change explicitly separates them into a dedicated header file.

Signed-off-by: Roman Kiryanov <rkir@google.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/platform/goldfish/goldfish_pipe.c
drivers/platform/goldfish/goldfish_pipe_qemu.h [new file with mode: 0644]

index 4e7e100dc7a09f2ee85e82d6c11e9cbbed8d04be..caf514aafb211e98ae37691088804975fed06bbc 100644 (file)
@@ -64,6 +64,7 @@
 #include <linux/dma-mapping.h>
 #include <linux/mm.h>
 #include <linux/acpi.h>
+#include "goldfish_pipe_qemu.h"
 
 /*
  * Update this when something changes in the driver's behavior so the host
@@ -74,74 +75,6 @@ enum {
        PIPE_CURRENT_DEVICE_VERSION = 2
 };
 
-/*
- * IMPORTANT: The following constants must match the ones used and defined
- * in external/qemu/hw/goldfish_pipe.c in the Android source tree.
- */
-
-/* List of bitflags returned in status of CMD_POLL command */
-enum PipePollFlags {
-       PIPE_POLL_IN    = 1 << 0,
-       PIPE_POLL_OUT   = 1 << 1,
-       PIPE_POLL_HUP   = 1 << 2
-};
-
-/*
- * Possible status values used to signal errors
- * see: goldfish_pipe_error_convert
- */
-enum PipeErrors {
-       PIPE_ERROR_INVAL  = -1,
-       PIPE_ERROR_AGAIN  = -2,
-       PIPE_ERROR_NOMEM  = -3,
-       PIPE_ERROR_IO     = -4
-};
-
-/* Bit-flags used to signal events from the emulator */
-enum PipeWakeFlags {
-       PIPE_WAKE_CLOSED = 1 << 0,  /* emulator closed pipe */
-       PIPE_WAKE_READ   = 1 << 1,  /* pipe can now be read from */
-       PIPE_WAKE_WRITE  = 1 << 2  /* pipe can now be written to */
-};
-
-/* Bit flags for the 'flags' field */
-enum PipeFlagsBits {
-       BIT_CLOSED_ON_HOST = 0,  /* pipe closed by host */
-       BIT_WAKE_ON_WRITE  = 1,  /* want to be woken on writes */
-       BIT_WAKE_ON_READ   = 2,  /* want to be woken on reads */
-};
-
-enum PipeRegs {
-       PIPE_REG_CMD = 0,
-
-       PIPE_REG_SIGNAL_BUFFER_HIGH = 4,
-       PIPE_REG_SIGNAL_BUFFER = 8,
-       PIPE_REG_SIGNAL_BUFFER_COUNT = 12,
-
-       PIPE_REG_OPEN_BUFFER_HIGH = 20,
-       PIPE_REG_OPEN_BUFFER = 24,
-
-       PIPE_REG_VERSION = 36,
-
-       PIPE_REG_GET_SIGNALLED = 48,
-};
-
-enum PipeCmdCode {
-       PIPE_CMD_OPEN = 1,      /* to be used by the pipe device itself */
-       PIPE_CMD_CLOSE,
-       PIPE_CMD_POLL,
-       PIPE_CMD_WRITE,
-       PIPE_CMD_WAKE_ON_WRITE,
-       PIPE_CMD_READ,
-       PIPE_CMD_WAKE_ON_READ,
-
-       /*
-        * TODO(zyy): implement a deferred read/write execution to allow
-        * parallel processing of pipe operations on the host.
-        */
-       PIPE_CMD_WAKE_ON_DONE_IO,
-};
-
 enum {
        MAX_BUFFERS_PER_COMMAND = 336,
        MAX_SIGNALLED_PIPES = 64,
diff --git a/drivers/platform/goldfish/goldfish_pipe_qemu.h b/drivers/platform/goldfish/goldfish_pipe_qemu.h
new file mode 100644 (file)
index 0000000..c272e11
--- /dev/null
@@ -0,0 +1,112 @@
+/* SPDX-License-Identifier: GPL-2.0 */
+/*
+ * Copyright (C) 2018 Google, Inc.
+ *
+ * This software is licensed under the terms of the GNU General Public
+ * License version 2, as published by the Free Software Foundation, and
+ * may be copied, distributed, and modified under those terms.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ */
+
+/*
+ * IMPORTANT: The following constants must match the ones used and defined in
+ * external/qemu/include/hw/misc/goldfish_pipe.h
+ */
+
+#ifndef GOLDFISH_PIPE_QEMU_H
+#define GOLDFISH_PIPE_QEMU_H
+
+/* List of bitflags returned in status of CMD_POLL command */
+enum PipePollFlags {
+       PIPE_POLL_IN    = 1 << 0,
+       PIPE_POLL_OUT   = 1 << 1,
+       PIPE_POLL_HUP   = 1 << 2
+};
+
+/* Possible status values used to signal errors */
+enum PipeErrors {
+       PIPE_ERROR_INVAL        = -1,
+       PIPE_ERROR_AGAIN        = -2,
+       PIPE_ERROR_NOMEM        = -3,
+       PIPE_ERROR_IO           = -4
+};
+
+/* Bit-flags used to signal events from the emulator */
+enum PipeWakeFlags {
+       /* emulator closed pipe */
+       PIPE_WAKE_CLOSED                = 1 << 0,
+
+       /* pipe can now be read from */
+       PIPE_WAKE_READ                  = 1 << 1,
+
+       /* pipe can now be written to */
+       PIPE_WAKE_WRITE                 = 1 << 2,
+
+       /* unlock this pipe's DMA buffer */
+       PIPE_WAKE_UNLOCK_DMA            = 1 << 3,
+
+       /* unlock DMA buffer of the pipe shared to this pipe */
+       PIPE_WAKE_UNLOCK_DMA_SHARED     = 1 << 4,
+};
+
+/* Possible pipe closing reasons */
+enum PipeCloseReason {
+       /* guest sent a close command */
+       PIPE_CLOSE_GRACEFUL             = 0,
+
+       /* guest rebooted, we're closing the pipes */
+       PIPE_CLOSE_REBOOT               = 1,
+
+       /* close old pipes on snapshot load */
+       PIPE_CLOSE_LOAD_SNAPSHOT        = 2,
+
+       /* some unrecoverable error on the pipe */
+       PIPE_CLOSE_ERROR                = 3,
+};
+
+/* Bit flags for the 'flags' field */
+enum PipeFlagsBits {
+       BIT_CLOSED_ON_HOST = 0,  /* pipe closed by host */
+       BIT_WAKE_ON_WRITE  = 1,  /* want to be woken on writes */
+       BIT_WAKE_ON_READ   = 2,  /* want to be woken on reads */
+};
+
+enum PipeRegs {
+       PIPE_REG_CMD = 0,
+
+       PIPE_REG_SIGNAL_BUFFER_HIGH = 4,
+       PIPE_REG_SIGNAL_BUFFER = 8,
+       PIPE_REG_SIGNAL_BUFFER_COUNT = 12,
+
+       PIPE_REG_OPEN_BUFFER_HIGH = 20,
+       PIPE_REG_OPEN_BUFFER = 24,
+
+       PIPE_REG_VERSION = 36,
+
+       PIPE_REG_GET_SIGNALLED = 48,
+};
+
+enum PipeCmdCode {
+       /* to be used by the pipe device itself */
+       PIPE_CMD_OPEN           = 1,
+
+       PIPE_CMD_CLOSE,
+       PIPE_CMD_POLL,
+       PIPE_CMD_WRITE,
+       PIPE_CMD_WAKE_ON_WRITE,
+       PIPE_CMD_READ,
+       PIPE_CMD_WAKE_ON_READ,
+
+       /*
+        * TODO(zyy): implement a deferred read/write execution to allow
+        * parallel processing of pipe operations on the host.
+        */
+       PIPE_CMD_WAKE_ON_DONE_IO,
+};
+
+#endif /* GOLDFISH_PIPE_QEMU_H */