From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by spool.mail.gandi.net (Postfix) with ESMTPS id E1086941DD3 for ; Sun, 26 Nov 2023 20:41:28 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=e+Ym12rrsY9hiDtjvZyLVfvUqSMUFGfxf8Z1IiUfBKg=; c=relaxed/simple; d=groups.io; h=MIME-Version:References:In-Reply-To:From:Date:Message-ID:Subject:To:Cc:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Reply-To:List-Unsubscribe-Post:List-Unsubscribe:Content-Type:Content-Transfer-Encoding; s=20140610; t=1701031287; v=1; b=OBBvCea9IHM80H0+t4cVg6GiGfuuqp9qfqmNds97hbRPbeaFkGtjmYgQL23LZPL+McifX3EP 308L6FgcohNxhaNwticaS4RwAV/OkcisJTmIUWxkUoOSmUgPV2o59d/laRTmxyv0KH90nYIWHlk kjQ31N6wwaKgc59iDLUzGcRs= X-Received: by 127.0.0.2 with SMTP id DWAhYY7687511xAHx3c8nHMI; Sun, 26 Nov 2023 12:41:27 -0800 X-Received: from mail-yb1-f180.google.com (mail-yb1-f180.google.com [209.85.219.180]) by mx.groups.io with SMTP id smtpd.web10.65630.1701031286691294160 for ; Sun, 26 Nov 2023 12:41:26 -0800 X-Received: by mail-yb1-f180.google.com with SMTP id 3f1490d57ef6-d9fe0a598d8so3133031276.2 for ; Sun, 26 Nov 2023 12:41:26 -0800 (PST) X-Gm-Message-State: 5G1feRUUEeWAD32mp3lN89R9x7686176AA= X-Google-Smtp-Source: AGHT+IFJM+fUyTYP09ZlvvPuY1VXKFMq1fSCRNNimoxA1EvEzJn4N/T8f6gixntgDZyKziAs3nZNdwTTNmkPOY5yoI0= X-Received: by 2002:a25:5344:0:b0:da0:c49a:5fed with SMTP id h65-20020a255344000000b00da0c49a5fedmr7461074ybb.50.1701031285832; Sun, 26 Nov 2023 12:41:25 -0800 (PST) MIME-Version: 1.0 References: <20231123000103.31216-1-mike.maslenkin@gmail.com> <20231123000103.31216-4-mike.maslenkin@gmail.com> <179A1BD569EE113B.9514@groups.io> In-Reply-To: From: "Mike Maslenkin" Date: Sun, 26 Nov 2023 23:40:49 +0300 Message-ID: Subject: Re: [edk2-devel] [PATCH v2 3/3] RedfishPkg: fix searching for the BMC-exposed USB NIC To: "Chang, Abner" Cc: "devel@edk2.groups.io" , Nickle Wang , "igork@ami.com" Precedence: Bulk List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,mike.maslenkin@gmail.com List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-GND-Status: LEGIT Authentication-Results: spool.mail.gandi.net; dkim=pass header.d=groups.io header.s=20140610 header.b=OBBvCea9; dmarc=fail reason="SPF not aligned (relaxed), DKIM not aligned (relaxed)" header.from=gmail.com (policy=none); spf=pass (spool.mail.gandi.net: domain of bounce@groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce@groups.io Hi Abner, I agree with your fixes. This patch can be dropped from this set. BTW, do I understand correctly, PR is not required because changes are triv= ial? Regards, Mike. On Thu, Nov 23, 2023 at 6:06=E2=80=AFAM Chang, Abner = wrote: > > [AMD Official Use Only - General] > > Good to go. We can merge this after code freeze. > > Abner > > > -----Original Message----- > > From: devel@edk2.groups.io On Behalf Of Chang, > > Abner via groups.io > > Sent: Thursday, November 23, 2023 9:20 AM > > To: Nickle Wang ; Mike Maslenkin > > ; devel@edk2.groups.io > > Cc: igork@ami.com > > Subject: Re: [edk2-devel] [PATCH v2 3/3] RedfishPkg: fix searching for = the > > BMC-exposed USB NIC > > > > [AMD Official Use Only - General] > > > > Caution: This message originated from an External Source. Use proper ca= ution > > when opening attachments, clicking links, or responding. > > > > > > [AMD Official Use Only - General] > > > > HI Nickle and Mike, > > I also have some fixes and revised BMC USB NIC library for searching BM= C > > exposed NIC. > > Please don't not merge this patch set just for now. I will review this = and > > compare with the changes I have. > > > > Thanks > > Abner > > > > > -----Original Message----- > > > From: Nickle Wang > > > Sent: Thursday, November 23, 2023 8:24 AM > > > To: Mike Maslenkin ; devel@edk2.groups.io > > > Cc: Chang, Abner ; igork@ami.com > > > Subject: RE: [PATCH v2 3/3] RedfishPkg: fix searching for the BMC-exp= osed > > > USB NIC > > > > > > Caution: This message originated from an External Source. Use proper > > caution > > > when opening attachments, clicking links, or responding. > > > > > > > > > Reviewed-by: Nickle Wang > > > > > > Regards, > > > Nickle > > > > > > > -----Original Message----- > > > > From: Mike Maslenkin > > > > Sent: Thursday, November 23, 2023 8:01 AM > > > > To: devel@edk2.groups.io > > > > Cc: abner.chang@amd.com; Nickle Wang ; > > > > igork@ami.com > > > > Subject: [PATCH v2 3/3] RedfishPkg: fix searching for the BMC-expos= ed > > USB > > > NIC > > > > > > > > External email: Use caution opening links or attachments > > > > > > > > > > > > According to RedfishPkg/Readme.md document: > > > > "The last byte of host-end USB NIC MAC address is the last byte of = BMC- > > end > > > USB > > > > NIC MAC address minus 1." > > > > > > > > It is necessary to subtract 1 from IpmiLanChannelMacAddress. > > > > > > > > Cc: Abner Chang > > > > Cc: Nickle Wang > > > > Cc: Igor Kulchytskyy > > > > Signed-off-by: Mike Maslenkin > > > > --- > > > > .../PlatformHostInterfaceBmcUsbNicLib.c | 4 = ++-- > > > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > > > > > diff --git > > > > > > > > > a/RedfishPkg/Library/PlatformHostInterfaceBmcUsbNicLib/PlatformHostInte= r > > > fac > > > > eBmcUsbNicLib.c > > > > > > > > > b/RedfishPkg/Library/PlatformHostInterfaceBmcUsbNicLib/PlatformHostInte= r > > > fac > > > > eBmcUsbNicLib.c > > > > index 95900579118b..20ec89d4fcb0 100644 > > > > --- > > > > > > > > > a/RedfishPkg/Library/PlatformHostInterfaceBmcUsbNicLib/PlatformHostInte= r > > > fac > > > > eBmcUsbNicLib.c > > > > +++ > > > b/RedfishPkg/Library/PlatformHostInterfaceBmcUsbNicLib/PlatformHostI > > > > +++ nterfaceBmcUsbNicLib.c > > > > @@ -738,8 +738,8 @@ HostInterfaceIpmiCheckMacAddress ( > > > > (VOID *)&IpmiLanChannelMacAddress.Addr, > > > > > > > > IpmiLanMacAddressSize - 1 > > > > > > > > ) !=3D 0) || > > > > > > > > - (IpmiLanChannelMacAddress.Addr[IpmiLanMacAddressSize - 1= ] !=3D > > > > > > > > - *(UsbNicInfo->MacAddress + IpmiLanMacAddressSize - 1) -= 1) > > > > > > > > + (IpmiLanChannelMacAddress.Addr[IpmiLanMacAddressSize - 1= ] - 1 > > > > + !=3D > > > > > > > > + *(UsbNicInfo->MacAddress + IpmiLanMacAddressSize - 1)) > > > > > > > > ) > > > > > > > > { > > > > > > > > DEBUG ((DEBUG_REDFISH_HOST_INTERFACE, " MAC address is = not > > > > matched.\n")); > > > > > > > > -- > > > > 2.32.0 (Apple Git-132) > > > > > > > >=20 > > > -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#111708): https://edk2.groups.io/g/devel/message/111708 Mute This Topic: https://groups.io/mt/102759080/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-