Говнокод #27516 — Си — Говнокод.ру

Си / Говнокод #27516

+4
1. 01
2. 02
3. 03
4. 04
5. 05
6. 06
7. 07
8. 08
9. 09
10. 10
11. 11
12. 12
13. 13
14. 14
15. 15
16. 16
17. 17
18. 18
19. 19
20. 20
21. 21
22. 22
23. 23
24. 24
25. 25
26. 26
27. 27
28. 28
29. 29
30. 30
31. 31
32. 32
33. 33
34. 34
35. 35
```
while (1) {
    uint16_t state = in(USB_BASE + USB_ISTR);
    if (state & 0x8000) {
        uint16_t epstat = in(USB_BASE + USB_EP0R);
        if (epstat & 0x8000) {
            // RX done
            if (epstat & 0x0800) {
                // setup
                if ((usbRead(0x180) == 0x80) && (usbRead(0x181) == 0x06)) {
                    uint16_t maxlen = (usbRead(0x187) << 8) | usbRead(0x186);
                    if (usbRead(0x183) == 0x01) {
                        // send device descriptor
                        copyToUsb(0x0100, deviceDescriptor, sizeof(deviceDescriptor));
                        if (sizeof(deviceDescriptor) < maxlen)
                            maxlen = sizeof(deviceDescriptor);
                        out(USB_SRAM_BASE + 0x04, 0x9000 + maxlen); // TX count
                        // STAT_TX=11 (valid)
                        if ((in(USB_BASE + USB_EP0R) & 0x0010) != 0x0010)
                            out(USB_BASE + USB_EP0R, 0x0210);
                        if ((in(USB_BASE + USB_EP0R) & 0x0020) != 0x0020)
                            out(USB_BASE + USB_EP0R, 0x0220);
                    } else if (usbRead(0x183) == 0x02) {
                        // send config descriptor
                        copyToUsb(0x0100, configDescriptor, sizeof(configDescriptor));
                        if (sizeof(configDescriptor) < maxlen)
                            maxlen = sizeof(configDescriptor);
                        out(USB_SRAM_BASE + 0x04, 0x9000 + maxlen); // TX count
                        // STAT_TX=11 (valid)
                        if ((in(USB_BASE + USB_EP0R) & 0x0010) != 0x0010)
                            out(USB_BASE + USB_EP0R, 0x0210);
                        if ((in(USB_BASE + USB_EP0R) & 0x0020) != 0x0020)
                            out(USB_BASE + USB_EP0R, 0x0220);
                    } else {

// ... и ещё сотня строк в том же духе ...
```
Иногда в меня вселяется дух PHP...

Запостил: bormand, 13 Июля 2021

Tweet
Комментарии (30) RSS
- bormand 13.07.2021 21:32 # 0
  
  Тот самый момент, когда под рукой не оказалось USB UART зато была STM'ка.
  Ответить
- bormand 13.07.2021 21:59 # 0
  // USBPRE=0 (USB=72MHz/1.5=48MHz) // PLLMUL=0111 (PLLOUT=8MHz*9=72MHz) // PLLXTPRE=0 (PLLIN=HSE=8MHz) // PLLSRC=1 (HSE) // PPRE2=000 (APB2=AHB=72MHz) // PPRE1=100 (APB1=AHB/2=36MHz) // HPRE=0000 (AHB=72MHz) // SW=0 (stay on HSI during PLL startup) out(RCC_BASE + RCC_CFGR, 0x001D0400); // PLLON=1 out(RCC_BASE + RCC_CR, in(RCC_BASE + RCC_CR) | 0x01000000); // Wait for PLLRDY while ((in(RCC_BASE + RCC_CR) & 0x02000000) == 0);
  Константы? Не, ня слышала.
  Ответить
- j123123 13.07.2021 22:57 # 0
  if ((in(USB_BASE + USB_EP0R) & 0x0010) != 0x0010) out(USB_BASE + USB_EP0R, 0x0210); if ((in(USB_BASE + USB_EP0R) & 0x0020) != 0x0020) out(USB_BASE + USB_EP0R, 0x0220); ...
  
  Я бы такую хуйню в макрос завернул
  
  #define SHIT(x) \ if ((in(USB_BASE + USB_EP0R) & (x)) != (x)) \ out(USB_BASE + USB_EP0R, 0x0200 | (x) );
  Ответить
  - bormand 13.07.2021 22:58 # +1
    
    Да вообще в функцию, наверное. А то оно там дофига раз встречается.
    Ответить
  - bormand 13.07.2021 23:48 # +1
    
    По идее эти портянки копипасты вообще должны быть каким-нибудь usb_respond(ep, data, size). Но что вышло то вышло.
    Ответить
    - ASD_77 14.07.2021 01:16 # +2
      
      ужос. есть еще в мире кто так пишет. все нормальные люди пишут так usb() << data << usb::pos(10) << bla bla;
      Ответить
      
      j123123 14.07.2021 03:38 # +8
      
      Это хуита одномерная, надо чтоб можно было направленные ациклические графы рисовать, типа
      
      (a) >> (b) >> (c) ^ ^ ^ ^ (d) (e) >> (g) ^ ^ ^ ^ (f) > > *
      
      Или даже чтоб в 3D!
      Ответить
      
      HEu3BECTHblu_nemyx 14.07.2021 03:47 # +3
      
      Есть такой язык:
      https://ru.wikipedia.org/wiki/Befunge
      Ответить
      
      j123123 14.07.2021 03:47 # +1
      
      Если серьезно, ничего такого хорошего я не вижу в этой вот "<<" хрени в тех же крестах с их std::cout
      Ответить
      
      HEu3BECTHblu_nemyx 14.07.2021 03:49 # +1
      
      Да, есть багор: можно спутать со сдвигом. Хоть скобки ставь повсюду, как в Лиспе.
      Ответить
      
      guest6 14.07.2021 05:20 # 0
      
      хорошего там то, что можно определить этот оператор на объекте
      Ответить
      
      bormand 14.07.2021 16:01 # 0
      
      Но писать его всегда приходится снаружи т.к. левый аргумент -- это стрим.
      Ответить
