On 12/08/2019 00:23, Chris Johns wrote: > On 10/8/19 4:33 am, Christian Mauderer wrote: >> thanks for the patch. It sounds like we will get a behaviour that is >> more similar to FreeBSD one with it. > > Yes. > >> But it has the potential to have a >> big influence on existing BSPs so it would be good to have some more >> feedback for it before merging it. > > Do you have specific BSPs in mind? If we agree this behaviour is more like > FreeBSD would that mean a BSP that breaks needs to be fixed? > > Chris
Hello Chris, I would consider a BSP broken that has a problem with the change. So yes: It should be fixed in that case. I haven't had a specific BSP in mind. Every BSP that currently uses an EARLY.*MODULE would be an candidate. Luckily it's not that many: > grep "EARLY.*MODULE" -r --exclude-dir=freebsd-org . -l ./freebsd/sys/sys/bus.h ./freebsd/sys/dev/iicbus/ofw_iicbus.c ./freebsd/sys/dev/pci/pci_pci.c ./freebsd/sys/dev/pci/pci.c ./freebsd/sys/dev/gpio/ofw_gpiobus.c ./freebsd/sys/dev/gpio/gpiobus.c ./freebsd/sys/dev/ofw/ofwbus.c ./freebsd/sys/dev/fdt/simplebus.c ./freebsd/sys/arm/freescale/imx/imx_gpio.c ./freebsd/sys/arm/ti/am335x/am335x_prcm.c ./freebsd/sys/arm/ti/ti_scm.c So: BBB, i.MX, everything using PCI It's not that much and most are active ones. So it shouldn't be a problem as long as the maintainers of these BSPs know that they should take a look at that if they have strange problems. Beagle is covered by Nils' work. For i.MX and some PCI BSPs we should give Sebastian a hint (just added him on CC). I'm not sure who else uses PCI. Maybe x86? Best regards Christian -- -------------------------------------------- embedded brains GmbH Herr Christian Mauderer Dornierstr. 4 D-82178 Puchheim Germany email: christian.maude...@embedded-brains.de Phone: +49-89-18 94 741 - 18 Fax: +49-89-18 94 741 - 08 PGP: Public key available on request. Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG. _______________________________________________ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/devel
