pull-request: can-next 2019-07-24
mbox series

Message ID 93540cba-184a-a9c5-f9d2-b1779a69a36f@pengutronix.de
State Accepted
Delegated to: David Miller
Headers show
Series
  • pull-request: can-next 2019-07-24
Related show

Pull-request

git://git.kernel.org/pub/scm/linux/kernel/git/mkl/linux-can-next.git tags/linux-can-next-for-5.4-20190724

Message

Marc Kleine-Budde July 24, 2019, 9 a.m. UTC
Hello David,

this is a pull request for net-next/master consisting of 26 patches.

The first two patches are by me. One adds missing files of the CAN
subsystem to the MAINTAINERS file, while the other sorts the
Makefile/Kconfig of the sja1000 drivers sub directory. In the next patch
Ji-Ze Hong (Peter Hong) provides a driver for the "Fintek PCIE to 2 CAN"
controller, based on the the sja1000 IP core.

Gustavo A. R. Silva's patch for the kvaser_usb driver introduces the use
of struct_size() instead of open coding it. Henning Colliander's patch
adds a driver for the "Kvaser PCIEcan" devices.

Another patch by Gustavo A. R. Silva marks expected switch fall-throughs
properly.

Dan Murphy provides 5 patches for the m_can. After cleanups a framework
is introduced so that the driver can be used from memory mapped IO as
well as SPI attached devices. Finally he adds a driver for the tcan4x5x
which uses this framework.

A series of 5 patches by Appana Durga Kedareswara rao for the xilinx_can
driver, first clean up,then add support for CANFD. Colin Ian King
contributes another cleanup for the xilinx_can driver.

Robert P. J. Day's patch corrects the brief history of the CAN protocol
given in the Kconfig menu entry.

2 patches by Dong Aisheng for the flexcan driver provide PE clock source
select support and dt-bindings description.
2 patches by Sean Nyekjaer for the flexcan driver provide add CAN
wakeup-source property and dt-bindings description.

Jeroen Hofstee's patch converts the ti_hecc driver to make use of the
rx-offload helper fixing a number of outstanding bugs.

The first patch of Oliver Hartkopp removes the now obsolete empty
ioctl() handler for the CAN protocols. The second patch adds SPDX
license identifiers for CAN subsystem.

regards,
Marc

---

The following changes since commit 3e3bb69589e482e0783f28d4cd1d8e56fda0bcbb:

  tc-testing: added tdc tests for [b|p]fifo qdisc (2019-07-23 14:08:15 -0700)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/mkl/linux-can-next.git tags/linux-can-next-for-5.4-20190724

for you to fetch changes up to fba76a58452694b9b13c07e48839fa84c75f57af:

  can: Add SPDX license identifiers for CAN subsystem (2019-07-24 10:31:55 +0200)

----------------------------------------------------------------
linux-can-next-for-5.4-20190724

----------------------------------------------------------------
Aisheng Dong (1):
      can: flexcan: implement can Runtime PM

Appana Durga Kedareswara rao (5):
      can: xilinx_can: Fix style issues
      can: xilinx_can: Fix kernel doc warnings
      can: xilinx_can: Fix flags field initialization for axi can and canps
      can: xilinx_can: Add cantype parameter in xcan_devtype_data struct
      can: xilinx_can: Add support for CANFD FD frames

Colin Ian King (1):
      can: xilinx_can: clean up indentation issue

Dan Murphy (5):
      can: m_can: Fix checkpatch issues on existing code
      can: m_can: Create a m_can platform framework
      can: m_can: Rename m_can_priv to m_can_classdev
      dt-bindings: can: tcan4x5x: Add DT bindings for TCAN4x5X driver
      can: tcan4x5x: Add tcan4x5x driver to the kernel

Dong Aisheng (2):
      dt-bindings: can: flexcan: add PE clock source property to device tree
      can: flexcan: add support for PE clock source select

Gustavo A. R. Silva (2):
      can: kvaser_usb: Use struct_size() in alloc_candev()
      can: mark expected switch fall-throughs

