Arnd Bergmann wrote: > My point is that you could have some extra code that calls your > qe_upload_firmware() when the device tree contains the blob but the boot > loader did not already load it.
The current design of the 'firmware' node is such that if present, that means that the firmware has already been uploaded. We can't use the device tree to tell the kernel which firmware to upload, because that is determined exclusively by the user's application. The drivers are ultimately responsible for deciding which firmware to use. > This helps e.g. for the case where you > want to be able to install a generic Linux distribution that is not > able to ship with your firmware blob in the kernel or the root file system. > Putting the blob in the device tree makes it easier to get to a running > system then. But where would the blob come from? Probably from flash or a TFTP server. In that case, the boot loader can still handle it. > You can argue that the boot loader can always load the firmware in the > first place, but then you wouldn't need an implementation of > qe_upload_firmware in the kernel any more. You might want to do runtime swapping of firmwares. One of the drawbacks of the QE microcode design is that you can only have one microcode uploaded at a time. If you need to have two different microcodes (e.g. Soft-UART and interworking), then you need to merge the source code for those two into a new microcode and compile that. You might need to do full reset of the QE, which would require re-uploading. You might not know until after the kernel boots which firmware you want. In reality, having the boot loader load the firmware will usually be the preferred approach, because that's simpler. Having it in both U-Boot and the kernel covers all situations, though. There would be no need for the bootloader to pass the firmware to the kernel. -- Timur Tabi Linux kernel developer at Freescale _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev
