From: Johan Hovold <johan@kernel.org>
To: Mark Glover <mark.glover@actisense.com>
Cc: "linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>
Subject: Re: ftdi_sio: Actisense PIDs constant Names
Date: Thu, 21 Dec 2023 09:14:52 +0100 [thread overview]
Message-ID: <ZYPz_MS9BDEJVx0S@hovoldconsulting.com> (raw)
In-Reply-To: <AS8PR10MB4424408E2BFBD055150629C9FA96A@AS8PR10MB4424.EURPRD10.PROD.OUTLOOK.COM>
On Wed, Dec 20, 2023 at 01:57:40PM +0000, Mark Glover wrote:
> This changes the constant names for unused USB PIDs (product identifiers)
> to reflect the new products now using the PIDs.
Commit messages should use imperative mode (see the process
documentation) so I changed this to:
Update the constant names for unused USB PIDs (product
identifiers) to reflect the new products now using the PIDs.
and similar in the commit summary (Subject) where I also added the
missing common prefix ("USB: serial: ").
And when revising patches you should indicate that in Subject (e.g.
"[PATCH v2]") and include a short changelog below the '---' line (which
is missing below, but goes below the SoB).
Next time, try using git format-patch to generate the patch.
> Signed-off-by: Mark Glover <mark.glover@actisense.com>
>
> diff -urN linux-6.6.4/drivers/usb/serial/ftdi_sio.c linux/drivers/usb/serial/ftdi_sio.c
> --- linux-6.6.4/drivers/usb/serial/ftdi_sio.c 2023-12-03 06:33:10.000000000 +0000
> +++ linux/drivers/usb/serial/ftdi_sio.c 2023-12-12 10:39:51.157327241 +0000
> @@ -1033,9 +1033,9 @@
> { USB_DEVICE(FTDI_VID, ACTISENSE_USG_PID) },
> { USB_DEVICE(FTDI_VID, ACTISENSE_NGT_PID) },
> { USB_DEVICE(FTDI_VID, ACTISENSE_NGW_PID) },
> - { USB_DEVICE(FTDI_VID, ACTISENSE_D9AC_PID) },
> - { USB_DEVICE(FTDI_VID, ACTISENSE_D9AD_PID) },
> - { USB_DEVICE(FTDI_VID, ACTISENSE_D9AE_PID) },
> + { USB_DEVICE(FTDI_VID, ACTISENSE_UID_PID) },
> + { USB_DEVICE(FTDI_VID, ACTISENSE_USA_PID) },
> + { USB_DEVICE(FTDI_VID, ACTISENSE_NGX_PID) },
> { USB_DEVICE(FTDI_VID, ACTISENSE_D9AF_PID) },
> { USB_DEVICE(FTDI_VID, CHETCO_SEAGAUGE_PID) },
> { USB_DEVICE(FTDI_VID, CHETCO_SEASWITCH_PID) },
For some reason that I was not able to determine, this hunk did not
apply to mainline despite that there hasn't really been any changes to
the surrounding code since 6.6.4 (which you used to generate this
patch).
I fixed it up manually this time. End result is here:
https://git.kernel.org/pub/scm/linux/kernel/git/johan/usb-serial.git/commit/?h=usb-linus&id=513d88a88e0203188a38f4647dd08170aebd85df
Johan
prev parent reply other threads:[~2023-12-21 8:14 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <AS8PR10MB4424F3366DA20492BA9B1605FABAA@AS8PR10MB4424.EURPRD10.PROD.OUTLOOK.COM>
[not found] ` <ZWB5VtdzyGsROpuO@hovoldconsulting.com>
[not found] ` <AS8PR10MB442424BD43EE4AECDDC93F1EFA8EA@AS8PR10MB4424.EURPRD10.PROD.OUTLOOK.COM>
[not found] ` <ZXq4A-Uht67KcJNV@hovoldconsulting.com>
2023-12-20 11:17 ` USB: ftdi_sio: Actisense PIDs constant Names Mark Glover
2023-12-20 13:26 ` Greg KH
2023-12-20 13:57 ` Mark Glover
2023-12-21 8:14 ` Johan Hovold [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=ZYPz_MS9BDEJVx0S@hovoldconsulting.com \
--to=johan@kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=mark.glover@actisense.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox