From: Hemanth Selam <hemanth.selam@gmail.com>
To: "James E.J. Bottomley" <James.Bottomley@HansenPartnership.com>,
"Martin K. Petersen" <mkp@kernel.org>,
Anil Gurumurthy <anil.gurumurthy@qlogic.com>,
Sudarsana Kalluru <sudarsana.kalluru@qlogic.com>,
Oliver Neukum <oliver@neukum.org>, Ali Akcaagac <aliakc@web.de>,
Jamie Lenehan <lenehan@twibble.org>,
Hannes Reinecke <hare@suse.de>,
Satish Kharat <satishkh@cisco.com>,
Sesidhar Baddela <sebaddel@cisco.com>,
Karan Tilak Kumar <kartilak@cisco.com>,
Don Brace <don.brace@microchip.com>,
Justin Tee <justin.tee@broadcom.com>,
Paul Ely <paul.ely@broadcom.com>,
Kashyap Desai <kashyap.desai@broadcom.com>,
Sumit Saxena <sumit.saxena@broadcom.com>,
Shivasharan S <shivasharan.srikanteshwara@broadcom.com>,
Chandrakanth patil <chandrakanth.patil@broadcom.com>,
Sathya Prakash <sathya.prakash@broadcom.com>,
Sreekanth Reddy <sreekanth.reddy@broadcom.com>,
Suganath Prabu Subramani <suganath-prabu.subramani@broadcom.com>,
Ranjan Kumar <ranjan.kumar@broadcom.com>,
Doug Gilbert <dgilbert@interlog.com>,
Narsimhulu Musini <nmusini@cisco.com>,
Matthew Wilcox <willy@infradead.org>
Cc: linux-kernel@vger.kernel.org, linux-scsi@vger.kernel.org,
storagedev@microchip.com, megaraidlinux.pdl@broadcom.com,
MPT-FusionLinux.pdl@broadcom.com
Subject: Re: [PATCH 2/2] scsi: fix repeated words in comments
Date: Mon, 7 Sep 2026 09:54:01 +0530 [thread overview]
Message-ID: <20260907042402.12893-5-hemanth.selam@gmail.com> (raw)
In-Reply-To: <20260904121449.24178-3-hemanth.selam@gmail.com>
Please drop this patch. Three of the changes in it are wrong, and I only
found that out after posting.
drivers/scsi/dc395x.c
"but we'll fill them in in case something else needs them"
"fill them in" and "in case" are separate; the sentence is correct as
it stands.
drivers/scsi/aic94xx/aic94xx_scb.c
"if no device is connected to the LED, is is dimmed (off)"
This is not a doubled "is", it is a misspelt "it is". Deleting one
word leaves "is dimmed" with no subject.
drivers/scsi/sg.c
"but is is possible that the app intended SG_DXFER_TO_DEV"
Same thing: it should read "but it is possible".
I was matching doubled words mechanically without reading the sentence
around them. Sorry for the noise.
If a corrected patch for the aic94xx and sg wording would be welcome I am
happy to send one, but I will not resend unless you ask for it.
prev parent reply other threads:[~2026-09-07 4:24 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20260904121449.24178-1-hemanth.selam@gmail.com>
2026-09-04 12:14 ` [PATCH 1/2] scsi: fix typos in comments Hemanth Selam
2026-09-04 12:26 ` sashiko-bot
2026-09-04 12:14 ` [PATCH 2/2] scsi: fix repeated words " Hemanth Selam
2026-09-07 4:24 ` Hemanth Selam [this message]
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=20260907042402.12893-5-hemanth.selam@gmail.com \
--to=hemanth.selam@gmail.com \
--cc=James.Bottomley@HansenPartnership.com \
--cc=MPT-FusionLinux.pdl@broadcom.com \
--cc=aliakc@web.de \
--cc=anil.gurumurthy@qlogic.com \
--cc=chandrakanth.patil@broadcom.com \
--cc=dgilbert@interlog.com \
--cc=don.brace@microchip.com \
--cc=hare@suse.de \
--cc=justin.tee@broadcom.com \
--cc=kartilak@cisco.com \
--cc=kashyap.desai@broadcom.com \
--cc=lenehan@twibble.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=megaraidlinux.pdl@broadcom.com \
--cc=mkp@kernel.org \
--cc=nmusini@cisco.com \
--cc=oliver@neukum.org \
--cc=paul.ely@broadcom.com \
--cc=ranjan.kumar@broadcom.com \
--cc=sathya.prakash@broadcom.com \
--cc=satishkh@cisco.com \
--cc=sebaddel@cisco.com \
--cc=shivasharan.srikanteshwara@broadcom.com \
--cc=sreekanth.reddy@broadcom.com \
--cc=storagedev@microchip.com \
--cc=sudarsana.kalluru@qlogic.com \
--cc=suganath-prabu.subramani@broadcom.com \
--cc=sumit.saxena@broadcom.com \
--cc=willy@infradead.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.