linux-stable.git/drivers/w1, branch linux-3.13.y

drivers/w1/masters/w1-gpio.c: use dev_get_platdata()

2013-11-15T00:32:21+00:00

Use the wrapper function for retrieving the platform data instead of
accessing dev->platform_data directly.  This is a cosmetic change to make
the code simpler and enhance the readability.

Signed-off-by: Jingoo Han 
Acked-by: Evgeniy Polyakov 
Cc: Greg KH 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

drivers: w1: make w1_slave::flags long to avoid memory corruption

2013-11-13T03:09:35+00:00

On architectures where long is more then 32 bits, modifying a 32-bit field
with set_bit (and other atomic bit operations) may cause bytes following
the field to by modified.

Because the endianness of the bits within a field is the native endianness
of the CPU[1], on big-endian machines, bit number zero is in the last byte
of the field.

Therefore, `set_bit(0, ptr)' on a 64-bit big-endian machine is roughly
equivalent to `((char *)ptr)[7] |= 1', and since w1 driver uses a 32-bit
field for holding the flags, this causes bytes beyond the field to be
modified.

[1] From Documentation/atomic_ops.txt:

    Native atomic bit operations are defined to operate on objects
    aligned to the size of an "unsigned long" C data type, and are
    least of that size.  The endianness of the bits within each
    "unsigned long" are the native endianness of the cpu.

Signed-off-by: Michal Nazarewicz 
Cc: Evgeniy Polyakov 
Cc: Greg KH 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

drivers/w1/masters/ds1wm.cuse dev_get_platdata()

2013-11-13T03:09:35+00:00

Use the wrapper function for retrieving the platform data instead of
accessing dev->platform_data directly.  This is a cosmetic change to make
the code simpler and enhance the readability.

Signed-off-by: Jingoo Han 
Acked-by: Evgeniy Polyakov 
Cc: Greg KH 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

w1-gpio: Use devm_* functions

2013-10-29T23:58:18+00:00

Signed-off-by: Markus Pargmann 
Acked-by: Evgeniy Polyakov 
Signed-off-by: Greg Kroah-Hartman

w1-gpio: Detect of_gpio_error for first gpio

2013-10-29T23:58:18+00:00

The first DT gpio is necessary for this driver, but errors returned for
of_get_gpio are ignored.

This patch adds a return value check for the first of_get_gpio.

Signed-off-by: Markus Pargmann 
Acked-by: Evgeniy Polyakov 
Signed-off-by: Greg Kroah-Hartman

Merge 3.12-rc6 into char-misc-next

2013-10-19T20:02:47+00:00

We want the fixes in here as well.

Signed-off-by: Greg Kroah-Hartman

w1: omap-hdq: remove deprecated IRQF_DISABLED

2013-10-16T19:36:10+00:00

This patch proposes to remove the use of the IRQF_DISABLED flag

It's a NOOP since 2.6.35 and it will be removed one day.

Signed-off-by: Michael Opdenacker 
Acked-by: Evgeniy Polyakov 
Signed-off-by: Greg Kroah-Hartman

w1: ds1wm: remove deprecated IRQF_DISABLED

2013-10-16T19:36:10+00:00

This patch proposes to remove the use of the IRQF_DISABLED flag

It's a NOOP since 2.6.35 and it will be removed one day.

Signed-off-by: Michael Opdenacker 
Acked-by: Evgeniy Polyakov 
Signed-off-by: Greg Kroah-Hartman

w1 - call request_module with w1 master mutex unlocked

2013-10-07T07:12:14+00:00

request_module for w1 slave modules needs to be called with the w1
master mutex unlocked. Because w1_attach_slave_device gets always(?)
called with mutex locked, we need to temporarily unlock the w1 master
mutex for the loading of the w1 slave module.

Signed-off by: Hans-Frieder Vogt 
Acked-by: Evgeniy Polyakov 
Cc: stable  # 3.11+

Signed-off-by: Greg Kroah-Hartman

w1 - fix fops in w1_bus_notify

2013-10-07T07:12:14+00:00

Introduce a check to make sure that fops are only called if they have
been defined by the slave module.

Without this check modules like w1_smem cause a NULL pointer dereference
bug.

Signed-off by: Hans-Frieder Vogt 
Acked-by: Evgeniy Polyakov 
Cc: stable  # 3.11+

Signed-off-by: Greg Kroah-Hartman