On 4/30/20 6:34 AM, Dmitry Osipenko wrote:
30.04.2020 01:00, Sowjanya Komatineni пишет:
+static int tegra_csi_init(struct host1x_client *client)
+{
+       struct tegra_csi *csi = host1x_client_to_csi(client);
+       struct tegra_video_device *vid = dev_get_drvdata(client->host);
+       int ret;
+
+       INIT_LIST_HEAD(&csi->csi_chans);
+
+       ret = tegra_csi_tpg_channels_alloc(csi);
+       if (ret < 0) {
+               dev_err(csi->dev,
+                       "failed to allocate tpg channels: %d\n", ret);
+               goto cleanup;
+       }
+
+       ret = tegra_csi_channels_init(csi);
+       if (ret < 0)
+               goto cleanup;
+
+       vid->csi = csi;
+
+       return 0;
+
+cleanup:
+       tegra_csi_channels_cleanup(csi);
+       pm_runtime_put_sync(csi->dev);
This pm_runtime_put_sync() should be removed.

Sorry, I had it correct in follow-up patches and missed this to remove in v12.

Will double check all changes before sending v12 once all v11 feedback from you is done.


+       return ret;
+}

Reply via email to