From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: None (no SPF record) identity=mailfrom; client-ip=2607:f8b0:4864:20::141; helo=mail-it1-x141.google.com; envelope-from=mw@semihalf.com; receiver=edk2-devel@lists.01.org Received: from mail-it1-x141.google.com (mail-it1-x141.google.com [IPv6:2607:f8b0:4864:20::141]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 4264F21A07A82 for ; Mon, 19 Nov 2018 23:22:29 -0800 (PST) Received: by mail-it1-x141.google.com with SMTP id b5so2059450iti.2 for ; Mon, 19 Nov 2018 23:22:29 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=semihalf-com.20150623.gappssmtp.com; s=20150623; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc:content-transfer-encoding; bh=8UuvQe2J0nXJ0/NxERiIJDRTCaFxyUi9pqLNIvvXF6M=; b=1yADu0Tg5eLMc1obwMnaAAzOahVRdiWFMTzOGEjKMY/eYSxn2cO7T4J4XbkVvN7004 Ly7qFz8mXQounPg7ZanLVL3ysxKJyZ0K8vLIIGp0BY7fxD2cL0BN5vEt6fpzjYrfWEkV bGYa/ZcHynMWm72BUODliARc2P0zOvwegJZZc+OwCr2dPLr5Tc4NhkIQEs5/4RO4FMa5 p+j9oZF8cYwXROabnTrjq+4XPKIhMPe5mtiu3+9Lij18x+tCtRtsnmWzq0M8/kkLQVIQ HBg9PiQlT22kYhLR6Fsrjqtwtm7NaqNmcDITKOSnTFpoDyXEUoZ0niNwBMhJWgDjY/4j bLiA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc:content-transfer-encoding; bh=8UuvQe2J0nXJ0/NxERiIJDRTCaFxyUi9pqLNIvvXF6M=; b=V1e9bDEKJm9s8KN7XNXAL+Ax9bEN0Vy1661UNJS5erS+mMahiOXQZln0Xy8SE+9for mnAQ8/3/9ZRGcs07eDu4FegObpyUs1vrZox+gFssN0R0iHmbp4gzlIBBQNjVODQwge9/ 59YliQXGPwSS+Fsi8GLwAzAOS2R62M7PNkZXZd2UasVF4S6JzKjlXFghSt5XGLN8zdSq ZFBgGzQFiFiPylnLgWL60bm3onGH0MqotHhW5wYSAlzMh+XdI2SEi0mWdWW4aB/VoA0w kisPwmfHC+2cF7pNZSjlUEn7a/nxJhNbMAEzdekLm/l2+frVy67bfM9NbJLZa2BwdFC7 iL/Q== X-Gm-Message-State: AA+aEWY73x/FGhpZ681GaboB3UKAqBbeDBjRjVgAfpTSfkNy6/HYdncf JiJ9OTPwHU5ZcYOFlylCf+8eZivqsnM+5rkqlM9Ggw== X-Google-Smtp-Source: AJdET5cmYXAtortKwUPMf4GqsvJW6piFOOaDHMNZtTPyM+RSM+unrzbfNXhQEz65izf9BIUeGN9DfLh0fpYKKbd+ImI= X-Received: by 2002:a05:660c:681:: with SMTP id n1mr1167427itk.37.1542698548119; Mon, 19 Nov 2018 23:22:28 -0800 (PST) MIME-Version: 1.0 References: <1541804487-27458-1-git-send-email-mw@semihalf.com> In-Reply-To: From: Marcin Wojtas Date: Tue, 20 Nov 2018 08:22:16 +0100 Message-ID: To: hao.a.wu@intel.com Cc: Tomasz Michalec , Grzegorz Jaszczyk , edk2-devel-01 , "Gao, Liming" , nadavh@marvell.com, "Kinney, Michael D" Subject: Re: [PATCH v4 0/4] SdMmcOverride extension X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 20 Nov 2018 07:22:29 -0000 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable wt., 20 lis 2018 o 06:59 Wu, Hao A napisa=C5=82(a): > > > -----Original Message----- > > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of > > Wu, Hao A > > Sent: Tuesday, November 13, 2018 4:29 PM > > To: Marcin Wojtas > > Cc: Tomasz Michalec; Grzegorz Jaszczyk; edk2-devel-01; Gao, Liming; > > nadavh@marvell.com; Kinney, Michael D > > Subject: Re: [edk2] [PATCH v4 0/4] SdMmcOverride extension > > > > > -----Original Message----- > > > From: Marcin Wojtas [mailto:mw@semihalf.com] > > > Sent: Tuesday, November 13, 2018 4:25 PM > > > To: Wu, Hao A > > > Cc: edk2-devel-01; Leif Lindholm; Kinney, Michael D; Gao, Liming; Ard > > > Biesheuvel; nadavh@marvell.com; jsd@semihalf.com; Tomasz Michalec; > > > Grzegorz Jaszczyk > > > Subject: Re: [PATCH v4 0/4] SdMmcOverride extension > > > > > > Hi Hao, > > > > > > wt., 13 lis 2018 o 08:38 Wu, Hao A napisa=C5=82(= a): > > > > > > > > Hi Marcin, > > > > > > > > The code changes look good to me. > > > > > > > > Could you please grant me some time for some additional tests for t= hese > > > > patches? > > > > > > Sure. > > > > > > > I will inform you with the results sometime next week. Thanks in ad= vance. > > > > > > > > > > Ard gave his RB to 2/4 and 3/4. Moreover he pointed a typo in 3/4 > > > commit message - should I repost, or could those be included when > > > applying the patches (unless you don't request any code change, of > > > course)? > > > > If no other issues, I will help to address the typo issue in patch 3/4. > > > > A little concern on the commit author information if I modify the commi= t, but > > I will let you know if a repost from you is needed. > > > > Best Regards, > > Hao Wu > > For the series: > Reviewed-by: Hao Wu > > Pushed at 49c9953425..7f3b0bad4b > Thanks a lot! Marcin > > Best Regards, > Hao Wu > > > > > > > > > Best regards, > > > Marcin > > > > > > > Best Regards, > > > > Hao Wu > > > > > > > > > > > > > -----Original Message----- > > > > > From: Marcin Wojtas [mailto:mw@semihalf.com] > > > > > Sent: Saturday, November 10, 2018 7:01 AM > > > > > To: edk2-devel@lists.01.org > > > > > Cc: leif.lindholm@linaro.org; Wu, Hao A; Kinney, Michael D; Gao, = Liming; > > > > > ard.biesheuvel@linaro.org; nadavh@marvell.com; mw@semihalf.com; > > > > > jsd@semihalf.com; tm@semihalf.com; jaz@semihalf.com > > > > > Subject: [PATCH v4 0/4] SdMmcOverride extension > > > > > > > > > > Hi, > > > > > > > > > > Although I could've waited for Hao's remarks, I think it may > > > > > be better if he takes a look at much cleaner code, which > > > > > addresses v3 review comments. > > > > > The newest version of the patchset cleans-up significantly > > > > > patches 2&3 by removing code duplication and other minor > > > > > improvements. > > > > > > > > > > Patches are available in the github: > > > > > https://github.com/MarvellEmbeddedProcessors/edk2-open- > > > > > platform/commits/sdmmc-override-upstream-r20181109 > > > > > > > > > > Please note that extending SdMmcOverride protocol was impacting > > > > > so far the only user of it (Synquacer controller). In paralel > > > > > edk2-platforms patchset, a patch can be found: > > > > > ("Silicon/SynQuacer/PlatformDxe: adjust to updated SdMmcOverride"= ) > > > > > which adjust to the new API. > > > > > https://github.com/MarvellEmbeddedProcessors/edk2-open- > > > > > platform/commits/xenon-upstream-r20181109 > > > > > > > > > > I'm looking forward to the comments and remarks. > > > > > > > > > > Best regards, > > > > > Marcin > > > > > > > > > > Changelog: > > > > > v3->v4 > > > > > * 2/4: > > > > > - avoid duplication by calling SdMmcOverride callback in > > > > > SdMmcHcUhsSignaling > > > > > > > > > > * 3/4: > > > > > - avoid duplication by calling SdMmcOverride callback in > > > > > EmmcSwitchClockFreq > > > > > > > > > > * 4/4: > > > > > - add Ard's RB > > > > > > > > > > v2->v3 > > > > > * 1/4: > > > > > - rename new parameter to PhaseData > > > > > - add Ard's RB > > > > > > > > > > * 2/4: > > > > > - s/Controler/Controller/ > > > > > - remove all references to MMC_SDR_50 mode > > > > > - rename and reorder MMC bus modes > > > > > - rename enum: s/SD_MMC_UHS_TIMING/SD_MMC_BUS_MODE/ > > > > > and move it to protocol header in order to drop including pri= vate one > > > > > - fix if condition in EmmcSwitchToHighSpeed > > > > > - call SdMmcHcUhsSignaling unconditionally before SdMmcOverride > > > > > callback, so that protocol producer can optionally modify onl= y quirky > > > > > timing mode values. > > > > > > > > > > *4/4 > > > > > - bump protocol version to 2 > > > > > - remove redundant assert from SdMmcPciHcDriverBindingStart > > > > > (BaseClkFreq is already checked in SdMmcHcInitClockFreq) > > > > > - update comment in SdMmcHcInitClockFreq > > > > > - restore original DumpCapabilityReg and append > > > > > > > > > > v1 -> v2 > > > > > * Rebase onto newest master > > > > > * 1/4 [new patch] - preparation for extending NotifyPhase > > > > > * 2/4 - UhsSignaling as a part of NotifyPhase > > > > > * 3/4 - SwitchClockFreqPost as a part of NotifyPhase > > > > > * 4/4 - Allow updating BaseClkFreq via Capability instead of the > > > > > independent callback. > > > > > > > > > > > > > > > Marcin Wojtas (2): > > > > > MdeModulePkg/SdMmcPciHcDxe: Add an optional parameter in > > > NotifyPhase > > > > > MdeModulePkg/SdMmcPciHcDxe: Allow overriding base clock > > frequency > > > > > > > > > > Tomasz Michalec (2): > > > > > MdeModulePkg/SdMmcPciHcDxe: Add UhsSignaling to > > SdMmcOverride > > > > > protocol > > > > > MdeModulePkg/SdMmcPciHcDxe: Add SwitchClockFreqPost to > > > > > SdMmcOverride > > > > > > > > > > MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdMmcPciHcDxe.h | 6 + > > > > > MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdMmcPciHci.h | 42 > > ++++++- > > > > > MdeModulePkg/Include/Protocol/SdMmcOverride.h | 29 ++++- > > > > > MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/EmmcDevice.c | 121 > > > ++++++++++- > > > > > -------- > > > > > MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdDevice.c | 35 ++++-- > > > > > MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdMmcPciHcDxe.c | 13 +- > > > > > MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdMmcPciHci.c | 124 > > > > > +++++++++++++++++--- > > > > > 7 files changed, 280 insertions(+), 90 deletions(-) > > > > > > > > > > -- > > > > > 2.7.4 > > > > > > _______________________________________________ > > edk2-devel mailing list > > edk2-devel@lists.01.org > > https://lists.01.org/mailman/listinfo/edk2-devel