ALSA: firewire-lib: Add handling output connection by CMP
authorTakashi Sakamoto <o-takashi@sakamocchi.jp>
Fri, 25 Apr 2014 13:44:56 +0000 (22:44 +0900)
committerTakashi Iwai <tiwai@suse.de>
Mon, 26 May 2014 12:22:37 +0000 (14:22 +0200)
This patch adds some macros, codes with condition of direction and new functions
to handle output connection. Once cmp_connection_init() is executed with its
direction, CMP input and output connection can be handled by the same way.

Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/firewire/cmp.c

index c7f7a3164390e6c746b1e77e3fa27769a953d92a..9da40d9e79daa8526e0e73c858eb486a4e2864c3 100644 (file)
 #define PCR_CHANNEL_MASK       0x003f0000
 #define PCR_CHANNEL_SHIFT      16
 
+/* oPCR specific fields */
+#define OPCR_XSPEED_MASK       0x00C00000
+#define OPCR_XSPEED_SHIFT      22
+#define OPCR_SPEED_MASK                0x0000C000
+#define OPCR_SPEED_SHIFT       14
+#define OPCR_OVERHEAD_ID_MASK  0x00003C00
+#define OPCR_OVERHEAD_ID_SHIFT 10
+
 enum bus_reset_handling {
        ABORT_ON_BUS_RESET,
        SUCCEED_ON_BUS_RESET,
@@ -41,10 +49,27 @@ void cmp_error(struct cmp_connection *c, const char *fmt, ...)
 
        va_start(va, fmt);
        dev_err(&c->resources.unit->device, "%cPCR%u: %pV",
-               'i', c->pcr_index, &(struct va_format){ fmt, &va });
+               (c->direction == CMP_INPUT) ? 'i' : 'o',
+               c->pcr_index, &(struct va_format){ fmt, &va });
        va_end(va);
 }
 
+static u64 mpr_address(struct cmp_connection *c)
+{
+       if (c->direction == CMP_INPUT)
+               return CSR_REGISTER_BASE + CSR_IMPR;
+       else
+               return CSR_REGISTER_BASE + CSR_OMPR;
+}
+
+static u64 pcr_address(struct cmp_connection *c)
+{
+       if (c->direction == CMP_INPUT)
+               return CSR_REGISTER_BASE + CSR_IPCR(c->pcr_index);
+       else
+               return CSR_REGISTER_BASE + CSR_OPCR(c->pcr_index);
+}
+
 static int pcr_modify(struct cmp_connection *c,
                      __be32 (*modify)(struct cmp_connection *c, __be32 old),
                      int (*check)(struct cmp_connection *c, __be32 pcr),
@@ -60,8 +85,7 @@ static int pcr_modify(struct cmp_connection *c,
 
                err = snd_fw_transaction(
                                c->resources.unit, TCODE_LOCK_COMPARE_SWAP,
-                               CSR_REGISTER_BASE + CSR_IPCR(c->pcr_index),
-                               buffer, 8,
+                               pcr_address(c), buffer, 8,
                                FW_FIXED_GENERATION | c->resources.generation);
 
                if (err < 0) {
@@ -101,9 +125,9 @@ int cmp_connection_init(struct cmp_connection *c,
        u32 mpr;
        int err;
 
+       c->direction = direction;
        err = snd_fw_transaction(unit, TCODE_READ_QUADLET_REQUEST,
-                                CSR_REGISTER_BASE + CSR_IMPR,
-                                &mpr_be, 4, 0);
+                                mpr_address(c), &mpr_be, 4, 0);
        if (err < 0)
                return err;
        mpr = be32_to_cpu(mpr_be);
@@ -151,6 +175,53 @@ static __be32 ipcr_set_modify(struct cmp_connection *c, __be32 ipcr)
        return ipcr;
 }
 
+static int get_overhead_id(struct cmp_connection *c)
+{
+       int id;
+
+       /*
+        * apply "oPCR overhead ID encoding"
+        * the encoding table can convert up to 512.
+        * here the value over 512 is converted as the same way as 512.
+        */
+       for (id = 1; id < 16; id++) {
+               if (c->resources.bandwidth_overhead < (id << 5))
+                       break;
+       }
+       if (id == 16)
+               id = 0;
+
+       return id;
+}
+
+static __be32 opcr_set_modify(struct cmp_connection *c, __be32 opcr)
+{
+       unsigned int spd, xspd;
+
+       /* generate speed and extended speed field value */
+       if (c->speed > SCODE_400) {
+               spd  = SCODE_800;
+               xspd = c->speed - SCODE_800;
+       } else {
+               spd = c->speed;
+               xspd = 0;
+       }
+
+       opcr &= ~cpu_to_be32(PCR_BCAST_CONN |
+                            PCR_P2P_CONN_MASK |
+                            OPCR_XSPEED_MASK |
+                            PCR_CHANNEL_MASK |
+                            OPCR_SPEED_MASK |
+                            OPCR_OVERHEAD_ID_MASK);
+       opcr |= cpu_to_be32(1 << PCR_P2P_CONN_SHIFT);
+       opcr |= cpu_to_be32(xspd << OPCR_XSPEED_SHIFT);
+       opcr |= cpu_to_be32(c->resources.channel << PCR_CHANNEL_SHIFT);
+       opcr |= cpu_to_be32(spd << OPCR_SPEED_SHIFT);
+       opcr |= cpu_to_be32(get_overhead_id(c) << OPCR_OVERHEAD_ID_SHIFT);
+
+       return opcr;
+}
+
 static int pcr_set_check(struct cmp_connection *c, __be32 pcr)
 {
        if (pcr & cpu_to_be32(PCR_BCAST_CONN |
@@ -196,8 +267,13 @@ retry_after_bus_reset:
        if (err < 0)
                goto err_mutex;
 
-       err = pcr_modify(c, ipcr_set_modify, pcr_set_check,
-                        ABORT_ON_BUS_RESET);
+       if (c->direction == CMP_OUTPUT)
+               err = pcr_modify(c, opcr_set_modify, pcr_set_check,
+                                ABORT_ON_BUS_RESET);
+       else
+               err = pcr_modify(c, ipcr_set_modify, pcr_set_check,
+                                ABORT_ON_BUS_RESET);
+
        if (err == -EAGAIN) {
                fw_iso_resources_free(&c->resources);
                goto retry_after_bus_reset;
@@ -245,8 +321,13 @@ int cmp_connection_update(struct cmp_connection *c)
        if (err < 0)
                goto err_unconnect;
 
-       err = pcr_modify(c, ipcr_set_modify, pcr_set_check,
-                        SUCCEED_ON_BUS_RESET);
+       if (c->direction == CMP_OUTPUT)
+               err = pcr_modify(c, opcr_set_modify, pcr_set_check,
+                                SUCCEED_ON_BUS_RESET);
+       else
+               err = pcr_modify(c, ipcr_set_modify, pcr_set_check,
+                                SUCCEED_ON_BUS_RESET);
+
        if (err < 0)
                goto err_resources;