From: Paul Walmsley <paul@pwsan.com> To: Tero Kristo <t-kristo@ti.com> Cc: linux-omap@vger.kernel.org, tony@atomide.com, nm@ti.com, rnayak@ti.com, bcousson@baylibre.com, mturquette@linaro.org, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org Subject: Re: [PATCHv10 03/41] CLK: ti: add init support for clock IP blocks Date: Tue, 17 Dec 2013 08:14:42 +0000 (UTC) [thread overview] Message-ID: <alpine.DEB.2.02.1312170745360.14039@utopia.booyaka.com> (raw) In-Reply-To: <1385453182-24421-4-git-send-email-t-kristo@ti.com> On Tue, 26 Nov 2013, Tero Kristo wrote: > ti_dt_clk_init_provider() can now be used to initialize the contents of > a single clock IP block. This parses all the clocks under the IP block > and calls the corresponding init function for them. > > This patch also introduces a helper function for the TI clock drivers > to get register info from DT and append the master IP info to this. > > Signed-off-by: Tero Kristo <t-kristo@ti.com> ... > diff --git a/drivers/clk/ti/clk.c b/drivers/clk/ti/clk.c > index ef1a7cd..63f85e9 100644 > --- a/drivers/clk/ti/clk.c > +++ b/drivers/clk/ti/clk.c > @@ -19,10 +19,15 @@ > #include <linux/clkdev.h> > #include <linux/clk/ti.h> > #include <linux/of.h> > +#include <linux/of_address.h> > +#include <linux/list.h> > > #undef pr_fmt > #define pr_fmt(fmt) "%s: " fmt, __func__ > > +extern struct of_device_id __clk_of_table[]; This results in a checkpatch.pl warning: WARNING: externs should be avoided in .c files #33: FILE: drivers/clk/ti/clk.c:28: +extern struct of_device_id __clk_of_table[]; Please make sure your patches are checkpatch.pl-clean, with the exception of the 80-column warnings and any const-related warnings related to the clock framework. > +static int ti_dt_clk_regmap_index; > + > /** > * ti_dt_clocks_register - register DT alias clocks during boot > * @oclks: list of clocks to register > @@ -53,3 +58,96 @@ void __init ti_dt_clocks_register(struct ti_dt_clk oclks[]) > } > } > } > + > +typedef int (*ti_of_clk_init_cb_t)(struct device_node *); Normally typedefs should be a red flag to reviewers due to Documentation/CodingStyle chapter 5. Still it seems this patch is just duplicating the way that the CCF does it, so I'm not too worried about it. > + > +struct clk_init_item { > + int index; > + struct device_node *np; > + ti_of_clk_init_cb_t init_cb; > + struct list_head node; > +}; > + > +static LIST_HEAD(retry_list); > + > +/** > + * ti_clk_get_reg_addr - get register address for a clock register > + * @node: device node for the clock > + * @index: register index from the clock node > + * > + * Builds clock register address from device tree information. This > + * is a struct of type clk_omap_reg. > + */ > +void __iomem *ti_clk_get_reg_addr(struct device_node *node, int index) > +{ > + struct clk_omap_reg *reg; > + u32 val; > + u32 tmp; > + > + reg = (struct clk_omap_reg *)&tmp; > + reg->index = ti_dt_clk_regmap_index; > + > + if (of_property_read_u32_index(node, "reg", index, &val)) { > + pr_err("%s must have reg[%d]!\n", node->name, index); > + return NULL; > + } > + > + reg->offset = val; > + > + return (void __iomem *)tmp; > +} > + > +/** > + * ti_dt_clk_init_provider - init master clock provider > + * @parent: master node > + * @index: internal index for clk_reg_ops > + * > + * Initializes a master clock IP block and its child clock nodes. > + * Regmap is provided for accessing the register space for the > + * IP block and all the clocks under it. > + */ > +void ti_dt_clk_init_provider(struct device_node *parent, int index) > +{ > + const struct of_device_id *match; > + struct device_node *np; > + ti_of_clk_init_cb_t clk_init_cb; > + struct clk_init_item *retry; > + struct clk_init_item *tmp; > + int ret; > + > + ti_dt_clk_regmap_index = index; > + > + for_each_child_of_node(parent, np) { > + match = of_match_node(__clk_of_table, np); > + if (!match) > + continue; > + clk_init_cb = (ti_of_clk_init_cb_t)match->data; > + pr_debug("%s: initializing: %s\n", __func__, np->name); > + ret = clk_init_cb(np); > + if (ret == -EAGAIN) { > + pr_debug("%s: adding to again list...\n", np->name); > + retry = kzalloc(sizeof(*retry), GFP_KERNEL); > + retry->np = np; > + retry->init_cb = clk_init_cb; > + list_add(&retry->node, &retry_list); > + } else if (ret) { > + pr_err("%s: clock init failed for %s (%d)!\n", __func__, > + np->name, ret); > + } > + } > + > + list_for_each_entry_safe(retry, tmp, &retry_list, node) { > + pr_debug("%s: retry-init: %s\n", __func__, retry->np->name); > + ti_dt_clk_regmap_index = retry->index; > + ret = retry->init_cb(retry->np); > + if (ret == -EAGAIN) { > + pr_debug("%s failed again?\n", retry->np->name); This is presumably a serious error condition and should be a pr_warn() or pr_err(), right? If retry_list won't be walked again, then it seems best to delete and free the list_entry no matter what the return value is from retry->init_cb(), since it's not like it will be retried. Otherwise the code will leak memory. > + } else { > + if (ret) > + pr_err("%s: clock init failed for %s (%d)!\n", > + __func__, retry->np->name, ret); > + list_del(&retry->node); > + kfree(retry); > + } > + } > +} > diff --git a/include/linux/clk/ti.h b/include/linux/clk/ti.h > index df94c24..d6bf530 100644 > --- a/include/linux/clk/ti.h > +++ b/include/linux/clk/ti.h > @@ -36,7 +36,21 @@ struct ti_dt_clk { > .node_name = name, \ > } > > +/* Maximum number of clock regmaps */ > +#define CLK_MAX_REGMAPS 4 > > +/** > + * struct clk_omap_reg - OMAP register declaration > + * @offset: offset from the master IP module base address > + * @index: index of the master IP module > + */ > +struct clk_omap_reg { > + u16 offset; > + u16 index; > +}; > + > +void __iomem *ti_clk_get_reg_addr(struct device_node *node, int index); > void ti_dt_clocks_register(struct ti_dt_clk *oclks); > +void ti_dt_clk_init_provider(struct device_node *np, int index); > > #endif > -- > 1.7.9.5 > - Paul
WARNING: multiple messages have this Message-ID (diff)
From: paul@pwsan.com (Paul Walmsley) To: linux-arm-kernel@lists.infradead.org Subject: [PATCHv10 03/41] CLK: ti: add init support for clock IP blocks Date: Tue, 17 Dec 2013 08:14:42 +0000 (UTC) [thread overview] Message-ID: <alpine.DEB.2.02.1312170745360.14039@utopia.booyaka.com> (raw) In-Reply-To: <1385453182-24421-4-git-send-email-t-kristo@ti.com> On Tue, 26 Nov 2013, Tero Kristo wrote: > ti_dt_clk_init_provider() can now be used to initialize the contents of > a single clock IP block. This parses all the clocks under the IP block > and calls the corresponding init function for them. > > This patch also introduces a helper function for the TI clock drivers > to get register info from DT and append the master IP info to this. > > Signed-off-by: Tero Kristo <t-kristo@ti.com> ... > diff --git a/drivers/clk/ti/clk.c b/drivers/clk/ti/clk.c > index ef1a7cd..63f85e9 100644 > --- a/drivers/clk/ti/clk.c > +++ b/drivers/clk/ti/clk.c > @@ -19,10 +19,15 @@ > #include <linux/clkdev.h> > #include <linux/clk/ti.h> > #include <linux/of.h> > +#include <linux/of_address.h> > +#include <linux/list.h> > > #undef pr_fmt > #define pr_fmt(fmt) "%s: " fmt, __func__ > > +extern struct of_device_id __clk_of_table[]; This results in a checkpatch.pl warning: WARNING: externs should be avoided in .c files #33: FILE: drivers/clk/ti/clk.c:28: +extern struct of_device_id __clk_of_table[]; Please make sure your patches are checkpatch.pl-clean, with the exception of the 80-column warnings and any const-related warnings related to the clock framework. > +static int ti_dt_clk_regmap_index; > + > /** > * ti_dt_clocks_register - register DT alias clocks during boot > * @oclks: list of clocks to register > @@ -53,3 +58,96 @@ void __init ti_dt_clocks_register(struct ti_dt_clk oclks[]) > } > } > } > + > +typedef int (*ti_of_clk_init_cb_t)(struct device_node *); Normally typedefs should be a red flag to reviewers due to Documentation/CodingStyle chapter 5. Still it seems this patch is just duplicating the way that the CCF does it, so I'm not too worried about it. > + > +struct clk_init_item { > + int index; > + struct device_node *np; > + ti_of_clk_init_cb_t init_cb; > + struct list_head node; > +}; > + > +static LIST_HEAD(retry_list); > + > +/** > + * ti_clk_get_reg_addr - get register address for a clock register > + * @node: device node for the clock > + * @index: register index from the clock node > + * > + * Builds clock register address from device tree information. This > + * is a struct of type clk_omap_reg. > + */ > +void __iomem *ti_clk_get_reg_addr(struct device_node *node, int index) > +{ > + struct clk_omap_reg *reg; > + u32 val; > + u32 tmp; > + > + reg = (struct clk_omap_reg *)&tmp; > + reg->index = ti_dt_clk_regmap_index; > + > + if (of_property_read_u32_index(node, "reg", index, &val)) { > + pr_err("%s must have reg[%d]!\n", node->name, index); > + return NULL; > + } > + > + reg->offset = val; > + > + return (void __iomem *)tmp; > +} > + > +/** > + * ti_dt_clk_init_provider - init master clock provider > + * @parent: master node > + * @index: internal index for clk_reg_ops > + * > + * Initializes a master clock IP block and its child clock nodes. > + * Regmap is provided for accessing the register space for the > + * IP block and all the clocks under it. > + */ > +void ti_dt_clk_init_provider(struct device_node *parent, int index) > +{ > + const struct of_device_id *match; > + struct device_node *np; > + ti_of_clk_init_cb_t clk_init_cb; > + struct clk_init_item *retry; > + struct clk_init_item *tmp; > + int ret; > + > + ti_dt_clk_regmap_index = index; > + > + for_each_child_of_node(parent, np) { > + match = of_match_node(__clk_of_table, np); > + if (!match) > + continue; > + clk_init_cb = (ti_of_clk_init_cb_t)match->data; > + pr_debug("%s: initializing: %s\n", __func__, np->name); > + ret = clk_init_cb(np); > + if (ret == -EAGAIN) { > + pr_debug("%s: adding to again list...\n", np->name); > + retry = kzalloc(sizeof(*retry), GFP_KERNEL); > + retry->np = np; > + retry->init_cb = clk_init_cb; > + list_add(&retry->node, &retry_list); > + } else if (ret) { > + pr_err("%s: clock init failed for %s (%d)!\n", __func__, > + np->name, ret); > + } > + } > + > + list_for_each_entry_safe(retry, tmp, &retry_list, node) { > + pr_debug("%s: retry-init: %s\n", __func__, retry->np->name); > + ti_dt_clk_regmap_index = retry->index; > + ret = retry->init_cb(retry->np); > + if (ret == -EAGAIN) { > + pr_debug("%s failed again?\n", retry->np->name); This is presumably a serious error condition and should be a pr_warn() or pr_err(), right? If retry_list won't be walked again, then it seems best to delete and free the list_entry no matter what the return value is from retry->init_cb(), since it's not like it will be retried. Otherwise the code will leak memory. > + } else { > + if (ret) > + pr_err("%s: clock init failed for %s (%d)!\n", > + __func__, retry->np->name, ret); > + list_del(&retry->node); > + kfree(retry); > + } > + } > +} > diff --git a/include/linux/clk/ti.h b/include/linux/clk/ti.h > index df94c24..d6bf530 100644 > --- a/include/linux/clk/ti.h > +++ b/include/linux/clk/ti.h > @@ -36,7 +36,21 @@ struct ti_dt_clk { > .node_name = name, \ > } > > +/* Maximum number of clock regmaps */ > +#define CLK_MAX_REGMAPS 4 > > +/** > + * struct clk_omap_reg - OMAP register declaration > + * @offset: offset from the master IP module base address > + * @index: index of the master IP module > + */ > +struct clk_omap_reg { > + u16 offset; > + u16 index; > +}; > + > +void __iomem *ti_clk_get_reg_addr(struct device_node *node, int index); > void ti_dt_clocks_register(struct ti_dt_clk *oclks); > +void ti_dt_clk_init_provider(struct device_node *np, int index); > > #endif > -- > 1.7.9.5 > - Paul
next prev parent reply other threads:[~2013-12-17 8:14 UTC|newest] Thread overview: 162+ messages / expand[flat|nested] mbox.gz Atom feed top 2013-11-26 8:05 [PATCHv10 00/41] ARM: TI SoC clock DT conversion Tero Kristo 2013-11-26 8:05 ` Tero Kristo 2013-11-26 8:05 ` [PATCHv10 01/41] clk: add support for platform specific clock I/O wrapper functions Tero Kristo 2013-11-26 8:05 ` Tero Kristo 2013-12-15 0:48 ` Mike Turquette 2013-12-15 0:48 ` Mike Turquette 2013-12-16 8:06 ` Tero Kristo 2013-12-16 8:06 ` Tero Kristo 2013-12-17 12:34 ` Paul Walmsley 2013-12-17 12:34 ` Paul Walmsley 2013-12-18 3:33 ` Paul Walmsley 2013-12-18 3:33 ` Paul Walmsley 2013-11-26 8:05 ` [PATCHv10 02/41] CLK: TI: add DT alias clock registration mechanism Tero Kristo 2013-11-26 8:05 ` Tero Kristo 2013-11-26 8:05 ` [PATCHv10 03/41] CLK: ti: add init support for clock IP blocks Tero Kristo 2013-11-26 8:05 ` Tero Kristo 2013-12-17 8:14 ` Paul Walmsley [this message] 2013-12-17 8:14 ` Paul Walmsley 2013-12-17 8:21 ` Tero Kristo 2013-12-17 8:21 ` Tero Kristo 2013-12-17 8:32 ` Paul Walmsley 2013-12-17 8:32 ` Paul Walmsley 2013-11-26 8:05 ` [PATCHv10 04/41] CLK: TI: Add DPLL clock support Tero Kristo 2013-11-26 8:05 ` Tero Kristo 2013-12-17 8:37 ` Paul Walmsley 2013-12-17 8:37 ` Paul Walmsley 2013-12-17 8:40 ` Paul Walmsley 2013-12-17 8:40 ` Paul Walmsley 2013-11-26 8:05 ` [PATCHv10 05/41] CLK: TI: add autoidle support Tero Kristo 2013-11-26 8:05 ` Tero Kristo 2013-11-26 8:05 ` [PATCHv10 06/41] clk: ti: add composite clock support Tero Kristo 2013-11-26 8:05 ` Tero Kristo 2013-11-26 8:05 ` [PATCHv10 07/41] CLK: ti: add support for ti divider-clock Tero Kristo 2013-11-26 8:05 ` Tero Kristo 2013-11-26 8:05 ` [PATCHv10 08/41] clk: ti: add support for TI fixed factor clock Tero Kristo 2013-11-26 8:05 ` Tero Kristo 2013-11-26 8:05 ` [PATCHv10 09/41] CLK: TI: add support for gate clock Tero Kristo 2013-11-26 8:05 ` Tero Kristo 2013-11-26 8:05 ` [PATCHv10 11/41] clk: ti: add support for basic mux clock Tero Kristo 2013-11-26 8:05 ` Tero Kristo 2013-11-26 8:05 ` [PATCHv10 12/41] CLK: TI: add omap4 clock init file Tero Kristo 2013-11-26 8:05 ` Tero Kristo 2013-12-17 9:30 ` Paul Walmsley 2013-12-17 9:30 ` Paul Walmsley 2013-11-26 8:05 ` [PATCHv10 14/41] CLK: TI: omap5: Initialize USB_DPLL at boot Tero Kristo 2013-11-26 8:05 ` Tero Kristo 2013-11-26 8:05 ` [PATCHv10 15/41] CLK: TI: DRA7: Add APLL support Tero Kristo 2013-11-26 8:05 ` Tero Kristo 2013-11-26 8:51 ` Alexander Aring 2013-11-26 8:51 ` Alexander Aring 2013-11-29 19:00 ` Tero Kristo 2013-11-29 19:00 ` Tero Kristo 2013-11-29 20:52 ` Alexander Aring 2013-11-29 20:52 ` Alexander Aring 2013-11-26 8:05 ` [PATCHv10 16/41] CLK: TI: add dra7 clock init file Tero Kristo 2013-11-26 8:05 ` Tero Kristo 2013-11-26 8:05 ` [PATCHv10 17/41] CLK: TI: add am33xx " Tero Kristo 2013-11-26 8:05 ` Tero Kristo 2013-11-26 8:06 ` [PATCHv10 19/41] CLK: TI: add omap3 " Tero Kristo 2013-11-26 8:06 ` Tero Kristo 2013-11-26 8:06 ` [PATCHv10 20/41] CLK: TI: add am43xx " Tero Kristo 2013-11-26 8:06 ` Tero Kristo 2013-11-26 8:06 ` [PATCHv10 21/41] ARM: dts: omap4 clock data Tero Kristo 2013-11-26 8:06 ` Tero Kristo 2013-12-17 9:44 ` Paul Walmsley 2013-12-17 9:44 ` Paul Walmsley 2013-12-17 9:57 ` Tero Kristo 2013-12-17 9:57 ` Tero Kristo 2013-12-20 11:15 ` Paul Walmsley 2013-12-20 11:15 ` Paul Walmsley 2013-11-26 8:06 ` [PATCHv10 23/41] ARM: dts: dra7 " Tero Kristo 2013-11-26 8:06 ` Tero Kristo 2013-12-17 9:46 ` Paul Walmsley 2013-12-17 9:46 ` Paul Walmsley 2013-11-26 8:06 ` [PATCHv10 24/41] ARM: dts: clk: Add apll related clocks Tero Kristo 2013-11-26 8:06 ` Tero Kristo 2013-11-26 8:06 ` [PATCHv10 26/41] ARM: dts: DRA7: Add PCIe related clock nodes Tero Kristo 2013-11-26 8:06 ` Tero Kristo 2013-11-26 8:06 ` [PATCHv10 27/41] ARM: dts: am33xx clock data Tero Kristo 2013-11-26 8:06 ` Tero Kristo 2013-12-17 9:48 ` Paul Walmsley 2013-12-17 9:48 ` Paul Walmsley 2013-11-26 8:06 ` [PATCHv10 28/41] ARM: dts: omap3 " Tero Kristo 2013-11-26 8:06 ` Tero Kristo 2013-12-17 9:50 ` Paul Walmsley 2013-12-17 9:50 ` Paul Walmsley 2013-11-26 8:06 ` [PATCHv10 29/41] ARM: dts: AM35xx: use DT " Tero Kristo 2013-11-26 8:06 ` Tero Kristo 2013-11-26 8:06 ` [PATCHv10 30/41] ARM: dts: am43xx " Tero Kristo 2013-11-26 8:06 ` Tero Kristo 2013-12-17 9:52 ` Paul Walmsley 2013-12-17 9:52 ` Paul Walmsley 2013-11-26 8:06 ` [PATCHv10 31/41] ARM: OMAP2+: clock: add support for regmap Tero Kristo 2013-11-26 8:06 ` Tero Kristo 2013-11-26 17:40 ` Tony Lindgren 2013-11-26 17:40 ` Tony Lindgren 2013-11-27 9:08 ` Tero Kristo 2013-11-27 9:08 ` Tero Kristo 2013-11-26 8:06 ` [PATCHv10 32/41] ARM: OMAP2+: clock: use driver API instead of direct memory read/write Tero Kristo 2013-11-26 8:06 ` Tero Kristo 2013-11-26 8:06 ` [PATCHv10 33/41] ARM: OMAP: hwmod: fix an incorrect clk type cast with _get_clkdm Tero Kristo 2013-11-26 8:06 ` Tero Kristo 2013-11-26 8:06 ` [PATCHv10 34/41] ARM: OMAP3: hwmod: initialize clkdm from clkdm_name Tero Kristo 2013-11-26 8:06 ` Tero Kristo 2013-11-26 8:06 ` [PATCHv10 35/41] ARM: OMAP2+: PRM: add support for initializing PRCM clock modules from DT Tero Kristo 2013-11-26 8:06 ` Tero Kristo [not found] ` <1385453182-24421-1-git-send-email-t-kristo-l0cyMroinI0@public.gmane.org> 2013-11-26 8:05 ` [PATCHv10 10/41] CLK: TI: add support for clockdomain binding Tero Kristo 2013-11-26 8:05 ` Tero Kristo 2013-12-15 4:23 ` Mike Turquette 2013-12-15 4:23 ` Mike Turquette 2013-12-16 8:13 ` Tero Kristo 2013-12-16 8:13 ` Tero Kristo 2013-12-18 3:07 ` Mike Turquette 2013-12-18 3:07 ` Mike Turquette 2013-11-26 8:05 ` [PATCHv10 13/41] CLK: TI: add omap5 clock init file Tero Kristo 2013-11-26 8:05 ` Tero Kristo 2013-11-26 8:05 ` [PATCHv10 18/41] CLK: TI: add interface clock support for OMAP3 Tero Kristo 2013-11-26 8:05 ` Tero Kristo 2013-11-26 8:06 ` [PATCHv10 22/41] ARM: dts: omap5 clock data Tero Kristo 2013-11-26 8:06 ` Tero Kristo 2013-12-16 10:51 ` Paul Walmsley 2013-12-16 10:51 ` Paul Walmsley 2013-12-16 10:57 ` Tero Kristo 2013-12-16 10:57 ` Tero Kristo 2013-12-17 9:46 ` Paul Walmsley 2013-12-17 9:46 ` Paul Walmsley 2013-11-26 8:06 ` [PATCHv10 25/41] ARM: dts: DRA7: Change apll_pcie_m2_ck to fixed factor clock Tero Kristo 2013-11-26 8:06 ` Tero Kristo 2013-11-26 8:06 ` [PATCHv10 36/41] ARM: OMAP2+: io: use new clock init API Tero Kristo 2013-11-26 8:06 ` Tero Kristo 2013-11-28 0:49 ` [PATCHv10 00/41] ARM: TI SoC clock DT conversion Nishanth Menon 2013-11-28 0:49 ` Nishanth Menon [not found] ` <52969313.6090207-l0cyMroinI0@public.gmane.org> 2013-11-28 18:58 ` Paul Walmsley 2013-11-28 18:58 ` Paul Walmsley 2013-11-29 17:12 ` Tony Lindgren 2013-11-29 17:12 ` Tony Lindgren 2013-11-29 18:59 ` Tero Kristo 2013-11-29 18:59 ` Tero Kristo 2013-11-26 8:06 ` [PATCHv10 37/41] ARM: OMAP4: remove old clock data and link in new clock init code Tero Kristo 2013-11-26 8:06 ` Tero Kristo 2013-11-26 8:06 ` [PATCHv10 38/41] ARM: OMAP: DRA7: Enable clock init Tero Kristo 2013-11-26 8:06 ` Tero Kristo 2013-11-26 8:06 ` [PATCHv10 39/41] ARM: AM43xx: " Tero Kristo 2013-11-26 8:06 ` Tero Kristo 2013-11-26 8:06 ` [PATCHv10 40/41] ARM: AM33xx: remove old clock data and link in new clock init code Tero Kristo 2013-11-26 8:06 ` Tero Kristo 2013-11-26 8:06 ` [PATCHv10 41/41] ARM: OMAP3: use DT clock init if DT data is available Tero Kristo 2013-11-26 8:06 ` Tero Kristo 2013-11-26 17:44 ` Tony Lindgren 2013-11-26 17:44 ` Tony Lindgren 2013-11-27 9:06 ` Tero Kristo 2013-11-27 9:06 ` Tero Kristo 2013-11-26 17:57 ` [PATCHv10 00/41] ARM: TI SoC clock DT conversion Tony Lindgren 2013-11-26 17:57 ` Tony Lindgren 2013-12-15 0:51 ` Mike Turquette 2013-12-15 0:51 ` Mike Turquette 2013-12-15 4:35 ` Mike Turquette 2013-12-15 4:35 ` Mike Turquette 2013-12-16 8:12 ` Tero Kristo 2013-12-16 8:12 ` Tero Kristo 2013-12-20 16:10 ` Felipe Balbi 2013-12-20 16:10 ` Felipe Balbi
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=alpine.DEB.2.02.1312170745360.14039@utopia.booyaka.com \ --to=paul@pwsan.com \ --cc=bcousson@baylibre.com \ --cc=devicetree@vger.kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-omap@vger.kernel.org \ --cc=mturquette@linaro.org \ --cc=nm@ti.com \ --cc=rnayak@ti.com \ --cc=t-kristo@ti.com \ --cc=tony@atomide.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: linkBe 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.