Re: [PATCH v3 2/5] w1_therm: adding sysfs entry to check device power

2020-04-29 Thread Evgeniy Polyakov
Hi 29.04.2020, 16:47, "Greg KH" : >>  +What: /sys/bus/w1/devices/.../w1_slave >>  +Date: Apr 2020 >>  +Contact: Akira Shimahara >>  +Description: >>  + (RW) return the temperature in 1/1000 degC. >>  + *read*: return 2 lines with the hexa output data sent on the >>  + bus, return the CRC check a

Re: [PATCH 2/2] w1: fix the resume command API

2019-03-19 Thread Evgeniy Polyakov
Hi everyone Mariusz, thank you for the patch, a small comment on it logic 19.03.2019, 16:21, "Jean-Francois Dagenais" : >>  --- >>  drivers/w1/w1_io.c | 11 +-- >>  1 file changed, 9 insertions(+), 2 deletions(-) >> >>  diff --git a/drivers/w1/w1_io.c b/drivers/w1/w1_io.c >>  index 0364d3

Re: [PATCH 18/18] w1: w1_io.c: fix a kernel-doc warning

2018-05-10 Thread Evgeniy Polyakov
Hi 09.05.2018, 16:11, "Jonathan Corbet" : > On Wed, 9 May 2018 09:32:18 -0300 > Mauro Carvalho Chehab wrote: > >>  > Looks good to me, thank you! >>  > What tree should this be forwarded to, or folks from linux doc will pick >> it up? >>  >

Re: [PATCH 18/18] w1: w1_io.c: fix a kernel-doc warning

2018-05-08 Thread Evgeniy Polyakov
ank you! What tree should this be forwarded to, or folks from linux doc will pick it up? Acked-by: Evgeniy Polyakov > --- >  drivers/w1/w1_io.c | 1 + >  1 file changed, 1 insertion(+) > > diff --git a/drivers/w1/w1_io.c b/drivers/w1/w1_io.c > index 075d120e7b88..0364d3329c5

Re: [PATCH v2] w1: mxc_w1: Enable clock before calling clk_get_rate() on it

2018-05-08 Thread Evgeniy Polyakov
i.MX27 / i.MX31") > Signed-off-by: Stefan Potyra Looks good to me, thank you! Greg, please pull it into your tree, this also sounds like stable material, is it? Acked-by: Evgeniy Polyakov > --- >  drivers/w1/masters/mxc_w1.c | 20 +--- >  1 file changed, 13 inserti

Re: [PATCH] 1wire: family module autoload fails because of upper/lower case mismatch.

2018-05-01 Thread Evgeniy Polyakov
e pull this patch into your tree. Should it be stable material? Acked-by: Evgeniy Polyakov > --- >   drivers/w1/w1.c | 2 +- >   1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/w1/w1.c b/drivers/w1/w1.c > index ab0931e..aa458f2 100644 > --- a/drivers/

Re: [PATCH] Enable clock before calling clk_get_rate() on it.

2018-04-30 Thread Evgeniy Polyakov
r driver for i.MX27 / i.MX31 > Signed-off-by: Stefan Potyra Acked-by: Evgeniy Polyakov Greg, please pull it into your tree. Is this a stable material? > --- >  drivers/w1/masters/mxc_w1.c | 8 >  1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/dr

Re: [PATCH] connector: add parent pid and tgid to coredump and exit events

2018-04-30 Thread Evgeniy Polyakov
Stefan, hi Sorry for delay. 26.04.2018, 15:04, "Stefan Strogin" : > Hi David, Evgeniy, > > Sorry to bother you, but could you please comment about the UAPI change and > the patch? With 4-bytes pid_t everything looks fine, and I do not know arch where pid is larger curre

Re: [PATCH] Fixed typo in onewire generic doc

2018-04-08 Thread Evgeniy Polyakov
Hi everyone I'm really sorry for that long delay. Was this patch accepted or should I push it upstream? 20.12.2017, 22:09, "Gergo Huszty" : > Onewire devices has 6 byte long unique serial numbers, 1 byte family > code and 1 byte CRC. Linux sysfs presents the device folder in the > form of family

Re: [PATCH] dma-mapping: don't clear GFP_ZERO in dma_alloc_attrs

2018-03-28 Thread Evgeniy Didin
east arc and s390 do not zero without the explicit __GFP_ZERO > argument. This patch fixed Ethernet issues on ARC HSDK. https://www.spinics.net/lists/kernel/msg2762054.html Tested-by: Evgeniy Didin > Fixes: 57bf5a8963f8 ("dma-mapping: clear harmful GFP_* flags in common code")

dma-mapping: clearing GFP_ZERO flag caused crashes of Ethernet on arc/hsdk board.

2018-03-27 Thread Evgeniy Didin
addr = 0; - page = alloc_pages(gfp, order); + page = alloc_pages(gfp | __GFP_ZERO, order); if (!page) return NULL; ->8--------- Best regards, Evgeniy Didin

[PATCH v4] mmc: dw_mmc: Fix the DTO/CTO timeout overflow calculation for 32-bit systems

2018-02-28 Thread Evgeniy Didin
t Gupta Reported-by: Vineet Gupta # ARC STAR 9001306872 HSDK, sdio: board crashes when copying big files Fixes: 9d9491a7da2a ("mmc: dw_mmc: Fix the DTO timeout calculation") Fixes: 4c2357f57dd5 ("mmc: dw_mmc: Fix the CTO timeout calculation") Signed-off-by: Evgeniy Didin CC:

Re: [PATCH 2/2 v3] mmc: dw_mmc: Fix the CTO overflow calculation for 32-bit systems

2018-02-26 Thread Evgeniy Didin
On Mon, 2018-02-26 at 18:53 +0200, Andy Shevchenko wrote: > On Mon, Feb 26, 2018 at 5:14 PM, Evgeniy Didin > wrote: > > On Mon, 2018-02-26 at 16:39 +0200, Andy Shevchenko wrote: > > > On Mon, Feb 26, 2018 at 4:34 PM, Evgeniy Didin > > > wrote: > > > >

Re: [PATCH 2/2 v3] mmc: dw_mmc: Fix the CTO overflow calculation for 32-bit systems

2018-02-26 Thread Evgeniy Didin
On Mon, 2018-02-26 at 16:39 +0200, Andy Shevchenko wrote: > On Mon, Feb 26, 2018 at 4:34 PM, Evgeniy Didin > wrote: > > In commit 4c2357f57dd5 ("mmc: dw_mmc: Fix the CTO timeout calculation") > > have been made changes which can cause multiply overflow for 32-bit syst

[PATCH 2/2 v3] mmc: dw_mmc: Fix the CTO overflow calculation for 32-bit systems

2018-02-26 Thread Evgeniy Didin
w. Signed-off-by: Evgeniy Didin CC: Alexey Brodkin CC: Eugeniy Paltsev CC: Douglas Anderson CC: Ulf Hansson CC: linux-kernel@vger.kernel.org CC: linux-snps-...@lists.infradead.org Cc: # 4c2357f57dd5 mmc: dw_mmc: Fix the CTO timeout calculation --- drivers/mmc/host/dw_mmc.c | 3 ++- 1 file

[PATCH 1/2 v3] mmc: dw_mmc: Fix the DTO timeout overflow calculation for 32-bit systems

2018-02-26 Thread Evgeniy Didin
end trace 2a58c9af6c25fe51 ]--- Lets cast this multiply to u64 type which prevents overflow. Tested-by: Vineet Gupta Fixes: ARC STAR 9001306872 HSDK, sdio: board crashes when copying big files Signed-off-by: Evgeniy Didin CC: Alexey Brodkin CC: Eugeniy Paltsev CC: Douglas Anderson CC: Ulf H

[PATCH 0/2 v3] mmc: dw_mmc: Fix DTO/STO timeout overflow calculation

2018-02-26 Thread Evgeniy Didin
and DIV_ROUND_UP_ULL helps with sum overflow. --- Changes since v2: -add fix for cto_ms Evgeniy Didin (2): mmc: dw_mmc: Fix the DTO timeout overflow calculation for 32-bit systems mmc: dw_mmc: Fix the CTO overflow calculation for 32-bit systems drivers/mmc/host/dw_mmc.c | 6 -- 1 file

[PATCH v2] mmc: dw_mmc: Fix the DTO timeout overflow calculation for 32-bit systems

2018-02-22 Thread Evgeniy Didin
end trace 2a58c9af6c25fe51 ]--- Lets cast this multiply to u64 type which prevents overflow. Tested-by: Vineet Gupta Fixes: ARC STAR 9001306872 HSDK, sdio: board crashes when copying big files Signed-off-by: Evgeniy Didin CC: Alexey Brodkin CC: Eugeniy Paltsev CC: Douglas Anderson CC: Ulf H

[PATCH] mmc: dw_mmc: Fix the DTO timeout overflow calculation for 32-bit systems

2018-02-21 Thread Evgeniy Didin
opying file from mmc. Lets cast this multiply to long long int which prevents overflow. Signed-off-by: Evgeniy Didin CC: Alexey Brodkin CC: Douglas Anderson CC: Ulf Hansson CC: linux-kernel@vger.kernel.org CC: linux-snps-...@lists.infradead.org --- drivers/mmc/host/dw_mmc.c | 2 +- 1 file change

Re: [PATCH] w1: w1-gpio: Convert to use GPIO descriptors

2017-12-05 Thread Evgeniy Polyakov
Hi Linus Sorry for late reply 20.11.2017, 11:47, "Linus Walleij" : > The w1 master driver includes a complete open drain emulation > reimplementation among other things. > > This converts the driver and all board files using it to use > GPIO descriptors associated with the device to look up the >

Re: [PATCH] w1: remove redundant assignments to search_bit and last_rn

2017-12-05 Thread Evgeniy Polyakov
rs tree if you haven't yet. > Signed-off-by: Colin Ian King Acked-by: Evgeniy Polyakov > --- >  drivers/w1/w1.c | 2 -- >  1 file changed, 2 deletions(-) > > diff --git a/drivers/w1/w1.c b/drivers/w1/w1.c > index 0c2a5a8327bd..c29f6c5dda3c 100644 > --- a/drivers/w1/w1

Re: [PATCH v2] w1: driver for serial linkUSB, linkOEM, LINK devices

2017-12-05 Thread Evgeniy Polyakov
documentation file). The > kernel patch is against linux-next. Please try and comment. > > v2 of the patch set changes the following: > - based on review input from Evgeniy (thanks!), introduced a helper function > to re-use more code and generally improved the style of the code (c

Re: [PATCH] w1 driver for serial linkUSB, linkOEM, LINK devices

2017-10-30 Thread Evgeniy Polyakov
Hi 17.10.2017, 23:01, "Hans-Frieder Vogt" : > attached is a driver that supports the serial OneWire masters from > iButtonLink(TM) in the w1 kernel driver. In order to be usable it needs an > updated userland tool (patch included in documentation file). The patch is > against linux-next. Please

Re: [PATCH] w1: keep balance of mutex locks and refcnts

2017-10-09 Thread Evgeniy Polyakov
sleeping waiting for the bus master to complete operation, and during this period family driver has to be referenced. But we can easily grab the reference earlier and then try to lock the bus, so the second variant will work. > -- > Alexey > > On 01.10.2017 08:55, Evgeniy Polyakov wro

Re: [PATCH v5 2/2] add w1_ds28e17 driver for the DS28E17 Onewire to I2C master bridge

2017-09-30 Thread Evgeniy Polyakov
to I2C master bridge. > > Signed-off-by: Jan Kandziora > Acked-by: Evgeniy Polyakov > --- > Changes in v5 against v4 in this subpatch: >   - adapted to linux-4.14-rc1 > > Changes in v4 against v3 in this subpatch: >   - adapted to linux-4.12.0 > > Changes in v3 against v

Re: [PATCH 10/10] [RFC] w1_netlink.h: add support for nested structs

2017-09-30 Thread Evgeniy Polyakov
Hi 26.09.2017, 20:59, "Mauro Carvalho Chehab" : > Describe nested struct/union fields > > NOTE: This is a pure test patch, meant to validate if the > parsing logic for nested structs is working properly. > > I've no idea if the random text I added there is correct! It looks correct, although I wo

Re: [PATCH] w1: keep balance of mutex locks and refcnts

2017-09-30 Thread Evgeniy Polyakov
Hi Alex 29.09.2017, 23:23, "Alexey Khoroshilov" : > w1_therm_eeprom() and w1_DS18B20_precision() decrement THERM_REFCNT > on error paths, while they did not increment it yet. > > read_therm() unlocks bus mutex on some error paths, > while it is not acquired. > > The patch makes sure all the functi

Re: [PATCH v2] w1:fix byteorder of W1_READ_ROM id under big-endian cpu

2017-09-05 Thread Evgeniy Polyakov
Hi 28.08.2017, 12:25, "chen.l...@zte.com.cn" : > Hi > > Q: > > But w1_reg_num has a different layout for be/le systems, isn't it enough? > > A: > > sure, it's right only under the assumption that 'rn' is a  correct layout id. > > Here's my example in be system which I encounter before. > > buf[0]

Re: [PATCH] w1: ds2490: constify usb_device_id and fix space before '[' error

2017-09-05 Thread Evgeniy Polyakov
space prohibited before open square bracket '['. > > Signed-off-by: Arvind Yadav Looks good to me, thank you kernel-janitors@ please queue this up Acked-by: Evgeniy Polyakov > --- >  drivers/w1/masters/ds2490.c | 2 +- >  1 file changed, 1 insertion(+), 1 deletion(-) >

Re: [PATCH v4 2/2] add w1_ds28e17 driver for the DS28E17 Onewire to I2C master bridge

2017-09-05 Thread Evgeniy Polyakov
previously since I missed it. Acked-by: Evgeniy Polyakov > --- > Changes in v4 against v3 in this subpatch: >   - adapted to linux-4.12.0 > > Changes in v3 against v2 in this subpatch: >   - fixed a bug in using the i2c_adapter_quirks structure > > Changes in v2 against v1 i

Re: [PATCH] connector: Delete an error message for a failed memory allocation in cn_queue_alloc_callback_entry()

2017-09-05 Thread Evgeniy Polyakov
ned-off-by: Markus Elfring Looks good to me, thanks Markus. There is virtually zero useful information in this print if we are in the situation, when kernel can not allocate a few bytes to run connector queue. Acked-by: Evgeniy Polyakov kernel-janitors@ please queue this patch up > --

Re: [PATCH v2] w1:fix byteorder of W1_READ_ROM id under big-endian cpu

2017-08-27 Thread Evgeniy Polyakov
Hi Chen 19.07.2017, 10:58, "Chen Lin" : > The byteorder of para rn(W1_READ_ROM id) pass to w1_slave_found must > be the same with the byterorder defined in struct w1_reg_num. > > The rn read from 'rv = w1_read_block(dev, (u8 *)&rn, 8)' is a byte > serial and not cpu endian relative, it need to cha

Re: [PATCH linux v5 0/3] Export 1-wire thermal sensors as hwmon device

2017-08-27 Thread Evgeniy Polyakov
Hi 18.07.2017, 02:09, "Jaghathiswari Rankappagounder Natarajan" : > Hi Greg, > Please pull in this patchset into the tree. Thanks! Here is a patchset, is it visible in your mailbox? If so, please pull it into your tree. > Summary of what this patchset does: > > Our board has 4 DS18B20 1-wire tem

Re: [PATCH v4 1/2] wire: export w1_touch_bit

2017-08-27 Thread Evgeniy Polyakov
v4 against v3 in this subpatch: >   - adapted to linux-4.12.0 Greg, please pull this patch series into the tree, it has been adopted to the latest vanilla tree to date. Acked-by: Evgeniy Polyakov

Re: [PATCH v4 0/5] w1: ds1wm: register access patch

2017-08-27 Thread Evgeniy Polyakov
Hi everyone 25.07.2017, 14:27, "Johannes Poehlmann" : > To make the ds1wm driver work on a powerpc architecture (big endian, 32bit) > with a register offset multiplier of 4 I had to make some changes to > Version 4 of the patchset >    work on Greg Kroah-Hartmanns comments: > o added changelog to

Re: [PATCH v2 1/2] power: supply: move HDQ interface for bq27xxx from w1 to power/supply

2017-08-27 Thread Evgeniy Polyakov
Hi everyone Sorry for long delay 25.07.2017, 16:48, "Sebastian Reichel" : > Here is a signed immutable branch for Andrew's patch, which moves > bq27000 w1 driver to the power-supply subsystem. I guess git will > figure everything out without this, but better safe than sorry :) > > The following c

Re: [PATCH linux v2 0/3] Export 1-wire thermal sensors as hwmon device

2017-07-13 Thread Evgeniy Polyakov
Hi 13.07.2017, 18:36, "Evgeniy Polyakov" : > I believe this is a resend of your previous patchet, isn't it? > Greg, if you hadn't yet, please pull it into the tree. Aaand it looks like we have a discussion now, so please postpone it for a while :)

Re: [PATCH linux v2 0/3] Export 1-wire thermal sensors as hwmon device

2017-07-13 Thread Evgeniy Polyakov
Hi 13.07.2017, 00:41, "Jaghathiswari Rankappagounder Natarajan" : > Our board has 4 DS18B20 1-wire temperature sensors. Each 1-wire bus and the > sensor under it is already configured against the Linux 1-wire driver > (called w1). They have a sysfs file(e.g. > /sys/bus/w1/devices/w1_bus_master1/28

Re: [PATCH v4 1/2] wire: export w1_touch_bit

2017-07-13 Thread Evgeniy Polyakov
later for writing drivers for other Onewire chips > which do single-bit communication. > > Signed-off-by: Jan Kandziora Greg, please pull this patchset into the tree, I missed it last time and it stuck in the queues for a too long. Thank you. Acked-by: Evgeniy Polyakov > --- > F

Re: [PATCH v2 0/4] w1: ds1wm: register access patch

2017-06-29 Thread Evgeniy Polyakov
Hi everyone Greg, please pull this into your tree 23.06.2017, 13:47, "Johannes Poehlmann" : > To make the ds1wm driver work on a powerpc architecture (big endian, 32bit) > with a register offset multiplier of 4 I had to make some changes to > >    drivers/w1/masters/ds1wm.c > and include/linu

Re: [PATCH V1] one wire ds1wm patch

2017-06-19 Thread Evgeniy Polyalkov
tional: add messages to make incorporation in mfd drivers > easier Patches look good, thank you. Greg, please pull them into your tree, is it ok to be sent as attachment? Acked-by: Evgeniy Polyakov

Re: [PATCH linux v1 0/3] Export 1-wire thermal sensors as hwmon device

2017-06-17 Thread Evgeniy Polyakov
Looks good to me, thank you. Acked-by: Evgeniy Polyakov

Re: [PATCH] w1: w1-gpio: Fix double-free of platform_data

2017-04-29 Thread Evgeniy Polyakov
Hi Alexey 25.03.2017, 20:08, "Alexey Ignatov" : > struct w1_gpio_platform_data was allocated using devres when using > device tree. Then it was assigned to dev.platform_data, which leaded > to double free on device removal by devres and by direct > kfree(platform_data) in platform_device_release)

Re: [PATCH v2 21/23] MAINTAINERS: Add file patterns for w1 device tree bindings

2017-03-13 Thread Evgeniy Polyakov
nto. If it is ok, Greg, please pull it into your one, or I see devicetree@ mail in copy... Stumbled > Signed-off-by: Geert Uytterhoeven > Cc: Evgeniy Polyakov > Cc: Greg Kroah-Hartman > --- > Please apply this patch directly if you want to be involved in device > tree bindin

Re: [PATCH v2 0/4] w1: add DS2438 support, documentation and small fixes

2017-02-23 Thread Evgeniy Polyakov
Hi everyone 23.02.2017, 09:38, "Mariusz Bialonczyk" : > This is my second version of my w1 patchset. > It mainly adds support for the DS2438. There is also a documentation > for it and also a missing one for DS2413. Looks good to me, thank you Greg, please pull it into your tree

Re: [PATCH] i2c: w1: Add devicetree match for DS2482

2017-01-23 Thread Evgeniy Polyakov
Hi Colin 22.01.2017, 20:55, "Colin Godsey" : > Adds the OF devicetree match for the DS2482 I2C-1wire master and adds a > listing to the I2C trivial devicetree bindings doc. Based on a proposed patch > by Hector Palacios. Looks good to me. Acked-by: Evgeniy Polyakov

Re: [PATCH] w1: omap_hdq: Free resources on error path

2017-01-23 Thread Evgeniy Polyakov
d by '_omap_hdq_reset()' instead of returning -EINVAL > unconditionally. > > Signed-off-by: Christophe JAILLET Looks good to me. If kernel-janitors@ will not push it upstream feel free to send to to GregKH and add me to copy. Acked-by: Evgeniy Polyakov > --- >  drivers

Re: [PATCH] [RFC] proc connector: add namespace events

2016-09-12 Thread Evgeniy Polyakov
Hi everyone 08.09.2016, 18:39, "Alban Crequy" : > The act of a process creating or joining a namespace via clone(), > unshare() or setns() is a useful signal for monitoring applications. > + if (old_ns->mnt_ns != new_ns->mnt_ns) > + proc_ns_connector(tsk, CLONE_NEWNS, PROC_NM_REASON_CLONE, old_mn

Re: [PATCH v2] w1: fix typo in parameter description

2016-08-12 Thread Evgeniy Polyakov
Hi 11.08.2016, 03:03, "Wei Yongjun" : > Fix typo in parameter description. > > Signed-off-by: Wei Yongjun Looks good to me, thank you Greg, please pull it into your tree Acked-by: Evgeniy Polyakov > --- > v1 -> v2: make it as one line, comments from Evgeniy Polya

Re: [PATCH] w1: fix timeout_us parameter description

2016-08-10 Thread Evgeniy Polyakov
Hi 10.08.2016, 06:22, "Wei Yongjun" : >>  08.08.2016, 16:52, "Wei Yongjun" : >>>  Fix 'timeout_us' parameter description. >>>   MODULE_PARM_DESC(timeout, "time in seconds between automatic slave >>> searches"); >>>   module_param_named(timeout_us, w1_timeout_us, int, 0); >>>  -MODULE_PARM_DESC(t

Re: [PATCH] w1: fix timeout_us parameter description

2016-08-09 Thread Evgeniy Polyakov
Hi 08.08.2016, 16:52, "Wei Yongjun" : > Fix 'timeout_us' parameter description. > > Signed-off-by: Wei Yongjun > --- >  drivers/w1/w1.c | 4 ++-- >  1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/w1/w1.c b/drivers/w1/w1.c > index bb34362..e213c67 100644 > --- a/drivers/w1

Re: [PATCH] w1: Remove unneeded header file

2016-06-28 Thread Evgeniy Polyakov
Hi 28.06.2016, 16:50, "Julia Lawall" : >>  24.06.2016, 09:18, "Amitoj Kaur Chawla" : >>  > The Coccinelle semantic patch used to make this change is as follows: >>  > @ includesmodule @ >>  > @@ >>  > >>  > #include >>  > >>  > @ depends on includesmodule @ >>  > @@ >>  > >>  > - #include >>  >

Re: [PATCH] w1: Remove unneeded header file

2016-06-28 Thread Evgeniy Polyakov
Hi 24.06.2016, 09:18, "Amitoj Kaur Chawla" : > Drop redundant include of moduleparam.h > > The Coccinelle semantic patch used to make this change is as follows: > @ includesmodule @ > @@ > > #include > > @ depends on includesmodule @ > @@ > > - #include > > Signed-off-by: Amitoj Kaur Chawla Lo

Re: [PATCH] connector: fix out-of-order cn_proc netlink message delivery

2016-06-28 Thread Evgeniy Polyakov
Hi Aaron 24.06.2016, 16:07, "Aaron Campbell" : > The proc connector messages include a sequence number, allowing userspace > programs to detect lost messages. However, performing this detection is > currently more difficult than necessary, since netlink messages can be > delivered to the applicati

Re: [PATCH] w1: add ability to set (SRAM) and store (EEPROM) configuration for temp sensors like DS18B20

2016-04-27 Thread Evgeniy Polyakov
precision when running on a kernel based device (unlike the Dallas > 1Wire library for e.g. Arduino, which supports writing the > configuration/scratchpad). This looks good to me, Greg, please pull it into your tree > Signed-off-by: x29a <0.x29...@gmail.com> Acked-by: Evgeniy

Re: [patch] w1: silence an uninitialized variable warning

2016-04-27 Thread Evgeniy Polyakov
Hi Dan 14.04.2016, 12:36, "Dan Carpenter" : > If kstrtoint() returns -ERANGE then "tmp" is uninitialized. Looks good to me, please pull it into janitors tree > Signed-off-by: Dan Carpenter Acked-by: Evgeniy Polaykov > diff --git a/drivers/w1/w1.c b/drivers/w1/

Re: [patch] ufs: silence uninitialized warnings

2016-03-22 Thread Evgeniy Dushistov
struct ufs_super_block_third *usb3) > { > - __fs64 tmp; > + __fs64 tmp = 0; > > switch (UFS_SB(sb)->s_flags & UFS_ST_MASK) { > case UFS_ST_SUNOS: In fact result of this function is saved, but not used. May be better from cleanup point of view will be remove these two functions and structure fields where their result was saved. -- /Evgeniy

Re: [PATCH] w1: w1_process() is not freezable kthread

2015-11-05 Thread Evgeniy Polyakov
Hi 28.10.2015, 08:26, "Jiri Kosina" : > On Tue, 27 Oct 2015, Evgeniy Polyakov wrote: > >>  > w1_process() calls try_to_freeze(), but the thread doesn't mark itself >>  > freezable through set_freezable(), so the try_to_freeze() call is useless. >> &g

Re: [PATCH] w1: w1_process() is not freezable kthread

2015-10-27 Thread Evgeniy Polyakov
Hi 26.10.2015, 08:53, "Jiri Kosina" : > From: Jiri Kosina > > w1_process() calls try_to_freeze(), but the thread doesn't mark itself > freezable through set_freezable(), so the try_to_freeze() call is useless. I believe it is better to mark it freezable, what do you think? Its task is useless if

Re: [PATCH] w1: Use module_pci_driver

2015-07-11 Thread Evgeniy Polyakov
Hi everyone Patch looks good to me, thank you Greg, please pull it into your tree Acked-by: Evgeniy Polyakov 07.07.2015, 09:55, "Vaishali Thakkar" : > Use module_pci_driver for drivers whose init and exit functions > only register and unregister, respectively. > Sign

Re: [PATCH] w1: Drop owner assignment from i2c_driver

2015-07-11 Thread Evgeniy Polyakov
Hi Krzysztof Patch looks good to me, thank you. Greg, please pull it into your tree. Acked-by: Evgeniy Polyakov 10.07.2015, 09:38, "Krzysztof Kozlowski" : > i2c_driver does not need to set an owner because i2c_register_driver() > will set it. > > Signed-off-b

Re: [patch] [patch 2/2 v2] w1: use correct lock on error in w1_seq_show()

2015-06-11 Thread Evgeniy Polyakov
therm') > Signed-off-by: Dan Carpenter Looks good to me, Greg please pull this serie into your tree, if you hadn't yet. Am I right that this is a stable tree material too? Acked-by: Evgeniy Polyakov -- To unsubscribe from this list: send the line "unsubscribe linux-kernel&quo

Re: [PATCH v2] w1: masters: omap_hdq: Add support for 1-wire mode

2015-06-11 Thread Evgeniy Polyakov
Hi 25.05.2015, 08:15, "Vignesh R" : >>  HDQ mode remains unchanged. >> >>  Signed-off-by: Vignesh R I have no experience with omap_hdq platform, but there are quite a few questions related to IO - you never check whether write was successful or read returned actually valid data, is it ok? I mea

Re: [PATCH] w1: ds2482: Add i2c module alias name

2015-05-29 Thread Evgeniy Polyakov
Hi everyone 27.05.2015, 01:38, "Ludek Hlavacek" : > Add i2c alias to enable autoloading of the module for device specified > in device-tree. > > Signed-off-by: Ludek Hlavacek This looks good to me, Greg, please pull to your tree. Thank you. Acked-by: Evgeniy Polyakov

Re: [PATCH] w1: Fix typo in MODULE_DESCRIPTION in matrox_w1.c

2015-05-21 Thread Evgeniy Polyakov
Hi 21.05.2015, 16:41, "Masanari Iida" : > This patch fix spelling typo in matrox_w1.c > > Signed-off-by: Masanari Iida Looks good to me, please pull it into the tree Acked-by: Evgeniy Polyakov > --- >  drivers/w1/masters/matrox_w1.c | 2 +- >  1 file changed

[PATCH] ads7846: do not ignore pendown-gpio flags

2015-05-20 Thread Evgeniy Dushistov
7846: fix ads7846 driver for work with ads7845". Signed-off-by: Evgeniy A. Dushistov --- drivers/input/touchscreen/ads7846.c | 19 ++- 1 file changed, 14 insertions(+), 5 deletions(-) diff --git a/drivers/input/touchscreen/ads7846.c b/drivers/input/touchscreen/ads7846.

[PATCH][v2] ads7846: fix ads7846 driver for work with ads7845

2015-05-19 Thread Evgeniy Dushistov
e_add_tail(x, m); @@ -1408,10 +1335,7 @@ static int ads7846_probe(struct spi_device *spi) * Take a first sample, leaving nPENIRQ active and vREF off; avoid * the touchscreen, in case it's not connected. */ - if (ts->model == 7845) - ads7845_

[PATCH] [RFC] ads7846: force driver to work with ads7845

2015-05-18 Thread Evgeniy Dushistov
if (0/*ts->model == 7845*/) ads7845_read12_ser(&spi->dev, PWRDOWN); else (void) ads7846_read12_ser(&spi->dev, READ_12BIT_SER(vaux)); -- 2.3.6 -- /Evgeniy ads7845(page 8): Figure 5 shows the typical operation of the ADS7845’s digital interface.

Re: [PATCH 3/3] max7359_keypad: implement DT bindings

2015-05-17 Thread Evgeniy Dushistov
On Fri, May 15, 2015 at 11:00:02PM +0200, Dmitry Torokhov wrote: > On Thu, May 14, 2015 at 05:38:03AM +0300, Evgeniy Dushistov wrote: > > + maxim,debounce_reg = /bits/ 8 <0x5F>; > > + maxim,ports_reg = /bits/ 8 <0xFF>; > > Specifying exact size

[PATCH 3/3] max7359_keypad: implement DT bindings

2015-05-13 Thread Evgeniy Dushistov
max7359_keypad: implement DT bindings Signed-off-by: Evgeniy A. Dushistov --- .../devicetree/bindings/input/max7359-keypad.txt | 33 drivers/input/keyboard/max7359_keypad.c| 60 -- 2 files changed, 90 insertions(+), 3 deletions(-) create mode

[PATCH 2/3] max7359_keypad: remove code duplication

2015-05-13 Thread Evgeniy Dushistov
: Evgeniy A. Dushistov --- drivers/input/keyboard/Kconfig | 1 + drivers/input/keyboard/max7359_keypad.c | 30 +- 2 files changed, 10 insertions(+), 21 deletions(-) diff --git a/drivers/input/keyboard/Kconfig b/drivers/input/keyboard/Kconfig index 106fbac..4d75062

[PATCH 1/3] max7359_keypad: Do not set MAX7359_CFG_INTERRUPT flag

2015-05-13 Thread Evgeniy Dushistov
this flag, we have to read FIFO until it become empty. But in interrupt we read FIFO just once. This lead to "keyboard" hang until reboot, if we press several keys, because of interrupt handler read just one "press" from FIFO and clear interrupt. Signed-off-by: Evgeniy A. Dus

Re: [PATCH v3] w1: introduce an ability to specify microseconds bus scanning intervals

2015-05-13 Thread Evgeniy Polyakov
by: Dmitry Khromov Greg, please pull it upstream Acked-by: Evgeniy Polyakov -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: [PATCH RESEND] w1: masters: omap_hdq: Add support for 1-wire mode

2015-05-13 Thread Evgeniy Polyakov
sent for v4.1 but was never picked up. >>  Here is the link to the previous patch >>  http://www.spinics.net/lists/linux-omap/msg116228.html > > Hi Andrew, Evgeniy Polyakov, > > Gentle ping > This patch has been waiting since 4.1 merge window. I believe it should go via omap t

Re: [PATCH] w1_therm reference count family data

2015-05-13 Thread Evgeniy Polyakov
e when it is no longer detected. > If that happens the sl->family_data will be freed and set to NULL > causing w1_slave_show to crash when it wakes up. > > Signed-off-by: David Fries > Reported-By: Thorsten Bschorr > Tested-by: Thorsten Bschorr > Acked-by: Evgeniy Polyakov

Re: [PATCH]Add support for DS28EA00 sequence to w1-therm

2015-05-02 Thread Evgeniy Polyakov
Hi 28.04.2015, 14:44, "Matt Campbell" : > This patch provides support for the DS28EA00 digital thermometer. Patch looks good, Greg, please pull this patch into you tree -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org

Re: [PATCH] signals: Generate warning when flush_signals() is called from non-kthread context

2015-05-02 Thread Evgeniy Polyakov
dlers in: > >   drivers/w1/slaves/w1_bq27000.c >   drivers/w1/slaves/w1_ds2406.c >   drivers/w1/slaves/w1_ds2408.c >   drivers/w1/slaves/w1_ds2413.c >   drivers/w1/slaves/w1_ds2423.c >   drivers/w1/slaves/w1_ds2431.c >   drivers/w1/slaves/w1_ds2433.c >   drivers/w1/slaves/w1_ds

Re: [PATCH] w1: slaves: Add dependency for W1_SLAVE_BQ27000

2015-04-30 Thread Evgeniy Polyakov
HI Marek 28.04.2015, 23:25, "Marek Belisko" : > bq2700 slave driver depends on bq27x00-battery driver thus adding this > dependency to config. When bq2700 is build as module bq27x00-battery > will be loaded automatically. > > Signed-off-by: Marek Belisko Patch looks good to me, but you should ad

Re: [PATCH]Add support for DS28AE00 sequence to w1-therm

2015-04-27 Thread Evgeniy Polyakov
Hi everyone 27.04.2015, 16:32, "Matt Campbell" : > This patch provides support for the DS28AE00 digital thermometer. Greg, please pull it into you tree, everything looks good to me -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vge

Re: [PATCH v2] w1: introduce an ability to specify microseconds bus scanning intervals

2015-04-22 Thread Evgeniy Polyakov
Hi Looks good to me. Greg, please pull it into your tree. Acked-by: Evgeniy Polyakov 23.04.2015, 00:56, "Dmitry Khromov" : > Signed-off-by: Dmitry Khromov > --- >  drivers/w1/w1.c | 17 - >  1 file changed, 16 insertions(+), 1 deletion(-) > > di

Re: [PATCH] Avoid null-pointer access in w1/slaves/w1_therm

2015-04-17 Thread Evgeniy Polyakov
Hi 16.04.2015, 15:00, "Thorsten Bschorr" : >> Let's push this patch upstream as a temporal fix until we are ready with the >> new solution. >> Thorsten, does it fix your crash? > > I'm running David's refcounting-patch for about 3 weeks now on my 3.18.y > kernel, and did not observe any problems

Re: [PATCH] Avoid null-pointer access in w1/slaves/w1_therm

2015-04-16 Thread Evgeniy Polyakov
Hi David 16.04.2015, 06:52, "David Fries" : > It has not been solved.  Evgeniy would like to make use of the sysfs > device management instead of the current reference counting, however I > haven't heard any volunteers to do that work.  I posted a quick fix > patch

Re: [PATCH] Avoid null-pointer access in w1/slaves/w1_therm

2015-03-18 Thread Evgeniy Polyakov
Hi 18.03.2015, 07:20, "David Fries" : >  static void w1_therm_remove_slave(struct w1_slave *sl) >  { > + int refcnt = atomic_sub_return(1, THERM_REFCNT(sl->family_data)); > + while(refcnt) { > + msleep(1000); > + refcnt = atomic_read(THERM_REFCNT(sl->family_data)); > + } >  kfree(sl->famil

Re: [PATCH] Avoid null-pointer access in w1/slaves/w1_therm

2015-03-14 Thread Evgeniy Polyakov
Hi David 12.03.2015, 03:54, "David Fries" : > Would that be removing all four refcnt, w1_slave, w1_master, > w1_family, w1_cb_block, or just some of them?  It sounds good to me, > if that had bugs there would be much more than just the w1 system > relying on it.  I don't know enough about that sys

Re: [PATCH] Avoid null-pointer access in w1/slaves/w1_therm

2015-03-10 Thread Evgeniy Polyakov
Hi 10.03.2015, 02:09, "David Fries" : > diff --git a/drivers/w1/slaves/w1_therm.c b/drivers/w1/slaves/w1_therm.c > index 1f11a20..39a9e6a 100644 > --- a/drivers/w1/slaves/w1_therm.c > +++ b/drivers/w1/slaves/w1_therm.c > @@ -59,9 +59,20 @@ MODULE_ALIAS("w1-family-" __stringify(W1_THERM_DS28EA00))

Re: [PATCH netfilter-next] xt_osf: Use continue to reduce indentation

2014-12-17 Thread Evgeniy Polyakov
ank you. Which tree should this patch go through? Please pull it in. Acked-by: Evgeniy Polyakov -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordo

Re: [patch V2 35/64] connector: Use ktime_get_ns()

2014-07-16 Thread Evgeniy Polyakov
Hi 17.07.2014, 01:04, "Thomas Gleixner" : > Replace the ever recurring: > ts = ktime_get_ts(); > ns = timespec_to_ns(&ts); > with > ns = ktime_get_ns(); > > Signed-off-by: Thomas Gleixner > Cc: Evgeniy Polyakov Looks good to me,

Re: [PATCH 0/3] "printk" related fixes for the 1-wire (w1) subsystem

2014-06-19 Thread Evgeniy Polyakov
_*" > function but also minor issues such as indentation and whitespace problems > in the immediate vicinity were addressed. Looks good to me Acked-by: Evgeniy Polyakov -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord.

Re: revised driver: w1_ds2406

2014-06-19 Thread Evgeniy Polyakov
y submitted. > > Signed-off-by: Scott Alfter Greg, this patch looks good, please pull it into your tree. If you already applied previous ds2406, let us know. Acked-by: Evgeniy Polyakov -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a messa

Re: new driver: w1_ds2406

2014-06-06 Thread Evgeniy Polyakov
Hi everyone Greg, please pull this patch into the tree 06.06.2014, 21:47, "Scott Alfter" : > On Thu, June 5, 2014 10:44, Evgeniy Polyakov wrote: >>  04.06.2014, 08:25, "Scott Alfter" : >>>  I have a project for which I needed support for the DS2406 address

Re: [PATCH] drivers/w1/w1_int.c: Fix style errors.

2014-05-31 Thread Evgeniy Polyakov
appropriate. Acked-by: Evgeniy Polyakov -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: [RFC] w1: fixes and bundling replies

2014-04-08 Thread Evgeniy Polyakov
Hi David 07.04.2014, 18:13, "David Fries" : > Evgeniy, > Could you review this set of patches in this thread?  Thanks. I've checked it and it looks good, please resend it to lkml and Greg Kroah-Hartman, he will pull it into the tree. Feel free to add my Acked-by. -- To

Re: [PATCH 1/4] w1: Remove excess dependencies on W1 for masters and slaves

2014-03-04 Thread Evgeniy Polyakov
Hi everyone 22.02.2014, 11:30, "Alexander Shiyan" : > Configuration for masters and slaves is included only if W1 symbol > enabled, so no reason to check it once more. > > Signed-off-by: Alexander Shiyan > --- >  drivers/w1/masters/Kconfig | 3 +-- >  drivers/w1/slaves/Kconfig  | 5 - >  2 file

Re: [PATCH] w1: bundle reply if the request was bundled

2014-02-20 Thread Evgeniy Polyakov
Hi 19.02.2014, 11:19, "David Fries" : > A program can bundle multiple messages and commands together when > making one wire requests, which is going to be much more efficient > than sending lots of little packets one write at a time.  With this > change the kernel will then bundle responses to req

Re: [PATCH] w1-gpio: handle of_get_gpio() returning -EPROBE_DEFER better

2014-02-18 Thread Evgeniy Polyakov
t > controller). In this case you still want to retry probing later instead > of ignoring the problem. > > Does that make it clearer? Well, yes, thank you :) Greg, please pull this patch into your tree. Acked-by: Evgeniy Polyakov -- To unsubscribe from this list: send the line "

Re: [patch v2] w1: small type cleanup in sysfs

2014-02-14 Thread Evgeniy Polyakov
his is more of a problem for static checkers than a real life issue, > but it's simple enough to fix. > > Acked-by: David Fries > Signed-off-by: Dan Carpenter Looks good to me, thank you Acked-by: Evgeniy Polyakov -- To unsubscribe from this list: send the line "unsubscribe linu

Re: [PATCH 16/16] hold bus_mutex in netlink and search

2014-01-15 Thread Evgeniy Polyakov
arch.  This was causing the ds2490 hardware to stop > responding when both netlink and /sys slaves were executing bus > commands at the same time. > > Signed-off-by: David Fries Looks good to me, thank you. Greg, please pull the whole set if you hadn't yet Acked-by: Evgeniy Polyakov

Re: [PATCH 00/14] w1: async netlink, search, fixes, and improvements

2013-12-29 Thread Evgeniy Polyakov
Hi 29.12.2013, 10:46, "David Fries" : > This patch series aims to extend and improve the netlink interface to > the one wire system.  Netlink is exposed as a socket interface which I've reviewed all your patches and everything looks good. Thank you David Acked-by: Ev

Re: [PATCH 4/4] w1: call put_device if device_register fails

2013-12-23 Thread Evgeniy Polyakov
Hi 19.12.2013, 03:47, "Greg KH" : > I don't seem to have this patch anywhere, can someone please resend it > to me? Andrew Morton picked it up http://www.spinics.net/lists/mm-commits/msg101101.html -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a messa

Re: [PATCH 4/4] w1: call put_device if device_register fails

2013-12-14 Thread Evgeniy Polyakov
device right after it fails. Looks good to me, thank you Greg, please pull it into your treee > Signed-off-by: Levente Kurusa Acked-by: Evgeniy Polyakov -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger

  1   2   3   4   5   6   7   8   9   10   >