* [PATCH 0/3] Add get/set ethernet address functions and ACPI MAC address pass through functionality to cdc_ncm driver
@ 2019-09-05 21:01 Charles.Hyde
2019-09-05 21:04 ` Mario.Limonciello
0 siblings, 1 reply; 5+ messages in thread
From: Charles.Hyde @ 2019-09-05 21:01 UTC (permalink / raw)
To: oliver, rjw, lenb
Cc: Mario.Limonciello, chip.programmer, nic_swsd, linux-usb, linux-acpi
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
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH 0/3] Add get/set ethernet address functions and ACPI MAC address pass through functionality to cdc_ncm driver
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
0 siblings, 1 reply; 5+ messages in thread
From: Mario.Limonciello @ 2019-09-05 21:04 UTC (permalink / raw)
To: Charles.Hyde, oliver, rjw, lenb
Cc: chip.programmer, nic_swsd, linux-usb, linux-acpi
> -----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
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH 0/3] Add get/set ethernet address functions and ACPI MAC address pass through functionality to cdc_ncm driver
2019-09-05 21:04 ` Mario.Limonciello
@ 2019-09-05 21:14 ` Charles.Hyde
2019-09-05 21:55 ` Greg KH
0 siblings, 1 reply; 5+ messages in thread
From: Charles.Hyde @ 2019-09-05 21:14 UTC (permalink / raw)
To: Mario.Limonciello, oliver, rjw, lenb
Cc: chip.programmer, nic_swsd, linux-usb, linux-acpi
> > -----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
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 0/3] Add get/set ethernet address functions and ACPI MAC address pass through functionality to cdc_ncm driver
2019-09-05 21:14 ` Charles.Hyde
@ 2019-09-05 21:55 ` Greg KH
0 siblings, 0 replies; 5+ messages in thread
From: Greg KH @ 2019-09-05 21:55 UTC (permalink / raw)
To: Charles.Hyde
Cc: Mario.Limonciello, oliver, rjw, lenb, chip.programmer, nic_swsd,
linux-usb, linux-acpi
On Thu, Sep 05, 2019 at 09:14:05PM +0000, Charles.Hyde@dellteam.com wrote:
> > > -----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.
Please resend the series with that information in it, and also properly
thread the emails so they are at least clustered together.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 0/3] Add get/set ethernet address functions and ACPI MAC address pass through functionality to cdc_ncm driver
@ 2019-08-30 19:36 Charles.Hyde
0 siblings, 0 replies; 5+ messages in thread
From: Charles.Hyde @ 2019-08-30 19:36 UTC (permalink / raw)
To: oliver, rjw, lenb; +Cc: linux-acpi, linux-usb, nic_swsd, Mario.Limonciello
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 | 141 ++++++++++++++++++++++++++++++-
drivers/net/usb/r8152.c | 44 +---------
include/acpi/acpi_mac_passthru.h | 29 +++++++
5 files changed, 234 insertions(+), 44 deletions(-)
create mode 100644 drivers/acpi/acpi_mac_passthru.c
create mode 100644 include/acpi/acpi_mac_passthru.h
--
2.20.1
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-09-05 21:55 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
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
2019-09-05 21:55 ` Greg KH
-- strict thread matches above, loose matches on Subject: below --
2019-08-30 19:36 Charles.Hyde
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).