diff for duplicates of <556D6BC7.8000308@topic.nl> diff --git a/a/1.txt b/N1/1.txt index b041626..530b981 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,4 +1,4 @@ -=EF=BB=BFOn 02-06-15 09:50, Paul Bolle wrote: +On 02-06-15 09:50, Paul Bolle wrote: > On Mon, 2015-06-01 at 12:13 +0200, Mike Looijmans wrote: >> --- /dev/null >> +++ b/drivers/clk/clk-cdce925.c @@ -13,31 +13,26 @@ > did a quick build of this file. That triggered some noise on x86_64. > Excerpt: > -> drivers/clk/clk-cdce925.c: In function =E2=80=98cdce925_regmap_i2c_write= -=E2=80=99: -> include/linux/dynamic_debug.h:64:16: warning: format =E2=80=98%u=E2=80=99= - expects argument of type =E2=80=98unsigned int=E2=80=99, but argument 5 ha= -s type =E2=80=98size_t=E2=80=99 [-Wformat=3D] +> drivers/clk/clk-cdce925.c: In function ‘cdce925_regmap_i2c_write’: +> include/linux/dynamic_debug.h:64:16: warning: format ‘%u’ expects argument of type ‘unsigned int’, but argument 5 has type ‘size_t’ [-Wformat=] > static struct _ddebug __aligned(8) \ > ^ > [...] -> drivers/clk/clk-cdce925.c:505:2: note: in expansion of macro =E2=80=98dev= -_dbg=E2=80=99 +> drivers/clk/clk-cdce925.c:505:2: note: in expansion of macro ‘dev_dbg’ > dev_dbg(&i2c->dev, "%s(%u) %#x %#x\n", __func__, count, > ^ > > A quick look at Documentation/printk-formats.txt suggested this (very > lightly tested) fix: > @@ -502,7 +502,7 @@ static int cdce925_regmap_i2c_write( -> reg_data[0] =3D CDCE925_I2C_COMMAND_BYTE_TRANSFER | ((u8 *)data)= -[0]; -> reg_data[1] =3D ((u8 *)data)[1]; +> reg_data[0] = CDCE925_I2C_COMMAND_BYTE_TRANSFER | ((u8 *)data)[0]; +> reg_data[1] = ((u8 *)data)[1]; > > - dev_dbg(&i2c->dev, "%s(%u) %#x %#x\n", __func__, count, > + dev_dbg(&i2c->dev, "%s(%zu) %#x %#x\n", __func__, count, > reg_data[0], reg_data[1]); > -> ret =3D i2c_master_send(i2c, reg_data, count); +> ret = i2c_master_send(i2c, reg_data, count); > >> +static int cdce925_regmap_i2c_read(void *context, >> + const void *reg, size_t reg_size, void *val, size_t val_size) @@ -48,12 +43,11 @@ _dbg=E2=80=99 > Likewise: > @@ -547,7 +547,7 @@ static int cdce925_regmap_i2c_read(void *context, > -> ret =3D i2c_transfer(i2c->adapter, xfer, 2); -> if (likely(ret =3D=3D 2)) { +> ret = i2c_transfer(i2c->adapter, xfer, 2); +> if (likely(ret == 2)) { > - dev_dbg(&i2c->dev, "%s(%u, %u) %#x %#x\n", __func__, > + dev_dbg(&i2c->dev, "%s(%zu, %zu) %#x %#x\n", __func__, -> reg_size, val_size, reg_data[0], *((u8 *= -)val)); +> reg_size, val_size, reg_data[0], *((u8 *)val)); > return 0; > } else if (ret < 0) > diff --git a/a/content_digest b/N1/content_digest index 18d314f..50488f1 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -4,14 +4,14 @@ "Subject\0Re: [PATCH v3] Add TI CDCE925 I2C controlled clock synthesizer driver\0" "Date\0Tue, 2 Jun 2015 10:39:35 +0200\0" "To\0Paul Bolle <pebolle@tiscali.nl>\0" - "Cc\0<linux-clk@vger.kernel.org>" - <sboyd@codeaurora.org> - <mturquette@linaro.org> - <linux-kernel@vger.kernel.org> - " <devicetree@vger.kernel.org>\0" + "Cc\0linux-clk@vger.kernel.org" + sboyd@codeaurora.org + mturquette@linaro.org + linux-kernel@vger.kernel.org + " devicetree@vger.kernel.org\0" "\00:1\0" "b\0" - "=EF=BB=BFOn 02-06-15 09:50, Paul Bolle wrote:\n" + "\357\273\277On 02-06-15 09:50, Paul Bolle wrote:\n" "> On Mon, 2015-06-01 at 12:13 +0200, Mike Looijmans wrote:\n" ">> --- /dev/null\n" ">> +++ b/drivers/clk/clk-cdce925.c\n" @@ -26,31 +26,26 @@ "> did a quick build of this file. That triggered some noise on x86_64.\n" "> Excerpt:\n" ">\n" - "> drivers/clk/clk-cdce925.c: In function =E2=80=98cdce925_regmap_i2c_write=\n" - "=E2=80=99:\n" - "> include/linux/dynamic_debug.h:64:16: warning: format =E2=80=98%u=E2=80=99=\n" - " expects argument of type =E2=80=98unsigned int=E2=80=99, but argument 5 ha=\n" - "s type =E2=80=98size_t=E2=80=99 [-Wformat=3D]\n" + "> drivers/clk/clk-cdce925.c: In function \342\200\230cdce925_regmap_i2c_write\342\200\231:\n" + "> include/linux/dynamic_debug.h:64:16: warning: format \342\200\230%u\342\200\231 expects argument of type \342\200\230unsigned int\342\200\231, but argument 5 has type \342\200\230size_t\342\200\231 [-Wformat=]\n" "> static struct _ddebug __aligned(8) \\\n" "> ^\n" "> [...]\n" - "> drivers/clk/clk-cdce925.c:505:2: note: in expansion of macro =E2=80=98dev=\n" - "_dbg=E2=80=99\n" + "> drivers/clk/clk-cdce925.c:505:2: note: in expansion of macro \342\200\230dev_dbg\342\200\231\n" "> dev_dbg(&i2c->dev, \"%s(%u) %#x %#x\\n\", __func__, count,\n" "> ^\n" ">\n" "> A quick look at Documentation/printk-formats.txt suggested this (very\n" "> lightly tested) fix:\n" "> @@ -502,7 +502,7 @@ static int cdce925_regmap_i2c_write(\n" - "> reg_data[0] =3D CDCE925_I2C_COMMAND_BYTE_TRANSFER | ((u8 *)data)=\n" - "[0];\n" - "> reg_data[1] =3D ((u8 *)data)[1];\n" + "> reg_data[0] = CDCE925_I2C_COMMAND_BYTE_TRANSFER | ((u8 *)data)[0];\n" + "> reg_data[1] = ((u8 *)data)[1];\n" ">\n" "> - dev_dbg(&i2c->dev, \"%s(%u) %#x %#x\\n\", __func__, count,\n" "> + dev_dbg(&i2c->dev, \"%s(%zu) %#x %#x\\n\", __func__, count,\n" "> reg_data[0], reg_data[1]);\n" ">\n" - "> ret =3D i2c_master_send(i2c, reg_data, count);\n" + "> ret = i2c_master_send(i2c, reg_data, count);\n" ">\n" ">> +static int cdce925_regmap_i2c_read(void *context,\n" ">> +\t const void *reg, size_t reg_size, void *val, size_t val_size)\n" @@ -61,12 +56,11 @@ "> Likewise:\n" "> @@ -547,7 +547,7 @@ static int cdce925_regmap_i2c_read(void *context,\n" ">\n" - "> ret =3D i2c_transfer(i2c->adapter, xfer, 2);\n" - "> if (likely(ret =3D=3D 2)) {\n" + "> ret = i2c_transfer(i2c->adapter, xfer, 2);\n" + "> if (likely(ret == 2)) {\n" "> - dev_dbg(&i2c->dev, \"%s(%u, %u) %#x %#x\\n\", __func__,\n" "> + dev_dbg(&i2c->dev, \"%s(%zu, %zu) %#x %#x\\n\", __func__,\n" - "> reg_size, val_size, reg_data[0], *((u8 *=\n" - ")val));\n" + "> reg_size, val_size, reg_data[0], *((u8 *)val));\n" "> return 0;\n" "> } else if (ret < 0)\n" ">\n" @@ -93,4 +87,4 @@ "\n" Please consider the environment before printing this e-mail -8308e742f2cb972004e402c4781184cf0bba6dc4e7f751109b1cba935fe7c642 +6e8faef600d4951a261d750578efd0ecd56dd0e4f6f962eb25b1f09aa6c806b7
diff --git a/a/1.txt b/N2/1.txt index b041626..530b981 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -1,4 +1,4 @@ -=EF=BB=BFOn 02-06-15 09:50, Paul Bolle wrote: +On 02-06-15 09:50, Paul Bolle wrote: > On Mon, 2015-06-01 at 12:13 +0200, Mike Looijmans wrote: >> --- /dev/null >> +++ b/drivers/clk/clk-cdce925.c @@ -13,31 +13,26 @@ > did a quick build of this file. That triggered some noise on x86_64. > Excerpt: > -> drivers/clk/clk-cdce925.c: In function =E2=80=98cdce925_regmap_i2c_write= -=E2=80=99: -> include/linux/dynamic_debug.h:64:16: warning: format =E2=80=98%u=E2=80=99= - expects argument of type =E2=80=98unsigned int=E2=80=99, but argument 5 ha= -s type =E2=80=98size_t=E2=80=99 [-Wformat=3D] +> drivers/clk/clk-cdce925.c: In function ‘cdce925_regmap_i2c_write’: +> include/linux/dynamic_debug.h:64:16: warning: format ‘%u’ expects argument of type ‘unsigned int’, but argument 5 has type ‘size_t’ [-Wformat=] > static struct _ddebug __aligned(8) \ > ^ > [...] -> drivers/clk/clk-cdce925.c:505:2: note: in expansion of macro =E2=80=98dev= -_dbg=E2=80=99 +> drivers/clk/clk-cdce925.c:505:2: note: in expansion of macro ‘dev_dbg’ > dev_dbg(&i2c->dev, "%s(%u) %#x %#x\n", __func__, count, > ^ > > A quick look at Documentation/printk-formats.txt suggested this (very > lightly tested) fix: > @@ -502,7 +502,7 @@ static int cdce925_regmap_i2c_write( -> reg_data[0] =3D CDCE925_I2C_COMMAND_BYTE_TRANSFER | ((u8 *)data)= -[0]; -> reg_data[1] =3D ((u8 *)data)[1]; +> reg_data[0] = CDCE925_I2C_COMMAND_BYTE_TRANSFER | ((u8 *)data)[0]; +> reg_data[1] = ((u8 *)data)[1]; > > - dev_dbg(&i2c->dev, "%s(%u) %#x %#x\n", __func__, count, > + dev_dbg(&i2c->dev, "%s(%zu) %#x %#x\n", __func__, count, > reg_data[0], reg_data[1]); > -> ret =3D i2c_master_send(i2c, reg_data, count); +> ret = i2c_master_send(i2c, reg_data, count); > >> +static int cdce925_regmap_i2c_read(void *context, >> + const void *reg, size_t reg_size, void *val, size_t val_size) @@ -48,12 +43,11 @@ _dbg=E2=80=99 > Likewise: > @@ -547,7 +547,7 @@ static int cdce925_regmap_i2c_read(void *context, > -> ret =3D i2c_transfer(i2c->adapter, xfer, 2); -> if (likely(ret =3D=3D 2)) { +> ret = i2c_transfer(i2c->adapter, xfer, 2); +> if (likely(ret == 2)) { > - dev_dbg(&i2c->dev, "%s(%u, %u) %#x %#x\n", __func__, > + dev_dbg(&i2c->dev, "%s(%zu, %zu) %#x %#x\n", __func__, -> reg_size, val_size, reg_data[0], *((u8 *= -)val)); +> reg_size, val_size, reg_data[0], *((u8 *)val)); > return 0; > } else if (ret < 0) > diff --git a/a/content_digest b/N2/content_digest index 18d314f..c848db9 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -11,7 +11,7 @@ " <devicetree@vger.kernel.org>\0" "\00:1\0" "b\0" - "=EF=BB=BFOn 02-06-15 09:50, Paul Bolle wrote:\n" + "\357\273\277On 02-06-15 09:50, Paul Bolle wrote:\n" "> On Mon, 2015-06-01 at 12:13 +0200, Mike Looijmans wrote:\n" ">> --- /dev/null\n" ">> +++ b/drivers/clk/clk-cdce925.c\n" @@ -26,31 +26,26 @@ "> did a quick build of this file. That triggered some noise on x86_64.\n" "> Excerpt:\n" ">\n" - "> drivers/clk/clk-cdce925.c: In function =E2=80=98cdce925_regmap_i2c_write=\n" - "=E2=80=99:\n" - "> include/linux/dynamic_debug.h:64:16: warning: format =E2=80=98%u=E2=80=99=\n" - " expects argument of type =E2=80=98unsigned int=E2=80=99, but argument 5 ha=\n" - "s type =E2=80=98size_t=E2=80=99 [-Wformat=3D]\n" + "> drivers/clk/clk-cdce925.c: In function \342\200\230cdce925_regmap_i2c_write\342\200\231:\n" + "> include/linux/dynamic_debug.h:64:16: warning: format \342\200\230%u\342\200\231 expects argument of type \342\200\230unsigned int\342\200\231, but argument 5 has type \342\200\230size_t\342\200\231 [-Wformat=]\n" "> static struct _ddebug __aligned(8) \\\n" "> ^\n" "> [...]\n" - "> drivers/clk/clk-cdce925.c:505:2: note: in expansion of macro =E2=80=98dev=\n" - "_dbg=E2=80=99\n" + "> drivers/clk/clk-cdce925.c:505:2: note: in expansion of macro \342\200\230dev_dbg\342\200\231\n" "> dev_dbg(&i2c->dev, \"%s(%u) %#x %#x\\n\", __func__, count,\n" "> ^\n" ">\n" "> A quick look at Documentation/printk-formats.txt suggested this (very\n" "> lightly tested) fix:\n" "> @@ -502,7 +502,7 @@ static int cdce925_regmap_i2c_write(\n" - "> reg_data[0] =3D CDCE925_I2C_COMMAND_BYTE_TRANSFER | ((u8 *)data)=\n" - "[0];\n" - "> reg_data[1] =3D ((u8 *)data)[1];\n" + "> reg_data[0] = CDCE925_I2C_COMMAND_BYTE_TRANSFER | ((u8 *)data)[0];\n" + "> reg_data[1] = ((u8 *)data)[1];\n" ">\n" "> - dev_dbg(&i2c->dev, \"%s(%u) %#x %#x\\n\", __func__, count,\n" "> + dev_dbg(&i2c->dev, \"%s(%zu) %#x %#x\\n\", __func__, count,\n" "> reg_data[0], reg_data[1]);\n" ">\n" - "> ret =3D i2c_master_send(i2c, reg_data, count);\n" + "> ret = i2c_master_send(i2c, reg_data, count);\n" ">\n" ">> +static int cdce925_regmap_i2c_read(void *context,\n" ">> +\t const void *reg, size_t reg_size, void *val, size_t val_size)\n" @@ -61,12 +56,11 @@ "> Likewise:\n" "> @@ -547,7 +547,7 @@ static int cdce925_regmap_i2c_read(void *context,\n" ">\n" - "> ret =3D i2c_transfer(i2c->adapter, xfer, 2);\n" - "> if (likely(ret =3D=3D 2)) {\n" + "> ret = i2c_transfer(i2c->adapter, xfer, 2);\n" + "> if (likely(ret == 2)) {\n" "> - dev_dbg(&i2c->dev, \"%s(%u, %u) %#x %#x\\n\", __func__,\n" "> + dev_dbg(&i2c->dev, \"%s(%zu, %zu) %#x %#x\\n\", __func__,\n" - "> reg_size, val_size, reg_data[0], *((u8 *=\n" - ")val));\n" + "> reg_size, val_size, reg_data[0], *((u8 *)val));\n" "> return 0;\n" "> } else if (ret < 0)\n" ">\n" @@ -93,4 +87,4 @@ "\n" Please consider the environment before printing this e-mail -8308e742f2cb972004e402c4781184cf0bba6dc4e7f751109b1cba935fe7c642 +9dc9ce511a75d94f70e51ac043689e573d302dd61d46823c3aef9913f3b043d1
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.