On Monday, November 27, 2017 05:11:45 PM Russell King - ARM Linux wrote: > On Tue, Oct 17, 2017 at 03:32:30PM +0200, Bartlomiej Zolnierkiewicz wrote: > > On Friday, September 29, 2017 11:50:04 AM Russell King - ARM Linux wrote: > > > Hi Bart, > > > > Hi Russell, > > > > > This series updates the sa1100fb fbdev driver's initialisation paths > > > to be more robust. In doing these updates, I realised that we omitted > > > to free some memory which was allocated for the framebuffer if > > > initialisation fails - something that was hidden due to the complex > > > cleanup that the driver performs. Switching to managed resources made > > > this more obvious. > > > > > > Tested on H3600 iPAQ and Assabet. > > > > > > drivers/video/fbdev/sa1100fb.c | 75 +++++++++++++++--------------------------- > > > drivers/video/fbdev/sa1100fb.h | 2 ++ > > > 2 files changed, 29 insertions(+), 48 deletions(-) > > > > I queued all patches for 4.15, thanks! > > Now that 4.15-rc1 is out, something tells me that was not actually > the case. Any ideas what happened? Could you please explain the issue that you are seeing a bit more? For me it looks all fine: $ git log --oneline v4.15-rc1 drivers/video/fbdev/sa1100fb.c cb6bc3f video: sa1100fb: move pseudo palette into sa1100fb_info structure 0ab7658 video: sa1100fb: fix video memory allocation leak c244f8e video: sa1100fb: clean up failure path 5634cba video: sa1100fb: use devm_gpio_request_one() f6fc8c9 video: sa1100fb: use devm_request_irq() df6b228 video: sa1100fb: use devm_ioremap_resource() e43064c video: sa1100fb: use devm_clk_get() ba1d36b video: sa1100fb: use devm_kzalloc() ... Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
