From: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
To: wg@grandegger.com, mkl@pengutronix.de, robh+dt@kernel.org
Cc: linux-can@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, dev.kurt@vandijck-laurijssen.be,
o.rempel@pengutronix.de
Subject: Re: [PATCH 0/6] Add support for MCP25XXFD SPI-CAN Network driver
Date: Tue, 15 Sep 2020 21:49:25 +0530 [thread overview]
Message-ID: <20200915161925.GA5660@linux> (raw)
In-Reply-To: <20200910133806.25077-1-manivannan.sadhasivam@linaro.org>
Hi,
On Thu, Sep 10, 2020 at 07:08:00PM +0530, Manivannan Sadhasivam wrote:
> Hello,
>
> This series is the continuation of the work done by Marc Kleine-Budde on
> adding support for Microchip MCP25XXFD SPI-CAN driver [1]. I've taken the
> patches from Marc's tree [2] and posted with an additional MAINTAINERS
> patch on top since he seems to be very busy. This series has been tested
> on RB5 board featuring MCP2518FD transceiver.
>
> Marc: I'd like to co-maintain this driver in upstream hence added myself
> as the co-maintainer. Shout at me if you do not want it! Also I've removed
> the v4x tag since I don't think all the versions are posted to mailing
> list.
>
> I'll add my review on top of this posting.
>
Just a quick question: I don't see any activity on this specific driver for
sometime (back in Martin days itself). Is it due to lack of reviewers or
it is due to the patch size (lines of code) so that nobody is interested
in reviewing?
How are we going to move forward?
Thanks,
Mani
> Thanks,
> Mani
>
> [1] https://www.spinics.net/lists/linux-can/msg03712.html
> [2] https://git.kernel.org/pub/scm/linux/kernel/git/mkl/linux-can-next.git/log/?h=mcp25xxfd-47
>
> Kurt Van Dijck (1):
> can: mcp25xxfd: add listen-only mode
>
> Manivannan Sadhasivam (1):
> MAINTAINERS: Add entry for Microchip MCP25XXFD SPI-CAN network driver
>
> Marc Kleine-Budde (3):
> can: rx-offload: can_rx_offload_add_manual(): add new initialization
> function
> can: mcp25xxfd: add regmap infrastructure
> can: mcp25xxfd: add driver for Microchip MCP25xxFD SPI CAN
>
> Oleksij Rempel (1):
> dt-bindings: can: mcp25xxfd: document device tree bindings
>
> .../bindings/net/can/microchip,mcp25xxfd.yaml | 79 +
> MAINTAINERS | 8 +
> drivers/net/can/rx-offload.c | 11 +
> drivers/net/can/spi/Kconfig | 2 +
> drivers/net/can/spi/Makefile | 1 +
> drivers/net/can/spi/mcp25xxfd/Kconfig | 17 +
> drivers/net/can/spi/mcp25xxfd/Makefile | 8 +
> .../net/can/spi/mcp25xxfd/mcp25xxfd-core.c | 2884 +++++++++++++++++
> .../net/can/spi/mcp25xxfd/mcp25xxfd-crc16.c | 89 +
> .../net/can/spi/mcp25xxfd/mcp25xxfd-regmap.c | 556 ++++
> drivers/net/can/spi/mcp25xxfd/mcp25xxfd.h | 828 +++++
> include/linux/can/rx-offload.h | 3 +
> 12 files changed, 4486 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/net/can/microchip,mcp25xxfd.yaml
> create mode 100644 drivers/net/can/spi/mcp25xxfd/Kconfig
> create mode 100644 drivers/net/can/spi/mcp25xxfd/Makefile
> create mode 100644 drivers/net/can/spi/mcp25xxfd/mcp25xxfd-core.c
> create mode 100644 drivers/net/can/spi/mcp25xxfd/mcp25xxfd-crc16.c
> create mode 100644 drivers/net/can/spi/mcp25xxfd/mcp25xxfd-regmap.c
> create mode 100644 drivers/net/can/spi/mcp25xxfd/mcp25xxfd.h
>
> --
> 2.17.1
>
next prev parent reply other threads:[~2020-09-15 22:29 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-09-10 13:38 [PATCH 0/6] Add support for MCP25XXFD SPI-CAN Network driver Manivannan Sadhasivam
2020-09-10 13:38 ` [PATCH 1/6] can: rx-offload: can_rx_offload_add_manual(): add new initialization function Manivannan Sadhasivam
2020-09-10 13:38 ` [PATCH 2/6] dt-bindings: can: mcp25xxfd: document device tree bindings Manivannan Sadhasivam
2020-09-22 19:32 ` Rob Herring
2020-09-29 9:46 ` Geert Uytterhoeven
2020-09-29 10:06 ` Marc Kleine-Budde
2020-09-29 11:25 ` Thomas.Kopp
2020-09-29 11:27 ` Marc Kleine-Budde
2020-09-10 13:38 ` [PATCH 3/6] can: mcp25xxfd: add regmap infrastructure Manivannan Sadhasivam
2020-09-10 13:38 ` [PATCH 4/6] can: mcp25xxfd: add driver for Microchip MCP25xxFD SPI CAN Manivannan Sadhasivam
2020-09-10 13:38 ` [PATCH 5/6] can: mcp25xxfd: add listen-only mode Manivannan Sadhasivam
2020-09-10 13:38 ` [PATCH 6/6] MAINTAINERS: Add entry for Microchip MCP25XXFD SPI-CAN network driver Manivannan Sadhasivam
2020-09-15 16:19 ` Manivannan Sadhasivam [this message]
2020-09-15 17:58 ` [PATCH 0/6] Add support for MCP25XXFD SPI-CAN Network driver Kurt Van Dijck
2020-09-16 4:07 ` Manivannan Sadhasivam
2020-09-16 13:59 ` Marc Kleine-Budde
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=20200915161925.GA5660@linux \
--to=manivannan.sadhasivam@linaro.org \
--cc=dev.kurt@vandijck-laurijssen.be \
--cc=devicetree@vger.kernel.org \
--cc=linux-can@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mkl@pengutronix.de \
--cc=o.rempel@pengutronix.de \
--cc=robh+dt@kernel.org \
--cc=wg@grandegger.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;
as well as URLs for NNTP newsgroup(s).