From: <Charles.Hyde@dellteam.com>
To: <Mario.Limonciello@dell.com>, <oliver@neukum.org>,
<rjw@rjwysocki.net>, <lenb@kernel.org>
Cc: <chip.programmer@gmail.com>, <nic_swsd@realtek.com>,
<linux-usb@vger.kernel.org>, <linux-acpi@vger.kernel.org>
Subject: RE: [PATCH 0/3] Add get/set ethernet address functions and ACPI MAC address pass through functionality to cdc_ncm driver
Date: Thu, 5 Sep 2019 21:14:05 +0000 [thread overview]
Message-ID: <67e340c684a64e528b04276f715567cc@AUSX13MPS307.AMER.DELL.COM> (raw)
In-Reply-To: <e41270463c46445d82415d86bebc9566@AUSX13MPC101.AMER.DELL.COM>
> > -----Original Message-----
> > From: Hyde, Charles - Dell Team
> > Sent: Thursday, September 5, 2019 4:02 PM
> > To: Oliver Neukum; "Rafael J. Wysocki"; Len Brown
> > Cc: Limonciello, Mario; chip.programmer@gmail.com; Realtek linux nic
> > maintainers; linux-usb@vger.kernel.org; linux-acpi@vger.kernel.org
> > Subject: [PATCH 0/3] Add get/set ethernet address functions and ACPI
> > MAC address pass through functionality to cdc_ncm driver
> >
> > In recent testing of a Dell Universal Dock D6000, I found that MAC
> > address pass through is not supported in the Linux drivers. However,
> > this same device is supported in Windows 10 (Pro) on my personal
> > computer, in as much as I was able to tell Windows to assign a new MAC
> > address of my choosing, and I saw through wireshark the new MAC
> > address was pushed out to the device. Afterward, Windows reported a
> > new IP address and I was able to view web pages.
> >
> > This series of patches give support to cdc_ncm USB based Ethernet
> > controllers for programming a MAC address to the device, and also to
> > retrieve the device's MAC address. This patch series further adds
> > ACPI MAC address pass through support specifically for the cdc_ncm
> > driver, and generally for any other driver that may need or want it,
> > in furtherance of Dell's enterprise IT policy efforts. It was this
> > latter that I initially found lacking when testing a D6000 with a Dell
> > laptop, and then I found ifconfig was unable to set a MAC address into
> > the device. These patches bring a similar level of functionality to
> > cdc_ncm driver as is available with the Realtek r8152 driver, and is available
> with Windows.
> >
> > The cdc_ncm driver limits the ACPI MAC address pass through support to
> > only the Dell Universal Dock D6000, so no other cdc_ncm device will be
> > impacted.
> >
> > Charles Hyde (3):
> > net: cdc_ncm: add get/set ethernet address functions
> > ACPI: move ACPI functionality out of r8152 driver
> > net: cdc_ncm: Add ACPI MAC address pass through functionality
> >
> > drivers/acpi/Makefile | 1 +
> > drivers/acpi/acpi_mac_passthru.c | 63 +++++++++++++
> > drivers/net/usb/cdc_ncm.c | 148 ++++++++++++++++++++++++++++---
> > drivers/net/usb/r8152.c | 44 +--------
> > include/acpi/acpi_mac_passthru.h | 29 ++++++
> > 5 files changed, 234 insertions(+), 51 deletions(-) create mode
> > 100644 drivers/acpi/acpi_mac_passthru.c create mode 100644
> > include/acpi/acpi_mac_passthru.h
> >
> > --
> > 2.20.1
>
> Typical practice is to make this new patch series prefixed with a v2. And to
> describe what has changed From v1 either in individual patches below the '--'
> or in the cover letter.
>
> So can you please describe what changed from previous submission in case it's
> not obvious to reviewers?
>
> For example:
>
> [PATCH v2 0/3] Add get/set ethernet address functions and ACPI MAC address
> pass through functionality to cdc_ncm driver
>
> Changes from v1 to v 2:
> * Changed foo to bar
Ah, my apologies.
What changed with today's patch series from what I proposed on Friday, August 30, is that I created a function named get_ethernet_addr() which replaces two instances where the same code snippet was located in the previous patch series. I also created a post reset function to set the MAC address, if there exists an ACPI MAC address pass through (MAPT). Oliver Neukum had requested a post reset function for this purpose.
Charles Hyde
next prev parent reply other threads:[~2019-09-05 21:14 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-09-05 21:01 [PATCH 0/3] Add get/set ethernet address functions and ACPI MAC address pass through functionality to cdc_ncm driver Charles.Hyde
2019-09-05 21:04 ` Mario.Limonciello
2019-09-05 21:14 ` Charles.Hyde [this message]
2019-09-05 21:55 ` Greg KH
-- strict thread matches above, loose matches on Subject: below --
2019-08-30 19:36 Charles.Hyde
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=67e340c684a64e528b04276f715567cc@AUSX13MPS307.AMER.DELL.COM \
--to=charles.hyde@dellteam.com \
--cc=Mario.Limonciello@dell.com \
--cc=chip.programmer@gmail.com \
--cc=lenb@kernel.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=nic_swsd@realtek.com \
--cc=oliver@neukum.org \
--cc=rjw@rjwysocki.net \
/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;
as well as URLs for NNTP newsgroup(s).