All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alejandro Colomar <alx@kernel.org>
To: Don Brace <don.brace@microchip.com>
Cc: linux-man@vger.kernel.org
Subject: Re: [PATCH v8 1/3] smartpqi: update host attribute descriptions
Date: Tue, 17 Oct 2023 01:32:56 +0200	[thread overview]
Message-ID: <ZS3IKF9XHOuQaMMt@debian> (raw)
In-Reply-To: <20231016160403.636655-2-don.brace@microchip.com>

[-- Attachment #1: Type: text/plain, Size: 1659 bytes --]

Hi Don,

On Mon, Oct 16, 2023 at 11:04:01AM -0500, Don Brace wrote:
> Minor description changes for host attributes.
> 
> Change the word "drives" to "volumes".

But why is volumes more appropriate?  Please justify.

Thanks,
Alex

> 
> Reviewed-by: Scott Benesh <scott.benesh@microchip.com>
> Reviewed-by: Scott Teel <scott.teel@microchip.com>
> Reviewed-by: Mike McGowen <mike.mcgowen@microchip.com>
> Reviewed-by: Kevin Barnett <kevin.barnett@microchip.com>
> Signed-off-by: Don Brace <don.brace@microchip.com>
> ---
>  man4/smartpqi.4 | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/man4/smartpqi.4 b/man4/smartpqi.4
> index 393545d3728d..1e5b6d76f548 100644
> --- a/man4/smartpqi.4
> +++ b/man4/smartpqi.4
> @@ -138,12 +138,12 @@ The host
>  attribute is a write-only attribute.
>  Writing to this attribute will cause the driver to scan for new,
>  changed, or removed devices (e.g., hot-plugged tape drives, or newly
> -configured or deleted logical drives) and notify the SCSI mid-layer of
> +configured or deleted logical volumes) and notify the SCSI mid-layer of
>  any changes detected.
>  Usually this action is triggered automatically by configuration
>  changes, so the user should not normally have to write to this file.
>  Doing so may be useful when hot-plugging devices such as tape drives or
> -entire storage boxes containing pre-configured logical drives.
> +entire storage boxes containing pre-configured logical volumes.
>  .TP
>  .IR /sys/class/scsi_host/host * /version
>  The host
> -- 
> 2.42.0.357.g59167d7d09
> 

-- 
<https://www.alejandro-colomar.es/>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

  reply	other threads:[~2023-10-16 23:33 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-10-16 16:04 [PATCH v8 0/3] smartpqi man page updates Don Brace
2023-10-16 16:04 ` [PATCH v8 1/3] smartpqi: update host attribute descriptions Don Brace
2023-10-16 23:32   ` Alejandro Colomar [this message]
2023-10-16 16:04 ` [PATCH v8 2/3] smartpqi: remove sysfs entry for version Don Brace
2023-10-16 16:04 ` [PATCH v8 3/3] smartpqi: add device attributes Don Brace
2023-10-16 23:52   ` Alejandro Colomar

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ZS3IKF9XHOuQaMMt@debian \
    --to=alx@kernel.org \
    --cc=don.brace@microchip.com \
    --cc=linux-man@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.