Re: [netdev-net:main 50/55] Error: arch/arm/boot/dts/stm32f746.dtsi:265.20-21 syntax error

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Marc,

On Wed, May 17, 2023 at 3:36 PM Marc Kleine-Budde <mkl@xxxxxxxxxxxxxx> wrote:
>
> Good Bot!
>
> On 17.05.2023 21:21:58, kernel test robot wrote:
> > tree:   git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net.git main
> > head:   6ad85ed0ebf7ece0f376950a6b3b3c6048093d35
> > commit: 0920ccdf41e3078a4dd2567eb905ea154bc826e6 [50/55] ARM: dts: stm32: add CAN support on stm32f746
> > config: arm-randconfig-r034-20230517
> > compiler: arm-linux-gnueabi-gcc (GCC) 12.1.0
> > reproduce (this is a W=1 build):
> >         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> >         chmod +x ~/bin/make.cross
> >         # https://git.kernel.org/pub/scm/linux/kernel/git/netdev/net.git/commit/?id=0920ccdf41e3078a4dd2567eb905ea154bc826e6
> >         git remote add netdev-net git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net.git
> >         git fetch --no-tags netdev-net main
> >         git checkout 0920ccdf41e3078a4dd2567eb905ea154bc826e6
> >         # save the config file
> >         mkdir build_dir && cp config build_dir/.config
> >         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=arm olddefconfig
> >         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=arm SHELL=/bin/bash
> >
> > If you fix the issue, kindly add following tag where applicable
> > | Reported-by: kernel test robot <lkp@xxxxxxxxx>
> > | Closes: https://lore.kernel.org/oe-kbuild-all/202305172130.eGGEUhpi-lkp@xxxxxxxxx/
> >
> > All errors (new ones prefixed by >>):
> >
> > >> Error: arch/arm/boot/dts/stm32f746.dtsi:265.20-21 syntax error
> > >> FATAL ERROR: Unable to parse input tree
>
> Doh!
>
> The stm32f7-rcc.h is missing the definition for CAN3.
>
> |               can3: can@40003400 {
> |                       compatible = "st,stm32f4-bxcan";
> |                       reg = <0x40003400 0x200>;
> |                       interrupts = <104>, <105>, <106>, <107>;
> |                       interrupt-names = "tx", "rx0", "rx1", "sce";
> |                       resets = <&rcc STM32F7_APB1_RESET(CAN3)>;
> |                       clocks = <&rcc 0 STM32F7_APB1_CLOCK(CAN3)>;
> |                       st,gcan = <&gcan3>;
> |                       status = "disabled";
> |               };
> |
> |               gcan3: gcan@40003600 {
> |                       compatible = "st,stm32f4-gcan", "syscon";
> |                       reg = <0x40003600 0x200>;
> |                       clocks = <&rcc 0 STM32F7_APB1_CLOCK(CAN3)>;
> |               };
> |
>
> | $ git grep CAN include/dt-bindings/mfd/stm32f7-rcc.h
> | include/dt-bindings/mfd/stm32f7-rcc.h:78:#define STM32F7_RCC_APB1_CAN1           25
> | include/dt-bindings/mfd/stm32f7-rcc.h:79:#define STM32F7_RCC_APB1_CAN2           26
>
> Dario, do you have a fix at hand?

in version 1,  the patch "dt-bindings: mfd: stm32f7: add binding
definition for CAN3"
was accepted by Lee Jones
(https://lore.kernel.org/lkml/20230424090229.GB8035@xxxxxxxxxx/#t)
So I removed it in version 2 as I described in its cover letter:
https://marc.info/?l=linux-arm-kernel&m=168262836731935&w=2
Can we ensure that the patch is merged before this series?

Thanks and regards,
Dario


>
> regards,
> Marc
>
> --
> Pengutronix e.K.                 | Marc Kleine-Budde          |
> Embedded Linux                   | https://www.pengutronix.de |
> Vertretung Nürnberg              | Phone: +49-5121-206917-129 |
> Amtsgericht Hildesheim, HRA 2686 | Fax:   +49-5121-206917-9   |



-- 

Dario Binacchi

Senior Embedded Linux Developer

dario.binacchi@xxxxxxxxxxxxxxxxxxxx

__________________________________


Amarula Solutions SRL

Via Le Canevare 30, 31100 Treviso, Veneto, IT

T. +39 042 243 5310
info@xxxxxxxxxxxxxxxxxxxx

www.amarulasolutions.com





[Index of Archives]     [Automotive Discussions]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]     [CAN Bus]

  Powered by Linux