diff for duplicates of <1486666949.2726.99.camel@synopsys.com> diff --git a/a/1.txt b/N1/1.txt index ceaf74f..afed9e6 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,10 +1,10 @@ Hi Florian, -On Wed, 2017-02-08@10:20 -0800, Florian Fainelli wrote: +On Wed, 2017-02-08 at 10:20 -0800, Florian Fainelli wrote: > On 02/08/2017 10:15 AM, David Miller wrote: > > -> > From: Alexey Brodkin <Alexey.Brodkin at synopsys.com> -> > Date: Mon,??6 Feb 2017 22:24:45 +0300 +> > From: Alexey Brodkin <Alexey.Brodkin@synopsys.com> +> > Date: Mon, 6 Feb 2017 22:24:45 +0300 > > > > > > > > Given there're default values mentioned in the PHY datasheet @@ -14,7 +14,7 @@ On Wed, 2017-02-08@10:20 -0800, Florian Fainelli wrote: > > > This allows to use minimalistic description in DT if no special > > > features are required. > > > -> > > Signed-off-by: Alexey Brodkin <abrodkin at synopsys.com> +> > > Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com> > > > > If these defaults are legitimate to use, then you should probably also > > set them in the non-CONFIG_OF_MDIO case implementation of @@ -30,7 +30,7 @@ On Wed, 2017-02-08@10:20 -0800, Florian Fainelli wrote: > dp83867_of_init() whose job is to set default values, if none have been > previously set through Device Tree. -But why do we need to return error code from?dp83867_of_init()? +But why do we need to return error code from dp83867_of_init()? The point is this function doesn't do any hardware setup as well, in fact it doesn't even reads anything from real hardware. diff --git a/a/content_digest b/N1/content_digest index 46de684..d664500 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,19 +1,28 @@ "ref\020170206192445.35829-1-abrodkin@synopsys.com\0" "ref\020170208.131519.1313210815958231832.davem@davemloft.net\0" "ref\0360cfba0-cc84-4378-025d-34e5eb1fc2c8@gmail.com\0" - "From\0Alexey.Brodkin@synopsys.com (Alexey Brodkin)\0" - "Subject\0[PATCH] net: phy: dp83867: Fall-back to default values of clock delay and FIFO depth\0" + "From\0Alexey Brodkin <Alexey.Brodkin@synopsys.com>\0" + "Subject\0Re: [PATCH] net: phy: dp83867: Fall-back to default values of clock delay and FIFO depth\0" "Date\0Thu, 9 Feb 2017 19:02:30 +0000\0" - "To\0linux-snps-arc@lists.infradead.org\0" + "To\0f.fainelli@gmail.com <f.fainelli@gmail.com>\0" + "Cc\0andrew@lunn.ch <andrew@lunn.ch>" + linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org> + nsekhar@ti.com <nsekhar@ti.com> + m-karicheri2@ti.com <m-karicheri2@ti.com> + mugunthanvnm@ti.com <mugunthanvnm@ti.com> + linux-snps-arc@lists.infradead.org <linux-snps-arc@lists.infradead.org> + grygorii.strashko@ti.com <grygorii.strashko@ti.com> + netdev@vger.kernel.org <netdev@vger.kernel.org> + " davem@davemloft.net <davem@davemloft.net>\0" "\00:1\0" "b\0" "Hi Florian,\n" "\n" - "On Wed, 2017-02-08@10:20 -0800, Florian Fainelli wrote:\n" + "On Wed, 2017-02-08 at 10:20 -0800, Florian Fainelli wrote:\n" "> On 02/08/2017 10:15 AM, David Miller wrote:\n" "> > \n" - "> > From: Alexey Brodkin <Alexey.Brodkin at synopsys.com>\n" - "> > Date: Mon,??6 Feb 2017 22:24:45 +0300\n" + "> > From: Alexey Brodkin <Alexey.Brodkin@synopsys.com>\n" + "> > Date: Mon,\302\240\302\2406 Feb 2017 22:24:45 +0300\n" "> > \n" "> > > \n" "> > > Given there're default values mentioned in the PHY datasheet\n" @@ -23,7 +32,7 @@ "> > > This allows to use minimalistic description in DT if no special\n" "> > > features are required.\n" "> > > \n" - "> > > Signed-off-by: Alexey Brodkin <abrodkin at synopsys.com>\n" + "> > > Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>\n" "> > \n" "> > If these defaults are legitimate to use, then you should probably also\n" "> > set them in the non-CONFIG_OF_MDIO case implementation of\n" @@ -39,7 +48,7 @@ "> dp83867_of_init() whose job is to set default values, if none have been\n" "> previously set through Device Tree.\n" "\n" - "But why do we need to return error code from?dp83867_of_init()?\n" + "But why do we need to return error code from\302\240dp83867_of_init()?\n" "The point is this function doesn't do any hardware setup as well,\n" "in fact it doesn't even reads anything from real hardware.\n" "\n" @@ -57,4 +66,4 @@ "\n" -Alexey -239c61978c0a32db2060409b428dcc2a329503090b2b0c131f88eb2d521a719f +212ad26a85ea6e8684da00496e24c627983fc4fa8eef92e264d6057ebf633b6f
diff --git a/a/1.txt b/N2/1.txt index ceaf74f..09287ef 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -1,10 +1,10 @@ Hi Florian, -On Wed, 2017-02-08@10:20 -0800, Florian Fainelli wrote: +On Wed, 2017-02-08 at 10:20 -0800, Florian Fainelli wrote: > On 02/08/2017 10:15 AM, David Miller wrote: > > -> > From: Alexey Brodkin <Alexey.Brodkin at synopsys.com> -> > Date: Mon,??6 Feb 2017 22:24:45 +0300 +> > From: Alexey Brodkin <Alexey.Brodkin@synopsys.com> +> > Date: Mon, 6 Feb 2017 22:24:45 +0300 > > > > > > > > Given there're default values mentioned in the PHY datasheet @@ -14,7 +14,7 @@ On Wed, 2017-02-08@10:20 -0800, Florian Fainelli wrote: > > > This allows to use minimalistic description in DT if no special > > > features are required. > > > -> > > Signed-off-by: Alexey Brodkin <abrodkin at synopsys.com> +> > > Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com> > > > > If these defaults are legitimate to use, then you should probably also > > set them in the non-CONFIG_OF_MDIO case implementation of @@ -30,7 +30,7 @@ On Wed, 2017-02-08@10:20 -0800, Florian Fainelli wrote: > dp83867_of_init() whose job is to set default values, if none have been > previously set through Device Tree. -But why do we need to return error code from?dp83867_of_init()? +But why do we need to return error code from dp83867_of_init()? The point is this function doesn't do any hardware setup as well, in fact it doesn't even reads anything from real hardware. @@ -47,3 +47,7 @@ I agree with David that it really worth to do the same settings for non_CONFIG_O case. -Alexey +_______________________________________________ +linux-snps-arc mailing list +linux-snps-arc@lists.infradead.org +http://lists.infradead.org/mailman/listinfo/linux-snps-arc diff --git a/a/content_digest b/N2/content_digest index 46de684..ecd486a 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -1,19 +1,28 @@ "ref\020170206192445.35829-1-abrodkin@synopsys.com\0" "ref\020170208.131519.1313210815958231832.davem@davemloft.net\0" "ref\0360cfba0-cc84-4378-025d-34e5eb1fc2c8@gmail.com\0" - "From\0Alexey.Brodkin@synopsys.com (Alexey Brodkin)\0" - "Subject\0[PATCH] net: phy: dp83867: Fall-back to default values of clock delay and FIFO depth\0" + "From\0Alexey Brodkin <Alexey.Brodkin@synopsys.com>\0" + "Subject\0Re: [PATCH] net: phy: dp83867: Fall-back to default values of clock delay and FIFO depth\0" "Date\0Thu, 9 Feb 2017 19:02:30 +0000\0" - "To\0linux-snps-arc@lists.infradead.org\0" + "To\0f.fainelli@gmail.com <f.fainelli@gmail.com>\0" + "Cc\0andrew@lunn.ch <andrew@lunn.ch>" + grygorii.strashko@ti.com <grygorii.strashko@ti.com> + mugunthanvnm@ti.com <mugunthanvnm@ti.com> + netdev@vger.kernel.org <netdev@vger.kernel.org> + nsekhar@ti.com <nsekhar@ti.com> + linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org> + m-karicheri2@ti.com <m-karicheri2@ti.com> + linux-snps-arc@lists.infradead.org <linux-snps-arc@lists.infradead.org> + " davem@davemloft.net <davem@davemloft.net>\0" "\00:1\0" "b\0" "Hi Florian,\n" "\n" - "On Wed, 2017-02-08@10:20 -0800, Florian Fainelli wrote:\n" + "On Wed, 2017-02-08 at 10:20 -0800, Florian Fainelli wrote:\n" "> On 02/08/2017 10:15 AM, David Miller wrote:\n" "> > \n" - "> > From: Alexey Brodkin <Alexey.Brodkin at synopsys.com>\n" - "> > Date: Mon,??6 Feb 2017 22:24:45 +0300\n" + "> > From: Alexey Brodkin <Alexey.Brodkin@synopsys.com>\n" + "> > Date: Mon,\302\240\302\2406 Feb 2017 22:24:45 +0300\n" "> > \n" "> > > \n" "> > > Given there're default values mentioned in the PHY datasheet\n" @@ -23,7 +32,7 @@ "> > > This allows to use minimalistic description in DT if no special\n" "> > > features are required.\n" "> > > \n" - "> > > Signed-off-by: Alexey Brodkin <abrodkin at synopsys.com>\n" + "> > > Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>\n" "> > \n" "> > If these defaults are legitimate to use, then you should probably also\n" "> > set them in the non-CONFIG_OF_MDIO case implementation of\n" @@ -39,7 +48,7 @@ "> dp83867_of_init() whose job is to set default values, if none have been\n" "> previously set through Device Tree.\n" "\n" - "But why do we need to return error code from?dp83867_of_init()?\n" + "But why do we need to return error code from\302\240dp83867_of_init()?\n" "The point is this function doesn't do any hardware setup as well,\n" "in fact it doesn't even reads anything from real hardware.\n" "\n" @@ -55,6 +64,10 @@ "I agree with David that it really worth to do the same settings for non_CONFIG_OF_MDIO\n" "case.\n" "\n" - -Alexey + "-Alexey\n" + "_______________________________________________\n" + "linux-snps-arc mailing list\n" + "linux-snps-arc@lists.infradead.org\n" + http://lists.infradead.org/mailman/listinfo/linux-snps-arc -239c61978c0a32db2060409b428dcc2a329503090b2b0c131f88eb2d521a719f +85a3641c5226826ca4c4319b9e088f5d0420ecec953885a8250210e244594185
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.