* AMLogic RTC series (second round)
@ 2016-06-27 16:44 Ben Dooks
2016-06-27 16:44 ` [PATCH v2 1/4] rtc: support for amlogic meson rtc Ben Dooks
` (4 more replies)
0 siblings, 5 replies; 8+ messages in thread
From: Ben Dooks @ 2016-06-27 16:44 UTC (permalink / raw)
To: linus-amlogic
Apologies, forgot to put the RTC driver through checkpatch
before sending. This gets rid of most of the warnings.
^ permalink raw reply [flat|nested] 8+ messages in thread* [PATCH v2 1/4] rtc: support for amlogic meson rtc 2016-06-27 16:44 AMLogic RTC series (second round) Ben Dooks @ 2016-06-27 16:44 ` Ben Dooks 2016-06-27 16:44 ` [PATCH v2 2/4] meson-rtc: add device-tree binding Ben Dooks ` (3 subsequent siblings) 4 siblings, 0 replies; 8+ messages in thread From: Ben Dooks @ 2016-06-27 16:44 UTC (permalink / raw) To: linus-amlogic Add support for the AMLogic meson8b/meson6 RTC block. Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk> --- drivers/rtc/Kconfig | 10 ++ drivers/rtc/Makefile | 1 + drivers/rtc/rtc-meson.c | 339 ++++++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 350 insertions(+) create mode 100644 drivers/rtc/rtc-meson.c diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig index 18639e0..ff1a25d 100644 --- a/drivers/rtc/Kconfig +++ b/drivers/rtc/Kconfig @@ -1210,6 +1210,16 @@ config RTC_DRV_IMXDI This driver can also be built as a module, if so, the module will be called "rtc-imxdi". +config RTC_DRV_MESON + tristate "AMLogic Meson Real Time Clock" + depends on ARCH_MESON || COMPILE_TEST + help + Support for AMLogic Meson RTC block. + + This driver can also be built as a module, if so, the module + will be called "rtc-meson". + + config RTC_DRV_OMAP tristate "TI OMAP Real Time Clock" depends on ARCH_OMAP || ARCH_DAVINCI || COMPILE_TEST diff --git a/drivers/rtc/Makefile b/drivers/rtc/Makefile index ea28337..0b4f106 100644 --- a/drivers/rtc/Makefile +++ b/drivers/rtc/Makefile @@ -92,6 +92,7 @@ obj-$(CONFIG_RTC_DRV_MAX8997) += rtc-max8997.o obj-$(CONFIG_RTC_DRV_MAX8998) += rtc-max8998.o obj-$(CONFIG_RTC_DRV_MC13XXX) += rtc-mc13xxx.o obj-$(CONFIG_RTC_DRV_MCP795) += rtc-mcp795.o +obj-$(CONFIG_RTC_DRV_MESON) += rtc-meson.o obj-$(CONFIG_RTC_DRV_MOXART) += rtc-moxart.o obj-$(CONFIG_RTC_DRV_MPC5121) += rtc-mpc5121.o obj-$(CONFIG_RTC_DRV_VRTC) += rtc-mrst.o diff --git a/drivers/rtc/rtc-meson.c b/drivers/rtc/rtc-meson.c new file mode 100644 index 0000000..8270bd9 --- /dev/null +++ b/drivers/rtc/rtc-meson.c @@ -0,0 +1,339 @@ +/* drivers/rtc/rtc-meson.c + * + * Copyright (c) 2015 Codethink Ltd + * Ben Dooks <ben.dooks@codethink.co.uk> + * Based on origin by Carlo Caione <carlo@endlessm.com> + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * published by the Free Software Foundation. +*/ + +#include <linux/module.h> +#include <linux/platform_device.h> +#include <linux/delay.h> +#include <linux/reset.h> +#include <linux/rtc.h> +#include <linux/io.h> +#include <linux/of.h> + +/* registers accessed from cpu bus */ +#define RTC_REG(x) ((x) * 4) /* rtc registers 0-4 */ + +#define LINE_SDI (1 << 2) +#define LINE_SEN (1 << 1) +#define LINE_SCLK (1 << 0) + +#define RTCREG0_START_SER BIT(17) +#define RTCREG0_WAIT_SER BIT(22) +#define RTC_REG0_DATA(__data) ((__data) << 24) + +#define RTCREG1_READY BIT(1) + +/* rtc registers accessed via rtc-serial interface */ +#define RTC_COUNTER (0) +#define RTC_SEC_ADJ (2) + +#define RTC_ADDR_BITS (3) /* number of address bits to send */ +#define RTC_DATA_BITS (32) /* number of data bits to tx/rx */ + +#define MESON_STATIC_BIAS_CUR (0x5 << 1) +#define MESON_STATIC_VOLTAGE (0x3 << 11) + +struct meson_rtc { + struct rtc_device *rtc; /* rtc device we created */ + struct device *dev; /* device we bound from */ + struct reset_control *reset; /* reset source */ + struct mutex lock; /* rtc lock */ + void __iomem *regs; /* rtc register access */ +}; + +static void meson_rtc_setline(struct meson_rtc *rtc, + unsigned int bit, unsigned int to) +{ + u32 reg0; + + reg0 = readl(rtc->regs + RTC_REG(0)); + if (to) + reg0 |= bit; + else + reg0 &= ~bit; + writel(reg0, rtc->regs + RTC_REG(0)); +} + +static void meson_rtc_sclk_pulse(struct meson_rtc *rtc) +{ + udelay(5); + meson_rtc_setline(rtc, LINE_SCLK, 0); + udelay(5); + meson_rtc_setline(rtc, LINE_SCLK, 1); +} + +static void meson_rtc_send_bit(struct meson_rtc *rtc, unsigned int bit) +{ + meson_rtc_setline(rtc, LINE_SDI, bit ? 1 : 0); + meson_rtc_sclk_pulse(rtc); +} + +static void meson_rtc_send_bits(struct meson_rtc *rtc, + u32 data, unsigned int nr) +{ + u32 bit = 1 << (nr - 1); + + while (bit) { + meson_rtc_send_bit(rtc, data & bit); + bit >>= 1; + } +} + +static void meson_rtc_set_dir(struct meson_rtc *rtc, u32 mode) +{ + meson_rtc_setline(rtc, LINE_SEN, 0); + meson_rtc_setline(rtc, LINE_SDI, 0); + meson_rtc_send_bit(rtc, mode); + meson_rtc_setline(rtc, LINE_SDI, 0); +} + +static u32 meson_rtc_read_sdo(struct meson_rtc *rtc) +{ + return readl(rtc->regs + RTC_REG(1)) & BIT(0); +} + +static u32 meson_rtc_get_data(struct meson_rtc *rtc) +{ + u32 val = 0; + int bit; + + for (bit = 0; bit < RTC_DATA_BITS; bit++) { + meson_rtc_sclk_pulse(rtc); + val <<= 1; + val |= meson_rtc_read_sdo(rtc); + } + + return val; +} + +static int meson_rtc_wait_bus(struct meson_rtc *rtc, unsigned int timeout_ms) +{ + unsigned long timeout = jiffies + msecs_to_jiffies(timeout_ms); + u32 val; + + while (time_before(jiffies, timeout)) { + val = readl(rtc->regs + RTC_REG(1)); + if (val & RTCREG1_READY) + return 1; + + dev_dbg(rtc->dev, "%s: reg0=%08x reg1=%08x\n", + __func__, readl(rtc->regs + RTC_REG(0)), val); + msleep(10); + } + + return 0; +} + +static int meson_rtc_get_bus(struct meson_rtc *rtc) +{ + int ret, retries = 3; + u32 val; + + /* prepare bus for transfers, set all lines low */ + val = readl(rtc->regs + RTC_REG(0)); + val &= ~(LINE_SDI | LINE_SEN | LINE_SCLK); + writel(val, rtc->regs + RTC_REG(0)); + + while (retries) { + if (meson_rtc_wait_bus(rtc, 300)) + return 0; + + dev_warn(rtc->dev, "failed to get bus, re-setting\n"); + + retries--; + ret = reset_control_reset(rtc->reset); + if (ret) + return ret; + } + + dev_err(rtc->dev, "%s: bus is not ready\n", __func__); + return -ETIMEDOUT; +} + +static int meson_rtc_read(struct meson_rtc *rtc, unsigned int reg, u32 *data) +{ + int ret; + + ret = meson_rtc_get_bus(rtc); + if (ret) + return ret; + + meson_rtc_setline(rtc, LINE_SEN, 1); + meson_rtc_send_bits(rtc, reg, RTC_ADDR_BITS); + meson_rtc_set_dir(rtc, 0); + *data = meson_rtc_get_data(rtc); + + return 0; +} + +static int meson_rtc_write(struct meson_rtc *rtc, unsigned int reg, u32 data) +{ + int ret; + + dev_dbg(rtc->dev, "%s: reg %d val %08x)\n", __func__, reg, data); + + ret = meson_rtc_get_bus(rtc); + if (ret) + return ret; + + meson_rtc_setline(rtc, LINE_SEN, 1); + meson_rtc_send_bits(rtc, data, RTC_DATA_BITS); + meson_rtc_send_bits(rtc, reg, RTC_ADDR_BITS); + meson_rtc_set_dir(rtc, 1); + + return 0; +} + +static int meson_rtc_gettime(struct device *dev, struct rtc_time *tm) +{ + struct meson_rtc *rtc = dev_get_drvdata(dev); + int ret; + u32 time; + + mutex_lock(&rtc->lock); + + ret = meson_rtc_read(rtc, RTC_COUNTER, &time); + if (!ret) { + rtc_time_to_tm(time, tm); + dev_dbg(dev, "read time %lu\n", (unsigned long)time); + } + + mutex_unlock(&rtc->lock); + return ret; +} + +static int meson_rtc_settime(struct device *dev, struct rtc_time *tm) +{ + struct meson_rtc *rtc = dev_get_drvdata(dev); + unsigned long time; + int ret; + + mutex_lock(&rtc->lock); + + rtc_tm_to_time(tm, &time); + ret = meson_rtc_write(rtc, RTC_COUNTER, time); + + mutex_unlock(&rtc->lock); + return ret; +} + +static const struct rtc_class_ops meson_rtc_ops = { + .read_time = meson_rtc_gettime, + .set_time = meson_rtc_settime, +}; + +static int meson_rtc_wait_serialiser(struct meson_rtc *rtc) +{ + unsigned long timeout = jiffies + msecs_to_jiffies(100); + + while (time_before(jiffies, timeout)) { + if (!(readl(rtc->regs + RTC_REG(0)) & RTCREG0_WAIT_SER)) + return 0; + msleep(10); + } + + return -ETIMEDOUT; +} + +static int meson_rtc_write_static(struct meson_rtc *rtc, u32 data) +{ + u32 tmp; + + mutex_lock(&rtc->lock); + + writel(data >> 8, rtc->regs + RTC_REG(4)); + + tmp = readl(rtc->regs + RTC_REG(0)); + tmp &= ~RTC_REG0_DATA(0xff); + tmp |= RTC_REG0_DATA(data); + tmp |= RTCREG0_START_SER; + + writel(tmp, rtc->regs + RTC_REG(0)); + + if (meson_rtc_wait_serialiser(rtc)) + return -ETIMEDOUT; + + dev_dbg(rtc->dev, "rtc_reg0 = %08x\n", readl(rtc->regs + RTC_REG(0))); + mutex_unlock(&rtc->lock); + + return 0; +} + +static int meson_rtc_probe(struct platform_device *pdev) +{ + struct meson_rtc *rtc; + struct device *dev = &pdev->dev; + struct resource *res; + u32 tm; + int ret; + + rtc = devm_kzalloc(dev, sizeof(struct meson_rtc), GFP_KERNEL); + if (!rtc) + return -ENOMEM; + + res = platform_get_resource(pdev, IORESOURCE_MEM, 0); + rtc->regs = devm_ioremap_resource(dev, res); + rtc->dev = dev; + + if (IS_ERR(rtc->regs)) + return PTR_ERR(rtc->regs); + + mutex_init(&rtc->lock); + platform_set_drvdata(pdev, rtc); + + ret = meson_rtc_write_static(rtc, + MESON_STATIC_BIAS_CUR | MESON_STATIC_VOLTAGE); + if (ret) { + dev_err(dev, "failed to set static values\n"); + return ret; + } + + rtc->reset = devm_reset_control_get(dev, NULL); + if (IS_ERR(rtc->reset)) + dev_warn(dev, "no reset available, rtc may not work\n"); + + /* check if we can read RTC counter, if not then the RTC is probably + * not functional. If it isn't probably best to not bind. + */ + ret = meson_rtc_read(rtc, RTC_COUNTER, &tm); + if (ret) { + dev_err(dev, "cannot read RTC counter, RTC not functional\n"); + return ret; + } + + rtc->rtc = devm_rtc_device_register(dev, "meson", + &meson_rtc_ops, THIS_MODULE); + if (IS_ERR(rtc->rtc)) + return PTR_ERR(rtc->rtc); + + return 0; +} + +static const struct of_device_id meson_rtc_dt_match[] = { + { .compatible = "amlogic,meson6-rtc", }, + { .compatible = "amlogic,meson8-rtc", }, + { .compatible = "amlogic,meson8b-rtc", }, + { }, +}; +MODULE_DEVICE_TABLE(of, meson_rtc_dt_match); + +static struct platform_driver meson_rtc_driver = { + .probe = meson_rtc_probe, + .driver = { + .name = "meson-rtc", + .of_match_table = of_match_ptr(meson_rtc_dt_match), + }, +}; +module_platform_driver(meson_rtc_driver); + +MODULE_DESCRIPTION("AMLogic MESON RTC Driver"); +MODULE_AUTHOR("Ben Dooks <ben.doosk@codethink.co.uk>"); +MODULE_LICENSE("GPL"); +MODULE_ALIAS("platform:meson-rtc"); -- 2.8.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH v2 2/4] meson-rtc: add device-tree binding 2016-06-27 16:44 AMLogic RTC series (second round) Ben Dooks 2016-06-27 16:44 ` [PATCH v2 1/4] rtc: support for amlogic meson rtc Ben Dooks @ 2016-06-27 16:44 ` Ben Dooks 2016-06-28 20:57 ` Rob Herring 2016-06-27 16:44 ` [PATCH v2 3/4] ARM: dts: amlogic: add rtc node for meson8b Ben Dooks ` (2 subsequent siblings) 4 siblings, 1 reply; 8+ messages in thread From: Ben Dooks @ 2016-06-27 16:44 UTC (permalink / raw) To: linus-amlogic Add documentation for the AMLogic Meson RTC. Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk> --- .../devicetree/bindings/rtc/rtc-amlogic.txt | 21 +++++++++++++++++++++ 1 file changed, 21 insertions(+) create mode 100644 Documentation/devicetree/bindings/rtc/rtc-amlogic.txt diff --git a/Documentation/devicetree/bindings/rtc/rtc-amlogic.txt b/Documentation/devicetree/bindings/rtc/rtc-amlogic.txt new file mode 100644 index 0000000..efc7672 --- /dev/null +++ b/Documentation/devicetree/bindings/rtc/rtc-amlogic.txt @@ -0,0 +1,21 @@ +* AMlogic Meson RTC + +Required properties: +- compatible: should be one of the following describing the hardware: + * "amlogic,meson6-rtc" + * "amlogic,meson8-rtc" + * "amlogic,meson8b-rtc" + +- reg: physical register space for the controller's memory mapped registers. + +Optional properties: +- resets: reset controller reference to allow reset of the controller +- interrupts: single interrupt reference if interrupt support needed + +Example: + + rtc: rtc at c8100740 { + compatible = "amlogic,meson6-rtc"; + reg = <0xc8100740 0x14>; + status = "disabled"; + }; -- 2.8.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH v2 2/4] meson-rtc: add device-tree binding 2016-06-27 16:44 ` [PATCH v2 2/4] meson-rtc: add device-tree binding Ben Dooks @ 2016-06-28 20:57 ` Rob Herring 0 siblings, 0 replies; 8+ messages in thread From: Rob Herring @ 2016-06-28 20:57 UTC (permalink / raw) To: linus-amlogic On Mon, Jun 27, 2016 at 05:44:07PM +0100, Ben Dooks wrote: > Add documentation for the AMLogic Meson RTC. > > Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk> > --- > .../devicetree/bindings/rtc/rtc-amlogic.txt | 21 +++++++++++++++++++++ > 1 file changed, 21 insertions(+) > create mode 100644 Documentation/devicetree/bindings/rtc/rtc-amlogic.txt Acked-by: Rob Herring <robh@kernel.org> ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v2 3/4] ARM: dts: amlogic: add rtc node for meson8b 2016-06-27 16:44 AMLogic RTC series (second round) Ben Dooks 2016-06-27 16:44 ` [PATCH v2 1/4] rtc: support for amlogic meson rtc Ben Dooks 2016-06-27 16:44 ` [PATCH v2 2/4] meson-rtc: add device-tree binding Ben Dooks @ 2016-06-27 16:44 ` Ben Dooks 2016-06-27 16:44 ` [PATCH v2 4/4] ARM: meson: enable RTC for ODroid-C1 Ben Dooks 2016-06-28 7:22 ` AMLogic RTC series (second round) Neil Armstrong 4 siblings, 0 replies; 8+ messages in thread From: Ben Dooks @ 2016-06-27 16:44 UTC (permalink / raw) To: linus-amlogic Add the DT node for the RTC support. Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk> --- arch/arm/boot/dts/meson8b.dtsi | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/arch/arm/boot/dts/meson8b.dtsi b/arch/arm/boot/dts/meson8b.dtsi index 2bfe401..ff97aa8 100644 --- a/arch/arm/boot/dts/meson8b.dtsi +++ b/arch/arm/boot/dts/meson8b.dtsi @@ -149,6 +149,13 @@ status = "disabled"; }; + rtc: rtc at c8100740 { + compatible = "amlogic,meson8b-rtc"; + reg = <0xc8100740 0x14>; + resets = <&reset RESET_RTC>; + status = "disabled"; + }; + clkc: clock-controller at c1104000 { #clock-cells = <1>; compatible = "amlogic,meson8b-clkc"; -- 2.8.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH v2 4/4] ARM: meson: enable RTC for ODroid-C1 2016-06-27 16:44 AMLogic RTC series (second round) Ben Dooks ` (2 preceding siblings ...) 2016-06-27 16:44 ` [PATCH v2 3/4] ARM: dts: amlogic: add rtc node for meson8b Ben Dooks @ 2016-06-27 16:44 ` Ben Dooks 2016-06-28 7:22 ` AMLogic RTC series (second round) Neil Armstrong 4 siblings, 0 replies; 8+ messages in thread From: Ben Dooks @ 2016-06-27 16:44 UTC (permalink / raw) To: linus-amlogic Now the RTC node has been added, enable it on the ODroid-C1 board. Note, the C1 does not have a battery by default, so the RTC will error out on those boards. The C1+ does have power to the RTC block whilst the external power is applied. Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk> --- arch/arm/boot/dts/meson8b-odroidc1.dts | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/arch/arm/boot/dts/meson8b-odroidc1.dts b/arch/arm/boot/dts/meson8b-odroidc1.dts index e50f1a1..5c7826f 100644 --- a/arch/arm/boot/dts/meson8b-odroidc1.dts +++ b/arch/arm/boot/dts/meson8b-odroidc1.dts @@ -76,3 +76,8 @@ pinctrl-0 = <&uart_ao_a_pins>; pinctrl-names = "default"; }; + +&rtc { + status = "okay"; +}; + -- 2.8.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* AMLogic RTC series (second round) 2016-06-27 16:44 AMLogic RTC series (second round) Ben Dooks ` (3 preceding siblings ...) 2016-06-27 16:44 ` [PATCH v2 4/4] ARM: meson: enable RTC for ODroid-C1 Ben Dooks @ 2016-06-28 7:22 ` Neil Armstrong 2016-06-28 10:19 ` Ben Dooks 4 siblings, 1 reply; 8+ messages in thread From: Neil Armstrong @ 2016-06-28 7:22 UTC (permalink / raw) To: linus-amlogic On 06/27/2016 06:44 PM, Ben Dooks wrote: > Apologies, forgot to put the RTC driver through checkpatch > before sending. This gets rid of most of the warnings. > > > _______________________________________________ > linux-amlogic mailing list > linux-amlogic at lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-amlogic > Hi Ben, The patchset looks good but misses some quick introduction on how the RTC works, it seems to be connected to an internal serial bus, do you have some informations about this ? Then, the cover letter of your patchset should contain much more informations that "git format-patch --cover-letter" generates. My personal workflow when I have a set ready is : * run checkpatch on C files and check my changes # scripts/checkpatch.pl -f path/to/my/driver.c * I identify the number of patches to include * I generate the patches with cover-letter, signoff and eventually subject-prefix, here for 4 commits # git format-patch --cover-letter -s -4 --subject-prefix "PATCH v2" -o my_drivers_patches_v2 * Then I complete the cover letter in my_drivers_patches_v2/0000-cover-letter * I also reference a small changelog between patchset version and identify the previous cover letter via it's message id Use the http://lkml.kernel.org service to reference a message-id, i.e this email I answer would be referenced by : http://lkml.kernel.org/r/1467045849-495-1-git-send-email-ben.dooks at codethink.co.uk while looking at the email headers at : Message-Id: <1467045849-495-1-git-send-email-ben.dooks@codethink.co.uk> * Then I look again all the patches and check all the descriptions * I re-run a checkpatch.pl on all the patches (and ignore errors on the cover letter) # scripts/checkpatch.pl my_drivers_patches_v2/* * I identify all the maintainers and mailing lists # scripts/get_maintainer.pl my_drivers_patches_v2/* * Personally, I fill the CC: and To: in the headers of each patches, only adding the devicetree mailing list on DT related patches * Then I send them via git send-email # git send-email my_drivers_patches_v2/* => here you can also select the recipients => you can also do a dry run or send yourself the patchset to check before final sending Another rule is time, please wait at least 1 week after the last review before send another run, and give yourself more time to check the integrity of the patchset. Neil ^ permalink raw reply [flat|nested] 8+ messages in thread
* AMLogic RTC series (second round) 2016-06-28 7:22 ` AMLogic RTC series (second round) Neil Armstrong @ 2016-06-28 10:19 ` Ben Dooks 0 siblings, 0 replies; 8+ messages in thread From: Ben Dooks @ 2016-06-28 10:19 UTC (permalink / raw) To: linus-amlogic On 28/06/16 08:22, Neil Armstrong wrote: > On 06/27/2016 06:44 PM, Ben Dooks wrote: >> Apologies, forgot to put the RTC driver through checkpatch >> before sending. This gets rid of most of the warnings. >> >> >> _______________________________________________ >> linux-amlogic mailing list >> linux-amlogic at lists.infradead.org >> http://lists.infradead.org/mailman/listinfo/linux-amlogic >> > > Hi Ben, > > The patchset looks good but misses some quick introduction on how > the RTC works, it seems to be connected to an internal serial bus, > do you have some informations about this ? I might add a few lines into the rtc-meson.c driver, but that's all detailed in the datasheet (for once this seems to be documented enough to be able to function) > Then, the cover letter of your patchset should contain much more informations > that "git format-patch --cover-letter" generates. I may just re-submit once patch to add the RTC and the binding documentation, and then wait until the reset controller bits have landed before submitting the > My personal workflow when I have a set ready is : > > * run checkpatch on C files and check my changes > # scripts/checkpatch.pl -f path/to/my/driver.c > * I identify the number of patches to include > * I generate the patches with cover-letter, signoff and eventually subject-prefix, here for 4 commits > # git format-patch --cover-letter -s -4 --subject-prefix "PATCH v2" -o my_drivers_patches_v2 > * Then I complete the cover letter in my_drivers_patches_v2/0000-cover-letter > * I also reference a small changelog between patchset version and identify the previous cover letter via it's message id > Use the http://lkml.kernel.org service to reference a message-id, i.e this email I answer would be referenced by : > http://lkml.kernel.org/r/1467045849-495-1-git-send-email-ben.dooks at codethink.co.uk I keep a copy of sent email anyway, so can get the message-ids out of it. > while looking at the email headers at : > Message-Id: <1467045849-495-1-git-send-email-ben.dooks@codethink.co.uk> > * Then I look again all the patches and check all the descriptions > * I re-run a checkpatch.pl on all the patches (and ignore errors on the cover letter) > # scripts/checkpatch.pl my_drivers_patches_v2/* > * I identify all the maintainers and mailing lists > # scripts/get_maintainer.pl my_drivers_patches_v2/* > * Personally, I fill the CC: and To: in the headers of each patches, only adding the devicetree mailing list on DT related patches > * Then I send them via git send-email > # git send-email my_drivers_patches_v2/* > => here you can also select the recipients > => you can also do a dry run or send yourself the patchset to check before final sending I got sparse and checkpatch confused, and only did sparse on the series. -- Ben Dooks http://www.codethink.co.uk/ Senior Engineer Codethink - Providing Genius ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2016-06-28 20:57 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-06-27 16:44 AMLogic RTC series (second round) Ben Dooks 2016-06-27 16:44 ` [PATCH v2 1/4] rtc: support for amlogic meson rtc Ben Dooks 2016-06-27 16:44 ` [PATCH v2 2/4] meson-rtc: add device-tree binding Ben Dooks 2016-06-28 20:57 ` Rob Herring 2016-06-27 16:44 ` [PATCH v2 3/4] ARM: dts: amlogic: add rtc node for meson8b Ben Dooks 2016-06-27 16:44 ` [PATCH v2 4/4] ARM: meson: enable RTC for ODroid-C1 Ben Dooks 2016-06-28 7:22 ` AMLogic RTC series (second round) Neil Armstrong 2016-06-28 10:19 ` Ben Dooks
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).