that.
--
+-+
| David Fries|
| [EMAIL PROTECTED] |
+-+
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More
o shut about
everything down to do that.
--
+-+
| David Fries|
| [EMAIL PROTECTED] |
+-+
-
To unsubscribe from this list: se
directory on the new mount was fine, but the problem
directory on the old mount was still stale.
Yes it is a ext2 file system being exported. It is using the kernel
nfs server.
--
+-+
| David Fries
work already.
--
+-+
| David Fries|
| [EMAIL PROTECTED] |
+-+
-
To unsubscribe from this list: send the line "unsubscribe linux-k
(dd9jn+kvh)
eth0: Intel: port 0x300, irq 3, hwaddr 00:A0:C9:7A:CE:D5
eth0: autonegotiation failed; using 10mbs
eth0: MII detected; using 10mbs
eth0: media 10BaseT, silicon revision 4
eth0: no IPv6 routers present
--
+-+
| David
all I have is, accepted by university, hope it gets there...
--
+-+
| David Fries|
| [EMAIL PROTECTED]|
+-+
-
To unsubscribe from
at /proc/interrupts.
Signed-off-by: David Fries [EMAIL PROTECTED]
--
David Fries [EMAIL PROTECTED]
http://fries.net/~david/ (PGP encryption key available)
diff --git a/include/linux/acct.h b/include/linux/acct.h
index 302eb72..86b848d 100644
--- a/include/linux/acct.h
+++ b/include/linux/acct.h
return when I unplug the joystick. Thanks for fixing the other
drivers, I didn't think of looking at them.
--
David Fries [EMAIL PROTECTED]
http://fries.net/~david/pgpkey.txt
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More
by the USB HUB even if I'm
only touching the chair and floor! The kernel then removes the old
joystick and redetects it as a new joystick.
I've also included a test program to verify that a blocking select,
poll, and read call will return if the joystick is unplugged.
--
David Fries [EMAIL PROTECTED
-by: David Fries da...@fries.net
Cc: Matthew Garrett m...@redhat.com
Cc: Alessandro Zummo a.zu...@towertech.it
Cc: Uwe Kleine-König u.kleine-koe...@pengutronix.de
Cc: Andrew Morton a...@linux-foundation.org
---
On Fri, Sep 07, 2012 at 04:56:26PM -0700, Andrew Morton wrote:
On Sat, 25 Aug 2012 21:19:31
...@canb.auug.org.au
--
David Fries da...@fries.netPGP pub CB1EE8F0
http://fries.net/~david/
signature.asc
Description: Digital signature
On Thu, Sep 13, 2012 at 11:34:29PM +1000, Stephen Rothwell wrote:
Hi David,
On Thu, 13 Sep 2012 08:24:08 -0500 David Fries da...@fries.net wrote:
On Thu, Sep 13, 2012 at 06:11:27PM +1000, Stephen Rothwell wrote:
Hi all,
After merging the final tree, today's linux-next build
.
This uses only CONFIG_RTC_HCTOSYS_DEVICE for conditional compilation
instead of it and CONFIG_RTC_HCTOSYS. rtc_hctosys_ret was moved to
class.c so rtc_hctosys can be removed with just a Kconfig change if
boot time setting isn't desired.
Signed-off-by: David Fries da...@fries.net
Cc: Matthew Garrett m
commit d05074b1beb66d40283228bd226f7244407375bb
Author: David Fries [EMAIL PROTECTED]
Date: Sat Jan 19 18:06:49 2008 -0600
The extra rom[0] check is flagging valid temperatures as invalid when
there is already a CRC data transmission check.
w1_therm_read_bin()
if (rom[8] == crc rom[0
commit c4ad974f3acef11e538e825ca0cb73be6faf2461
Author: David Fries [EMAIL PROTECTED]
Date: Sun Nov 4 18:14:21 2007 -0600
Corrected the decoding of negative C temperatures. The code did a binary
OR of two bytes to make a 16 bit value, but assignd it to an integer.
This caused the value
. The
.001 C does truncate one bit of precision from the ds18b20 by the way.
45 01 4b 46 7f ff 0b 10 84 : crc=84 YES
45 01 4b 46 7f ff 0b 10 84 t=20312
--
David Fries [EMAIL PROTECTED]
http://fries.net/~david/ (PGP encryption key available)
--
To unsubscribe from this list: send the line
= 719.96 ?F
The range for the sensor is -55 to 125 C, if an application didn't
care about precision they could store it in a signed 8 bit value just
fine.
--
David Fries [EMAIL PROTECTED]
http://fries.net/~david/ (PGP encryption key available)
--
To unsubscribe from this list: send the line unsubscribe
. It would be
easier to parse. If the conversion fails return 0 bytes. Just an
idea, but if someone wants it they can write the patch.
--
David Fries [EMAIL PROTECTED]
http://fries.net/~david/ (PGP encryption key available)
--
To unsubscribe from this list: send the line unsubscribe linux-kernel
as
it will now be 1000 times bigger. Fortunately there can't be that
many users out there, or some of these bugs will have been fixed by
now, such as the negative C error (see previous patch) that makes me
think the ds18b20 is the better choice to change because of the
current bugs.
Signed-off-by: David Fries
if they are detected without the driver constantly
scanning the bus for the same old devices. By the way the default bus
scan takes 7.7 seconds to complete and then sleeps 10 seconds before
the next scan leaving the bus idle for only 56% of the time.
--
David Fries [EMAIL PROTECTED]
http
.
Signed-off-by: David Fries [EMAIL PROTECTED]
Index: drivers/net/Kconfig
===
RCS file:
/home/david/kernel/k/spacedout/patches/linux/drivers/net/Attic/Kconfig,v
retrieving revision 1.1.2.1
diff -u -r1.1.2.1 Kconfig
--- drivers/net
On Fri, Jul 13, 2007 at 09:11:28PM -0700, Kok, Auke wrote:
[adding netdev]
David Fries wrote:
When I did a software suspend to disk then resumed the Intel network
card using eepro100 driver would be unable to transmit packets. I
tracked this down and found a register write after the print
On Tue, Nov 12, 2013 at 05:07:14AM +0400, Evgeny Boger wrote:
+David Fries da...@fries.net
Hi David,
Would you please comment on this?
On Mon, Nov 11, 2013 at 06:36:54PM +0400, Evgeny Boger wrote:
Strong pullup is emulated by driving pin logic high after write
command when
using tri
On Wed, Nov 13, 2013 at 05:15:52AM +0400, Evgeny Boger wrote:
11/12/2013 12:01 PM, David Fries:
On Tue, Nov 12, 2013 at 05:07:14AM +0400, Evgeny Boger wrote:
+David Fries da...@fries.net
Hi David,
Would you please comment on this?
On Mon, Nov 11, 2013 at 06:36:54PM +0400, Evgeny Boger
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 requests that were
bundled, where it's probably
one despite the tmp 1 check.
Acked-by: David Fries da...@fries.net
On Tue, Feb 11, 2014 at 07:08:26PM +0300, Dan Carpenter wrote:
On 64 bit systems, a large value for tmp could be truncated so we
still end up with a -max_slave_count which is less than one despite the
tmp 1 check
On Fri, Feb 21, 2014 at 01:07:28AM +0400, Evgeniy Polyakov wrote:
Your approach and patch seem correct, but I worry about how old
commands are processed. Do I get it right, that replies to old
non-bundle commands are queued back instead of sending immediately,
but since it is not bundle but
.
Signed-off-by: David Fries da...@fries.net
---
drivers/w1/w1_netlink.c | 44 ++--
1 file changed, 26 insertions(+), 18 deletions(-)
diff --git a/drivers/w1/w1_netlink.c b/drivers/w1/w1_netlink.c
index 5234964..a02704a 100644
--- a/drivers/w1/w1_netlink.c
identifies there being multiple of
the next.
Signed-off-by: David Fries da...@fries.net
---
Documentation/connector/connector.txt | 13 ++---
drivers/connector/connector.c | 17 +++--
include/linux/connector.h |1 +
3 files changed, 26 insertions(+), 5
such as the data returned from a read.
Signed-off-by: David Fries da...@fries.net
Some of the common variable names have been standardized as follows.
struct cn_msg *cn
struct w1_netlink_msg *msg
struct w1_netlink_cmd *cmd
struct w1_master *dev
When an argument and a function scope variable would collide, add
--
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/
--
David Fries da...@fries.netPGP pub CB1EE8F0
http
On Wed, May 07, 2014 at 01:42:12AM +0400, Alexey Khoroshilov wrote:
On 01.05.2014 07:48, David Fries wrote:
On Thu, May 01, 2014 at 12:37:58AM +0400, Alexey Khoroshilov wrote:
w1_process_callbacks() expects to be called with dev-list_mutex held,
but it is the fact only in w1_process
Acked-by: David Fries da...@fries.net
On Wed, May 07, 2014 at 01:26:04AM +0400, Alexey Khoroshilov wrote:
w1_process_callbacks() expects to be called with dev-list_mutex held,
but it is the fact only in w1_process(). __w1_remove_master_device()
calls w1_process_callbacks() after it releases
I could submit these patches as in, which would require the previous
set, or I could merge the documentation into the previous set and
resubmit them all since they haven't made it into the kernel tree yet.
Opinions?
Here's a small refcnt fix, skipping sending non-error messages, and
documentation
Signed-off-by: David Fries da...@fries.net
---
Documentation/connector/connector.txt | 10 +++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/Documentation/connector/connector.txt
b/Documentation/connector/connector.txt
index e5c5f5e..9bdfc1a 100644
--- a/Documentation
Otherwise there's an extra reply being sent out for each async
message. Some commands such as W1_CMD_LIST_SLAVES will be identical
except one message has data and the other doesn't making it difficult
for a program to know if all the slaves just vanished or what happened.
Signed-off-by: David
.
Signed-off-by: David Fries da...@fries.net
---
drivers/w1/w1_netlink.c | 44 ++--
1 file changed, 26 insertions(+), 18 deletions(-)
diff --git a/drivers/w1/w1_netlink.c b/drivers/w1/w1_netlink.c
index 5234964..a02704a 100644
--- a/drivers/w1/w1_netlink.c
I wasn't sure on the length, so I looked it up and documented it.
Signed-off-by: David Fries da...@fries.net
---
drivers/w1/w1_netlink.h | 25 +
1 file changed, 25 insertions(+)
diff --git a/drivers/w1/w1_netlink.h b/drivers/w1/w1_netlink.h
index 1e9504e..c646a98
On Tue, Feb 04, 2014 at 03:59:38AM +0400, z...@ioremap.net wrote:
Hi
03.02.2014, 05:15, David Fries da...@fries.net:
I could submit these patches as in, which would require the previous
set, or I could merge the documentation into the previous set and
resubmit them all since
netlink and /sys slaves were executing bus
commands at the same time.
Signed-off-by: David Fries da...@fries.net
---
This fixes existing bugs, tacking it to the end of the previous patch
series.
drivers/w1/masters/ds1wm.c |4 +++-
drivers/w1/masters/ds2490.c |8 ++--
drivers/w1
15.01.2014, 08:52, David Fries da...@fries.net:
The bus_mutex needs to be taken to serialize access to a specific bus.
netlink wasn't updated when bus_mutex was added and was calling
without that lock held, and not all of the masters were holding the
bus_mutex in a search. This was causing
This patch series aims to extend and improve the netlink interface to
the one wire system. Netlink is exposed as a socket interface which
is normally assumed to be asynchronous which only blocks waiting on
incoming packets or a fulls end buffer, but were executed on the
calling thread. A bus
,
there is space.
Instead, if there isn't space send and clear the buffer, then there is
always space for the slave id.
Signed-off-by: David Fries da...@fries.net
Cc: sta...@vger.kernel.org
Acked-by: Evgeniy Polyakov z...@ioremap.net
---
drivers/w1/w1_netlink.c | 25 +
1 file
early if the
hardware is removed in the middle of a search.
Signed-off-by: David Fries da...@fries.net
Acked-by: Evgeniy Polyakov z...@ioremap.net
Cc: Marcin Jurkowski marci...@gmail.com
Cc: Josh Boyer jwbo...@gmail.com
Cc: Sven Geggus li...@fuchsschwanzdomain.de
---
drivers/w1/w1.c |3
It's valid to set the search count to 0 to stop searching, so don't
wake up the search thread to not search.
Signed-off-by: David Fries da...@fries.net
Acked-by: Evgeniy Polyakov z...@ioremap.net
---
drivers/w1/w1.c |4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git
Search will detect at most max_slave_count devices per run, if there
are more pick up the next search where the previous left off.
Signed-off-by: David Fries da...@fries.net
Acked-by: Evgeniy Polyakov z...@ioremap.net
---
drivers/w1/w1.c | 18 +++---
drivers/w1/w1.h |3 +++
2
the first time the count is reached and
there were more devices to discover to let the user know why not
all the devices were found.
Signed-off-by: David Fries da...@fries.net
Acked-by: Evgeniy Polyakov z...@ioremap.net
---
drivers/w1/w1.c | 30 --
drivers/w1/w1.h |2
such as add/remove master/slave devices.
Signed-off-by: David Fries da...@fries.net
Acked-by: Evgeniy Polyakov z...@ioremap.net
---
drivers/w1/w1.h |5 +
drivers/w1/w1_netlink.c | 42 +-
2 files changed, 30 insertions(+), 17 deletions(-)
diff --git
This allows replying only to the requestor portid while still
supporting broadcasting. Pass 0 to portid for the previous behavior.
Signed-off-by: David Fries da...@fries.net
Acked-by: Evgeniy Polyakov z...@ioremap.net
---
Documentation/connector/cn_test.c |2 +-
drivers/connector
memory when needed, so move kzalloc into w1_get_slaves
where it was used.
Signed-off-by: David Fries da...@fries.net
Acked-by: Evgeniy Polyakov z...@ioremap.net
---
drivers/w1/w1.c |6 +--
drivers/w1/w1.h |3 ++
drivers/w1/w1_netlink.c | 125
Netlink is a socket interface and is expected to be asynchronous.
Clients can now make w1 requests without blocking by making use of the
w1_master thread to process netlink commands which was previously only
used for doing an automatic bus search.
Signed-off-by: David Fries da...@fries.net
Acked
Use a #define for the usb vendor request type, clear the status
byte and use that instead of a magic offset in checking if idle.
Signed-off-by: David Fries da...@fries.net
Acked-by: Evgeniy Polyakov z...@ioremap.net
---
drivers/w1/masters/ds2490.c | 23 +++
1 file changed
as it is an interrupt pipe
(bulk worked, it was just technically the wrong call).
Signed-off-by: David Fries da...@fries.net
Acked-by: Evgeniy Polyakov z...@ioremap.net
---
drivers/w1/masters/ds2490.c | 22 ++
1 file changed, 14 insertions(+), 8 deletions(-)
diff --git
went from 23.16 seconds to about 3 seconds,
this takes the time for the search down to .307346 seconds.
Signed-off-by: David Fries da...@fries.net
Acked-by: Evgeniy Polyakov z...@ioremap.net
---
Documentation/w1/masters/ds2490 |2 -
drivers/w1/masters/ds2490.c | 108
to do with that printed temperature value.
Modify the code to store the last good conversion in family_data,
which is designed for custom data structures.
Signed-off-by: David Fries da...@fries.net
Acked-by: Evgeniy Polyakov z...@ioremap.net
---
drivers/w1/slaves/w1_therm.c | 21
netlink and /sys slaves were executing bus
commands at the same time.
Signed-off-by: David Fries da...@fries.net
Acked-by: Evgeniy Polyakov z...@ioremap.net
---
drivers/w1/masters/ds1wm.c |4 +++-
drivers/w1/masters/ds2490.c |8 ++--
drivers/w1/w1_netlink.c | 13 +++--
3
Switch the code documentation format style to DocBook format, enable
DocBook documentation generation, and fix some comments.
Signed-off-by: David Fries da...@fries.net
Acked-by: Evgeniy Polyakov z...@ioremap.net
---
Documentation/DocBook/Makefile |2 +-
Documentation/DocBook/w1.tmpl | 101
merged upstream won't be updated unless I have
another set of changes.
cd5f76716a37ebef3e8b771d672ba552a25dafd5
git://github.com/dfries/linux.git w1_rework
--
David Fries da...@fries.netPGP pub CB1EE8F0
http://fries.net/~david/
--
To unsubscribe from this list: send the line unsubscribe linux
It's valid to set the search count to 0 to stop searching, so don't
wake up the search thread to not search.
Signed-off-by: David Fries da...@fries.net
Cc: Evgeniy Polyakov z...@ioremap.net
---
drivers/w1/w1.c |4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/w1
,
there is space.
Instead, if there isn't space send and clear the buffer, then there is
always space for the slave id.
Signed-off-by: David Fries da...@fries.net
Cc: sta...@vger.kernel.org
Cc: Evgeniy Polyakov z...@ioremap.net
---
drivers/w1/w1_netlink.c | 25 +
1 file
early if the
hardware is removed in the middle of a search.
Signed-off-by: David Fries da...@fries.net
Cc: Evgeniy Polyakov z...@ioremap.net
Cc: Marcin Jurkowski marci...@gmail.com
Cc: Josh Boyer jwbo...@gmail.com
Cc: Sven Geggus li...@fuchsschwanzdomain.de
---
drivers/w1/w1.c |3 +--
drivers
Switch the code documentation format style to DocBook format, enable
DocBook documentation generation, and fix some comments.
Signed-off-by: David Fries da...@fries.net
Cc: Evgeniy Polyakov z...@ioremap.net
---
Documentation/DocBook/Makefile |2 +-
Documentation/DocBook/w1.tmpl | 101
memory when needed, so move kzalloc into w1_get_slaves
where it was used.
Signed-off-by: David Fries da...@fries.net
Cc: Evgeniy Polyakov z...@ioremap.net
---
drivers/w1/w1.c |6 +--
drivers/w1/w1.h |3 ++
drivers/w1/w1_netlink.c | 125
Use a #define for the usb vendor request type, clear the status
byte and use that instead of a magic offset in checking if idle.
Signed-off-by: David Fries da...@fries.net
Cc: Evgeniy Polyakov z...@ioremap.net
---
drivers/w1/masters/ds2490.c | 23 +++
1 file changed, 15
Netlink is a socket interface and is expected to be asynchronous.
Clients can now make w1 requests without blocking by making use of the
w1_master thread to process netlink commands which was previously only
used for doing an automatic bus search.
Signed-off-by: David Fries da...@fries.net
Cc
This allows replying only to the requestor portid while still
supporting broadcasting. Pass 0 to portid for the previous behavior.
Signed-off-by: David Fries da...@fries.net
Cc: Evgeniy Polyakov z...@ioremap.net
---
Documentation/connector/cn_test.c |2 +-
drivers/connector/cn_proc.c
went from 23.16 seconds to about 3 seconds,
this takes the time for the search down to .307346 seconds.
Signed-off-by: David Fries da...@fries.net
Cc: Evgeniy Polyakov z...@ioremap.net
---
Documentation/w1/masters/ds2490 |2 -
drivers/w1/masters/ds2490.c | 108
the first time the count is reached and
there were more devices to discover to let the user know why not
all the devices were found.
Signed-off-by: David Fries da...@fries.net
Cc: Evgeniy Polyakov z...@ioremap.net
---
drivers/w1/w1.c | 30 --
drivers/w1/w1.h |2 ++
2
such as add/remove master/slave devices.
Signed-off-by: David Fries da...@fries.net
Cc: Evgeniy Polyakov z...@ioremap.net
---
drivers/w1/w1.h |5 +
drivers/w1/w1_netlink.c | 42 +-
2 files changed, 30 insertions(+), 17 deletions(-)
diff --git
to do with that printed temperature value.
Modify the code to store the last good conversion in family_data,
which is designed for custom data structures.
Signed-off-by: David Fries da...@fries.net
Cc: Evgeniy Polyakov z...@ioremap.net
---
drivers/w1/slaves/w1_therm.c | 21
as it is an interrupt pipe
(bulk worked, it was just technically the wrong call).
Signed-off-by: David Fries da...@fries.net
Cc: Evgeniy Polyakov z...@ioremap.net
---
drivers/w1/masters/ds2490.c | 22 ++
1 file changed, 14 insertions(+), 8 deletions(-)
diff --git a/drivers/w1
This patch series aims to extend and improve the netlink interface to
the one wire system. Netlink is exposed as a socket interface which
is normally assumed to be asynchronous which only blocks waiting on
incoming packets or a fulls end buffer, but were executed on the
calling thread. A bus
Search will detect at most max_slave_count devices per run, if there
are more pick up the next search where the previous left off.
Signed-off-by: David Fries da...@fries.net
Cc: Evgeniy Polyakov z...@ioremap.net
---
drivers/w1/w1.c | 18 +++---
drivers/w1/w1.h |3 +++
2 files
w1_rework
--
David Fries da...@fries.netPGP pub CB1EE8F0
http://fries.net/~david/
--
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
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/
--
David Fries da...@fries.netPGP pub CB1EE8F0
http://fries.net/~david/
--
To unsubscribe from this list: send the line unsubscribe
Evgeniy,
Could you review this set of patches in this thread? Thanks.
On Sat, Mar 22, 2014 at 08:27:44PM -0500, David Fries wrote:
On Fri, Feb 21, 2014 at 01:07:28AM +0400, Evgeniy Polyakov wrote:
Your approach and patch seem correct, but I worry about how old
commands are processed. Do I
A program can bundle a sequence of commands in one netlink packet but
the kernel was having to reply in many different packets, this adds a
flag W1_CN_BUNDLE in the cn_msg.flags to allow the kernel to bundle
the replies in to one message. This is opt in to avoid breaking
programs that aren't
such as the data returned from a read.
Some of the common variable names have been standardized as follows.
struct cn_msg *cn
struct w1_netlink_msg *msg
struct w1_netlink_cmd *cmd
struct w1_master *dev
When an argument and a function scope variable would collide, add req_
to the argument.
Signed-off-by: David
.
Signed-off-by: David Fries da...@fries.net
Acked-by: Evgeniy Polyakov z...@ioremap.net
---
drivers/w1/w1_netlink.c | 44 ++--
1 file changed, 26 insertions(+), 18 deletions(-)
diff --git a/drivers/w1/w1_netlink.c b/drivers/w1/w1_netlink.c
index 5234964
identifies there being multiple of
the next.
Signed-off-by: David Fries da...@fries.net
Acked-by: Evgeniy Polyakov z...@ioremap.net
---
Documentation/connector/connector.txt | 13 ++---
drivers/connector/connector.c | 17 +++--
include/linux/connector.h
window. Let me know if I should rewrite it for
stable and which kernel version.
The other two patches are more feature based changes.
On Tue, Apr 08, 2014 at 10:37:07PM -0500, David Fries wrote:
If the message type is W1_MASTER_CMD or W1_SLAVE_CMD, then a reference
is taken when searching
are setup before the add
goes out.
Belisko Marek reported this change fixed the deadlock he was seeing on
ARM device tree, while testing on my x86-64 system never saw the
deadlock.
Reported-by: Belisko Marek marek.beli...@gmail.com
Signed-off-by: David Fries da...@fries.net
---
Evgeniy any
+0200, Belisko Marek wrote:
Hi,
On Mon, Mar 24, 2014 at 1:01 PM, Evgeniy Polyakov z...@ioremap.net wrote:
Hi everyone
24.03.2014, 01:50, David Fries da...@fries.net:
On Mon, Mar 17, 2014 at 10:38:13PM +0100, Belisko Marek wrote:
booting latest 3.14-rc7 on gta04 board gives following
On Wed, Feb 05, 2014 at 03:48:45AM +0400, z...@ioremap.net wrote:
Hi
04.02.2014, 09:51, David Fries da...@fries.net:
Help me understand what the protocol is supposed to be. Assuming
there aren't any errors, is there supposed to be a
w1_netlink_send_error generated reply per netlink
On Sat, Feb 08, 2014 at 01:23:43AM +0400, z...@ioremap.net wrote:
Hi
07.02.2014, 10:00, David Fries da...@fries.net:
Here's a patch to implement that. Is this what you have in mind?
From 4ed65d81b0121a8c191a9833d041484e9097198b Mon Sep 17 00:00:00 2001
From: David Fries da
This reverts ds1wm.c from commit d3a8a9dbb903c73a7ec2deae4c9b7d74b6834f4c.
Of the three files changed ds1wm.c ds2490.c and w1_netlink.c, it turns out
ds1wm.c was locking bus_mutex, but inside the loop and I missed it.
Reverting ds1wm.c to the previous version.
Signed-off-by: David Fries da
Greg Kroah-Hartman,
These issues were found by Dan Carpenter with a static checker and will check
message buffer lengths from userspace or avoid length overflows. Evgeniy
Polyakov has given his ack, and they can be applied to the stable branch as
well.
[PATCH 1/2] cn: verify msg-len before
Acked-by: Evgeniy Polyakov z...@ioremap.net
Signed-off-by: David Fries da...@fries.net
---
drivers/connector/connector.c |6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/connector/connector.c b/drivers/connector/connector.c
index f612d68..30f5228 100644
--- a/drivers/connector
Reported-by: Dan Carpenter dan.carpen...@oracle.com
Acked-by: Evgeniy Polyakov z...@ioremap.net
Signed-off-by: David Fries da...@fries.net
---
drivers/w1/w1_netlink.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/w1/w1_netlink.c b/drivers/w1/w1_netlink.c
index
Reported-by: Dan Carpenter dan.carpen...@oracle.com
Signed-off-by: David Fries da...@fries.net
---
drivers/w1/w1_netlink.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/w1/w1_netlink.c b/drivers/w1/w1_netlink.c
index dd96562..881597a 100644
--- a/drivers/w1
These issues were found by Dan Carpenter with a static checker.
Evgeniy are you okay with these?
[PATCH 1/2] cn: verify msg-len before making callback
[PATCH 2/2] w1: avoid potential u16 overflow
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to
Signed-off-by: David Fries da...@fries.net
---
drivers/connector/connector.c |6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/connector/connector.c b/drivers/connector/connector.c
index f612d68..30f5228 100644
--- a/drivers/connector/connector.c
+++ b/drivers/connector/connector.c
on this.
Mariusz Gorski (2):
w1: slaves: w1_therm: Extract read_rom function
w1: slaves: w1_therm: Add temp attribute
drivers/w1/slaves/w1_therm.c | 83
++--
1 file changed, 65 insertions(+), 18 deletions(-)
--
2.2.1
--
David Fries da...@fries.net
--
David Fries da...@fries.netPGP pub CB1EE8F0
http://fries.net/~david/
--
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
wanted to point out that this returns without unlocking bus_mutex.
--
David Fries da...@fries.netPGP pub CB1EE8F0
http://fries.net/~david/
--
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
On Sat, Mar 14, 2015 at 11:55:16PM +0300, Evgeniy Polyakov wrote:
Hi David
12.03.2015, 03:54, David Fries da...@fries.net:
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
On Wed, Mar 18, 2015 at 06:18:53PM +0300, Evgeniy Polyakov wrote:
Hi
18.03.2015, 07:20, David Fries da...@fries.net:
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
, but keeping the original sleep-time tm.
Oops, sorry, it got lost in the shuffle, here's the first patch again
(the others were for debugging and increase that time and so wouldn't
go upstream anyway).
From 777f5fd75f5f99a3352863e83d226c7b65ebdaa4 Mon Sep 17 00:00:00 2001
From: David Fries da
a timed lock or
try lock first.
--
David Fries da...@fries.netPGP pub CB1EE8F0
http://fries.net/~david/
--
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
On Tue, Mar 10, 2015 at 04:52:00PM +0300, Evgeniy Polyakov wrote:
Hi
10.03.2015, 02:09, David Fries da...@fries.net:
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
1 - 100 of 228 matches
Mail list logo