как обработать установочный пакет usb Опции

[sergey sva]

Разбираюсь с работой усб немного не понятно,с пакетами,
к примеру: программа приступила к обработки прерывания, определила какая конечная точка вызвала это прерывание,
определила, что поступил установочный пакет if (save_reg_csrX & AT91C_UDP_RXSETUP) {
какие действия нужно проделать дальше что бы обработать этот пакет? Или для at91sam7 его не надо обрабатывать?

[aaarrr]

какие действия нужно проделать дальше что бы обработать этот пакет?

Дальше нужно узнать, что от нас хочет хост, т.е. разобрать пакет в соответствии с разделом 9 (USB Device Framework) спецификации USB.

[sergey sva]

понятно , будем разбирать.
Есть простой вопрос наверно по си, не хочется темы размножать с простыми вопросами, напишу здесь.
вот объявляем новый тип SetPacked который состоит из 8 байт:

Код

typedef struct{
BYTE  bmReqestType;
BYTE  bReqest;
DWORD wValue;
DWORD wIndex;
DWORD wLenght;
}SetPacked;

Вот еще объявляем? что то, как это будет работать не пойму, пример из книги Агурова.

Код

typedef union{
SetPacked  setup;
BYTE   b[8];
WORD  wREqest;
}usbSetPacked;

объясните если не затруднит.

[aaarrr]

вот объявляем новый тип SetPacked который состоит из 8 байт:

А DWORD у Вас - это short? Как-то нестандартно очень.

Вот еще объявляем? что то, как это будет работать не пойму, пример из книги Агурова.

Я бы не стал вообще заморачиваться структурами и юнионами для setup-пакета. Зачем?

[rezident]

Какой-то немного странный union, вы wREqest[4] может быть хотели написать? Но раз так написали, то в памяти переменные будут расположены следующим образом (если принять порядок следования как у "больших индейцев")

Код

1-й байт, смещение 0: bmReqestType    - b[0] - мл. байт wREqest
2-й байт, смещение 1: bReqest         - b[1] - ст. байт wREqest
3-й байт, смещение 2: мл.байт wValue  - b[2] - ничего
4-й байт, смещение 3: ст.байт wValue  - b[3] - ничего
5-й байт, смещение 4: мл.байт wIndex  - b[4] - ничего
6-й байт, смещение 5: ст.байт wIndex  - b[5] - ничего
7-й байт, смещение 6: мл.байт wLenght - b[6] - ничего
8-й байт, смещение 7: ст.байт wLenght - b[7] - ничего

[sergey sva]

А DWORD у Вас - это short? Как-то нестандартно очень.

Опечатка вышла WORD там должен быть, unsigned short.

[aaarrr]

если принять порядок следования как у "больших индейцев"

А написано для little. Вот поэтому лучше работать с байтами, тем более, что их всего восемь, а поля в setup-пакете не меняются.

[sergey sva]

Вот набросал не большой разборщик установочного пакета, может что здесь нужно подправить ?

CODE

//-------------------------------------------------------------------------------
void PROCES_SETUP_PACKET(DWORD EP){
DWORD LenByt, n;
BYTE SetPacket[8];
WORD bReqest;
/* */
EP &=0x0F;
/* */
LenByt = (AT91C_BASE_UDP->UDP_CSR[EP] >> 16) & 0x07FF;
for (n = 0; (n < LenByt && n < 8); n++) {
SetPacket[n] = (BYTE) AT91C_BASE_UDP->UDP_FDR[EP];
}//end for
/* */
bReqest = SetPacket[0] & 0xFF ;
bReqest = (bReqest<<8) | (SetPacket[1] & 0xFF);
/* */
switch(bReqest){
case GET_STATUS_DEVICE:

break;
case GET_STATUS_INTERF:

break;
case GET_STATUS_ENDPNT:

break;
case GET_DESCRIPTOR_DEVICE:
case GET_DESCRIPTOR_INTERF:
case GET_DESCRIPTOR_ENDPNT:

break;
case GET_CONFIGURATION:

break;
case SET_CONFIGURATION:

break;
case SET_ADDRESS:

break;

default :

break;
}//end switch

}//end PROCES_SETUP_PACKET
//-------------------------------------------------------------------------------

[aaarrr]

Ну, это пока еще не разборщик

Не вижу смысла объединять поля bmRequestType и bRequest в одной переменной - будет весьма неудобно работать.

[sergey sva]

Ну, это пока еще не разборщик smile.gif

Будем потихоньку разбираться с этими разборщиками.
Вот наваял для разборки запроса от хоста Get_stat_dev,
может что тоже нужно подправить, может в обращениях к регистрам что подправить?

CODE

//-------------------------------------------------------------------------------
void end_point_read(DWORD EP, BYTE *dat){
DWORD LenByt, n;
/* */
EP &=0x0F;
LenByt = (AT91C_BASE_UDP->UDP_CSR[EP] >> 16) & 0x07FF;
for (n = 0; n < LenByt; n++) {
*dat++ = (BYTE) AT91C_BASE_UDP->UDP_FDR[EP];
}//end for
/* */
}//end
//-------------------------------------------------------------------------------
DWORD end_point_write(DWORD EP, BYTE dat[], DWORD len) {
DWORD n;
/* */
EP &= 0x0F;
/* */
if (AT91C_BASE_UDP->UDP_CSR[EP] & AT91C_UDP_TXPKTRDY) {
if ((DualBankEP & (1 << EP)) && (TxDataBank[EP] == 0)) {
TxDataBank[EP] = 1;
} else {
return 0;
}
}//end if AT91C_UDP_TXPKTRDY


for (n = 0; n < len; n++){
AT91C_BASE_UDP->UDP_FDR[EP] = dat[n];
}
AT91C_BASE_UDP->UDP_CSR[EP] |= AT91C_UDP_TXPKTRDY;
while(!(AT91C_BASE_UDP->UDP_CSR[EP] & AT91C_UDP_TXCOMP));
AT91C_BASE_UDP->UDP_CSR[EP] &= ~AT91C_UDP_TXCOMP;
return len;
}
//-------------------------------------------------------------------------------
void Get_stat_dev(DWORD EP){
BYTE OPT[2];
EP &=0x0F;
/* */
OPT[0] = (POW_BUS | IGNOR_SIG);
OPT[1] = 0;
AT91C_BASE_UDP->UDP_CSR[EP] |= AT91C_UDP_DIR;

end_point_write(EP,OPT,2);

while(!(AT91C_BASE_UDP->UDP_CSR[EP] & AT91C_UDP_RXSETUP));
AT91C_BASE_UDP->UDP_CSR[EP] &= ~AT91C_UDP_RXSETUP;
AT91C_BASE_UDP->UDP_CSR[EP] &= ~AT91C_UDP_DIR;
}//end void Get_stat_dev(void)
//-------------------------------------------------------------------------------

[aaarrr]

Вот наваял для разборки запроса от хоста Get_stat_dev,

Логичнее было бы начинать с GET_DESCRIPTOR.

может что тоже нужно подправить, может в обращениях к регистрам что подправить?

Обратите внимание на методику и последовательность установки/снятия битов регистра UDP_CSR в датшите. В частности, DIR нужно ставить до снятия RXSETUP и т.п.

[sergey sva]

исправил. Еще есть вопрос читаю по ходу книгу агурова, есть не ясность не большая, на картинке.
там функция getstatendpoint в ней объявляется переменная wIndex для чего она здесь?

[aaarrr]

для чего она здесь?

Наверное, осталась от copy-paste. Компилятор должен выдать ворнинг.

[sergey sva]

Вот функция которая записывает в конечную точку байты.:

CODE

//-------------------------------------------------------------------------------
DWORD end_point_write(DWORD EP, BYTE dat[], DWORD len) {
DWORD n;
/* */
EP &= 0x0F;
/* */
if (AT91C_BASE_UDP->UDP_CSR[EP] & AT91C_UDP_TXPKTRDY) {
if ((DualBankEP & (1 << EP)) && (TxDataBank[EP] == 0)) {
TxDataBank[EP] = 1;
} else {
return 0;
}
}//end if AT91C_UDP_TXPKTRDY


for (n = 0; n < len; n++){
AT91C_BASE_UDP->UDP_FDR[EP] = dat[n];
}
AT91C_BASE_UDP->UDP_CSR[EP] |= AT91C_UDP_TXPKTRDY;
while(!(AT91C_BASE_UDP->UDP_CSR[EP] & AT91C_UDP_TXCOMP));
AT91C_BASE_UDP->UDP_CSR[EP] &= ~AT91C_UDP_TXCOMP;
return len;
}
//-------------------------------------------------------------------------------

сначала здесь записываются в буфер конечной точки байты , после устанавливается флаг окончания записи,
далеше в цикле while ожидаю когда хост подтвердит получение байт, установкой флага AT91C_UDP_TXCOMP,
а если этого не произойдет, то программа зависнет в этом месте.
Как получше это обработать, может еще какие флаги проверять, или таймаут добавить ?

Причина редактирования: Уменьшение видимого размера исходника.

[aaarrr]

Код

  EP &= 0x0F;

Лучше сделать здесь реальную проверку, или не делать вообще.

Код

  TxDataBank[EP] = 1;

А где это сбрасывается?

Код

  AT91C_BASE_UDP->UDP_CSR[EP] |= AT91C_UDP_TXPKTRDY;

Обратите внимание на Warning в описании UDP_CSR. В данном случае он важен.

Код

  while(!(AT91C_BASE_UDP->UDP_CSR[EP] & AT91C_UDP_TXCOMP));

Простым ожиданием TXCOMP убиваются все преимущества двойной буферизации.


Совет: не пытайтесь написать универсальные процедуры обслуживания конечных точек -
ничего, кроме лишнего геморроя, это не принесет.