From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=209.85.214.68; helo=mail-it0-f68.google.com; envelope-from=mlureau@redhat.com; receiver=edk2-devel@lists.01.org Received: from mail-it0-f68.google.com (mail-it0-f68.google.com [209.85.214.68]) (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 7373C2063D76E for ; Thu, 17 May 2018 07:43:10 -0700 (PDT) Received: by mail-it0-f68.google.com with SMTP id 70-v6so9179044ity.2 for ; Thu, 17 May 2018 07:43:10 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc:content-transfer-encoding; bh=D+JAfkhE7cJtItuyR609ICH1muGxjF738LyRYcbnAIM=; b=iCewb3hnOIOvCp0iwTnafE+UUh0P+VefaJSgPMsU18DwmHj7CoNoZn66KZG5k/VzEC 0PYfMqrxpWMV4nXD3NLOGMhDVF/6z96Zqs3tuHmrbKhO4Ly+5gtke3UWhbSWt802Nesx dYyPVRJZn7LyOtaVkBxSrxULDtMaBH5fRJbPE3CCWx0ZOcdXjlj6c7kPaj32ELw/ttlc i7ufsFkXYGeKHz7NI2BFmBOuuhjY5yRQEKOftLcKdZL0QlipHNIwydx+ju/opH/Uc4RJ eNfiZO5cN3a2Ez5YFqTARbiLEeStWurcmqcxDLXyIGsIN1tmgeR5dYWEBzWoRSwMp3fV taIg== X-Gm-Message-State: ALKqPwe8rCX+8kol3p5+YPKXFdyTxtlsHxZeqR5px5DJaZCFWYCOwxQc UKzAssFOIumr3ED3T8lV4EMR5WakIp3LrTR2+wka9A== X-Google-Smtp-Source: AB8JxZqqESBzcfLM9o13+XvERbOMRZrCvW8MX2SCroUSuBaL4OKhTYEe426Yle2GqqWvNM0ZJc+l8pHZZaXlyJoR3XQ= X-Received: by 2002:a24:1301:: with SMTP id 1-v6mr3037212itz.55.1526568189347; Thu, 17 May 2018 07:43:09 -0700 (PDT) MIME-Version: 1.0 Received: by 2002:a4f:1503:0:0:0:0:0 with HTTP; Thu, 17 May 2018 07:43:08 -0700 (PDT) In-Reply-To: <229d0059-2ab4-bf2d-cbf5-9dca4ff978bd@redhat.com> References: <20180515123007.10164-1-marcandre.lureau@redhat.com> <229d0059-2ab4-bf2d-cbf5-9dca4ff978bd@redhat.com> From: =?UTF-8?B?TWFyYy1BbmRyw6kgTHVyZWF1?= Date: Thu, 17 May 2018 16:43:08 +0200 Message-ID: To: Laszlo Ersek Cc: edk2-devel@lists.01.org, qemu-devel , javierm@redhat.com, pjones@redhat.com, jiewen.yao@intel.com Subject: Re: [PATCH 0/4] RFC: ovmf: Add support for TPM Physical Presence interface X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.26 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 17 May 2018 14:43:10 -0000 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Hi On Wed, May 16, 2018 at 11:29 AM, Laszlo Ersek wrote: > Hi Marc-Andr=C3=A9, > > On 05/15/18 14:30, marcandre.lureau@redhat.com wrote: >> From: Marc-Andr=C3=A9 Lureau >> >> Hi, >> >> The following series adds basic TPM PPI 1.3 support for OVMF-on-QEMU >> with TPM2 (I haven't tested TPM1, for lack of interest). > > I got the review of this patch series added to my TODO list, but I'll > have to ask for your patience. :( > > From an extremely superficial skim: > > * please use the > > TopDirPkg/ModuleName: blah blah blah > > subject format, or more generally, if a module cannot be identified, > > TopDirPkg: blah blah blah > done > * the subject line and the commit message shouldn't be wider than 74 > chars; > that should be ok > * edk2 uses two spaces for general indentation, and I'm noticing some > inconsistency there (4 spaces like in QEMU). yes, I tried to respect that, but sometime fail (emacs c-basic-offset 2 isn't great with comments) > > * Please consider formatting the patches with "--find-copies-harder" > (although I can look at them with the same option after fetching the > series from your repo). This option is usually helpful for reviewers > when cloning and modifying modules cross-package. Hmm, I didn't know that option, ok > > * Please consider adopting the git settings at > , > in particular: > > - "--stat=3D1000 --stat-graph-width=3D20", so that pathnames are not > truncated in the diffstats, > I use git-publish very often. I had to modify it to pass those options (https://github.com/stefanha/git-publish/pull/48) > - the "xfuncname"-related settings, so that git diff hunk headers @@ > are useful for DSC and INF files too, > This is already in my .git/config, I hope it takes it by default in format-patch? > - the diff order file, so that files are listed in patches in logical > order, going from abstract / descriptive (.inf, .h) to concrete / > imperative (.c). > ok > Not much of a review, I know; this is all I can offer right now. If you > have the time to respin just with these superficial changes, that might > make my life easier. If you prefer to delay them, that's 100% fine too. > I am going to resend with the style fixes.