#include <stdio.h>

typedef union {
    unsigned char c[2];
    unsigned short n;
} W_16;

void swap_w16( unsigned short *x );

int main(void)
{
        unsigned short x = 0x55aa;

        printf("%hx\n", x);
        swap_w16(&x);
        printf("%hx\n", x);
      
        return (0);
}

void swap_w16( unsigned short *x )
{
        unsigned char w;
      
        w = ((W_16 *)x)->c[1];
        ((W_16 *)x)->c[1] = ((W_16 *)x)->c[0] ;
        ((W_16 *)x)->c[0] = w;
}

> void swap_w16( unsigned short *x )
> {
>     unsigned char w;
>     
>     w = ((W_16 *)x)->c[1];
>     ((W_16 *)x)->c[1] = ((W_16 *)x)->c[0] ;
>     ((W_16 *)x)->c[0] = w;
> }


私はこの解に不満があります。union のメンバー n が使われていません。
unsigned short へのポインタを W_16 へのポインタにキャストすることで、
*x を n に重ね合わせていると強弁することもできますが、キャストを使う
のなら、次のように書けば済みます。

void swap_w16(unsigned short *x)
{
    unsigned char w, *p;

    p = (unsigned char *)x;
    w = p[1];
    p[1] = p[0];
    p[0] = w;
}

キャストを使わず union だけを使うとすれば、

void swap_w16(unsigned short *x)
{
    W_16 w1, w2;

    w1.n = *x;
    w2.c[0] = w1.c[1];
    w2.c[1] = w1.c[0];
    *x = w2.n;
}

void swap_w16(unsigned short *x)
{
    W_16 w1, w2;

    w1.n = *x;
    w2.c[0] = w1.c[1];
    w2.c[1] = w1.c[0];
    *x = w2.n;
}

unsigned short swap_w16(unsigned short x)
{
    W_16 w;
    w.c[0] = ((W_16 *)&x)->c[1];
    w.c[1] = ((W_16 *)&x)->c[0];
    return (w.n);
}