Re: [PATCH 1/2] Input: cs40l26: Support for CS40L26 Boosted Haptic Amplifier

[Fred Treven]

Hi Jeff,

I had one last question for you before I send the next version of this submission.

>> +
>> + return 0;
>> +}
>> +EXPORT_SYMBOL_GPL(cs40l26_remove);
>> +
>> +static int cs40l26_suspend(struct device *dev)
>> +{
>> + struct cs40l26_private *cs40l26 = dev_get_drvdata(dev);
>> + int ret;
>> +
>> + mutex_lock(&cs40l26->lock);
>> + dev_dbg(dev, "%s: Enabling hibernation\n", __func__);
>> +
>> + ret = cs40l26_allow_hiber(cs40l26);
>> +
>> + mutex_unlock(&cs40l26->lock);
>> +
>> + return ret;
>> +}
>> +
>> +static int cs40l26_sys_suspend(struct device *dev)
>> +{
>> + struct i2c_client *i2c_client = to_i2c_client(dev);
>> +
>> + dev_dbg(dev, "System suspend, disabling IRQ\n");
>> +
>> + disable_irq(i2c_client->irq);
>> +
>> + return 0;
>> +}
>> +
>> +static int cs40l26_sys_suspend_noirq(struct device *dev)
>> +{
>> + struct i2c_client *i2c_client = to_i2c_client(dev);
>> +
>> + dev_dbg(dev, "Late system suspend, re-enabling IRQ\n");
>> + enable_irq(i2c_client->irq);
>> +
>> + return 0;
>> +}
>> +
>> +static int cs40l26_resume(struct device *dev)
>> +{
>> + dev_dbg(dev, "%s: Disabling hibernation\n", __func__);
>> +
>> + return cs40l26_prevent_hiber(dev_get_drvdata(dev));
>> +}
>> +
>> +static int cs40l26_sys_resume(struct device *dev)
>> +{
>> + struct i2c_client *i2c_client = to_i2c_client(dev);
>> +
>> + dev_dbg(dev, "System resume, re-enabling IRQ\n");
>> +
>> + enable_irq(i2c_client->irq);
>> +
>> + return 0;
>> +}
>> +
>> +static int cs40l26_sys_resume_noirq(struct device *dev)
>> +{
>> + struct i2c_client *i2c_client = to_i2c_client(dev);
>> +
>> + dev_dbg(dev, "Early system resume, disabling IRQ\n");
>> +
>> + disable_irq(i2c_client->irq);
>> +
>> + return 0;
>> +}
>> +
>> +const struct dev_pm_ops cs40l26_pm_ops = {
>> + SET_RUNTIME_PM_OPS(cs40l26_suspend, cs40l26_resume, NULL)
>> + SET_SYSTEM_SLEEP_PM_OPS(cs40l26_sys_suspend, cs40l26_sys_resume)
>> + SET_NOIRQ_SYSTEM_SLEEP_PM_OPS(cs40l26_sys_suspend_noirq, cs40l26_sys_resume_noirq)
>> +};
>> +EXPORT_SYMBOL_GPL(cs40l26_pm_ops);
> 
> Please use latest macros (e.g. DEFINE_SIMPLE_DEV_PM_OPS).

When looking at these *_PM_OPS* macros that replace the deprecated versions, it is unclear to me how to maintain support for *_sys_* and
*_sys_*_noirq* functions. Would these all need to be separately defined via DEFINE_SIMPLE_DEV_PM_OPS?
Would the *_sys_* definitions still be defined through a struct i.e.
const struct dev_pm_ops cs40l26_sys_pm_ops which is then exported as it is in my initial submission? 
I’m unsure how to handle these cases with the latest macros.

Appreciate any guidance you can provide.

Best regards,
Fred