2022-09-18 10:32:00

by Jilin Yuan

[permalink] [raw]
Subject: [PATCH] of: device: fix repeated words in comments

Delete the redundant word 'of'.

Signed-off-by: Jilin Yuan <[email protected]>
---
drivers/of/device.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/of/device.c b/drivers/of/device.c
index 75b6cbffa755..6158173b92ad 100644
--- a/drivers/of/device.c
+++ b/drivers/of/device.c
@@ -19,7 +19,7 @@

/**
* of_match_device - Tell if a struct device matches an of_device_id list
- * @matches: array of of device match structures to search in
+ * @matches: array of device match structures to search in
* @dev: the of device structure to match against
*
* Used by a driver to check whether an platform_device present in the
--
2.36.1


2022-09-26 19:06:01

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH] of: device: fix repeated words in comments

On Sun, Sep 18, 2022 at 05:49:15PM +0800, Jilin Yuan wrote:
> Delete the redundant word 'of'.

Again? Sigh.

Please read:

https://lore.kernel.org/all/CAL_JsqL4GvgFYzGUfhW5pvm4wYGrFaj6gHOYZjnOMuk2zCz67w@mail.gmail.com/

>
> Signed-off-by: Jilin Yuan <[email protected]>

Are you sure you are the author?

> ---
> drivers/of/device.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/of/device.c b/drivers/of/device.c
> index 75b6cbffa755..6158173b92ad 100644
> --- a/drivers/of/device.c
> +++ b/drivers/of/device.c
> @@ -19,7 +19,7 @@
>
> /**
> * of_match_device - Tell if a struct device matches an of_device_id list
> - * @matches: array of of device match structures to search in
> + * @matches: array of device match structures to search in
> * @dev: the of device structure to match against
> *
> * Used by a driver to check whether an platform_device present in the
> --
> 2.36.1
>
>

2022-09-27 02:36:17

by Bagas Sanjaya

[permalink] [raw]
Subject: Re: [PATCH] of: device: fix repeated words in comments

On Mon, Sep 26, 2022 at 01:58:52PM -0500, Rob Herring wrote:
> On Sun, Sep 18, 2022 at 05:49:15PM +0800, Jilin Yuan wrote:
> > Delete the redundant word 'of'.
>
> Again? Sigh.
>
> Please read:
>
> https://lore.kernel.org/all/CAL_JsqL4GvgFYzGUfhW5pvm4wYGrFaj6gHOYZjnOMuk2zCz67w@mail.gmail.com/
>
> >
> > Signed-off-by: Jilin Yuan <[email protected]>
>
> Are you sure you are the author?
>

Hi Rob,

Some reviewers sent comments for similar patches from @cdjrlc.com
people (including myself at [1]), with some of them requesting changes.
However, there are never any responses to these reviews from @cdjrlc.com
people, just like ZTE developers sending through cgel.zte ignored
review comments [2] (try searching `f:"cdjrlc.com" AND s:"Re:"`, it returns
nothing for now).

Thanks.

[1]: https://lore.kernel.org/lkml/[email protected]/
[2]: https://lore.kernel.org/lkml/[email protected]/

--
An old man doll... just what I always wanted! - Clara


Attachments:
(No filename) (0.98 kB)
signature.asc (235.00 B)
Download all attachments

2022-09-27 13:02:26

by Bagas Sanjaya

[permalink] [raw]
Subject: Re: [PATCH] of: device: fix repeated words in comments

On 9/27/22 19:29, Rob Herring wrote:
>> Hi Rob,
>>
>> Some reviewers sent comments for similar patches from @cdjrlc.com
>> people (including myself at [1]), with some of them requesting changes.
>> However, there are never any responses to these reviews from @cdjrlc.com
>> people, just like ZTE developers sending through cgel.zte ignored
>> review comments [2] (try searching `f:"cdjrlc.com" AND s:"Re:"`, it returns
>> nothing for now).
>
> I can't decide whether to fix it correctly myself or leave it to see
> how many times I can get the same patch. :)
>

Hi Rob,

I'm not talking about whether this patch should be picked up or not, but
rather I'm pointing out the ignoring review behavior of @cdjrlc.com people.

Thanks.

--
An old man doll... just what I always wanted! - Clara

2022-09-27 13:51:26

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH] of: device: fix repeated words in comments

On Mon, Sep 26, 2022 at 9:04 PM Bagas Sanjaya <[email protected]> wrote:
>
> On Mon, Sep 26, 2022 at 01:58:52PM -0500, Rob Herring wrote:
> > On Sun, Sep 18, 2022 at 05:49:15PM +0800, Jilin Yuan wrote:
> > > Delete the redundant word 'of'.
> >
> > Again? Sigh.
> >
> > Please read:
> >
> > https://lore.kernel.org/all/CAL_JsqL4GvgFYzGUfhW5pvm4wYGrFaj6gHOYZjnOMuk2zCz67w@mail.gmail.com/
> >
> > >
> > > Signed-off-by: Jilin Yuan <[email protected]>
> >
> > Are you sure you are the author?
> >
>
> Hi Rob,
>
> Some reviewers sent comments for similar patches from @cdjrlc.com
> people (including myself at [1]), with some of them requesting changes.
> However, there are never any responses to these reviews from @cdjrlc.com
> people, just like ZTE developers sending through cgel.zte ignored
> review comments [2] (try searching `f:"cdjrlc.com" AND s:"Re:"`, it returns
> nothing for now).

I can't decide whether to fix it correctly myself or leave it to see
how many times I can get the same patch. :)

Rob

2022-09-27 14:22:23

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH] of: device: fix repeated words in comments

On Tue, Sep 27, 2022 at 7:47 AM Bagas Sanjaya <[email protected]> wrote:
>
> On 9/27/22 19:29, Rob Herring wrote:
> >> Hi Rob,
> >>
> >> Some reviewers sent comments for similar patches from @cdjrlc.com
> >> people (including myself at [1]), with some of them requesting changes.
> >> However, there are never any responses to these reviews from @cdjrlc.com
> >> people, just like ZTE developers sending through cgel.zte ignored
> >> review comments [2] (try searching `f:"cdjrlc.com" AND s:"Re:"`, it returns
> >> nothing for now).
> >
> > I can't decide whether to fix it correctly myself or leave it to see
> > how many times I can get the same patch. :)
> >
>
> Hi Rob,
>
> I'm not talking about whether this patch should be picked up or not, but
> rather I'm pointing out the ignoring review behavior of @cdjrlc.com people.

Yes, I get that. Not responding seems to be a growing behavior not
just on these trivial type patches. (Though replying with 'ok' to
every review comment is equally annoying.) I suppose there could be
some filtering happening such that replies are never received.

Rob