| Message ID | 20240305174933.1370708-6-andriy.shevchenko@linux.intel.com |
|---|---|
| State | Accepted |
| Commit | 888eefd91cbfe143e2f57f4463832861950457a0 |
| Headers | show |
| Series | media: ir-spi: A few cleanups | expand |
On Wed, Mar 06, 2024 at 03:35:56PM +0200, Andy Shevchenko wrote: > On Tue, Mar 05, 2024 at 11:52:15PM +0100, Andi Shyti wrote: > > On Tue, Mar 05, 2024 at 07:48:30PM +0200, Andy Shevchenko wrote: > > ... > > > > +static int ir_spi_tx(struct rc_dev *dev, unsigned int *buffer, unsigned int count) > > > > this goes over 80 characters, though. Not an error, but not worth > > a change either. > > It's only 82 characters and I consider this as an improvement in readability. > It's quite pity that some of the subsystems are too conservative, hope this > one is not an obstacle for them. > > > I'm not going block the patch as the rest is OK. > > > > Reviewed-by: Andi Shyti <andi.shyti@kernel.org> > > Thank you! > > Btw, don't you want to either add your entry into .mailcap and/or update your > email address in this source file (and maybe others)? I Cc'ed you only after > I looked closer to the sources... yes, will do... I also noticed that lots of updates are missing... I've been far from all this for too long, so that I have still a lot of catch up to do :-) Thanks, Andy! Andi
diff --git a/drivers/media/rc/ir-spi.c b/drivers/media/rc/ir-spi.c index 801de3d108cc..8fc8e496e6aa 100644 --- a/drivers/media/rc/ir-spi.c +++ b/drivers/media/rc/ir-spi.c @@ -36,8 +36,7 @@ struct ir_spi_data { struct regulator *regulator; }; -static int ir_spi_tx(struct rc_dev *dev, - unsigned int *buffer, unsigned int count) +static int ir_spi_tx(struct rc_dev *dev, unsigned int *buffer, unsigned int count) { int i; int ret; @@ -57,7 +56,7 @@ static int ir_spi_tx(struct rc_dev *dev, return -EINVAL; /* - * the first value in buffer is a pulse, so that 0, 2, 4, ... + * The first value in buffer is a pulse, so that 0, 2, 4, ... * contain a pulse duration. On the contrary, 1, 3, 5, ... * contain a space duration. */ @@ -146,9 +145,9 @@ static int ir_spi_probe(struct spi_device *spi) if (ret) dc = 50; - /* ir_spi_set_duty_cycle cannot fail, - * it returns int to be compatible with the - * rc->s_tx_duty_cycle function + /* + * ir_spi_set_duty_cycle() cannot fail, it returns int + * to be compatible with the rc->s_tx_duty_cycle function. */ ir_spi_set_duty_cycle(idata->rc, dc); @@ -177,7 +176,6 @@ static struct spi_driver ir_spi_driver = { .of_match_table = ir_spi_of_match, }, }; - module_spi_driver(ir_spi_driver); MODULE_AUTHOR("Andi Shyti <andi@etezian.org>");
Unify the indentation and multi-line comment style. Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> --- drivers/media/rc/ir-spi.c | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-)