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 960FA740038 for ; Thu, 16 Nov 2023 17:11:32 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=6FGsJ7ZnU3ZGpLhKPVXJLN2D5Ntdi2IQq66G/cCtB54=; c=relaxed/simple; d=groups.io; h=Date:From:To:CC:Subject:Message-ID:References:MIME-Version:In-Reply-To:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Reply-To:List-Unsubscribe-Post:List-Unsubscribe:Content-Type:Content-Disposition; s=20140610; t=1700154691; v=1; b=AmId2oTCylT75/ME6WEY3oS38I2YY3Yi9KHJ8Rq6Cr4/743pfqFtIDvAz7B2ynS5KpPU0CrR CuA1CYLSeK93vcaT35fZDUBZVIAhPNaYWkJDPZg6DH2bhLanuinFjm5CJCun0HLrKPzJhH/i2n6 1JDX734Uz5yzzoTJzqksqwOU= X-Received: by 127.0.0.2 with SMTP id HeOqYY7687511xOKr9liJr2m; Thu, 16 Nov 2023 09:11:31 -0800 X-Received: from mx0b-0031df01.pphosted.com (mx0b-0031df01.pphosted.com [205.220.180.131]) by mx.groups.io with SMTP id smtpd.web11.12341.1700154690615098671 for ; Thu, 16 Nov 2023 09:11:30 -0800 X-Received: from pps.filterd (m0279869.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.17.1.19/8.17.1.19) with ESMTP id 3AGFvgXY011232; Thu, 16 Nov 2023 17:11:29 GMT X-Received: from nasanppmta05.qualcomm.com (i-global254.qualcomm.com [199.106.103.254]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 3ud6ecj4nx-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Thu, 16 Nov 2023 17:11:29 +0000 X-Received: from nasanex01c.na.qualcomm.com (nasanex01c.na.qualcomm.com [10.45.79.139]) by NASANPPMTA05.qualcomm.com (8.17.1.5/8.17.1.5) with ESMTPS id 3AGHBSHA018430 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Thu, 16 Nov 2023 17:11:28 GMT X-Received: from qc-i7.hemma.eciton.net (10.80.80.8) by nasanex01c.na.qualcomm.com (10.45.79.139) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1118.40; Thu, 16 Nov 2023 09:11:26 -0800 Date: Thu, 16 Nov 2023 17:11:23 +0000 From: "Leif Lindholm" To: CC: , , Subject: Re: [edk2-devel] [edk2-platforms PATCH v2 0/1] Marvell package restructure Message-ID: References: <20231102034552.4515-1-ndhillon@marvell.com> MIME-Version: 1.0 In-Reply-To: <20231102034552.4515-1-ndhillon@marvell.com> X-Originating-IP: [10.80.80.8] X-ClientProxiedBy: nasanex01a.na.qualcomm.com (10.52.223.231) To nasanex01c.na.qualcomm.com (10.45.79.139) X-QCInternal: smtphost X-Proofpoint-GUID: 3f4LBoN8nv8WSgM-ye2d7z-YyU73ieKp X-Proofpoint-ORIG-GUID: 3f4LBoN8nv8WSgM-ye2d7z-YyU73ieKp 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,quic_llindhol@quicinc.com List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: X-Gm-Message-State: yp3GwEhOahVDCqeVzGj1j1LHx7686176AA= Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline X-GND-Status: LEGIT Authentication-Results: spool.mail.gandi.net; dkim=pass header.d=groups.io header.s=20140610 header.b=AmId2oTC; dmarc=fail reason="SPF not aligned (relaxed), DKIM not aligned (relaxed)" header.from=quicinc.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 Apologies, I managed to misfile this. Reviewed-by: Leif Lindholm Typo in subject line fixed, pushed as ec5de71d83f3. Thanks! / Leif On Thu, Nov 02, 2023 at 09:15:51 +0530, ndhillon@marvell.com wrote: > From: Narinder Dhillon > > Current Silicon/Marvell package structure does not allow sharing of > components that are common to different SoC's. This restructure will > increase shared code and better seperation. > > Credit to Leif Lindholm for providing this new structure. > > v2: > - Remove three lines for libraries that were erroneously introduced > into MarvellSiliconPkg.dec file. These libraries are present in > Armada7k8k.dsc.inc > - Removed trailing new line at the end of MarvellSiliconPkg.dec > - Squashed patches 2,3,4 into 1 as they don't build separately > > v1: > - Original patch series restructuring Marvell package > > > Narinder Dhillon (1): > Silicon/Marvell: Retructure package > > .../Marvell/Armada70x0Db/Armada70x0Db.dsc | 108 ++++----- > .../Armada70x0DbBoardDescLib.inf | 2 +- > .../NonDiscoverableInitLib.inf | 2 +- > .../Marvell/Armada80x0Db/Armada80x0Db.dsc | 133 ++++++----- > .../Armada80x0DbBoardDescLib.inf | 2 +- > .../NonDiscoverableInitLib.inf | 2 +- > .../Cn9130DbABoardDescLib.inf | 2 +- > .../Cn9132DbABoardDescLib.inf | 2 +- > Platform/Marvell/Cn913xDb/Cn9130DbA.dsc.inc | 100 ++++----- > Platform/Marvell/Cn913xDb/Cn9131DbA.dsc.inc | 66 +++--- > Platform/Marvell/Cn913xDb/Cn9132DbA.dsc.inc | 66 +++--- > Platform/Marvell/Cn913xDb/Cn913xDbA.dsc | 8 +- > .../NonDiscoverableInitLib.inf | 2 +- > .../Armada80x0McBin/Armada80x0McBin.dsc | 116 +++++----- > .../Armada80x0McBinBoardDescLib.inf | 2 +- > .../NonDiscoverableInitLib.inf | 2 +- > .../BoardDescriptionLib.inf | 2 +- > .../Cn913xCEx7Eval/Cn9130Eval.dsc.inc | 40 ++-- > .../Cn913xCEx7Eval/Cn9131Eval.dsc.inc | 56 ++--- > .../Cn913xCEx7Eval/Cn9132Eval.dsc.inc | 56 ++--- > .../Cn913xCEx7Eval/Cn913xCEx7.dsc.inc | 60 ++--- > .../Cn913xCEx7Eval/Cn913xCEx7Eval.dsc | 6 +- > .../NonDiscoverableInitLib.inf | 2 +- > .../Applications/EepromCmd/EepromCmd.inf | 2 +- > .../Applications/FirmwareUpdate/FUpdate.inf | 6 +- > .../Applications/SpiTool/SpiFlashCmd.inf | 6 +- > .../Armada7k8k/AcpiTables/Armada70x0Db.inf | 2 +- > .../Armada7k8k/AcpiTables/Armada80x0Db.inf | 2 +- > .../Armada7k8k/AcpiTables/Armada80x0McBin.inf | 2 +- > Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc | 22 +- > .../Armada7k8kRngDxe/Armada7k8kRngDxe.inf | 4 +- > .../Drivers/PlatInitDxe/PlatInitDxe.inf | 6 +- > .../PlatformFlashAccessLib.inf | 6 +- > .../Library/Armada7k8kLib/Armada7k8kLib.inf | 4 +- > .../Armada7k8kMemoryInitPeiLib.inf | 14 +- > .../PciHostBridgeLib.inf | 2 +- > .../Armada7k8kPciSegmentLib/PciSegmentLib.inf | 2 +- > .../Armada7k8kSampleAtResetLib.inf | 2 +- > .../Armada7k8kSoCDescLib.inf | 4 +- > .../RealTimeClockLib/RealTimeClockLib.inf | 4 +- > .../Marvell/Documentation/PortingGuide.txt | 114 +++++----- > .../Drivers/BoardDesc/MvBoardDescDxe.inf | 18 +- > .../Drivers/Gpio/MvGpioDxe/MvGpioDxe.inf | 2 +- > .../Gpio/MvPca95xxDxe/MvPca95xxDxe.inf | 2 +- > .../Drivers/I2c/MvEepromDxe/MvEepromDxe.inf | 6 +- > .../Marvell/Drivers/I2c/MvI2cDxe/MvI2cDxe.inf | 14 +- > .../Drivers/Net/MvMdioDxe/MvMdioDxe.inf | 2 +- > .../Marvell/Drivers/Net/MvPhyDxe/MvPhyDxe.inf | 12 +- > Silicon/Marvell/Drivers/Net/Pp2Dxe/Pp2Dxe.inf | 16 +- > .../NonDiscoverableDxe/NonDiscoverableDxe.inf | 2 +- > .../Drivers/SdMmc/XenonDxe/XenonDxe.inf | 2 +- > .../SmbiosPlatformDxe/SmbiosPlatformDxe.inf | 14 +- > .../Marvell/Drivers/Spi/MvFvbDxe/MvFvbDxe.inf | 8 +- > .../Spi/MvSpiFlashDxe/MvSpiFlashDxe.inf | 2 +- > .../Spi/MvSpiOrionDxe/MvSpiOrionDxe.inf | 8 +- > .../Marvell/Library/ComPhyLib/ComPhyLib.inf | 28 +-- > Silicon/Marvell/Library/IcuLib/IcuLib.inf | 4 +- > Silicon/Marvell/Library/MppLib/MppLib.inf | 94 ++++---- > .../Marvell/Library/MvGpioLib/MvGpioLib.inf | 2 +- > .../Marvell/Library/UtmiPhyLib/UtmiPhyLib.inf | 2 +- > Silicon/Marvell/Marvell.dec | 208 ------------------ > .../Include/IndustryStandard/MvSmc.h | 0 > .../Include/Library/ArmadaBoardDescLib.h | 0 > .../Include/Library/ArmadaIcuLib.h | 0 > .../Include/Library/ArmadaSoCDescLib.h | 0 > .../Include/Library/MppLib.h | 0 > .../Include/Library/MvComPhyLib.h | 0 > .../Include/Library/MvGpioLib.h | 0 > .../Include/Library/NonDiscoverableInitLib.h | 0 > .../Include/Library/SampleAtResetLib.h | 0 > .../Include/Library/UtmiPhyLib.h | 0 > .../Include/Protocol/BoardDesc.h | 0 > .../Include/Protocol/Eeprom.h | 0 > .../Include/Protocol/Mdio.h | 0 > .../Include/Protocol/MvI2c.h | 0 > .../Include/Protocol/MvPhy.h | 0 > .../Include/Protocol/Spi.h | 0 > .../Include/Protocol/SpiFlash.h | 0 > .../MarvellSiliconPkg/MarvellSiliconPkg.dec | 207 +++++++++++++++++ > .../OcteonTx/AcpiTables/T91/Cn9130DbA.inf | 2 +- > .../OcteonTx/AcpiTables/T91/Cn9131DbA.inf | 2 +- > .../AcpiTables/T91/Cn913xCEx7Eval.inf | 2 +- > 82 files changed, 848 insertions(+), 852 deletions(-) > delete mode 100644 Silicon/Marvell/Marvell.dec > rename Silicon/Marvell/{ => MarvellSiliconPkg}/Include/IndustryStandard/MvSmc.h (100%) > rename Silicon/Marvell/{ => MarvellSiliconPkg}/Include/Library/ArmadaBoardDescLib.h (100%) > rename Silicon/Marvell/{ => MarvellSiliconPkg}/Include/Library/ArmadaIcuLib.h (100%) > rename Silicon/Marvell/{ => MarvellSiliconPkg}/Include/Library/ArmadaSoCDescLib.h (100%) > rename Silicon/Marvell/{ => MarvellSiliconPkg}/Include/Library/MppLib.h (100%) > rename Silicon/Marvell/{ => MarvellSiliconPkg}/Include/Library/MvComPhyLib.h (100%) > rename Silicon/Marvell/{ => MarvellSiliconPkg}/Include/Library/MvGpioLib.h (100%) > rename Silicon/Marvell/{ => MarvellSiliconPkg}/Include/Library/NonDiscoverableInitLib.h (100%) > rename Silicon/Marvell/{ => MarvellSiliconPkg}/Include/Library/SampleAtResetLib.h (100%) > rename Silicon/Marvell/{ => MarvellSiliconPkg}/Include/Library/UtmiPhyLib.h (100%) > rename Silicon/Marvell/{ => MarvellSiliconPkg}/Include/Protocol/BoardDesc.h (100%) > rename Silicon/Marvell/{ => MarvellSiliconPkg}/Include/Protocol/Eeprom.h (100%) > rename Silicon/Marvell/{ => MarvellSiliconPkg}/Include/Protocol/Mdio.h (100%) > rename Silicon/Marvell/{ => MarvellSiliconPkg}/Include/Protocol/MvI2c.h (100%) > rename Silicon/Marvell/{ => MarvellSiliconPkg}/Include/Protocol/MvPhy.h (100%) > rename Silicon/Marvell/{ => MarvellSiliconPkg}/Include/Protocol/Spi.h (100%) > rename Silicon/Marvell/{ => MarvellSiliconPkg}/Include/Protocol/SpiFlash.h (100%) > create mode 100644 Silicon/Marvell/MarvellSiliconPkg/MarvellSiliconPkg.dec > > > base-commit: c1d04ecedf3246ec7c2e0cc12658c9c6a26f91ac > -- > 2.34.1 > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#111321): https://edk2.groups.io/g/devel/message/111321 Mute This Topic: https://groups.io/mt/102335606/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/leave/12367111/7686176/1913456212/xyzzy [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=-