protocol: Separate device add from boot-option add messages

We want to cater for situations where boot options may be discovered
some time after we get notificiation about devices. For instance,
discovering boot options from DHCP configuration parameters. In this
case, we'll need to notify UIs of boot options appear some time after
the device (and/or other boot options on the same device) has appeared.

This change adds a new protocol message type,
PB_PROTOCOL_ACTION_BOOT_OPTION_ADD. We also rename
PB_PROTOCOL_ACTION_ADD to make it clear that it is just for devices.

The discover server is updated to send boot option add events at device
discover time, but we are now able to decouple this later.

We also update the clients to handle the boot option add events
separately.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

1 files changed, 13 insertions, 4 deletions

diff --git a/lib/pb-protocol/pb-protocol.h b/lib/pb-protocol/pb-protocol.h
index de2ae7c..35e96f0 100644
--- a/lib/pb-protocol/pb-protocol.h
+++ b/lib/pb-protocol/pb-protocol.h
@@ -12,9 +12,11 @@
 #define PB_PROTOCOL_MAX_PAYLOAD_SIZE (64 * 1024)
 
 enum pb_protocol_action {
-	PB_PROTOCOL_ACTION_ADD		= 0x1,
-	PB_PROTOCOL_ACTION_REMOVE	= 0x2,
-	PB_PROTOCOL_ACTION_BOOT		= 0x3,
+	PB_PROTOCOL_ACTION_DEVICE_ADD		= 0x1,
+	PB_PROTOCOL_ACTION_BOOT_OPTION_ADD	= 0x2,
+	PB_PROTOCOL_ACTION_DEVICE_REMOVE	= 0x3,
+/*	PB_PROTOCOL_ACTION_BOOT_OPTION_REMOVE	= 0x4, */
+	PB_PROTOCOL_ACTION_BOOT			= 0x5,
 };
 
 struct pb_protocol_message {
@@ -26,6 +28,7 @@ struct pb_protocol_message {
 void pb_protocol_dump_device(const struct device *dev, const char *text,
 	FILE *stream);
 int pb_protocol_device_len(const struct device *dev);
+int pb_protocol_boot_option_len(const struct boot_option *opt);
 int pb_protocol_boot_len(const struct boot_command *boot);
 int pb_protocol_device_cmp(const struct device *a, const struct device *b);
 
@@ -36,7 +39,10 @@ int pb_protocol_serialise_string(char *pos, const char *str);
 char *pb_protocol_deserialise_string(void *ctx,
 		const struct pb_protocol_message *message);
 
-int pb_protocol_serialise_device(const struct device *dev, char *buf, int buf_len);
+int pb_protocol_serialise_device(const struct device *dev,
+		char *buf, int buf_len);
+int pb_protocol_serialise_boot_option(const struct boot_option *opt,
+		char *buf, int buf_len);
 int pb_protocol_serialise_boot_command(const struct boot_command *boot,
 		char *buf, int buf_len);
 
@@ -50,6 +56,9 @@ struct pb_protocol_message *pb_protocol_read_message(void *ctx, int fd);
 int pb_protocol_deserialise_device(struct device *dev,
 		const struct pb_protocol_message *message);
 
+int pb_protocol_deserialise_boot_option(struct boot_option *opt,
+		const struct pb_protocol_message *message);
+
 int pb_protocol_deserialise_boot_command(struct boot_command *cmd,
 		const struct pb_protocol_message *message);