On Thu, 28 Nov 2019 at 03:54, Suzuki Kuruppassery Poulose <suzuki.poulose@xxxxxxx> wrote: > > On 19/11/2019 23:19, Mike Leach wrote: > > Adds in sysfs programming support for the CTI function register sets. > > Allows direct manipulation of channel / trigger association registers. > > > > Reviewed-by: Mathieu Poirier <mathieu.poirier@xxxxxxxxxx> > > Signed-off-by: Mike Leach <mike.leach@xxxxxxxxxx> > > > > +/* > > + * #define CTI_DEBUG_INTEGRATION_CTRL to enable the access to the integration > > + * control registers. Normally only used to investigate connection data. > > + */ > > On a second thought, I have some comments on this symbol. > > Given that the integration control registers may be useful for people to > find the device connections, I strongly feel that this is provided > via a CONFIG symbol rather than a debug symbol within the code. Device connections can be discovered with the dynamic sysfs connection entries added as part of patch 09. In cases where that is not sufficient and people really need to use the integration control registers they are probably instrumenting the code anyway. > > i.e, CONFIG_CTI_DEBUG_INTEGRATION_CTRL, to help the people better. > Codewise this doesn't make much difference, but it certainly makes > it more easier for people to use it. I agree that code-wise it doesn't make much difference but I'm really not convinced it makes the driver easier to use, and one needs to recompile their kernel for production systems anyway. Thanks, Mathieu > > We have used debug symbols elsewhere in the drivers for pure functional > debugging purposes. However I feel this is case is superior. > > > Cheers > Suzuki
