 
            
            
            
            
                7 Mar
                
                    2023
                
            
            
                7 Mar
                
                '23
                
            
            
            
        
    
                4:28 p.m.
            
        +static int amd_resume_child_device(struct device *dev, void *data) +{
- struct sdw_slave *slave = dev_to_sdw_dev(dev);
- int ret;
- if (!slave->probed) {
dev_dbg(dev, "skipping device, no probed driver\n");
return 0;- }
- if (!slave->dev_num_sticky) {
dev_dbg(dev, "skipping device, never detected on bus\n");
return 0;- }
- if (!pm_runtime_suspended(dev))
return 0;- ret = pm_request_resume(dev);
I still don't get why the test above was needed. It's racy and brings limited benefits.
- if (ret < 0)
dev_err(dev, "pm_request_resume failed: %d\n", ret);- return ret;
+}