Henning Colliander (1):
      can: kvaser_pciefd: Add driver for Kvaser PCIEcan devices

Jeroen Hofstee (1):
      can: ti_hecc: use timestamp based rx-offloading

Ji-Ze Hong (Peter Hong) (1):
      can: sja1000: f81601: add Fintek F81601 support

Marc Kleine-Budde (2):
      MAINTAINERS: can: add missing files to CAN NETWORK DRIVERS and CAN NETWORK LAYER
      can: sja1000: Makefile/Kconfig: sort alphabetically

Oliver Hartkopp (2):
      can: remove obsolete empty ioctl() handler
      can: Add SPDX license identifiers for CAN subsystem

Robert P. J. Day (1):
      can: Kconfig: correct history of the CAN protocol

Sean Nyekjaer (2):
      dt-bindings: can: flexcan: add can wakeup property
      can: flexcan: add support for DT property 'wakeup-source'

 .../devicetree/bindings/net/can/fsl-flexcan.txt    |   10 +
 .../devicetree/bindings/net/can/tcan4x5x.txt       |   37 +
 MAINTAINERS                                        |    5 +
 drivers/net/can/Kconfig                            |   13 +
 drivers/net/can/Makefile                           |    1 +
 drivers/net/can/at91_can.c                         |    6 +-
 drivers/net/can/flexcan.c                          |  136 +-
 drivers/net/can/kvaser_pciefd.c                    | 1912 ++++++++++++++++++++
 drivers/net/can/m_can/Kconfig                      |   22 +-
 drivers/net/can/m_can/Makefile                     |    2 +
 drivers/net/can/m_can/m_can.c                      | 1079 +++++------
 drivers/net/can/m_can/m_can.h                      |  110 ++
 drivers/net/can/m_can/m_can_platform.c             |  202 +++
 drivers/net/can/m_can/tcan4x5x.c                   |  532 ++++++
 drivers/net/can/peak_canfd/peak_pciefd_main.c      |    2 +-
 drivers/net/can/sja1000/Kconfig                    |   79 +-
 drivers/net/can/sja1000/Makefile                   |   11 +-
 drivers/net/can/sja1000/f81601.c                   |  212 +++
 drivers/net/can/spi/mcp251x.c                      |    3 +-
 drivers/net/can/ti_hecc.c                          |  191 +-
 drivers/net/can/usb/kvaser_usb/kvaser_usb_core.c   |    3 +-
 drivers/net/can/usb/peak_usb/pcan_usb.c            |    2 +-
 drivers/net/can/xilinx_can.c                       |  293 ++-
 include/linux/can/core.h                           |    3 +-
 include/linux/can/skb.h                            |    2 +-
 net/can/Kconfig                                    |   11 +-
 net/can/af_can.c                                   |   10 +-
 net/can/af_can.h                                   |    1 +
 net/can/bcm.c                                      |    3 +-
 net/can/gw.c                                       |    1 +
 net/can/proc.c                                     |    1 +
 net/can/raw.c                                      |    3 +-
 32 files changed, 4098 insertions(+), 800 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/net/can/tcan4x5x.txt
 create mode 100644 drivers/net/can/kvaser_pciefd.c
 create mode 100644 drivers/net/can/m_can/m_can.h
 create mode 100644 drivers/net/can/m_can/m_can_platform.c
 create mode 100644 drivers/net/can/m_can/tcan4x5x.c
 create mode 100644 drivers/net/can/sja1000/f81601.c

Comments

Oliver Hartkopp July 24, 2019, 9:22 a.m. UTC | #1
Hi Marc,

why didn't you include the CAN FD support for the can-gw?

[PATCH 1/2] can: gw: use struct canfd_frame as internal data structure
https://marc.info/?l=linux-can&m=156388681922741&w=2

[PATCH 2/2] can: gw: add support for CAN FD frames
https://marc.info/?l=linux-can&m=156388682022742&w=2

The patches have already been sent in January as RFC and I did extensive 
testing since then.

Regards,
Oliver


