public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Subramanian, Sriram" <sriram-s@hpe.com>
To: Jiaxin Wu <jiaxin.wu@intel.com>,
	"edk2-devel@lists.01.org" <edk2-devel@lists.01.org>
Cc: "Hegde, Nagaraj P" <nagaraj-p.hegde@hpe.com>,
	Ye Ting <ting.ye@intel.com>,  Fu Siyuan <siyuan.fu@intel.com>
Subject: Re: [Patch 0/2] Ignore duplicated DNS address check
Date: Thu, 23 Feb 2017 10:25:00 +0000	[thread overview]
Message-ID: <DF4PR84MB00415067FFC39EA6FDD2E08E80530@DF4PR84MB0041.NAMPRD84.PROD.OUTLOOK.COM> (raw)
In-Reply-To: <1487827857-38844-1-git-send-email-jiaxin.wu@intel.com>

Looks ok to me. Nagaraj will also test with these changes and provide feedback.

Series Reviewed-by: Sriram Subramanian <sriram-s@hpe.com>

-----Original Message-----
From: Jiaxin Wu [mailto:jiaxin.wu@intel.com] 
Sent: Thursday, February 23, 2017 11:01 AM
To: edk2-devel@lists.01.org
Cc: Hegde, Nagaraj P <nagaraj-p.hegde@hpe.com>; Subramanian, Sriram <sriram-s@hpe.com>; Ye Ting <ting.ye@intel.com>; Fu Siyuan <siyuan.fu@intel.com>; Wu Jiaxin <jiaxin.wu@intel.com>
Subject: [Patch 0/2] Ignore duplicated DNS address check

Having duplicated DNS server IPs specified is not an ideal
configuration, but not an error condition. This patch is to
remove the duplicated DNS address check to allow the same DNS
address setting in SetData().

Cc: Hegde Nagaraj P <nagaraj-p.hegde@hpe.com>
Cc: Subramanian Sriram <sriram-s@hpe.com>
Cc: Ye Ting <ting.ye@intel.com>
Cc: Fu Siyuan <siyuan.fu@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Wu Jiaxin <jiaxin.wu@intel.com>

Jiaxin Wu (2):
  MdeModulePkg/Ip4Dxe: Ignore duplicated DNS address check
  NetworkPkg/Ip6Dxe: Ignore duplicated DNS address check

 MdeModulePkg/Universal/Network/Ip4Dxe/Ip4Config2Impl.c | 10 ----------
 NetworkPkg/Ip6Dxe/Ip6ConfigImpl.c                      | 10 ----------
 2 files changed, 20 deletions(-)

-- 
1.9.5.msysgit.1



  parent reply	other threads:[~2017-02-23 10:25 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-02-23  5:30 [Patch 0/2] Ignore duplicated DNS address check Jiaxin Wu
2017-02-23  5:30 ` [Patch 1/2] MdeModulePkg/Ip4Dxe: " Jiaxin Wu
2017-02-23  5:30 ` [Patch 2/2] NetworkPkg/Ip6Dxe: " Jiaxin Wu
2017-02-23 10:25 ` Subramanian, Sriram [this message]
2017-02-27  6:37 ` [Patch 0/2] " Hegde, Nagaraj P

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=DF4PR84MB00415067FFC39EA6FDD2E08E80530@DF4PR84MB0041.NAMPRD84.PROD.OUTLOOK.COM \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox