)]}'
{
  "commit": "4e6fd26dcf5d70a7700a4c15d8f239103bdce91d",
  "tree": "8e86749506bdcea2a07243f7e4efaaf462ce865e",
  "parents": [
    "148ad68b3d0dd0f0068ff5f6f47bc360011c429e"
  ],
  "author": {
    "name": "Linus Walleij",
    "email": "linus.walleij@linaro.org",
    "time": "Fri Feb 12 09:22:11 2016 +0100"
  },
  "committer": {
    "name": "Linus Walleij",
    "email": "linus.walleij@linaro.org",
    "time": "Tue Feb 16 00:19:54 2016 +0100"
  },
  "message": "pinctrl: sirf/atlas7: stop poking around in GPIO internals\n\nThis code is poking around in the gpio_chip:s internal structures\nto achieve some kind of pin to GPIO mappings.\n\n- It is wrong to poke around in these structs and the pinctrl\n  maintainer was stupid to let it pass unnoticed, mea culpa.\n\n- The right interface to use is gpiochip_add_pin_range()\n\n- The code appears unused: the pin control part of the driver\n  is not adding any ranges, so we\u0027re iterating over an empty\n  list. Maybe it is poking around in some other pin controllers\n  GPIO ranges, and that\u0027s just totally wrong, again use\n  gpiochip_add_pin_range() and specify the right pin\n  controller.\n\nCc: Barry Song \u003cbaohua@kernel.org\u003e\nCc: Guoying Zhang \u003cGuoying.Zhang@csr.com\u003e\nCc: Wei Chen \u003cWei.Chen@csr.com\u003e\nSigned-off-by: Linus Walleij \u003clinus.walleij@linaro.org\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "beb024c31a5d8b5ffb115fa94283dc7909219f7c",
      "old_mode": 33188,
      "old_path": "drivers/pinctrl/sirf/pinctrl-atlas7.c",
      "new_id": "3d233fc3448ad4675a08a52b7ca4c44a11a4737b",
      "new_mode": 33188,
      "new_path": "drivers/pinctrl/sirf/pinctrl-atlas7.c"
    }
  ]
}