Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1283148 > unrolled thread
| Started by | "Andrew F. Davis" <afd@ti.com> |
|---|---|
| First post | 2015-12-03 17:00 +0100 |
| Last post | 2015-12-04 09:50 +0100 |
| Articles | 2 — 2 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [PATCH v7 0/5] mfd: tps65912: Driver rewrite with DT support "Andrew F. Davis" <afd@ti.com> - 2015-12-03 17:00 +0100
Re: [PATCH v7 0/5] mfd: tps65912: Driver rewrite with DT support Lee Jones <lee.jones@linaro.org> - 2015-12-04 09:50 +0100
| From | "Andrew F. Davis" <afd@ti.com> |
|---|---|
| Date | 2015-12-03 17:00 +0100 |
| Subject | Re: [PATCH v7 0/5] mfd: tps65912: Driver rewrite with DT support |
| Message-ID | <qBEsy-75q-5@gated-at.bofh.it> |
On 11/24/2015 10:26 AM, Lee Jones wrote: > On Wed, 18 Nov 2015, Andrew F. Davis wrote: > >> In an effort to cleanup this driver and add Device Tree support >> the driver has been rewritten based on new driver styles and >> modern kernel driver helpers. This has nearly halved the lines >> of code while keeping all previous functionality. >> >> Platform file based initialization has been dropped as there is >> no examples of this use in the kernel. >> >> v1 can be found here: [1] v2: [2] v3: [3] v4: [4] v5: [5] v6: [6] >> >> Changes from v6: >> - Removed compatible strings from DT sub-nodes >> - Rearranged DT bindings >> - Small fixes >> >> Changes from v5: >> - Small formatting changes to DT Docs >> - Converted to_tps65912_gpio from macro to inline function >> >> Changes from v4: >> - Use mfd core to add sub-devices >> >> Changes from v3: >> - Reorganized regulator driver and related DT node >> - Other small fixes as discussed in v3 thread >> >> Changes from v2: >> - Split the series further into subsystems >> >> Changes from v1: >> - Split the rewrite into delete/create patches >> - Several small fixes as discussed in v1 thread >> >> [1] http://www.spinics.net/lists/devicetree/msg93863.html >> [2] http://www.spinics.net/lists/devicetree/msg95003.html >> [3] http://www.spinics.net/lists/devicetree/msg95133.html >> [4] http://www.spinics.net/lists/devicetree/msg96109.html >> [5] http://www.spinics.net/lists/devicetree/msg100601.html >> [6] https://lkml.org/lkml/2015/10/30/690 >> >> Andrew F. Davis (5): >> Documentation: tps65912: Add DT bindings for the TPS65912 PMIC >> mfd: tps65912: Remove old driver in preparation for new driver >> mfd: tps65912: Add driver for the TPS65912 PMIC >> regulator: tps65912: Add regulator driver for the TPS65912 PMIC >> gpio: tps65912: Add GPIO driver for the TPS65912 PMIC >> >> Documentation/devicetree/bindings/mfd/tps65912.txt | 50 ++ >> drivers/gpio/Kconfig | 2 +- >> drivers/gpio/gpio-tps65912.c | 317 ++++----- >> drivers/mfd/Kconfig | 20 +- >> drivers/mfd/Makefile | 3 +- >> drivers/mfd/tps65912-core.c | 290 ++++----- >> drivers/mfd/tps65912-i2c.c | 219 +++---- >> drivers/mfd/tps65912-irq.c | 217 ------- >> drivers/mfd/tps65912-spi.c | 219 +++---- >> drivers/regulator/Kconfig | 2 +- >> drivers/regulator/tps65912-regulator.c | 710 +++++---------------- >> include/linux/mfd/tps65912.h | 208 +++--- > > Just waiting for the regulator Ack now, right? > I believe so, is there a friendly way to poke Mark Brown so this doesn't get caught in his filter? This is almost identical to the regulator he applied the other day from me so I don't imagine there will be much problem with this patch. >> 12 files changed, 780 insertions(+), 1477 deletions(-) >> create mode 100644 Documentation/devicetree/bindings/mfd/tps65912.txt >> rewrite drivers/gpio/gpio-tps65912.c (74%) >> rewrite drivers/mfd/tps65912-core.c (95%) >> rewrite drivers/mfd/tps65912-i2c.c (93%) >> delete mode 100644 drivers/mfd/tps65912-irq.c >> rewrite drivers/mfd/tps65912-spi.c (92%) >> rewrite drivers/regulator/tps65912-regulator.c (94%) >> >> -- >> 1.9.1 >> > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [next] | [standalone]
| From | Lee Jones <lee.jones@linaro.org> |
|---|---|
| Date | 2015-12-04 09:50 +0100 |
| Message-ID | <qBUdZ-kv-11@gated-at.bofh.it> |
| In reply to | #1283148 |
On Thu, 03 Dec 2015, Andrew F. Davis wrote: > On 11/24/2015 10:26 AM, Lee Jones wrote: > >On Wed, 18 Nov 2015, Andrew F. Davis wrote: > > > >>In an effort to cleanup this driver and add Device Tree support > >>the driver has been rewritten based on new driver styles and > >>modern kernel driver helpers. This has nearly halved the lines > >>of code while keeping all previous functionality. > >> > >>Platform file based initialization has been dropped as there is > >>no examples of this use in the kernel. > >> > >>v1 can be found here: [1] v2: [2] v3: [3] v4: [4] v5: [5] v6: [6] > >> > >>Changes from v6: > >> - Removed compatible strings from DT sub-nodes > >> - Rearranged DT bindings > >> - Small fixes > >> > >>Changes from v5: > >> - Small formatting changes to DT Docs > >> - Converted to_tps65912_gpio from macro to inline function > >> > >>Changes from v4: > >> - Use mfd core to add sub-devices > >> > >>Changes from v3: > >> - Reorganized regulator driver and related DT node > >> - Other small fixes as discussed in v3 thread > >> > >>Changes from v2: > >> - Split the series further into subsystems > >> > >>Changes from v1: > >> - Split the rewrite into delete/create patches > >> - Several small fixes as discussed in v1 thread > >> > >>[1] http://www.spinics.net/lists/devicetree/msg93863.html > >>[2] http://www.spinics.net/lists/devicetree/msg95003.html > >>[3] http://www.spinics.net/lists/devicetree/msg95133.html > >>[4] http://www.spinics.net/lists/devicetree/msg96109.html > >>[5] http://www.spinics.net/lists/devicetree/msg100601.html > >>[6] https://lkml.org/lkml/2015/10/30/690 > >> > >>Andrew F. Davis (5): > >> Documentation: tps65912: Add DT bindings for the TPS65912 PMIC > >> mfd: tps65912: Remove old driver in preparation for new driver > >> mfd: tps65912: Add driver for the TPS65912 PMIC > >> regulator: tps65912: Add regulator driver for the TPS65912 PMIC > >> gpio: tps65912: Add GPIO driver for the TPS65912 PMIC > >> > >> Documentation/devicetree/bindings/mfd/tps65912.txt | 50 ++ > >> drivers/gpio/Kconfig | 2 +- > >> drivers/gpio/gpio-tps65912.c | 317 ++++----- > >> drivers/mfd/Kconfig | 20 +- > >> drivers/mfd/Makefile | 3 +- > >> drivers/mfd/tps65912-core.c | 290 ++++----- > >> drivers/mfd/tps65912-i2c.c | 219 +++---- > >> drivers/mfd/tps65912-irq.c | 217 ------- > >> drivers/mfd/tps65912-spi.c | 219 +++---- > >> drivers/regulator/Kconfig | 2 +- > >> drivers/regulator/tps65912-regulator.c | 710 +++++---------------- > >> include/linux/mfd/tps65912.h | 208 +++--- > > > >Just waiting for the regulator Ack now, right? > > > > I believe so, is there a friendly way to poke Mark Brown so this doesn't > get caught in his filter? This is almost identical to the regulator he > applied the other day from me so I don't imagine there will be much problem > with this patch. Just repost the set with all the remaining Acks and explain in the cover letter that you're only missing one Ack. I can prod Mark from that point. > >> 12 files changed, 780 insertions(+), 1477 deletions(-) > >> create mode 100644 Documentation/devicetree/bindings/mfd/tps65912.txt > >> rewrite drivers/gpio/gpio-tps65912.c (74%) > >> rewrite drivers/mfd/tps65912-core.c (95%) > >> rewrite drivers/mfd/tps65912-i2c.c (93%) > >> delete mode 100644 drivers/mfd/tps65912-irq.c > >> rewrite drivers/mfd/tps65912-spi.c (92%) > >> rewrite drivers/regulator/tps65912-regulator.c (94%) > >> > > -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web