On Sat, Oct 27, 2018 at 10:26:56AM +0200, Sam Ravnborg wrote: > Hi Andrey, > > On Fri, Oct 26, 2018 at 06:32:29PM -0700, Andrey Smirnov wrote: > > Both dev_request_mem_region() and dev_request_mem_region_err_null() > > implement exactly the same functionality different only in error > > reporting value. Change the code to make use of a common generic > > function whose return type can be specified via an argument. > > Same story as PATCH 6/6 where a helper functions does > two things depending on a bool. > For simple functions like this my personal preference > is to keep the small functions that are more explicit. Hmm, seems coffee did not kick in yet. There is only this single patch with the boolean argument. Sorry for the noise. Sam _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox
