[<prev] [next>] [day] [month] [year] [list]
Message-ID: <d9899157-99f3-6382-8693-8d1eb2dcaeba@xs4all.nl>
Date: Tue, 16 Mar 2021 09:27:06 +0100
From: Hans Verkuil <hverkuil-cisco@...all.nl>
To: Xiaofeng Cao <cxfcosmos@...il.com>
Cc: gustavoars@...nel.org, gregkh@...uxfoundation.org,
linux-media@...r.kernel.org, linux-kernel@...r.kernel.org,
Xiaofeng Cao <caoxiaofeng@...ong.com>, mchehab@...nel.org
Subject: Re: [PATCH] Rectify spelling and grammar
On 16/03/2021 09:16, Xiaofeng Cao wrote:
> Hi Hans Verkuil,
>
> I'm sorry to make you confused. Please use Xiaofeng Cao <caoxiaofeng@...ong.com <mailto:caoxiaofeng@...ong.com>>
Thanks, I've updated this to keep checkpatch happy.
BTW, for your next patch: it is good practice to add a prefix of the driver you
are patching, so:
[PATCH] radio-si476x: rectify spelling and grammar
That way we know in which driver the change is.
Regards,
Hans
>
> Gmail.com is my personal email while yulong.com <http://yulong.com> is my company email which is in trouble when communicating with the
> Linux kernel.
>
> According to relevant regulations, I have to use the company mailbox to indicate company information.
>
> Best regards,
>
> Xiaofeng
>
> Hans Verkuil <hverkuil-cisco@...all.nl <mailto:hverkuil-cisco@...all.nl>> 于2021年3月16日周二 下午3:45写道:
>
> Hi Xiaofeng Cao,
>
> The patch is good, but scripts/checkpatch.pl <http://checkpatch.pl> complains about a mismatch:
>
> WARNING: From:/Signed-off-by: email address mismatch: 'From: Xiaofeng Cao <cxfcosmos@...il.com <mailto:cxfcosmos@...il.com>>' !=
> 'Signed-off-by: Xiaofeng Cao
> <caoxiaofeng@...ong.com <mailto:caoxiaofeng@...ong.com>>'
>
> Which is the one I should use?
>
> Regards,
>
> Hans
>
> On 11/03/2021 08:20, Xiaofeng Cao wrote:
> > Change 'inaccesable' to 'inaccessible'
> > Change 'detrmine' to 'determine'
> > Delete 'in' grammatically
> >
> > Signed-off-by: Xiaofeng Cao <caoxiaofeng@...ong.com <mailto:caoxiaofeng@...ong.com>>
> > ---
> > drivers/media/radio/radio-si476x.c | 6 +++---
> > 1 file changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/media/radio/radio-si476x.c b/drivers/media/radio/radio-si476x.c
> > index 23997425bdb5..b39a68f83c5f 100644
> > --- a/drivers/media/radio/radio-si476x.c
> > +++ b/drivers/media/radio/radio-si476x.c
> > @@ -152,7 +152,7 @@ static struct v4l2_ctrl_config si476x_ctrls[] = {
> >
> > /*
> > * SI476X during its station seeking(or tuning) process uses several
> > - * parameters to detrmine if "the station" is valid:
> > + * parameters to determine if "the station" is valid:
> > *
> > * - Signal's SNR(in dBuV) must be lower than
> > * #V4L2_CID_SI476X_SNR_THRESHOLD
> > @@ -255,7 +255,7 @@ struct si476x_radio;
> > *
> > * This table holds pointers to functions implementing particular
> > * operations depending on the mode in which the tuner chip was
> > - * configured to start in. If the function is not supported
> > + * configured to start. If the function is not supported
> > * corresponding element is set to #NULL.
> > *
> > * @tune_freq: Tune chip to a specific frequency
> > @@ -917,7 +917,7 @@ static int si476x_radio_s_ctrl(struct v4l2_ctrl *ctrl)
> > case V4L2_CID_RDS_RECEPTION:
> > /*
> > * It looks like RDS related properties are
> > - * inaccesable when tuner is in AM mode, so cache the
> > + * inaccessible when tuner is in AM mode, so cache the
> > * changes
> > */
> > if (si476x_core_is_in_am_receiver_mode(radio->core))
> >
>
Powered by blists - more mailing lists