diff for duplicates of <1460553825.6620.134.camel@linux.intel.com> diff --git a/a/1.txt b/N1/1.txt index 23088fb..1e335d3 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,7 +1,7 @@ On Wed, 2016-04-13 at 20:16 +0800, Jisheng Zhang wrote: > Dear Andy, > -> On Thu, 7 Apr 2016 21:05:35 +0300 Andy Shevchenko wrote: +> On Thu, 7 Apr 2016 21:05:35 +0300 Andy Shevchenko??wrote: > > > > > On Wed, Apr 6, 2016 at 10:28 AM, Jisheng Zhang <jszhang@marvell.com> @@ -20,7 +20,7 @@ On Wed, 2016-04-13 at 20:16 +0800, Jisheng Zhang wrote: > > > get hcnt/lcnt values rather than the sda/scl falling time from > > > ACPI > > > method, we want similar feature for DT platforms. -> > > +> > > ? > > Instead of duplicating some words maybe better to explicitly define > > two groups of parameters and one which supersedes the other. > I'm sorry, I can't catch your meaning. Could you please kindly give @@ -30,12 +30,12 @@ On Wed, 2016-04-13 at 20:16 +0800, Jisheng Zhang wrote: > --- + - i2c-ss-scl-high-count : should contain the standard speed i2c > > > clock SCL high -> > > + count. +> > > +???count. > > > If defined, the i2c-scl-falling-time-ns and i2c-sda-falling-time- > > > ns -> > > + will be ignored. +> > > +???will be ignored. I'm referring to the above sentence. Something like @@ -50,25 +50,25 @@ The second group of parameters supersedes the first one. > > > + > > > + - i2c-ss-scl-low-count : should contain the standard speed i2c > > > clock SCL low -> > > + count. If defined, the i2c-scl-falling-time-ns and i2c-sda- +> > > +???count. If defined, the i2c-scl-falling-time-ns and i2c-sda- > > > falling-time-ns -> > > + will be ignored. +> > > +???will be ignored. > > > + > > > + - i2c-fs-scl-high-count : should contain the fast speed i2c > > > clock SCL high -> > > + count. If defined, the i2c-scl-falling-time-ns and i2c-sda- +> > > +???count. If defined, the i2c-scl-falling-time-ns and i2c-sda- > > > falling-time-ns -> > > + will be ignored. +> > > +???will be ignored. > > > + > > > + - i2c-fs-scl-low-count : should contain the fast speed i2c clock > > > SCL low -> > > + count. If defined, the i2c-scl-falling-time-ns and i2c-sda- +> > > +???count. If defined, the i2c-scl-falling-time-ns and i2c-sda- > > > falling-time-ns -> > > + will be ignored. +> > > +???will be ignored. > > > + -> > > Example : +> > > ?Example : > > > -> > > i2c@f0000 { +> > > ????????i2c at f0000 { > > > diff --git a/drivers/i2c/busses/i2c-designware-platdrv.c > > > b/drivers/i2c/busses/i2c-designware-platdrv.c > > > index d656657..8739a60 100644 @@ -76,28 +76,28 @@ The second group of parameters supersedes the first one. > > > +++ b/drivers/i2c/busses/i2c-designware-platdrv.c > > > @@ -188,6 +188,14 @@ static int dw_i2c_plat_probe(struct > > > platform_device *pdev) -> > > &dev->scl_falling_time); -> > > device_property_read_u32(&pdev->dev, "clock- +> > > ?????????????????????????????????????????&dev->scl_falling_time); +> > > ????????????????device_property_read_u32(&pdev->dev, "clock- > > > frequency", -> > > &clk_freq); -> > > + device_property_read_u16(&pdev->dev, "i2c-ss-scl- +> > > ?????????????????????????????????????????&clk_freq); +> > > +???????????????device_property_read_u16(&pdev->dev, "i2c-ss-scl- > > > high-count", -> > > + &dev->ss_hcnt); -> > > + device_property_read_u16(&pdev->dev, "i2c-ss-scl- +> > > +????????????????????????????????????????&dev->ss_hcnt); +> > > +???????????????device_property_read_u16(&pdev->dev, "i2c-ss-scl- > > > low-count", -> > > + &dev->ss_lcnt); -> > > + device_property_read_u16(&pdev->dev, "i2c-fs-scl- +> > > +????????????????????????????????????????&dev->ss_lcnt); +> > > +???????????????device_property_read_u16(&pdev->dev, "i2c-fs-scl- > > > high-count", -> > > + &dev->fs_hcnt); -> > > + device_property_read_u16(&pdev->dev, "i2c-fs-scl- +> > > +????????????????????????????????????????&dev->fs_hcnt); +> > > +???????????????device_property_read_u16(&pdev->dev, "i2c-fs-scl- > > > low-count", -> > > + &dev->fs_lcnt); -> > > } +> > > +????????????????????????????????????????&dev->fs_lcnt); +> > > ????????} > > > -> > > if (has_acpi_companion(&pdev->dev)) +> > > ????????if (has_acpi_companion(&pdev->dev)) > > > -- > > > 2.8.0.rc3 -> > > +> > > ? > > > > diff --git a/a/content_digest b/N1/content_digest index 1dfa9e1..8deed60 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,29 +1,16 @@ "ref\01459927680-5480-1-git-send-email-jszhang@marvell.com\0" "ref\0CAHp75VcHNdAUeob2iR6U1HbUQzEccdjPy8ffuG4mB_=ZX7EokQ@mail.gmail.com\0" "ref\020160413201637.284a3673@xhacker\0" - "From\0Andy Shevchenko <andriy.shevchenko@linux.intel.com>\0" - "Subject\0Re: [PATCH] i2c: designware-platdrv: get fast/std speed scl high/low count from DT\0" + "From\0andriy.shevchenko@linux.intel.com (Andy Shevchenko)\0" + "Subject\0[PATCH] i2c: designware-platdrv: get fast/std speed scl high/low count from DT\0" "Date\0Wed, 13 Apr 2016 16:23:45 +0300\0" - "To\0Jisheng Zhang <jszhang@marvell.com>" - " Andy Shevchenko <andy.shevchenko@gmail.com>\0" - "Cc\0Wolfram Sang <wsa@the-dreams.de>" - Rob Herring <robh+dt@kernel.org> - Pawel Moll <pawel.moll@arm.com> - Mark Rutland <mark.rutland@arm.com> - ijc+devicetree <ijc+devicetree@hellion.org.uk> - Kumar Gala <galak@codeaurora.org> - Jarkko Nikula <jarkko.nikula@linux.intel.com> - Mika Westerberg <mika.westerberg@linux.intel.com> - linux-i2c@vger.kernel.org - devicetree <devicetree@vger.kernel.org> - linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org> - " linux-arm Mailing List <linux-arm-kernel@lists.infradead.org>\0" + "To\0linux-arm-kernel@lists.infradead.org\0" "\00:1\0" "b\0" "On Wed, 2016-04-13 at 20:16 +0800, Jisheng Zhang wrote:\n" "> Dear Andy,\n" "> \n" - "> On Thu, 7 Apr 2016 21:05:35 +0300 Andy Shevchenko\302\240\302\240wrote:\n" + "> On Thu, 7 Apr 2016 21:05:35 +0300 Andy Shevchenko??wrote:\n" "> \n" "> > \n" "> > On Wed, Apr 6, 2016 at 10:28 AM, Jisheng Zhang <jszhang@marvell.com>\n" @@ -42,7 +29,7 @@ "> > > get hcnt/lcnt values rather than the sda/scl falling time from\n" "> > > ACPI\n" "> > > method, we want similar feature for DT platforms.\n" - "> > > \302\240\n" + "> > > ?\n" "> > Instead of duplicating some words maybe better to explicitly define\n" "> > two groups of parameters and one which supersedes the other.\n" "> I'm sorry, I can't catch your meaning. Could you please kindly give\n" @@ -52,12 +39,12 @@ "\n" "> --- + - i2c-ss-scl-high-count : should contain the standard speed i2c\n" "> > > clock SCL high\n" - "> > > +\302\240\302\240\302\240count. \n" + "> > > +???count. \n" "\n" "\n" "> > > If defined, the i2c-scl-falling-time-ns and i2c-sda-falling-time-\n" "> > > ns\n" - "> > > +\302\240\302\240\302\240will be ignored.\n" + "> > > +???will be ignored.\n" "\n" "I'm referring to the above sentence. Something like\n" "\n" @@ -72,25 +59,25 @@ "> > > +\n" "> > > + - i2c-ss-scl-low-count : should contain the standard speed i2c\n" "> > > clock SCL low\n" - "> > > +\302\240\302\240\302\240count. If defined, the i2c-scl-falling-time-ns and i2c-sda-\n" + "> > > +???count. If defined, the i2c-scl-falling-time-ns and i2c-sda-\n" "> > > falling-time-ns\n" - "> > > +\302\240\302\240\302\240will be ignored.\n" + "> > > +???will be ignored.\n" "> > > +\n" "> > > + - i2c-fs-scl-high-count : should contain the fast speed i2c\n" "> > > clock SCL high\n" - "> > > +\302\240\302\240\302\240count. If defined, the i2c-scl-falling-time-ns and i2c-sda-\n" + "> > > +???count. If defined, the i2c-scl-falling-time-ns and i2c-sda-\n" "> > > falling-time-ns\n" - "> > > +\302\240\302\240\302\240will be ignored.\n" + "> > > +???will be ignored.\n" "> > > +\n" "> > > + - i2c-fs-scl-low-count : should contain the fast speed i2c clock\n" "> > > SCL low\n" - "> > > +\302\240\302\240\302\240count. If defined, the i2c-scl-falling-time-ns and i2c-sda-\n" + "> > > +???count. If defined, the i2c-scl-falling-time-ns and i2c-sda-\n" "> > > falling-time-ns\n" - "> > > +\302\240\302\240\302\240will be ignored.\n" + "> > > +???will be ignored.\n" "> > > +\n" - "> > > \302\240Example :\n" + "> > > ?Example :\n" "> > > \n" - "> > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240i2c@f0000 {\n" + "> > > ????????i2c at f0000 {\n" "> > > diff --git a/drivers/i2c/busses/i2c-designware-platdrv.c\n" "> > > b/drivers/i2c/busses/i2c-designware-platdrv.c\n" "> > > index d656657..8739a60 100644\n" @@ -98,28 +85,28 @@ "> > > +++ b/drivers/i2c/busses/i2c-designware-platdrv.c\n" "> > > @@ -188,6 +188,14 @@ static int dw_i2c_plat_probe(struct\n" "> > > platform_device *pdev)\n" - "> > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240&dev->scl_falling_time);\n" - "> > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240device_property_read_u32(&pdev->dev, \"clock-\n" + "> > > ?????????????????????????????????????????&dev->scl_falling_time);\n" + "> > > ????????????????device_property_read_u32(&pdev->dev, \"clock-\n" "> > > frequency\",\n" - "> > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240&clk_freq);\n" - "> > > +\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240device_property_read_u16(&pdev->dev, \"i2c-ss-scl-\n" + "> > > ?????????????????????????????????????????&clk_freq);\n" + "> > > +???????????????device_property_read_u16(&pdev->dev, \"i2c-ss-scl-\n" "> > > high-count\",\n" - "> > > +\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240&dev->ss_hcnt);\n" - "> > > +\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240device_property_read_u16(&pdev->dev, \"i2c-ss-scl-\n" + "> > > +????????????????????????????????????????&dev->ss_hcnt);\n" + "> > > +???????????????device_property_read_u16(&pdev->dev, \"i2c-ss-scl-\n" "> > > low-count\",\n" - "> > > +\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240&dev->ss_lcnt);\n" - "> > > +\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240device_property_read_u16(&pdev->dev, \"i2c-fs-scl-\n" + "> > > +????????????????????????????????????????&dev->ss_lcnt);\n" + "> > > +???????????????device_property_read_u16(&pdev->dev, \"i2c-fs-scl-\n" "> > > high-count\",\n" - "> > > +\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240&dev->fs_hcnt);\n" - "> > > +\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240device_property_read_u16(&pdev->dev, \"i2c-fs-scl-\n" + "> > > +????????????????????????????????????????&dev->fs_hcnt);\n" + "> > > +???????????????device_property_read_u16(&pdev->dev, \"i2c-fs-scl-\n" "> > > low-count\",\n" - "> > > +\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240&dev->fs_lcnt);\n" - "> > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240}\n" + "> > > +????????????????????????????????????????&dev->fs_lcnt);\n" + "> > > ????????}\n" "> > > \n" - "> > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240if (has_acpi_companion(&pdev->dev))\n" + "> > > ????????if (has_acpi_companion(&pdev->dev))\n" "> > > --\n" "> > > 2.8.0.rc3\n" - "> > > \302\240\n" + "> > > ?\n" "> > \n" "> > \n" "\n" @@ -127,4 +114,4 @@ "Andy Shevchenko <andriy.shevchenko@linux.intel.com>\n" Intel Finland Oy -83d2ced72e3470f80a1887c6735c241a212146fc907db0df3e4f66dee21821bf +4720600dd03084d174cd11332f83e973487ab3a8397c46e5b47829d9fbb79784
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.