stm32f070f6px - Githubissues

bindassdost commented 5 years ago

i am trying to get your library working on my stm32f070f6 chip from past two days. i have read previous issue which says stm32l052 code should work on my chipset. beacuse startup code for my chip was missing in cdc_startup.c file. i had added it as below

#elif defined(STM32F070x6)
    /* set flash latency 1WS */
    _BMD(FLASH->ACR, FLASH_ACR_LATENCY, 1);
    if ((RCC->CFGR & RCC_CFGR_SWS) == RCC_CFGR_SWS_PLL) /* (1) */
    {
        RCC->CFGR &= (uint32_t) (~RCC_CFGR_SW); /* (2) */
        while ((RCC->CFGR & RCC_CFGR_SWS) != RCC_CFGR_SWS_HSI) /* (3) */
        {
            /* For robust implementation, add here time-out management */
        }
    }
    RCC->CR &= (uint32_t)(~RCC_CR_PLLON);/* (4) */
    while((RCC->CR & RCC_CR_PLLRDY) != 0) /* (5) */
    {
        /* For robust implementation, add here time-out management */
    }
    RCC->CFGR = (RCC->CFGR & (~RCC_CFGR_PLLMUL)) | (RCC_CFGR_PLLMUL6); /* (6) */
    RCC->CR |= RCC_CR_PLLON; /* (7) */
    while((RCC->CR & RCC_CR_PLLRDY) == 0) /* (8) */
    {
        /* For robust implementation, add here time-out management */
    }
    RCC->CFGR |= (uint32_t) (RCC_CFGR_SW_PLL); /* (9) */
    while ((RCC->CFGR & RCC_CFGR_SWS) != RCC_CFGR_SWS_PLL) /* (10) */
    {
        /* For robust implementation, add here time-out management */
    }
    RCC->CFGR3 |= RCC_CFGR3_USBSW_PLLCLK;
#endif

but i am still unable to get it working. this clock setup was working with cubemx project so i think my startup code does set and enable usb clock through pll to 48Mhz.

defines that i was using are STM32F0, STM32F070x6, USBD_SOF_DISABLED

default linker script for stm32f070f6px was used with --specs=nosys.specs

please help me

MohammedNoureldin commented 5 years ago

Please reformat the code, it is unreadable. Do you have an oscilloscope available?

bindassdost commented 5 years ago

I had fixed formatting the code. I do not have an oscilloscope available. Startup code was taken from stm32f070f6p6 reference manual. same code was tested on CubeMX project where it was working fine but with huge 22KB compile size with only bare-bone CDC loopback. that is why this library is important for me because i had less than 10KB flash available with cubemx

dmitrystu commented 5 years ago

RCC setup looks correct. According to datasheet PA11 and PA12 must be switched to USB mode automatically. Also this https://github.com/dmitrystu/libusb_stm32/issues/29 maybe helpful

dmitrystu commented 5 years ago

Checked RM0360. Found 'Figure 11. Clock tree (STM32F070x6, STM32F070xB and STM32F030xC)' doesn't meet '7.4.2 Clock configuration register (RCC_CFGR) bit 16' (HSI vs HSI/2 as PLL in). Try to set PLLMUL to 12.

MohammedNoureldin commented 5 years ago

That is exactly why I asked him for an oscilloscope, I had always doubts about configuring the clock to the correct frequency, till I got an oscilloscope and validated the frequency using the MCO pins.

bindassdost commented 5 years ago

First of all Thank you @dmitrystu for taking time to reply.

i had read reference manual one more time to have bit deeper understanding. i had modified startup code as below but still my usb doesnt get recognized.

//
    _BMD(RCC->CR, RCC_CR_HSITRIM, RCC_CR_HSITRIM_4); // SET TRIM TO 16
    _BST(RCC->CR, RCC_CR_HSION); // TURN ON HSI
    _WBS(RCC->CR, RCC_CR_HSIRDY); // WAIT FOR READY
//
    _BMD(RCC->CFGR, RCC_CFGR_SW, RCC_CFGR_SW_HSI); // SWITCH TO HSI
    _WVL(RCC->CFGR, RCC_CFGR_SWS, RCC_CFGR_SWS_HSI); // WAIT FOR HSI SWITCH
//
    _BMD(RCC->CFGR, RCC_CFGR_HPRE, RCC_CFGR_HPRE_DIV1);
    _BMD(RCC->CFGR, RCC_CFGR_PPRE, RCC_CFGR_PPRE_DIV1);
//
    _BST(FLASH->ACR, FLASH_ACR_PRFTBE);// ENABLE PREFETCH
    _BST(FLASH->ACR, FLASH_ACR_LATENCY); // SET LATENCY TO 1 BECAUSE WE WILL SET FREQUENCY TO 48MHz
//
    _BCL(RCC->CR, RCC_CR_PLLON); // TURN OFF PLL
    _WBC(RCC->CR, RCC_CR_PLLRDY); // WAIT FOR PLL OFF
//
    _BMD(RCC->CFGR, RCC_CFGR_PLLSRC, RCC_CFGR_PLLSRC_HSI_DIV2); // SET SOURCE TO HSI/2 = 4MHz
    _BMD(RCC->CFGR, RCC_CFGR_PLLMUL, RCC_CFGR_PLLMUL12); // SET MULTIPLY TO 12 4*12 = 48MHz
//
    _BST(RCC->CR, RCC_CR_PLLON); // TURN ON PLL
    _WBS(RCC->CR, RCC_CR_PLLRDY); // WAIT FOR PLL ON
//
    _BMD(RCC->CFGR, RCC_CFGR_SW, RCC_CFGR_SW_PLL); // SWITCH TO PLL
    _WVL(RCC->CFGR, RCC_CFGR_SWS, RCC_CFGR_SWS_PLL); // WAIT FOR PLL SWITCH
//
    _BMD(RCC->CFGR3, RCC_CFGR3_USBSW, RCC_CFGR3_USBSW_PLLCLK); // SET USB CLOCK TO PLL
    _BST(SYSCFG->CFGR1, SYSCFG_CFGR1_PA11_PA12_RMP); // ENABLE REMAP
    _BST(RCC->APB2ENR, RCC_APB2ENR_SYSCFGCOMPEN);
    _BST(RCC->AHBENR, RCC_AHBENR_GPIOAEN);
    _BST(RCC->APB1ENR, RCC_APB1ENR_USBEN);

any suggestion, i am out of clue now

dmitrystu commented 5 years ago

I have no F070 in my lab, Will order STM32F070RBT6 as a replacement of F103 on bluepill.