lkml.org 
[lkml]   [2018]   [May]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 3.18 155/185] hwmon: (pmbus/adm1275) Accept negative page register values
    Date
    3.18-stable review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Guenter Roeck <linux@roeck-us.net>

    [ Upstream commit ecb29abd4cb0670c616fb563a078f25d777ce530 ]

    A negative page register value means that no page needs to be
    selected. This is used by status register read operations and needs
    to be accepted. The failure to do so so results in missed status
    and limit registers.

    Fixes: da8e48ab483e1 ("hwmon: (pmbus) Always call _pmbus_read_byte in core driver")
    Signed-off-by: Guenter Roeck <linux@roeck-us.net>
    Signed-off-by: Sasha Levin <alexander.levin@microsoft.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
    ---
    drivers/hwmon/pmbus/adm1275.c | 4 ++--
    1 file changed, 2 insertions(+), 2 deletions(-)

    --- a/drivers/hwmon/pmbus/adm1275.c
    +++ b/drivers/hwmon/pmbus/adm1275.c
    @@ -67,7 +67,7 @@ static int adm1275_read_word_data(struct
    const struct adm1275_data *data = to_adm1275_data(info);
    int ret = 0;

    - if (page)
    + if (page > 0)
    return -ENXIO;

    switch (reg) {
    @@ -144,7 +144,7 @@ static int adm1275_write_word_data(struc
    {
    int ret;

    - if (page)
    + if (page > 0)
    return -ENXIO;

    switch (reg) {

    \
     
     \ /
      Last update: 2018-05-28 18:18    [W:4.108 / U:0.020 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site