- JloJle4Ka 14.07.2021 04:17 # +1
  
  Оглянись вокруг посмотри назад
  Духи с тобою связаться хотят.....
  Ответить
  - j123123 14.07.2021 15:59 # +1
    
    Призрачный голос со стороны параши
    https://en.wikipedia.org/wiki/Hanako-san
    Ответить
- guest6 14.07.2021 05:18 # +1
  
  ненавижу лесенки такие
  
  >usbRead(0x181) == 0x06
  могу простить
  а лесенки нет
  
  >USB_BASE + USB_EP0R
  тоже лучше вынести
  Ответить
  - bormand 14.07.2021 16:02 # 0
    
    > ненавижу лесенки такие
    
    Ты не поверишь... я тоже ;)
    
    Но говнопрототип на то и говнопрототип.
    Ответить
    - MAKAKA 14.07.2021 16:55 # 0
      
      а твой IDE экстрактить не умеет?
      Ответить
      
      bormand 14.07.2021 17:00 # +1
      
      Экстракт говна на вкус ничуть не лучше, чем оно само...
      Ответить
      
      MAKAKA 14.07.2021 17:01 # +1
      
      ну ты бы хотя бы разобрал лесенку
      
      На самом деле код довольно прямолинейный, и не выглядит таким уж адом (если знать предметную область конечно)
      Ответить
      
      bormand 14.07.2021 17:03 # 0
      
      А смысл разбирать лесенку в коде, который написан на выброс?
      
      Я специально пишу прототипы в таком стиле, чтобы вообще не возникало желания их рефакторить.
      Ответить
      
      Desktop 14.07.2021 17:10 # 0
      
      интересный подход. и как, работает?
      Ответить
      
      bormand 14.07.2021 17:19 # 0
      
      Вполне. Ну суть в том, что иначе я буду пытаться придумать красивую архитектуру и сделать конфетку. А для первого прототипа эта красота будет только мешать т.к. я ещё толком не понимаю предметную область.
      
      Поэтому хуяк-хуяк и в продакшен. А потом можно красиво переписать с нуля.
      
      Там, где ты всё уже знаешь и умеешь, смысла в такой методологии нету, наверное. Можно сразу чистовик ебашить.
      Ответить
      
      MAKAKA 14.07.2021 17:44 # +1
      
      а, ну я тоже иногда пишу на коленке куски говнокода когда что-то изучаю (API например), но я их даже не коммичу обычно
      
      Иногда полезно три раза с ноля одно и тоже написать: каждый следующий раз код будет чище, а ты лучше поймешь API
      
      Но это только маленькие участочки кода конечно, не целые приложения
      Ответить
      
      bormand 14.07.2021 17:11 # 0
      
      > если знать предметную область конечно
      
      Если знать предметную область, то тут всё ещё хуже.
      
      Я деталей уже не помню, но вот эти ожидания флажков синхронные и могут застрять на несколько миллисекунд. Или вообще повиснуть если софт со стороны компа упал. И плату придётся ребутать.
      
      Т.е. говно тут от корней до самых кончиков.
      Ответить
      
      MAKAKA 14.07.2021 17:15 # 0
      
      Я не знаю где это запускается, и насколько валидно зависать на несколько миллисек
      
      А как правильно? Через прерывания?
      Ответить
      
      bormand 14.07.2021 17:18 # 0
      
      > через прерывания
      
      По хорошему -- да. while (true) с поллингом флажков ещё и не особо энергосберегающий.
      Ответить
      
      MAKAKA 14.07.2021 17:19 # 0
      
      да, бизивейт такой малоприятный, и жрущий цпу
      понял тебя
      
      Есть кстати примеры бизивейта вполне обоснованные, например спинлоки в ядре или дрееевний софт на однозадачной ОС и ЦПУ без ACPI/APM
      
      Есть даже алгоритмы синхронизации (Петерсона и Деккера вроде) на бизивейте
      Ответить
      
      bormand 14.07.2021 17:30 # 0
      
      Не, ну в теории и в бизивейт можно въебать какой-нибудь wfi чтобы проц спал до следующего прерывания.
      Ответить
      
      MAKAKA 14.07.2021 17:42 # 0
      
      этим ты решишь проблему бесполезной траты питания, глобального потепления и таянья ледников
      
      но ты не один на цпу, угу
      Ответить
      
      bormand 14.07.2021 20:17 # 0
      
      В этом случае в общем-то один. Там ещё обработчик таймерного прерывания, но он более приоритетный и ему пофиг.
      Ответить
Добавить комментарий
Ошибка компиляции комментария:

Гости могут высказаться только в понедельник, среду, четверг или воскресение
Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

А не использовать ли нам bbcode?

[b]жирный[/b] — жирный

[i]курсив[/i] — курсив

[u]подчеркнутый[/u] — подчеркнутый

[s]перечеркнутый[/s] — перечеркнутый

[blink]мигающий[/blink] — мигающий

[color=red]цвет[/color] — цвет (подробнее)

[size=20]размер[/size] — размер (подробнее)

[code=<language>]some code[/code] (подробнее)

Проверочный код: *

Говнокод: по колено в коде.

Си / Говнокод #27516

Комментарии (30) RSS

Добавить комментарий