All the mail mirrored from lore.kernel.org
 help / color / mirror / Atom feed
From: Gerhard Bertelsmann <info@gerhard-bertelsmann.de>
To: wg@grandegger.com, mkl@pengutronix.de,
	maxime.ripard@free-electrons.com, linux-can@vger.kernel.org,
	netdev@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org
Cc: Gerhard Bertelsmann <info@gerhard-bertelsmann.de>
Subject: [PATCH v6 0/3] can: Allwinner A10/A20 CAN Controller support - Summary
Date: Mon, 14 Sep 2015 14:54:56 +0200	[thread overview]
Message-ID: <1442235299-5885-1-git-send-email-info@gerhard-bertelsmann.de> (raw)

Hi,

please find attached the next try. Thanks to Marc an Maxime for their reviews.
To be honest I made the defconfig the best of my belief.

Please test and report bugs if exists.

[PATCH v6 1/3] Device Tree Binding Documentation
[PATCH v6 2/3] Defconfigs
[PATCH v6 3/3] Kernel Module

History:
    V6: renamed the driver to sun4i as suggested by Maxime Ripard
        removed module version
        removed suspend and resume
        moved clk enable from can_start into open / should be balanced
          between enabling and disabling now
        freeing resources on error
        
    v5: fix license
        modify prefix to mode select defines
        enable and disable clock in sunxican_get_berr_counter
        delete set_normal_mode at the end of sunxi_can_start
        removed sunxican_id_table
        use devm_clk_get instead of clk_get
        use devm_ioremap_resource to simplify probe and remove
        make set-normal-mode and set-reset-mode more readable
        
    v4: defines prefixed with SUNXI_
        sunxi_can_write_cmdreg tweaked
        loops in set_xxx_mode reworked
        add return value to set_xxx_mode
        sunxican_start_xmit reworked
        struct platform_driver stripped
        moved set_bittiming into open
        moved clock start into open
        add clock stop to close
        suspend reworked
        resume reworked
        fixed double counting bug

    v3: changed error state change handling (thx Andri for the hint)
        use bittiming function correct (no need to call it)
        strip down priv (suggested by Marc)
        scripts/checkpatch.pl-> no matches anymore
        sparse -> no errors or warnings anymore
    v2: cleaning
    v1: initial

Signed-off-by: Gerhard Bertelsmann <info@gerhard-bertelsmann.de>
---

 .../devicetree/bindings/net/can/sun4i_can.txt      |  37 +
 arch/arm/configs/multi_v7_defconfig                |   1 +
 arch/arm/configs/sunxi_defconfig                   |   2 +
 drivers/net/can/Kconfig                            |  10 +
 drivers/net/can/Makefile                           |   1 +
 drivers/net/can/sun4i_can.c                        | 824 +++++++++++++++++++++
 6 files changed, 875 insertions(+)


WARNING: multiple messages have this Message-ID (diff)
From: info@gerhard-bertelsmann.de (Gerhard Bertelsmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v6 0/3] can: Allwinner A10/A20 CAN Controller support - Summary
Date: Mon, 14 Sep 2015 14:54:56 +0200	[thread overview]
Message-ID: <1442235299-5885-1-git-send-email-info@gerhard-bertelsmann.de> (raw)

Hi,

please find attached the next try. Thanks to Marc an Maxime for their reviews.
To be honest I made the defconfig the best of my belief.

Please test and report bugs if exists.

[PATCH v6 1/3] Device Tree Binding Documentation
[PATCH v6 2/3] Defconfigs
[PATCH v6 3/3] Kernel Module

History:
    V6: renamed the driver to sun4i as suggested by Maxime Ripard
        removed module version
        removed suspend and resume
        moved clk enable from can_start into open / should be balanced
          between enabling and disabling now
        freeing resources on error
        
    v5: fix license
        modify prefix to mode select defines
        enable and disable clock in sunxican_get_berr_counter
        delete set_normal_mode at the end of sunxi_can_start
        removed sunxican_id_table
        use devm_clk_get instead of clk_get
        use devm_ioremap_resource to simplify probe and remove
        make set-normal-mode and set-reset-mode more readable
        
    v4: defines prefixed with SUNXI_
        sunxi_can_write_cmdreg tweaked
        loops in set_xxx_mode reworked
        add return value to set_xxx_mode
        sunxican_start_xmit reworked
        struct platform_driver stripped
        moved set_bittiming into open
        moved clock start into open
        add clock stop to close
        suspend reworked
        resume reworked
        fixed double counting bug

    v3: changed error state change handling (thx Andri for the hint)
        use bittiming function correct (no need to call it)
        strip down priv (suggested by Marc)
        scripts/checkpatch.pl-> no matches anymore
        sparse -> no errors or warnings anymore
    v2: cleaning
    v1: initial

Signed-off-by: Gerhard Bertelsmann <info@gerhard-bertelsmann.de>
---

 .../devicetree/bindings/net/can/sun4i_can.txt      |  37 +
 arch/arm/configs/multi_v7_defconfig                |   1 +
 arch/arm/configs/sunxi_defconfig                   |   2 +
 drivers/net/can/Kconfig                            |  10 +
 drivers/net/can/Makefile                           |   1 +
 drivers/net/can/sun4i_can.c                        | 824 +++++++++++++++++++++
 6 files changed, 875 insertions(+)

             reply	other threads:[~2015-09-14 13:02 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-09-14 12:54 Gerhard Bertelsmann [this message]
2015-09-14 12:54 ` [PATCH v6 0/3] can: Allwinner A10/A20 CAN Controller support - Summary Gerhard Bertelsmann
2015-09-14 12:54 ` [PATCH v6 1/3] can: Allwinner A10/A20 CAN Controller support - Devicetree bindings Gerhard Bertelsmann
2015-09-14 12:54   ` Gerhard Bertelsmann
2015-09-14 12:58   ` Marc Kleine-Budde
2015-09-14 12:58     ` Marc Kleine-Budde
2015-09-14 13:30     ` Maxime Ripard
2015-09-14 13:30       ` Maxime Ripard
2015-09-14 12:54 ` [PATCH v6 2/3] can: Allwinner A10/A20 CAN Controller support - Defconfigs Gerhard Bertelsmann
2015-09-14 12:54   ` Gerhard Bertelsmann
2015-09-14 13:04   ` Marc Kleine-Budde
2015-09-14 13:04     ` Marc Kleine-Budde
2015-09-14 12:54 ` [PATCH v6 3/3] can: Allwinner A10/A20 CAN Controller support - Kernel module Gerhard Bertelsmann
2015-09-14 12:54   ` Gerhard Bertelsmann
2015-09-14 13:53   ` Marc Kleine-Budde
2015-09-14 13:53     ` 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=1442235299-5885-1-git-send-email-info@gerhard-bertelsmann.de \
    --to=info@gerhard-bertelsmann.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-can@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=maxime.ripard@free-electrons.com \
    --cc=mkl@pengutronix.de \
    --cc=netdev@vger.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.