On 10/01/2019 14:57, Alexander Schroth wrote:
According to the Linux coding guidelines, defines should be written in uppercase. This patch converts all define-statements in the ni_pcidio.c file to uppercase, thus matching the coding style of the kernel. Signed-off-by: Alexander Schroth <alexander.schroth@xxxxxx> Signed-off-by: Marco Ammon <marco.ammon@xxxxxx> --- v2: After feedback, renamed defines we had proposed to delete in the original patch series. v3: rephrased subject line to add correct tags --- drivers/staging/comedi/drivers/ni_pcidio.c | 444 +++++++++++---------- 1 file changed, 223 insertions(+), 221 deletions(-)
Thanks for resending. The patch looks fine, thanks. Reviewed-by: Ian Abbott <abbotti@xxxxxxxxx> -- -=( Ian Abbott <abbotti@xxxxxxxxx> || Web: www.mev.co.uk )=- -=( MEV Ltd. is a company registered in England & Wales. )=- -=( Registered number: 02862268. Registered address: )=- -=( 15 West Park Road, Bramhall, STOCKPORT, SK7 3JZ, UK. )=- _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel