On 22/11/2018 15:14, Ben Dooks wrote: > On 22/11/2018 09:26, Jon Hunter wrote: >> On Wed, Nov 21, 2018 at 4:15 PM Ben Dooks <ben.dooks@xxxxxxxxxxxxxxx> >> wrote: >>> >>> The name field is used for "apbdma.%d" which is rarely going to be >>> more than 10 bytes, so reduce the size from 30 to 12. This is only >>> being used by the interrupt registration, so is not critical to the >>> operation of the driver either. >> >> Do you mean 'never' above instead of 'rarely'? The max number of >> channels is never more than 2 digits and so the change is fine with >> me. So for the change ... > > I've never seen a system with more than 64 channels in it, but it didn't > seem worth trying to get it below 12 anyway. I see. If you look at the 'nr_channels' definition for the various Tegra devices that use this driver, there is nothing more than 32. So for this driver the max is 32. Newer Tegra devices are not using this anymore and so in the case of this specific driver we can say 'never' (although I understand now you meant rarely in the general case). Cheers Jon -- nvpublic