On 24.07.19 11:00, Marc Kleine-Budde wrote:
> Hello David,
> 
> this is a pull request for net-next/master consisting of 26 patches.
> 
> The first two patches are by me. One adds missing files of the CAN
> subsystem to the MAINTAINERS file, while the other sorts the
> Makefile/Kconfig of the sja1000 drivers sub directory. In the next patch
> Ji-Ze Hong (Peter Hong) provides a driver for the "Fintek PCIE to 2 CAN"
> controller, based on the the sja1000 IP core.
> 
> Gustavo A. R. Silva's patch for the kvaser_usb driver introduces the use
> of struct_size() instead of open coding it. Henning Colliander's patch
> adds a driver for the "Kvaser PCIEcan" devices.
> 
> Another patch by Gustavo A. R. Silva marks expected switch fall-throughs
> properly.
> 
> Dan Murphy provides 5 patches for the m_can. After cleanups a framework
> is introduced so that the driver can be used from memory mapped IO as
> well as SPI attached devices. Finally he adds a driver for the tcan4x5x
> which uses this framework.
> 
> A series of 5 patches by Appana Durga Kedareswara rao for the xilinx_can
> driver, first clean up,then add support for CANFD. Colin Ian King
> contributes another cleanup for the xilinx_can driver.
> 
> Robert P. J. Day's patch corrects the brief history of the CAN protocol
> given in the Kconfig menu entry.
> 
> 2 patches by Dong Aisheng for the flexcan driver provide PE clock source
> select support and dt-bindings description.
> 2 patches by Sean Nyekjaer for the flexcan driver provide add CAN
> wakeup-source property and dt-bindings description.
> 
> Jeroen Hofstee's patch converts the ti_hecc driver to make use of the
> rx-offload helper fixing a number of outstanding bugs.
> 
> The first patch of Oliver Hartkopp removes the now obsolete empty
> ioctl() handler for the CAN protocols. The second patch adds SPDX
> license identifiers for CAN subsystem.
> 
> regards,
> Marc
> 
> ---
> 
> The following changes since commit 3e3bb69589e482e0783f28d4cd1d8e56fda0bcbb:
> 
>    tc-testing: added tdc tests for [b|p]fifo qdisc (2019-07-23 14:08:15 -0700)
> 
> are available in the Git repository at:
> 
>    git://git.kernel.org/pub/scm/linux/kernel/git/mkl/linux-can-next.git tags/linux-can-next-for-5.4-20190724
> 
> for you to fetch changes up to fba76a58452694b9b13c07e48839fa84c75f57af:
> 
>    can: Add SPDX license identifiers for CAN subsystem (2019-07-24 10:31:55 +0200)
> 
> ----------------------------------------------------------------
> linux-can-next-for-5.4-20190724
> 
> ----------------------------------------------------------------
> Aisheng Dong (1):
>        can: flexcan: implement can Runtime PM
> 
> Appana Durga Kedareswara rao (5):
>        can: xilinx_can: Fix style issues
>        can: xilinx_can: Fix kernel doc warnings
>        can: xilinx_can: Fix flags field initialization for axi can and canps
>        can: xilinx_can: Add cantype parameter in xcan_devtype_data struct
>        can: xilinx_can: Add support for CANFD FD frames
> 
> Colin Ian King (1):
>        can: xilinx_can: clean up indentation issue
> 
> Dan Murphy (5):
>        can: m_can: Fix checkpatch issues on existing code
>        can: m_can: Create a m_can platform framework
>        can: m_can: Rename m_can_priv to m_can_classdev
>        dt-bindings: can: tcan4x5x: Add DT bindings for TCAN4x5X driver
>        can: tcan4x5x: Add tcan4x5x driver to the kernel
> 
> Dong Aisheng (2):
>        dt-bindings: can: flexcan: add PE clock source property to device tree
>        can: flexcan: add support for PE clock source select
> 
> Gustavo A. R. Silva (2):
>        can: kvaser_usb: Use struct_size() in alloc_candev()
>        can: mark expected switch fall-throughs
> 
> Henning Colliander (1):
>        can: kvaser_pciefd: Add driver for Kvaser PCIEcan devices
> 
> Jeroen Hofstee (1):
>        can: ti_hecc: use timestamp based rx-offloading
> 
> Ji-Ze Hong (Peter Hong) (1):
>        can: sja1000: f81601: add Fintek F81601 support
> 
> Marc Kleine-Budde (2):
>        MAINTAINERS: can: add missing files to CAN NETWORK DRIVERS and CAN NETWORK LAYER
>        can: sja1000: Makefile/Kconfig: sort alphabetically
> 
> Oliver Hartkopp (2):
>        can: remove obsolete empty ioctl() handler
>        can: Add SPDX license identifiers for CAN subsystem
> 
> Robert P. J. Day (1):
>        can: Kconfig: correct history of the CAN protocol
> 
> Sean Nyekjaer (2):
>        dt-bindings: can: flexcan: add can wakeup property
>        can: flexcan: add support for DT property 'wakeup-source'
> 
>   .../devicetree/bindings/net/can/fsl-flexcan.txt    |   10 +
>   .../devicetree/bindings/net/can/tcan4x5x.txt       |   37 +
>   MAINTAINERS                                        |    5 +
>   drivers/net/can/Kconfig                            |   13 +
>   drivers/net/can/Makefile                           |    1 +
>   drivers/net/can/at91_can.c                         |    6 +-
>   drivers/net/can/flexcan.c                          |  136 +-
>   drivers/net/can/kvaser_pciefd.c                    | 1912 ++++++++++++++++++++
>   drivers/net/can/m_can/Kconfig                      |   22 +-
>   drivers/net/can/m_can/Makefile                     |    2 +
>   drivers/net/can/m_can/m_can.c                      | 1079 +++++------
>   drivers/net/can/m_can/m_can.h                      |  110 ++
>   drivers/net/can/m_can/m_can_platform.c             |  202 +++
>   drivers/net/can/m_can/tcan4x5x.c                   |  532 ++++++
>   drivers/net/can/peak_canfd/peak_pciefd_main.c      |    2 +-
>   drivers/net/can/sja1000/Kconfig                    |   79 +-
>   drivers/net/can/sja1000/Makefile                   |   11 +-
>   drivers/net/can/sja1000/f81601.c                   |  212 +++
>   drivers/net/can/spi/mcp251x.c                      |    3 +-
>   drivers/net/can/ti_hecc.c                          |  191 +-
>   drivers/net/can/usb/kvaser_usb/kvaser_usb_core.c   |    3 +-
>   drivers/net/can/usb/peak_usb/pcan_usb.c            |    2 +-
>   drivers/net/can/xilinx_can.c                       |  293 ++-
>   include/linux/can/core.h                           |    3 +-
>   include/linux/can/skb.h                            |    2 +-
>   net/can/Kconfig                                    |   11 +-
>   net/can/af_can.c                                   |   10 +-
>   net/can/af_can.h                                   |    1 +
>   net/can/bcm.c                                      |    3 +-
>   net/can/gw.c                                       |    1 +
>   net/can/proc.c                                     |    1 +
>   net/can/raw.c                                      |    3 +-
>   32 files changed, 4098 insertions(+), 800 deletions(-)
>   create mode 100644 Documentation/devicetree/bindings/net/can/tcan4x5x.txt
>   create mode 100644 drivers/net/can/kvaser_pciefd.c
>   create mode 100644 drivers/net/can/m_can/m_can.h
>   create mode 100644 drivers/net/can/m_can/m_can_platform.c
>   create mode 100644 drivers/net/can/m_can/tcan4x5x.c
>   create mode 100644 drivers/net/can/sja1000/f81601.c
>
Stéphane Grosjean July 24, 2019, 9:24 a.m. UTC | #2
Hello Mark,

