Re: [PATCH v2 0/4] backlight: fix device-tree node lookups
On Fri, 11 May 2018, Johan Hovold wrote: > Hi Lee, > > On Mon, Nov 20, 2017 at 11:45:43AM +0100, Johan Hovold wrote: > > A number of drivers have been using the wrong OF helper when doing > > child-node > > lookups during probe. This meant that they were doing tree-wide searches > > rather > > than matching on child nodes and that the parent node could end up being > > prematurely freed. > > > Johan Hovold (4): > > backlight: as3711_bl: fix device-tree node lookup > > backlight: max8925_bl: fix device-tree node lookup > > backlight: tps65217_bl: fix device-tree node lookup > > backlight: as3711_bl: fix device-tree node leaks > > It appears these four have been sitting in your for-backlight-fixes > branch for half a year now without being forwarded to Linus. > > Don't get many backlight fixes these days, huh? ;) Ha! To be honest I think those patches are the only 4 -fixes I've *ever* had for Backlight. :) I have pulled them into the main branch, so they won't be forgotten again. Sorry for the delay. -- Lee Jones [李琼斯] Linaro Services Technical Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH v2 0/4] backlight: fix device-tree node lookups
Hi Lee, On Mon, Nov 20, 2017 at 11:45:43AM +0100, Johan Hovold wrote: > A number of drivers have been using the wrong OF helper when doing child-node > lookups during probe. This meant that they were doing tree-wide searches > rather > than matching on child nodes and that the parent node could end up being > prematurely freed. > Johan Hovold (4): > backlight: as3711_bl: fix device-tree node lookup > backlight: max8925_bl: fix device-tree node lookup > backlight: tps65217_bl: fix device-tree node lookup > backlight: as3711_bl: fix device-tree node leaks It appears these four have been sitting in your for-backlight-fixes branch for half a year now without being forwarded to Linus. Don't get many backlight fixes these days, huh? ;) Thanks, Johan ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
[PATCH v2 0/4] backlight: fix device-tree node lookups
A number of drivers have been using the wrong OF helper when doing child-node lookups during probe. This meant that they were doing tree-wide searches rather than matching on child nodes and that the parent node could end up being prematurely freed. Johan v2 - add patch 4/4 which fixes two node leaks in as3711_bl instead of marking the leaks using FIXMEs in patch 1/4 Johan Hovold (4): backlight: as3711_bl: fix device-tree node lookup backlight: max8925_bl: fix device-tree node lookup backlight: tps65217_bl: fix device-tree node lookup backlight: as3711_bl: fix device-tree node leaks drivers/video/backlight/as3711_bl.c | 45 ++- drivers/video/backlight/max8925_bl.c | 4 +++- drivers/video/backlight/tps65217_bl.c | 4 ++-- include/linux/mfd/as3711.h| 4 ++-- 4 files changed, 36 insertions(+), 21 deletions(-) -- 2.15.0 ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
