Re: [PATCH 1/6] soundwire: fix startup sequence for Intel/Cadence

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




@@ -1043,8 +1043,6 @@ static int intel_probe(struct platform_device *pdev)
      if (ret)
          goto err_init;
-    ret = sdw_cdns_enable_interrupt(&sdw->cdns);
-
      /* Read the PDI config and initialize cadence PDI */
      intel_pdi_init(sdw, &config);
      ret = sdw_cdns_pdi_init(&sdw->cdns, config);
@@ -1062,6 +1060,18 @@ static int intel_probe(struct platform_device *pdev)
          goto err_init;
      }
+    ret = sdw_cdns_enable_interrupt(&sdw->cdns);
+    if (ret < 0) {
+        dev_err(sdw->cdns.dev, "cannot enable interrupts\n");
+        goto err_init;
+    }
+
+    ret = sdw_cdns_exit_reset(&sdw->cdns);
+    if (ret < 0) {
+        dev_err(sdw->cdns.dev, "unable to exit bus reset sequence\n");
+        goto err_init;

Don't you want to disable interrupts at least... before you return
error? err_init does bus cleanup and not controller one

yes good point, let me look at this.

The existing code has no interrupt disable sequence.

I will add this improved error handling in a follow-up patch, after the capability to disable interrupts is added.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel




[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux