Hi,
On 31-08-18 11:57, Hans de Goede wrote:
Hi,
On 30-08-18 23:05, Wolfram Sang wrote:
On Wed, Aug 29, 2018 at 05:27:53PM +0200, Hans de Goede wrote:
Commit a3d411fb38c0 ("i2c: designware: Disable pm for PMIC i2c-bus even if
there is no _SEM method"), always set the pm_disabled flag on the I2C7
controller, even if its bus was not shared with the PUNIT.
This was a workaround for various suspend/resume issues, after the
following 2 commits this workaround is no longer necessary:
Commit 541527728341 ("PM: i2c-designware-platdrv: Suspend/resume at the
late/early stages")
Commit e6ce0ce34f65 ("ACPI / LPSS: Add device link for CHT SD card
dependency on I2C")
Therefor this commit removes this workaround.
After this commit the pm_disabled flag is only used to indicate that the
bus is shared with the PUNIT and after other recent changes we no longer
call dev_pm_syscore_device(dev, true), so we are no longer actually
disabling (non-runtime) pm, so this commit also renames the flag to
shared_with_punit to better reflect what it is for.
Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx>
Waiting for tags here... I assume it is OK to have applied patch 1/2
from the previous series independently of this patch?
Yes that is fine, given that that is a bugfix for a regression
introduced in 4.18 it is actually a good thing that you've already
applied patch 1/2. I assume you've added this to a fixes branch?
Never mind I just saw you said you put it in for current.
Regards,
Hans
