linux-kernel - Re: [PATCH] Correcting the spelling mistakes in "Documentation/ABI/testing/sysfs-bus-cxl" and "Documentation/ABI/testing/sysfs-bus-coresight-devices-tmc"

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <53f6cac5-cdfd-42e3-88e4-9725575adff4@arm.com>
Date: Tue, 27 Feb 2024 11:17:33 +0000
From: Suzuki K Poulose <suzuki.poulose@....com>
To: prabhav kumar <pvkumar5749404@...il.com>
Cc: mike.leach@...aro.org, james.clark@....com, linux-kernel@...r.kernel.org,
 linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH] Correcting the spelling mistakes in
 "Documentation/ABI/testing/sysfs-bus-cxl" and
 "Documentation/ABI/testing/sysfs-bus-coresight-devices-tmc"

On 26/02/2024 18:58, prabhav kumar wrote:
> Thanks for commenting on the patch.
> 
> Will generate another patch for corresponding subsystem.
> I had ran the get_maintainer script on the CXL subsystem , but didn't 
> get any maintainer's name other than the commit_signers

Another nit: Please do not top post your replies. Use inline replies.

https://en.wikipedia.org/wiki/Posting_style#Interleaved_style


Suzuki
> 
> Thanks,
> Prabhav
> 
> On Mon, Feb 26, 2024 at 9:25 PM Suzuki K Poulose <suzuki.poulose@....com 
> <mailto:suzuki.poulose@....com>> wrote:
> 
>     Hi Prabhav,
> 
> 
>     On 20/02/2024 21:42, Prabhav Kumar Vaish wrote:
> 
>     Thanks for sending the patch. Please find some comments.
> 
>      > Signed-off-by: Prabhav Kumar Vaish <pvkumar5749404@...il.com
>     <mailto:pvkumar5749404@...il.com>>
> 
>     Only one "Signed-off-by:" tag is required, the second one below
>     is at the desired location. So, please drop the above.
> 
> 
>      >
>      > Changes:
>      >       - "avaialble" corrected to "available" in
>     "Documentation/ABI/testing/sysfs-bus-coresight-devices-tmc"
>      >       - "firwmare" corrected to "firmware" in
>     "Documentation/ABI/testing/sysfs-bus-cxl"
> 
>     It is advisable to split this patch into two, as they affect files
>     maintained by different subsystems. i.e., coresight and CXL. Since
>     they go via different maintainers, it is always better to split
>     the patch (especially when there is no functional dependency, like
>     this case).
> 
>     Also, I see that you haven't copied any of the maintainers for the CXL
>     subsystem. Running "scripts/get_maintainer.pl
>     <http://get_maintainer.pl>" on the affected file
>     or the patch should give you the email addresses for the concerned
>     maintainers/list.
> 
>     e.g:
> 
>     $ scripts/get_maintainer.pl <http://get_maintainer.pl>
>     Documentation/ABI/testing/sysfs-bus-cxl
> 
>     So, please resend the patch after splitting, to the respective
>     maintainers/reviewers/list.
> 
>     Suzuki
> 
>      > Signed-off-by: Prabhav Kumar Vaish <pvkumar5749404@...il.com
>     <mailto:pvkumar5749404@...il.com>>
>      > ---
>      >   Documentation/ABI/testing/sysfs-bus-coresight-devices-tmc | 2 +-
>      >   Documentation/ABI/testing/sysfs-bus-cxl                   | 2 +-
>      >   2 files changed, 2 insertions(+), 2 deletions(-)
>      >
>      > diff --git
>     a/Documentation/ABI/testing/sysfs-bus-coresight-devices-tmc
>     b/Documentation/ABI/testing/sysfs-bus-coresight-devices-tmc
>      > index 96aafa66b4a5..339cec3b2f1a 100644
>      > --- a/Documentation/ABI/testing/sysfs-bus-coresight-devices-tmc
>      > +++ b/Documentation/ABI/testing/sysfs-bus-coresight-devices-tmc
>      > @@ -97,7 +97,7 @@ Date:               August 2023
>      >   KernelVersion:      6.7
>      >   Contact:    Anshuman Khandual <anshuman.khandual@....com
>     <mailto:anshuman.khandual@....com>>
>      >   Description:        (Read) Shows all supported Coresight
>     TMC-ETR buffer modes available
>      > -             for the users to configure explicitly. This file is
>     avaialble only
>      > +             for the users to configure explicitly. This file is
>     available only
>      >               for TMC ETR devices.
>      >
>      >   What:             
>       /sys/bus/coresight/devices/<memory_map>.tmc/buf_mode_preferred
>      > diff --git a/Documentation/ABI/testing/sysfs-bus-cxl
>     b/Documentation/ABI/testing/sysfs-bus-cxl
>      > index fff2581b8033..bbf6de5a4ca1 100644
>      > --- a/Documentation/ABI/testing/sysfs-bus-cxl
>      > +++ b/Documentation/ABI/testing/sysfs-bus-cxl
>      > @@ -224,7 +224,7 @@ Description:
>      >               decoding a Host Physical Address range. Note that
>     this number
>      >               may be elevated without any regionX objects active
>     or even
>      >               enumerated, as this may be due to decoders
>     established by
>      > -             platform firwmare or a previous kernel (kexec).
>      > +             platform firmware or a previous kernel (kexec).
>      >
>      >
>      >   What:               /sys/bus/cxl/devices/decoderX.Y
>