| Message ID | 8776296d079b3b4d67d4421656238757a8ad373d.1556046082.git.leonard.crestez@nxp.com (mailing list archive) |
|---|---|
| State | Accepted, archived |
| Headers | show |
| Series | clk: imx6sll: Fix mispelling uart4_serial as serail | expand |
On Tue, Apr 23, 2019 at 07:05:08PM +0000, Leonard Crestez wrote: > This looks like a copy-paste error. This string is not referenced > anywhere so it's safe to rename it. > > Signed-off-by: Leonard Crestez <leonard.crestez@nxp.com> I see this keep coming, so have to ask you to fix your patch sender. The base64 encoding makes it very difficult to apply patch. I have done manual applying for your patches a couple of times, and I thought that's just accident. It seems not. Please fix and resend. Shawn > --- > drivers/clk/imx/clk-imx6sll.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/clk/imx/clk-imx6sll.c b/drivers/clk/imx/clk-imx6sll.c > index 3bd2044cf25c..9def76df0879 100644 > --- a/drivers/clk/imx/clk-imx6sll.c > +++ b/drivers/clk/imx/clk-imx6sll.c > @@ -266,11 +266,11 @@ static void __init imx6sll_clocks_init(struct device_node *ccm_node) > clks[IMX6SLL_CLK_EPIT1] = imx_clk_gate2("epit1", "perclk", base + 0x6c, 12); > clks[IMX6SLL_CLK_EPIT2] = imx_clk_gate2("epit2", "perclk", base + 0x6c, 14); > clks[IMX6SLL_CLK_GPT_BUS] = imx_clk_gate2("gpt1_bus", "perclk", base + 0x6c, 20); > clks[IMX6SLL_CLK_GPT_SERIAL] = imx_clk_gate2("gpt1_serial", "perclk", base + 0x6c, 22); > clks[IMX6SLL_CLK_UART4_IPG] = imx_clk_gate2("uart4_ipg", "ipg", base + 0x6c, 24); > - clks[IMX6SLL_CLK_UART4_SERIAL] = imx_clk_gate2("uart4_serail", "uart_podf", base + 0x6c, 24); > + clks[IMX6SLL_CLK_UART4_SERIAL] = imx_clk_gate2("uart4_serial", "uart_podf", base + 0x6c, 24); > clks[IMX6SLL_CLK_GPIO1] = imx_clk_gate2("gpio1", "ipg", base + 0x6c, 26); > clks[IMX6SLL_CLK_GPIO5] = imx_clk_gate2("gpio5", "ipg", base + 0x6c, 30); > > /* CCGR2 */ > clks[IMX6SLL_CLK_GPIO6] = imx_clk_gate2("gpio6", "ipg", base + 0x70, 0); > -- > 2.17.1 >
Quoting Shawn Guo (2019-05-09 20:27:47) > On Tue, Apr 23, 2019 at 07:05:08PM +0000, Leonard Crestez wrote: > > This looks like a copy-paste error. This string is not referenced > > anywhere so it's safe to rename it. > > > > Signed-off-by: Leonard Crestez <leonard.crestez@nxp.com> > > I see this keep coming, so have to ask you to fix your patch sender. > The base64 encoding makes it very difficult to apply patch. I have done > manual applying for your patches a couple of times, and I thought that's > just accident. It seems not. Please fix and resend. > I already applied this one. Sorry, forgot to send the email.
On Fri, May 10, 2019 at 10:37:23AM -0700, Stephen Boyd wrote: > Quoting Shawn Guo (2019-05-09 20:27:47) > > On Tue, Apr 23, 2019 at 07:05:08PM +0000, Leonard Crestez wrote: > > > This looks like a copy-paste error. This string is not referenced > > > anywhere so it's safe to rename it. > > > > > > Signed-off-by: Leonard Crestez <leonard.crestez@nxp.com> > > > > I see this keep coming, so have to ask you to fix your patch sender. > > The base64 encoding makes it very difficult to apply patch. I have done > > manual applying for your patches a couple of times, and I thought that's > > just accident. It seems not. Please fix and resend. > > > > I already applied this one. Sorry, forgot to send the email. No problem. Thanks for notification. Shawn
diff --git a/drivers/clk/imx/clk-imx6sll.c b/drivers/clk/imx/clk-imx6sll.c index 3bd2044cf25c..9def76df0879 100644 --- a/drivers/clk/imx/clk-imx6sll.c +++ b/drivers/clk/imx/clk-imx6sll.c @@ -266,11 +266,11 @@ static void __init imx6sll_clocks_init(struct device_node *ccm_node) clks[IMX6SLL_CLK_EPIT1] = imx_clk_gate2("epit1", "perclk", base + 0x6c, 12); clks[IMX6SLL_CLK_EPIT2] = imx_clk_gate2("epit2", "perclk", base + 0x6c, 14); clks[IMX6SLL_CLK_GPT_BUS] = imx_clk_gate2("gpt1_bus", "perclk", base + 0x6c, 20); clks[IMX6SLL_CLK_GPT_SERIAL] = imx_clk_gate2("gpt1_serial", "perclk", base + 0x6c, 22); clks[IMX6SLL_CLK_UART4_IPG] = imx_clk_gate2("uart4_ipg", "ipg", base + 0x6c, 24); - clks[IMX6SLL_CLK_UART4_SERIAL] = imx_clk_gate2("uart4_serail", "uart_podf", base + 0x6c, 24); + clks[IMX6SLL_CLK_UART4_SERIAL] = imx_clk_gate2("uart4_serial", "uart_podf", base + 0x6c, 24); clks[IMX6SLL_CLK_GPIO1] = imx_clk_gate2("gpio1", "ipg", base + 0x6c, 26); clks[IMX6SLL_CLK_GPIO5] = imx_clk_gate2("gpio5", "ipg", base + 0x6c, 30); /* CCGR2 */ clks[IMX6SLL_CLK_GPIO6] = imx_clk_gate2("gpio6", "ipg", base + 0x70, 0);
This looks like a copy-paste error. This string is not referenced anywhere so it's safe to rename it. Signed-off-by: Leonard Crestez <leonard.crestez@nxp.com> --- drivers/clk/imx/clk-imx6sll.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)