I hope you're fine. Did you see the attached patch I've sent earlier this month?

Regards,

— Stéphane

> -----Original Message-----
> From: linux-can-owner@vger.kernel.org <linux-can-
> owner@vger.kernel.org> On Behalf Of Marc Kleine-Budde
> Sent: mercredi 24 juillet 2019 11:00
> To: netdev@vger.kernel.org
> Cc: davem@davemloft.net; kernel@pengutronix.de; linux-
> can@vger.kernel.org
> Subject: pull-request: can-next 2019-07-24
>
> Hello David,
>
> this is a pull request for net-next/master consisting of 26 patches.
>
> The first two patches are by me. One adds missing files of the CAN
> subsystem to the MAINTAINERS file, while the other sorts the
> Makefile/Kconfig of the sja1000 drivers sub directory. In the next patch Ji-Ze
> Hong (Peter Hong) provides a driver for the "Fintek PCIE to 2 CAN"
> controller, based on the the sja1000 IP core.
>
> Gustavo A. R. Silva's patch for the kvaser_usb driver introduces the use of
> struct_size() instead of open coding it. Henning Colliander's patch adds a
> driver for the "Kvaser PCIEcan" devices.
>
> Another patch by Gustavo A. R. Silva marks expected switch fall-throughs
> properly.
>
> Dan Murphy provides 5 patches for the m_can. After cleanups a framework
> is introduced so that the driver can be used from memory mapped IO as well
> as SPI attached devices. Finally he adds a driver for the tcan4x5x which uses
> this framework.
>
> A series of 5 patches by Appana Durga Kedareswara rao for the xilinx_can
> driver, first clean up,then add support for CANFD. Colin Ian King contributes
> another cleanup for the xilinx_can driver.
>
> Robert P. J. Day's patch corrects the brief history of the CAN protocol given in
> the Kconfig menu entry.
>
> 2 patches by Dong Aisheng for the flexcan driver provide PE clock source
> select support and dt-bindings description.
> 2 patches by Sean Nyekjaer for the flexcan driver provide add CAN wakeup-
> source property and dt-bindings description.
>
> Jeroen Hofstee's patch converts the ti_hecc driver to make use of the rx-
> offload helper fixing a number of outstanding bugs.
>
> The first patch of Oliver Hartkopp removes the now obsolete empty
> ioctl() handler for the CAN protocols. The second patch adds SPDX license
> identifiers for CAN subsystem.
>
> regards,
> Marc
>
> ---
>
> The following changes since commit
> 3e3bb69589e482e0783f28d4cd1d8e56fda0bcbb:
>
>   tc-testing: added tdc tests for [b|p]fifo qdisc (2019-07-23 14:08:15 -0700)
>
> are available in the Git repository at:
>
>   git://git.kernel.org/pub/scm/linux/kernel/git/mkl/linux-can-next.git
> tags/linux-can-next-for-5.4-20190724
>
> for you to fetch changes up to fba76a58452694b9b13c07e48839fa84c75f57af:
>
>   can: Add SPDX license identifiers for CAN subsystem (2019-07-24 10:31:55
> +0200)
>
> ----------------------------------------------------------------
> linux-can-next-for-5.4-20190724
>
> ----------------------------------------------------------------
> Aisheng Dong (1):
>       can: flexcan: implement can Runtime PM
>
> Appana Durga Kedareswara rao (5):
>       can: xilinx_can: Fix style issues
>       can: xilinx_can: Fix kernel doc warnings
>       can: xilinx_can: Fix flags field initialization for axi can and canps
>       can: xilinx_can: Add cantype parameter in xcan_devtype_data struct
>       can: xilinx_can: Add support for CANFD FD frames
>
> Colin Ian King (1):
>       can: xilinx_can: clean up indentation issue
>
> Dan Murphy (5):
>       can: m_can: Fix checkpatch issues on existing code
>       can: m_can: Create a m_can platform framework
>       can: m_can: Rename m_can_priv to m_can_classdev
>       dt-bindings: can: tcan4x5x: Add DT bindings for TCAN4x5X driver
>       can: tcan4x5x: Add tcan4x5x driver to the kernel
>
> Dong Aisheng (2):
>       dt-bindings: can: flexcan: add PE clock source property to device tree
>       can: flexcan: add support for PE clock source select
>
> Gustavo A. R. Silva (2):
>       can: kvaser_usb: Use struct_size() in alloc_candev()
>       can: mark expected switch fall-throughs
>
> Henning Colliander (1):
>       can: kvaser_pciefd: Add driver for Kvaser PCIEcan devices
>
> Jeroen Hofstee (1):
>       can: ti_hecc: use timestamp based rx-offloading
>
> Ji-Ze Hong (Peter Hong) (1):
>       can: sja1000: f81601: add Fintek F81601 support
>
> Marc Kleine-Budde (2):
>       MAINTAINERS: can: add missing files to CAN NETWORK DRIVERS and CAN
> NETWORK LAYER
>       can: sja1000: Makefile/Kconfig: sort alphabetically
>
> Oliver Hartkopp (2):
>       can: remove obsolete empty ioctl() handler
>       can: Add SPDX license identifiers for CAN subsystem
>
> Robert P. J. Day (1):
>       can: Kconfig: correct history of the CAN protocol
>
> Sean Nyekjaer (2):
>       dt-bindings: can: flexcan: add can wakeup property
>       can: flexcan: add support for DT property 'wakeup-source'
>
>  .../devicetree/bindings/net/can/fsl-flexcan.txt    |   10 +
>  .../devicetree/bindings/net/can/tcan4x5x.txt       |   37 +
>  MAINTAINERS                                        |    5 +
>  drivers/net/can/Kconfig                            |   13 +
>  drivers/net/can/Makefile                           |    1 +
>  drivers/net/can/at91_can.c                         |    6 +-
>  drivers/net/can/flexcan.c                          |  136 +-
>  drivers/net/can/kvaser_pciefd.c                    | 1912 ++++++++++++++++++++
>  drivers/net/can/m_can/Kconfig                      |   22 +-
>  drivers/net/can/m_can/Makefile                     |    2 +
>  drivers/net/can/m_can/m_can.c                      | 1079 +++++------
>  drivers/net/can/m_can/m_can.h                      |  110 ++
>  drivers/net/can/m_can/m_can_platform.c             |  202 +++
>  drivers/net/can/m_can/tcan4x5x.c                   |  532 ++++++
>  drivers/net/can/peak_canfd/peak_pciefd_main.c      |    2 +-
>  drivers/net/can/sja1000/Kconfig                    |   79 +-
>  drivers/net/can/sja1000/Makefile                   |   11 +-
>  drivers/net/can/sja1000/f81601.c                   |  212 +++
>  drivers/net/can/spi/mcp251x.c                      |    3 +-
>  drivers/net/can/ti_hecc.c                          |  191 +-
>  drivers/net/can/usb/kvaser_usb/kvaser_usb_core.c   |    3 +-
>  drivers/net/can/usb/peak_usb/pcan_usb.c            |    2 +-
>  drivers/net/can/xilinx_can.c                       |  293 ++-
>  include/linux/can/core.h                           |    3 +-
>  include/linux/can/skb.h                            |    2 +-
>  net/can/Kconfig                                    |   11 +-
>  net/can/af_can.c                                   |   10 +-
>  net/can/af_can.h                                   |    1 +
>  net/can/bcm.c                                      |    3 +-
>  net/can/gw.c                                       |    1 +
>  net/can/proc.c                                     |    1 +
>  net/can/raw.c                                      |    3 +-
>  32 files changed, 4098 insertions(+), 800 deletions(-)  create mode 100644
> Documentation/devicetree/bindings/net/can/tcan4x5x.txt
>  create mode 100644 drivers/net/can/kvaser_pciefd.c  create mode 100644
> drivers/net/can/m_can/m_can.h  create mode 100644
> drivers/net/can/m_can/m_can_platform.c
>  create mode 100644 drivers/net/can/m_can/tcan4x5x.c  create mode
> 100644 drivers/net/can/sja1000/f81601.c
>
> --
> Pengutronix e.K.                  | Marc Kleine-Budde           |
> Industrial Linux Solutions        | Phone: +49-231-2826-924     |
> Vertretung West/Dortmund          | Fax:   +49-5121-206917-5555 |-
> Amtsgericht Hildesheim, HRA 2686  | http://www.pengutronix.de   |
>
>
>
>


