)]}'
{
  "commit": "1c9c73a61a4d69392a253d1af3f0aec608af6484",
  "tree": "87c68d3855eba59c189b42b4eaf2943545942375",
  "parents": [
    "340b20faae9680f7abb28129c30990074fdc73d2"
  ],
  "author": {
    "name": "Anastasia Klimchuk",
    "email": "aklm@chromium.org",
    "time": "Fri Apr 30 11:25:44 2021 +1000"
  },
  "committer": {
    "name": "Nico Huber",
    "email": "nico.h@gmx.de",
    "time": "Sun Jan 22 00:55:18 2023 +0000"
  },
  "message": "pickit2_spi.c: Separate shutdown from failed init cleanup\n\nShutdown function was covering two different jobs here: 1) the actual\nshutdown which is run at the end of the driver\u0027s lifecycle and\n2) cleanup in cases when initialisation failed. Now, shutdown is only\ndoing its main job (#1), and the driver itself is doing cleanup\nwhen init fails (#2).\n\nThe good thing is that now resources are released/closed immediately\nin cases when init fails (vs shutdown function which was run at some\npoint later), and the driver leaves clean space after itself if init\nfails.\n\nAnd very importantly this unlocks API change which plans to move\nregister_shutdown inside register master API, see\nhttps://review.coreboot.org/c/flashrom/+/51761\n\nChange-Id: I1b672b33169a7a1b6ceab190ad3f48c2f35c3a1f\nSigned-off-by: Anastasia Klimchuk \u003caklm@chromium.org\u003e\nOriginal-Reviewed-on: https://review.coreboot.org/c/flashrom/+/52773\nOriginal-Reviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\nOriginal-Reviewed-by: Angel Pons \u003cth3fanbus@gmail.com\u003e\nReviewed-on: https://review.coreboot.org/c/flashrom-stable/+/72175\nTested-by: build bot (Jenkins) \u003cno-reply@coreboot.org\u003e\nReviewed-by: Nico Huber \u003cnico.h@gmx.de\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "e364b06cc0c102de5b2589c02777355962b1909d",
      "old_mode": 33188,
      "old_path": "pickit2_spi.c",
      "new_id": "b8d3f7667fd34c388b0086e7d26983072e63665a",
      "new_mode": 33188,
      "new_path": "pickit2_spi.c"
    }
  ]
}