Hi Prabhakar, On Mon, Mar 2, 2020 at 4:19 AM Lad, Prabhakar <prabhakar.csengg@xxxxxxxxx> wrote: > > Should this be a device tree property instead? > I did give a thought about it, but making this as DT property would > make it more stiff. In case a system has two OV5645 and we want to operate each OV5645 with a different virtual channel, it will not be possible with the module_param approach. Using a device tree property would make it possible though, so I think it makes more sense to use a device tree property for this. Thanks