Hi Thierry, On Tue, Jan 16, 2018 at 2:16 AM, Thierry Escande <thierry.escande at collabora.com> wrote: > From: Tomasz Figa <tfiga at chromium.org> > > Current implementation of unbind dereferences the drvdata pointer > assuming that it's its own data, however the ownership belongs to the > analogix code, which means that the pointer is dereferenced with wrong > type. Fix this by using the recently added platform data .cleanup() > callback to do Rockchip-specific things at unbind. First of all, thanks a lot for taking care of these patches. Much appreciated. One thing to note, though, is that the four patches (31, 32, 33 and 39 in this series), which we initially made for fixing the problems related, were later reverted and replaced by a cleaner solution. Please take a look at this series on our gerrit: https://chromium-review.googlesource.com/c/chromiumos/third_party/kernel/+/442390/ https://chromium-review.googlesource.com/c/chromiumos/third_party/kernel/+/442392/ https://chromium-review.googlesource.com/c/chromiumos/third_party/kernel/+/442391/ Perhaps we could skip this intermediate solution and just go with the above 3 patches? Best regards, Tomasz
