diff for duplicates of <1501238415.29303.279.camel@linux.intel.com> diff --git a/a/1.txt b/N1/1.txt index 997169b..2eb73ae 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,6 +1,6 @@ On Fri, 2017-07-28 at 01:16 +0900, Masahiro Yamada wrote: > Several drivers call to_platform_device() to get platform_device -> and pass it to platform_get_drvdata(). In platform_get_drvdata(), +> and pass it to platform_get_drvdata().??In platform_get_drvdata(), > the platform_device is converted back to struct device again. > > Use dev_get_drvdata() to avoid platform_device/device dance. @@ -14,7 +14,7 @@ Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> (for DesignWare only) -> drivers/i2c/busses/i2c-designware-platdrv.c | 6 ++---- +> ?drivers/i2c/busses/i2c-designware-platdrv.c | 6 ++---- > diff --git a/drivers/i2c/busses/i2c-designware-platdrv.c > b/drivers/i2c/busses/i2c-designware-platdrv.c @@ -23,25 +23,25 @@ Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > +++ b/drivers/i2c/busses/i2c-designware-platdrv.c > @@ -428,8 +428,7 @@ static void dw_i2c_plat_complete(struct device > *dev) -> #ifdef CONFIG_PM -> static int dw_i2c_plat_suspend(struct device *dev) -> { +> ?#ifdef CONFIG_PM +> ?static int dw_i2c_plat_suspend(struct device *dev) +> ?{ > - struct platform_device *pdev = to_platform_device(dev); > - struct dw_i2c_dev *i_dev = platform_get_drvdata(pdev); > + struct dw_i2c_dev *i_dev = dev_get_drvdata(dev); -> -> i_dev->disable(i_dev); -> i2c_dw_plat_prepare_clk(i_dev, false); +> ? +> ? i_dev->disable(i_dev); +> ? i2c_dw_plat_prepare_clk(i_dev, false); > @@ -439,8 +438,7 @@ static int dw_i2c_plat_suspend(struct device *dev) -> -> static int dw_i2c_plat_resume(struct device *dev) -> { +> ? +> ?static int dw_i2c_plat_resume(struct device *dev) +> ?{ > - struct platform_device *pdev = to_platform_device(dev); > - struct dw_i2c_dev *i_dev = platform_get_drvdata(pdev); > + struct dw_i2c_dev *i_dev = dev_get_drvdata(dev); -> -> i2c_dw_plat_prepare_clk(i_dev, true); -> i_dev->init(i_dev); +> ? +> ? i2c_dw_plat_prepare_clk(i_dev, true); +> ? i_dev->init(i_dev); > -- diff --git a/a/content_digest b/N1/content_digest index 57ac183..7b796c1 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,33 +1,13 @@ "ref\01501172184-17102-1-git-send-email-yamada.masahiro@socionext.com\0" - "From\0Andy Shevchenko <andriy.shevchenko@linux.intel.com>\0" - "Subject\0Re: [PATCH] i2c: use dev_get_drvdata() to get private data in suspend/resume hooks\0" + "From\0andriy.shevchenko@linux.intel.com (Andy Shevchenko)\0" + "Subject\0[PATCH] i2c: use dev_get_drvdata() to get private data in suspend/resume hooks\0" "Date\0Fri, 28 Jul 2017 13:40:15 +0300\0" - "To\0Masahiro Yamada <yamada.masahiro@socionext.com>" - " linux-i2c@vger.kernel.org\0" - "Cc\0S\303\266ren Brinkmann <soren.brinkmann@xilinx.com>" - Mika Westerberg <mika.westerberg@linux.intel.com> - Michal Simek <michal.simek@xilinx.com> - linux-arm-kernel@lists.infradead.org - linux-kernel@vger.kernel.org - Sekhar Nori <nsekhar@ti.com> - Scott Branden <sbranden@broadcom.com> - Wolfram Sang <wsa@the-dreams.de> - Joachim Eastwood <manabian@gmail.com> - Kukjin Kim <kgene@kernel.org> - bcm-kernel-feedback-list@broadcom.com - Krzysztof Kozlowski <krzk@kernel.org> - Barry Song <baohua@kernel.org> - Jon Mason <jonmason@broadcom.com> - Kevin Hilman <khilman@kernel.org> - Jarkko Nikula <jarkko.nikula@linux.intel.com> - linux-samsung-soc@vger.kernel.org - Ray Jui <rjui@broadcom.com> - " Patrice Chotard <patrice.chotard@st.com>\0" + "To\0linux-arm-kernel@lists.infradead.org\0" "\00:1\0" "b\0" "On Fri, 2017-07-28 at 01:16 +0900, Masahiro Yamada wrote:\n" "> Several drivers call to_platform_device() to get platform_device\n" - "> and pass it to platform_get_drvdata().\302\240\302\240In platform_get_drvdata(),\n" + "> and pass it to platform_get_drvdata().??In platform_get_drvdata(),\n" "> the platform_device is converted back to struct device again.\n" "> \n" "> Use dev_get_drvdata() to avoid platform_device/device dance.\n" @@ -41,7 +21,7 @@ "\n" "(for DesignWare only)\n" "\n" - "> \302\240drivers/i2c/busses/i2c-designware-platdrv.c | 6 ++----\n" + "> ?drivers/i2c/busses/i2c-designware-platdrv.c | 6 ++----\n" "\n" "> diff --git a/drivers/i2c/busses/i2c-designware-platdrv.c\n" "> b/drivers/i2c/busses/i2c-designware-platdrv.c\n" @@ -50,29 +30,29 @@ "> +++ b/drivers/i2c/busses/i2c-designware-platdrv.c\n" "> @@ -428,8 +428,7 @@ static void dw_i2c_plat_complete(struct device\n" "> *dev)\n" - "> \302\240#ifdef CONFIG_PM\n" - "> \302\240static int dw_i2c_plat_suspend(struct device *dev)\n" - "> \302\240{\n" + "> ?#ifdef CONFIG_PM\n" + "> ?static int dw_i2c_plat_suspend(struct device *dev)\n" + "> ?{\n" "> -\tstruct platform_device *pdev = to_platform_device(dev);\n" "> -\tstruct dw_i2c_dev *i_dev = platform_get_drvdata(pdev);\n" "> +\tstruct dw_i2c_dev *i_dev = dev_get_drvdata(dev);\n" - "> \302\240\n" - "> \302\240\ti_dev->disable(i_dev);\n" - "> \302\240\ti2c_dw_plat_prepare_clk(i_dev, false);\n" + "> ?\n" + "> ?\ti_dev->disable(i_dev);\n" + "> ?\ti2c_dw_plat_prepare_clk(i_dev, false);\n" "> @@ -439,8 +438,7 @@ static int dw_i2c_plat_suspend(struct device *dev)\n" - "> \302\240\n" - "> \302\240static int dw_i2c_plat_resume(struct device *dev)\n" - "> \302\240{\n" + "> ?\n" + "> ?static int dw_i2c_plat_resume(struct device *dev)\n" + "> ?{\n" "> -\tstruct platform_device *pdev = to_platform_device(dev);\n" "> -\tstruct dw_i2c_dev *i_dev = platform_get_drvdata(pdev);\n" "> +\tstruct dw_i2c_dev *i_dev = dev_get_drvdata(dev);\n" - "> \302\240\n" - "> \302\240\ti2c_dw_plat_prepare_clk(i_dev, true);\n" - "> \302\240\ti_dev->init(i_dev);\n" + "> ?\n" + "> ?\ti2c_dw_plat_prepare_clk(i_dev, true);\n" + "> ?\ti_dev->init(i_dev);\n" "> \n" "\n" "-- \n" "Andy Shevchenko <andriy.shevchenko@linux.intel.com>\n" Intel Finland Oy -cdfeb56ff0814c76831e706fe874dc4407d46efcd8d3fa2a6e8dafafa05092d1 +64719862b91ddbe27ba4c24559691e8825ccc649beea3d4c4d717d7b048f88c4
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.