On Tue, May 02, 2023 at 01:32:35PM +0800, Shenghao Ding wrote: > Create tasdevice firmware lib. > drivers/firmware/Kconfig | 1 + > drivers/firmware/Makefile | 1 + > drivers/firmware/ti/Kconfig | 5 + > drivers/firmware/ti/Makefile | 3 + > drivers/firmware/ti/tasdevice-fmw.c | 2380 +++++++++++++++++++++++++++ > 5 files changed, 2390 insertions(+) Given how large this part of the code for these devices is it definitely makes sense to split it into a separate commit like you've done but are there any non-audio devices in this series which will share the same firmware style? If not then it probably makes sense to keep the code in sound/soc/codecs, though a separate file would still make sense. There's some devices that do keep firmware interface code in the firmware directory but in those cases the devices have other, non-audio, functionality which also uses the firmware (eg, always on monitoring) but I've not seen any of the tas devices like that. If there are some then the split you've made here makes sense.
Attachment:
signature.asc
Description: PGP signature