lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 31 Jan 2020 17:34:59 +0300 From: Dmitry Osipenko <digetx@...il.com> To: Sameer Pujar <spujar@...dia.com>, perex@...ex.cz, tiwai@...e.com, robh+dt@...nel.org Cc: broonie@...nel.org, lgirdwood@...il.com, thierry.reding@...il.com, jonathanh@...dia.com, alsa-devel@...a-project.org, devicetree@...r.kernel.org, linux-tegra@...r.kernel.org, linux-kernel@...r.kernel.org, sharadg@...dia.com, mkumard@...dia.com, viswanathl@...dia.com, rlokhande@...dia.com, dramesh@...dia.com, atalambedu@...dia.com Subject: Re: [PATCH v2 5/9] ASoC: tegra: add Tegra210 based AHUB driver 31.01.2020 06:12, Sameer Pujar пишет: > > > On 1/30/2020 10:55 PM, Dmitry Osipenko wrote: >> External email: Use caution opening links or attachments >> >> >> 30.01.2020 13:33, Sameer Pujar пишет: >> ... >>> +#include <linux/clk.h> >>> +#include <linux/device.h> >>> +#include <linux/module.h> >>> +#include <linux/of_platform.h> >>> +#include <linux/platform_device.h> >>> +#include <linux/pm_runtime.h> >>> +#include <linux/regmap.h> >>> +#include <sound/soc.h> >>> +#include "tegra210_ahub.h" >> Nit: I'd separate the per-directory includes with a blank line, like it >> done by the most of existing drivers, for consistency. > > Hi Dmitry, > > I'd assume these are the only review comments (which includes your other > replies for v2 5/9) for the series. > Let me know otherwise. > > It would help me to discuss all the issues at once, finalize changes for > v3 and so that we can avoid multiple iterations. Hello Sameer, I'll take another look at the patches around Monday. In general, you should wait for a week or two if you're expecting to get more comments. There is now more than month for you to finish this series. If you have any other patches that depend on this series, then you may send them as RFC and explicitly state that they depend on another series.
Powered by blists - more mailing lists