On Tue, Mar 19, 2024 at 07:44:33AM +0100, Felix N. Kimbu wrote: > Correct typos in pi433_if.c comments to address the following checkpatch checks; > > CHECK: 'interace' may be misspelled - perhaps 'interface'? > #13: FILE: drivers/staging/pi433/pi433_if.c:13: > + * HopeRf with a similar interace - e. g. RFM69HCW, RFM12, RFM95, ... > ^^^^^^^^ > > CHECK: 'ebedded' may be misspelled - perhaps 'embedded'? > #71: FILE: drivers/staging/pi433/pi433_if.c:71: > + * so we have just one rx config, ebedded in device struct > ^^^^^^^ > > CHECK: 'reenabled' may be misspelled - perhaps 're-enabled'? > #650: FILE: drivers/staging/pi433/pi433_if.c:650: > + * irq will be reenabled after tx config is set > ^^^^^^^^^ > > CHECK: 'pendig' may be misspelled - perhaps 'pending'? > #926: FILE: drivers/staging/pi433/pi433_if.c:926: > + /* during pendig read request, change of config not allowed */ > ^^^^^^ > > Signed-off-by: Felix N. Kimbu <felixkimbu1@xxxxxxxxx> Hi Felix, This commit msg and log can be trimmed. Please take a look at samples in staging or here on the Outreachy list like this: https://lore.kernel.org/outreachy/00be5f2a97b0c899279bd8f9cd27634186b77b9d.1666299151.git.drv@xxxxxxxxx/ Checkpatch complained: WARNING: Prefer a maximum 75 chars per line (possible unwrapped commit description?) #311: Correct typos in pi433_if.c comments to address the following checkpatch checks; WARNING: Commit log lines starting with '#' are dropped by git as comments #314: #13: FILE: drivers/staging/pi433/pi433_if.c:13: WARNING: 'interace' may be misspelled - perhaps 'interface'? #315: + * HopeRf with a similar interace - e. g. RFM69HCW, RFM12, RFM95, ... ^^^^^^^^ WARNING: Commit log lines starting with '#' are dropped by git as comments #319: #71: FILE: drivers/staging/pi433/pi433_if.c:71: WARNING: 'ebedded' may be misspelled - perhaps 'embedded'? #320: + * so we have just one rx config, ebedded in device struct ^^^^^^^ WARNING: Commit log lines starting with '#' are dropped by git as comments #324: #650: FILE: drivers/staging/pi433/pi433_if.c:650: WARNING: 'reenabled' may be misspelled - perhaps 're-enabled'? #325: + * irq will be reenabled after tx config is set ^^^^^^^^^ WARNING: Commit log lines starting with '#' are dropped by git as comments #329: #926: FILE: drivers/staging/pi433/pi433_if.c:926: WARNING: 'pendig' may be misspelled - perhaps 'pending'? #330: + /* during pendig read request, change of config not allowed */ ^^^^^^ total: 0 errors, 9 warnings, 0 checks, 32 lines checked NOTE: For some of the reported defects, checkpatch may be able to mechanically convert to the typical style using --fix or --fix-inplace. /home/cxluser/patches/outreachy/felix-spell.patch has style problems, please review. NOTE: If any of the errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS. > --- > drivers/staging/pi433/pi433_if.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c > index b6c4917d515e..81de98c0245a 100644 > --- a/drivers/staging/pi433/pi433_if.c > +++ b/drivers/staging/pi433/pi433_if.c > @@ -10,7 +10,7 @@ > * devices, basing on HopeRfs rf69. > * > * The driver can also be extended, to support other modules of > - * HopeRf with a similar interace - e. g. RFM69HCW, RFM12, RFM95, ... > + * HopeRf with a similar interface - e. g. RFM69HCW, RFM12, RFM95, ... > * > * Copyright (C) 2016 Wolf-Entwicklungen > * Marcus Wolf <linux@xxxxxxxxxxxxxxxxxxxxx> > @@ -68,7 +68,7 @@ static const struct class pi433_class = { > */ > /* > * rx config is device specific > - * so we have just one rx config, ebedded in device struct > + * so we have just one rx config, embedded in device struct > */ > struct pi433_device { > /* device handling related values */ > @@ -647,7 +647,7 @@ static int pi433_tx_thread(void *data) > > /* > * prevent race conditions > - * irq will be reenabled after tx config is set > + * irq will be re-enabled after tx config is set > */ > disable_irq(device->irq_num[DIO0]); > device->tx_active = true; > @@ -923,7 +923,7 @@ static long pi433_ioctl(struct file *filp, unsigned int cmd, unsigned long arg) > case PI433_IOC_WR_RX_CFG: > mutex_lock(&device->rx_lock); > > - /* during pendig read request, change of config not allowed */ > + /* during pending read request, change of config not allowed */ > if (device->rx_active) { > mutex_unlock(&device->rx_lock); > return -EAGAIN; > -- > 2.34.1 > >