summaryrefslogtreecommitdiffstats
path: root/apps
diff options
context:
space:
mode:
authorBertrik Sikken <bertrik@sikken.nl>2011-04-21 22:36:10 +0000
committerBertrik Sikken <bertrik@sikken.nl>2011-04-21 22:36:10 +0000
commitd038db9420de69bdcd3859272c216ed4a0a2e142 (patch)
treefd39a97e528b25b3e8e659d98cd0e65a7d58207b /apps
parent8f8f9a5bde4750fadca0a4b24800e3e46fb848a3 (diff)
downloadrockbox-d038db9420de69bdcd3859272c216ed4a0a2e142.tar.gz
rockbox-d038db9420de69bdcd3859272c216ed4a0a2e142.zip
iap: introduce cmd_ok_mode0 and cmd_ok_mode4 functions to reduce code duplication
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@29757 a1c6a512-1295-4272-9138-f99709370657
Diffstat (limited to 'apps')
-rw-r--r--apps/iap.c49
1 files changed, 25 insertions, 24 deletions
diff --git a/apps/iap.c b/apps/iap.c
index 7447c67ba9..48344cd4cc 100644
--- a/apps/iap.c
+++ b/apps/iap.c
@@ -218,6 +218,13 @@ static void iap_set_remote_volume(void)
iap_send_pkt(data, sizeof(data));
}
+static void cmd_ok_mode0(unsigned char cmd)
+{
+ unsigned char data[] = {0x00, 0x02, 0x00, 0x13};
+ data[3] = cmd; /* respond with cmd */
+ iap_send_pkt(data, sizeof(data));
+}
+
static void iap_handlepkt_mode0(void)
{
unsigned int cmd = serbuf[2];
@@ -260,8 +267,7 @@ static void iap_handlepkt_mode0(void)
case 0x13:
{
- unsigned char data[] = {0x00, 0x02, 0x00, 0x13};
- iap_send_pkt(data, sizeof(data));
+ cmd_ok_mode0(cmd);
if (serbuf[6] == 0x35)
/* FM transmitter sends this: */
@@ -322,8 +328,7 @@ static void iap_handlepkt_mode0(void)
unsigned char data[] = {0x00, 0x02, 0x06,
0x05, 0x00, 0x00, 0x0B, 0xB8, 0x28};
iap_send_pkt(data, sizeof(data));
- unsigned char data2[] = {0x00, 0x02, 0x00, 0x05};
- iap_send_pkt(data2, sizeof(data2));
+ cmd_ok_mode0(cmd);
break;
}
@@ -349,9 +354,7 @@ static void iap_handlepkt_mode0(void)
/* default response is with cmd ok packet */
default:
{
- unsigned char data[] = {0x00, 0x02, 0x00, 0x00};
- data[3] = cmd; /* respond with cmd */
- iap_send_pkt(data, sizeof(data));
+ cmd_ok_mode0(cmd);
break;
}
}
@@ -492,6 +495,14 @@ static void iap_handlepkt_mode3(void)
}
}
+static void cmd_ok_mode4(unsigned int cmd)
+{
+ unsigned char data[] = {0x04, 0x00, 0x01, 0x00, 0x00, 0x00};
+ data[4] = (cmd >> 8) & 0xFF;
+ data[5] = (cmd >> 0) & 0xFF;
+ iap_send_pkt(data, sizeof(data));
+}
+
static void iap_handlepkt_mode4(void)
{
unsigned int cmd = (serbuf[2] << 8) | serbuf[3];
@@ -510,8 +521,7 @@ static void iap_handlepkt_mode4(void)
{
iap_updateflag = serbuf[4] ? 0 : 1;
/* respond with cmd ok packet */
- unsigned char data[] = {0x04, 0x00, 0x01, 0x00, 0x00, 0x0B};
- iap_send_pkt(data, sizeof(data));
+ cmd_ok_mode4(cmd);
break;
}
/* Get iPod size? */
@@ -634,9 +644,8 @@ static void iap_handlepkt_mode4(void)
case 0x0026:
{
iap_pollspeed = serbuf[4] ? 1 : 0;
- /*responsed with cmd ok packet */
- unsigned char data[] = {0x04, 0x00, 0x01, 0x00, 0x00, 0x26};
- iap_send_pkt(data, sizeof(data));
+ /*respond with cmd ok packet */
+ cmd_ok_mode4(cmd);
break;
}
/* AiR playback control */
@@ -715,8 +724,7 @@ static void iap_handlepkt_mode4(void)
/* respond with cmd ok packet */
- unsigned char data[] = {0x04, 0x00, 0x01, 0x00, 0x00, 0x2E};
- iap_send_pkt(data, sizeof(data));
+ cmd_ok_mode4(cmd);
break;
}
/* Get repeat mode */
@@ -751,8 +759,7 @@ static void iap_handlepkt_mode4(void)
}
/* respond with cmd ok packet */
- unsigned char data[] = {0x04, 0x00, 0x01, 0x00, 0x00, 0x31};
- iap_send_pkt(data, sizeof(data));
+ cmd_ok_mode4(cmd);
break;
}
/* Get Screen Size */
@@ -790,19 +797,13 @@ static void iap_handlepkt_mode4(void)
audio_resume();
/* respond with cmd ok packet */
- unsigned char data[] = {0x04, 0x00, 0x01, 0x00, 0x00, 0x00};
- data[4] = serbuf[2];
- data[5] = serbuf[3];
- iap_send_pkt(data, sizeof(data));
+ cmd_ok_mode4(cmd);
break;
}
default:
{
/* default response is with cmd ok packet */
- unsigned char data[] = {0x04, 0x00, 0x01, 0x00, 0x00, 0x00};
- data[4] = serbuf[2];
- data[5] = serbuf[3];
- iap_send_pkt(data, sizeof(data));
+ cmd_ok_mode4(cmd);
break;
}
}