25.04.2020 01:00, Sowjanya Komatineni пишет: > > On 4/24/20 8:07 AM, Dmitry Osipenko wrote: >> External email: Use caution opening links or attachments >> >> >> 24.04.2020 06:55, Sowjanya Komatineni пишет: >> >> Is this driver compiled as a single kernel module file? >> >>> +MODULE_AUTHOR("Sowjanya Komatineni <skomatineni@xxxxxxxxxx>"); >>> +MODULE_DESCRIPTION("NVIDIA Tegra CSI Device Driver"); >>> +MODULE_LICENSE("GPL v2"); >> ... >>> +MODULE_AUTHOR("Sowjanya Komatineni <skomatineni@xxxxxxxxxx>"); >>> +MODULE_DESCRIPTION("NVIDIA Tegra Video Input Device Driver"); >>> +MODULE_LICENSE("GPL v2"); >> I don't think that these macros are needed in that case. >> The video.c should be enough, isn't it? > yes these can be removed It will be nice to factor out the Tegra210-specific VI/CSI OPS into a separate driver module (say tegra210-vi) to ease supporting of other Tegra versions. Of course this could be done later on, although I suppose the amount of hassle could be reduced if it's done from the start.