On Tue, Oct 17, 2017 at 12:07:37AM +0200, Rafael J. Wysocki wrote:

On Monday, October 16, 2017 8:31:22 AM CEST Greg Kroah-Hartman wrote:

quoted

On Mon, Oct 16, 2017 at 03:29:02AM +0200, Rafael J. Wysocki wrote:

quoted

+static inline void dev_pm_set_driver_flags(struct device *dev, unsigned int flags)
+{
+	dev->power.driver_flags = flags;
+}

Should this function just set the specific bit?  Or is it going to be ok
to set the whole value, meaning you aren't going to care about turning
on and off specific flags over the lifetime of the driver/device, you
are just going to set them once and then just test them as needed?

The idea is to set them once and they should not be touched again until
the driver (or device) goes away, so that would be the whole value at once
(and one of the i2c-designware-platdrv patches actually sets multiple flags
in one go).

Ok, thanks.