From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from saturn.retrosnub.co.uk ([178.18.118.26]:41256 "EHLO saturn.retrosnub.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752856AbcHOOqq (ORCPT ); Mon, 15 Aug 2016 10:46:46 -0400 Subject: Re: [PATCH] iio: pressure: bmp280: Fix incorrect pointer access in PM callbacks To: Jarkko Nikula , Linus Walleij References: <1471002271-11426-1-git-send-email-jarkko.nikula@linux.intel.com> Cc: "linux-iio@vger.kernel.org" , Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald-Stadler From: Jonathan Cameron Message-ID: Date: Mon, 15 Aug 2016 15:46:44 +0100 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Sender: linux-iio-owner@vger.kernel.org List-Id: linux-iio@vger.kernel.org On 12/08/16 13:36, Jarkko Nikula wrote: > On 08/12/2016 03:21 PM, Linus Walleij wrote: >> On Fri, Aug 12, 2016 at 1:44 PM, Jarkko Nikula >> wrote: >> >>> BMP280 runtime PM callbacks are incorrectly casting dev_get_drvdata() to >>> struct bmp280_data * while bmp280_common_probe() sets driver data to >>> struct iio_dev *. Fix this and get the struct bmp280_data * from private >>> data of struct iio_dev. >>> >>> Fixes: 3d838118c6aa ("iio: pressure: bmp280: add power management") >>> Signed-off-by: Jarkko Nikula >>> --- >>> For v4.8-rc. >> >> Sent a fix for this already two weeks ago: >> http://marc.info/?l=linux-iio&m=146965159217655&w=2 >> >> Just waiting for it to be picked up and merged. >> > Ok, please feel free to add my tested by to your patch then > > Tested-by: Jarkko Nikula Added.