--
PEAK-System Technik GmbH
Sitz der Gesellschaft Darmstadt - HRB 9183
Geschaeftsfuehrung: Alexander Gach / Uwe Wilhelm
Unsere Datenschutzerklaerung mit wichtigen Hinweisen
zur Behandlung personenbezogener Daten finden Sie unter
www.peak-system.com/Datenschutz.483.0.html
When closing the CAN device while tx skbs are inflight, echo skb could be
released twice. By calling close_candev() before unlinking all pending
tx urbs, then the internal echo_skb[] array is fully and correctly cleared
before the USB write callback and, therefore, can_get_echo_skb() are called,
for each aborted URB.

Signed-off-by: Stephane Grosjean <s.grosjean@peak-system.com>
---
 drivers/net/can/usb/peak_usb/pcan_usb_core.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/net/can/usb/peak_usb/pcan_usb_core.c b/drivers/net/can/usb/peak_usb/pcan_usb_core.c
index 611f9d31be5d..740ef47eab01 100644
--- a/drivers/net/can/usb/peak_usb/pcan_usb_core.c
+++ b/drivers/net/can/usb/peak_usb/pcan_usb_core.c
@@ -576,16 +576,16 @@ static int peak_usb_ndo_stop(struct net_device *netdev)
        dev->state &= ~PCAN_USB_STATE_STARTED;
        netif_stop_queue(netdev);

+       close_candev(netdev);
+
+       dev->can.state = CAN_STATE_STOPPED;
+
        /* unlink all pending urbs and free used memory */
        peak_usb_unlink_all_urbs(dev);

        if (dev->adapter->dev_stop)
                dev->adapter->dev_stop(dev);

-       close_candev(netdev);
-
-       dev->can.state = CAN_STATE_STOPPED;
-
        /* can set bus off now */
        if (dev->adapter->dev_set_bus) {
                int err = dev->adapter->dev_set_bus(dev, 0);
--
2.20.1


--
PEAK-System Technik GmbH
Sitz der Gesellschaft Darmstadt - HRB 9183
Geschaeftsfuehrung: Alexander Gach / Uwe Wilhelm
Unsere Datenschutzerklaerung mit wichtigen Hinweisen
zur Behandlung personenbezogener Daten finden Sie unter
www.peak-system.com/Datenschutz.483.0.html
Marc Kleine-Budde July 24, 2019, 9:26 a.m. UTC | #3
On 7/24/19 11:22 AM, Oliver Hartkopp wrote:
> why didn't you include the CAN FD support for the can-gw?

I wanted to have a look at the patches. But the other patches are
already reviewed.

I'll do another pull-request this week.

> [PATCH 1/2] can: gw: use struct canfd_frame as internal data structure
> https://marc.info/?l=linux-can&m=156388681922741&w=2
> 
> [PATCH 2/2] can: gw: add support for CAN FD frames
> https://marc.info/?l=linux-can&m=156388682022742&w=2
> 
> The patches have already been sent in January as RFC and I did extensive 
> testing since then.

Good.

regards,
Marc
Marc Kleine-Budde July 24, 2019, 9:31 a.m. UTC | #4
On 7/24/19 11:24 AM, Stéphane Grosjean wrote:
> I hope you're fine.

Yes, the summer holidays of our $CUSTOMERS gave me a bit more time for CAN.

> Did you see the attached patch I've sent earlier this month?
Yes, it will be included in "linux-can-fixes-for-5.3-20190724".

https://git.kernel.org/pub/scm/linux/kernel/git/mkl/linux-can.git/log/?h=linux-can-fixes-for-5.3-20190724

regards,
Marc
David Miller July 24, 2019, 6:39 p.m. UTC | #5
From: Marc Kleine-Budde <mkl@pengutronix.de>
Date: Wed, 24 Jul 2019 11:00:24 +0200

> this is a pull request for net-next/master consisting of 26 patches.

Pulled, thanks Marc.