powerpc/pseries: Factor HVSI header struct in packet definitions
authorBenjamin Herrenschmidt <benh@kernel.crashing.org>
Thu, 16 Jun 2011 15:08:24 +0000 (15:08 +0000)
committerBenjamin Herrenschmidt <benh@kernel.crashing.org>
Wed, 29 Jun 2011 07:48:30 +0000 (17:48 +1000)
Embed the struct hvsi_header in the various packet definitions
rather than open coding it multiple times. Will help provide
stronger type checking.

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
arch/powerpc/include/asm/hvsi.h
drivers/tty/hvc/hvsi.c

index f13125a..ab2ddd7 100644 (file)
@@ -29,16 +29,12 @@ struct hvsi_header {
 } __attribute__((packed));
 
 struct hvsi_data {
-       uint8_t  type;
-       uint8_t  len;
-       uint16_t seqno;
+       struct hvsi_header hdr;
        uint8_t  data[HVSI_MAX_OUTGOING_DATA];
 } __attribute__((packed));
 
 struct hvsi_control {
-       uint8_t  type;
-       uint8_t  len;
-       uint16_t seqno;
+       struct hvsi_header hdr;
        uint16_t verb;
        /* optional depending on verb: */
        uint32_t word;
@@ -46,16 +42,12 @@ struct hvsi_control {
 } __attribute__((packed));
 
 struct hvsi_query {
-       uint8_t  type;
-       uint8_t  len;
-       uint16_t seqno;
+       struct hvsi_header hdr;
        uint16_t verb;
 } __attribute__((packed));
 
 struct hvsi_query_response {
-       uint8_t  type;
-       uint8_t  len;
-       uint16_t seqno;
+       struct hvsi_header hdr;
        uint16_t verb;
        uint16_t query_seqno;
        union {
index 0b35793..c94e2f5 100644 (file)
@@ -295,18 +295,18 @@ static int hvsi_version_respond(struct hvsi_struct *hp, uint16_t query_seqno)
        struct hvsi_query_response packet __ALIGNED__;
        int wrote;
 
-       packet.type = VS_QUERY_RESPONSE_PACKET_HEADER;
-       packet.len = sizeof(struct hvsi_query_response);
-       packet.seqno = atomic_inc_return(&hp->seqno);
+       packet.hdr.type = VS_QUERY_RESPONSE_PACKET_HEADER;
+       packet.hdr.len = sizeof(struct hvsi_query_response);
+       packet.hdr.seqno = atomic_inc_return(&hp->seqno);
        packet.verb = VSV_SEND_VERSION_NUMBER;
        packet.u.version = HVSI_VERSION;
        packet.query_seqno = query_seqno+1;
 
-       pr_debug("%s: sending %i bytes\n", __func__, packet.len);
-       dbg_dump_hex((uint8_t*)&packet, packet.len);
+       pr_debug("%s: sending %i bytes\n", __func__, packet.hdr.len);
+       dbg_dump_hex((uint8_t*)&packet, packet.hdr.len);
 
-       wrote = hvc_put_chars(hp->vtermno, (char *)&packet, packet.len);
-       if (wrote != packet.len) {
+       wrote = hvc_put_chars(hp->vtermno, (char *)&packet, packet.hdr.len);
+       if (wrote != packet.hdr.len) {
                printk(KERN_ERR "hvsi%i: couldn't send query response!\n",
                        hp->index);
                return -EIO;
@@ -321,7 +321,7 @@ static void hvsi_recv_query(struct hvsi_struct *hp, uint8_t *packet)
 
        switch (hp->state) {
                case HVSI_WAIT_FOR_VER_QUERY:
-                       hvsi_version_respond(hp, query->seqno);
+                       hvsi_version_respond(hp, query->hdr.seqno);
                        __set_state(hp, HVSI_OPEN);
                        break;
                default:
@@ -579,16 +579,16 @@ static int hvsi_query(struct hvsi_struct *hp, uint16_t verb)
        struct hvsi_query packet __ALIGNED__;
        int wrote;
 
-       packet.type = VS_QUERY_PACKET_HEADER;
-       packet.len = sizeof(struct hvsi_query);
-       packet.seqno = atomic_inc_return(&hp->seqno);
+       packet.hdr.type = VS_QUERY_PACKET_HEADER;
+       packet.hdr.len = sizeof(struct hvsi_query);
+       packet.hdr.seqno = atomic_inc_return(&hp->seqno);
        packet.verb = verb;
 
-       pr_debug("%s: sending %i bytes\n", __func__, packet.len);
-       dbg_dump_hex((uint8_t*)&packet, packet.len);
+       pr_debug("%s: sending %i bytes\n", __func__, packet.hdr.len);
+       dbg_dump_hex((uint8_t*)&packet, packet.hdr.len);
 
-       wrote = hvc_put_chars(hp->vtermno, (char *)&packet, packet.len);
-       if (wrote != packet.len) {
+       wrote = hvc_put_chars(hp->vtermno, (char *)&packet, packet.hdr.len);
+       if (wrote != packet.hdr.len) {
                printk(KERN_ERR "hvsi%i: couldn't send query (%i)!\n", hp->index,
                        wrote);
                return -EIO;
@@ -622,20 +622,20 @@ static int hvsi_set_mctrl(struct hvsi_struct *hp, uint16_t mctrl)
        struct hvsi_control packet __ALIGNED__;
        int wrote;
 
-       packet.type = VS_CONTROL_PACKET_HEADER,
-       packet.seqno = atomic_inc_return(&hp->seqno);
-       packet.len = sizeof(struct hvsi_control);
+       packet.hdr.type = VS_CONTROL_PACKET_HEADER,
+       packet.hdr.seqno = atomic_inc_return(&hp->seqno);
+       packet.hdr.len = sizeof(struct hvsi_control);
        packet.verb = VSV_SET_MODEM_CTL;
        packet.mask = HVSI_TSDTR;
 
        if (mctrl & TIOCM_DTR)
                packet.word = HVSI_TSDTR;
 
-       pr_debug("%s: sending %i bytes\n", __func__, packet.len);
-       dbg_dump_hex((uint8_t*)&packet, packet.len);
+       pr_debug("%s: sending %i bytes\n", __func__, packet.hdr.len);
+       dbg_dump_hex((uint8_t*)&packet, packet.hdr.len);
 
-       wrote = hvc_put_chars(hp->vtermno, (char *)&packet, packet.len);
-       if (wrote != packet.len) {
+       wrote = hvc_put_chars(hp->vtermno, (char *)&packet, packet.hdr.len);
+       if (wrote != packet.hdr.len) {
                printk(KERN_ERR "hvsi%i: couldn't set DTR!\n", hp->index);
                return -EIO;
        }
@@ -705,13 +705,13 @@ static int hvsi_put_chars(struct hvsi_struct *hp, const char *buf, int count)
 
        BUG_ON(count > HVSI_MAX_OUTGOING_DATA);
 
-       packet.type = VS_DATA_PACKET_HEADER;
-       packet.seqno = atomic_inc_return(&hp->seqno);
-       packet.len = count + sizeof(struct hvsi_header);
+       packet.hdr.type = VS_DATA_PACKET_HEADER;
+       packet.hdr.seqno = atomic_inc_return(&hp->seqno);
+       packet.hdr.len = count + sizeof(struct hvsi_header);
        memcpy(&packet.data, buf, count);
 
-       ret = hvc_put_chars(hp->vtermno, (char *)&packet, packet.len);
-       if (ret == packet.len) {
+       ret = hvc_put_chars(hp->vtermno, (char *)&packet, packet.hdr.len);
+       if (ret == packet.hdr.len) {
                /* return the number of chars written, not the packet length */
                return count;
        }
@@ -722,15 +722,15 @@ static void hvsi_close_protocol(struct hvsi_struct *hp)
 {
        struct hvsi_control packet __ALIGNED__;
 
-       packet.type = VS_CONTROL_PACKET_HEADER;
-       packet.seqno = atomic_inc_return(&hp->seqno);
-       packet.len = 6;
+       packet.hdr.type = VS_CONTROL_PACKET_HEADER;
+       packet.hdr.seqno = atomic_inc_return(&hp->seqno);
+       packet.hdr.len = 6;
        packet.verb = VSV_CLOSE_PROTOCOL;
 
-       pr_debug("%s: sending %i bytes\n", __func__, packet.len);
-       dbg_dump_hex((uint8_t*)&packet, packet.len);
+       pr_debug("%s: sending %i bytes\n", __func__, packet.hdr.len);
+       dbg_dump_hex((uint8_t*)&packet, packet.hdr.len);
 
-       hvc_put_chars(hp->vtermno, (char *)&packet, packet.len);
+       hvc_put_chars(hp->vtermno, (char *)&packet, packet.hdr.len);
 }
 
 static int hvsi_open(struct tty_struct *tty, struct file *filp)