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, 9 Jun 2017 13:21:17 +0530 From: Kishon Vijay Abraham I <kishon@...com> To: Ulf Hansson <ulf.hansson@...aro.org> CC: Tony Lindgren <tony@...mide.com>, Rob Herring <robh+dt@...nel.org>, Sekhar Nori <nsekhar@...com>, linux-omap <linux-omap@...r.kernel.org>, "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>, "linux-arm-kernel@...ts.infradead.org" <linux-arm-kernel@...ts.infradead.org>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, "linux-mmc@...r.kernel.org" <linux-mmc@...r.kernel.org> Subject: Re: [PATCH 0/3] omap_hsmmc: use mmc_regulator_get_supply() to get regulators Hi Uffe, On Thursday 08 June 2017 12:45 PM, Ulf Hansson wrote: > On 7 June 2017 at 10:36, Kishon Vijay Abraham I <kishon@...com> wrote: >> This series deprecates "vmmc_aux" supply property and uses the generic >> "vqmmc" property. >> >> Not sure how exactly to handle omap3-overo-base.dtsi which already has >> both vqmmc-supply and vmmc_aux-supply without breaking old dt >> compatibility. (vqmmc-supply seems to have been added there by mistake >> from 3.15). >> >> In the dt series that will follow, will replace all vmmc_aux-supply >> with vqmmc-supply (In omap3-overo-base.dtsi remove the existing >> vqmmc-supply and then replace vmmc_aux-supply with vqmmc-supply). >> >> Kishon Vijay Abraham I (3): >> mmc: host: omap_hsmmc: Do not initialize MMC regulators to NULL on >> error >> mmc: host: omap_hsmmc: use mmc_regulator_get_supply() to get >> regulators >> Documentation: dt-bindings: Document deprecation of "vmmc_aux" and >> using "vqmmc" >> >> .../devicetree/bindings/mmc/ti-omap-hsmmc.txt | 2 +- >> drivers/mmc/host/omap_hsmmc.c | 55 +++++++++------------- >> 2 files changed, 24 insertions(+), 33 deletions(-) >> >> -- >> 2.11.0 >> > > Besides the minor comment for patch2 this looks good to me. > > In case there are no other comments, and that you agree with my > suggested change, you don't need to re-spin. Your suggested change looks good. Thanks Kishon
Powered by blists - more mailing lists