RE: [PATCH 02/11] staging: kpc2000: add separate show functions for kpc_uio_class device attributes.

2019-05-16 Thread Matt Sickler
>-Original Message- >From: devel On Behalf Of >Define separate simple show functions for each attribute instead of having a >one big one containing a chain of conditionals. > >+static ssize_t s2c_dma_ch_show(struct device *dev, >+ struct device_attribute

Re: [PATCH 02/11] staging: kpc2000: add separate show functions for kpc_uio_class device attributes.

2019-05-16 Thread Jeremy Sowden
On 2019-05-16, at 22:45:33 +0200, Greg KH wrote: > On Thu, May 16, 2019 at 09:04:02PM +0100, Jeremy Sowden wrote: > > Define separate simple show functions for each attribute instead of > > having a one big one containing a chain of conditionals. > > There's nothing wrong with a change of

Re: [PATCH 02/11] staging: kpc2000: add separate show functions for kpc_uio_class device attributes.

2019-05-16 Thread Jeremy Sowden
On 2019-05-16, at 20:17:51 +, Matt Sickler wrote: > >-Original Message- > >From: devel On Behalf Of > >Define separate simple show functions for each attribute instead of having a > >one big one containing a chain of conditionals. > > > >+static ssize_t s2c_dma_ch_show(struct device

Re: [PATCH 02/11] staging: kpc2000: add separate show functions for kpc_uio_class device attributes.

2019-05-16 Thread Greg KH
On Thu, May 16, 2019 at 09:04:02PM +0100, Jeremy Sowden wrote: > Define separate simple show functions for each attribute instead of > having a one big one containing a chain of conditionals. There's nothing wrong with a change of contitionals, if you do it right :) > Signed-off-by: Jeremy