I have a patch to fix the memory leak but I haven't been able to test it yet because I am remote right now and I accidentally bootlooped the AMD device I am working on. I will have this tested early next week. Here is the patch for anyone interested.
Curtis Malainey | Software Engineer | cujomalainey@google.com | 650-898-3849
On Fri, Jan 25, 2019 at 3:26 AM Mark Brown broonie@kernel.org wrote:
On Thu, Jan 24, 2019 at 01:07:17PM -0600, Pierre-Louis Bossart wrote:
Thanks for the feedback, will send a formal patch with the helper and machine driver changes after I test more with the legacy drivers. Do you have a preference for one patch that deals with multiple machines drivers in one shot, or individual patches? The latter are nicer for backports (e.g. for Chrome), the former nicer for maintainers...
More patches is good, it doesn't make a huge difference if I get one big patch or a series of repetitive patches - big serieses are more of an issue if they're all different patches needing individual review.