Re: [PATCH 0/8] can: ems_pci: Add support for CPC-PCIe v3

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Vincent,

Thank you for your comments! I will send a v2 series soon.

Gerhard

Am 20.01.23 um 00:28 schrieb Vincent Mailhol:
Hi Gerhard,

I had a quick look at your series. I have a couple of nitpicks. The
rest looks good.

On Fri. 20 Jan. 2023 at 01:02, Gerhard Uttenthaler
<uttenthaler@xxxxxxxxxxxxxxxx> wrote:
The CPC-PCIe v3 uses a Asix AX99100 instead of the discontinued
PLX PCI9030 bridge chip. This patch series adds support for this
card version and cleaned some code styling issues.

Gerhard Uttenthaler (8):
   Fixed code style, copyright and email address
   Added Asix AX99100 definitions
   Initialize BAR registers
   Added read/write register and post irq functions
   Initialize CAN controller base addresses
   Added IRQ enable
   Deassert hardware reset
   Added MODULE_AUTHOR

   Fixed code style, copyright and email address
     ^^^^^
   Added Asix AX99100 definitions
     ^^^^^
   Initialize BAR registers
   Added read/write register and post irq functions
     ^^^^^
   Initialize CAN controller base addresses
   Added IRQ enable
     ^^^^^
   Deassert hardware reset
   Added MODULE_AUTHOR
     ^^^^^

For the titles, please use imperative (e.g. add) instead of past
tense (e.g. Added). This also applies to the description.


  drivers/net/can/sja1000/ems_pci.c | 152 ++++++++++++++++++++++--------
  1 file changed, 113 insertions(+), 39 deletions(-)

--
2.35.3

--
EMS Dr. Thomas Wuensche e.K.
Sonnenhang 3
85304 Ilmmuenster
HR Ingolstadt, HRA 170106

Phone: +49-8441-490260
Fax  : +49-8441-81860
http://www.ems-wuensche.com
-- --

--
EMS Dr. Thomas Wuensche e.K.
Sonnenhang 3
85304 Ilmmuenster
HR Ingolstadt, HRA 170106

Phone: +49-8441-490260
Fax  : +49-8441-81860
http://www.ems-wuensche.com



[Index of Archives]     [Automotive Discussions]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]     [CAN Bus]

  Powered by Linux