From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: None (no SPF record) identity=mailfrom; client-ip=2a00:1450:4864:20::542; helo=mail-ed1-x542.google.com; envelope-from=mw@semihalf.com; receiver=edk2-devel@lists.01.org Received: from mail-ed1-x542.google.com (mail-ed1-x542.google.com [IPv6:2a00:1450:4864:20::542]) (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 0EE582118DC4D for ; Sun, 18 Nov 2018 22:54:10 -0800 (PST) Received: by mail-ed1-x542.google.com with SMTP id f4so11668351edq.10 for ; Sun, 18 Nov 2018 22:54:10 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=semihalf-com.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id; bh=DfxjQBTMHBADbz7V2WmwL8VqpHlVIkl2Z4n0tTEb4QE=; b=cVpZ157SP+gSO4mW8ZxRuQmEgD/t6Fb9BJ/78//YCwGObUZgWpxBSXtjiyx1VvUz+u 1XF88FJOSoOWHTd04Ype6w46DTN9WSOTm0CHYG3IkrhIEM/ilU5IQ/2bvcer+eBSo0ss Mr/ZyitpFlnPZmMa/Yk8bbqI0b58vosj9uCFwLTgJ9yXh68gChKLXtS9WOdW0E2FJ/ut 7M+CVhss0ANQJOlYXb0ASP9jXaCBL/I9oU9+ijhbukjV+Wy+c1EmLoxv2QA+SJIODDZv Bz+ELTPYlbu+U6Ab6hXEwVJF8i4kLmyboVwNMOfaE8z8/Ctp8blweQHe+yhlzvZhisUJ e6oQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=DfxjQBTMHBADbz7V2WmwL8VqpHlVIkl2Z4n0tTEb4QE=; b=GNoApHowaCxaJzVngdGLG+qG0Wdo52chwds9FolRwcCyCucKu1ediiVig7kusnrhj1 j0XlAZQb6lNfv6DAF8jr1XPLzeLiC0CSylP19h4NIsRuR6CKrNgYQDRFo1ertG+58Mhz 6dEh4wr+aV2FcB5/k4+0d4WmkO0MNjf92f3OdDAKhVAgf3LIcAkxixhteM/HiKaCAMpL GDpPwz5chUMYGnZrgcoorROSY1ZOBEQis7PexF4UxYtq/h/7Ugd1odmiwfuFHiBqVuBI qCvhTg6h315bWvB0LdwsUNfMq2T5yH3JUJRt35sdVvVYRNZFb5zTGHUJCXk+APkijDYg tDWQ== X-Gm-Message-State: AGRZ1gK3LO4OOb8TCF8Oa8+rksBqhxM/+Q+Z7IQnVFKK+6aM09ScotXL F/qz13uDeInKWrBavNPNhvNkLJJOwbQ= X-Google-Smtp-Source: AJdET5fOB7PynJc0auCejCbsBYjJjwq2EaWBoAi4Q/dg49W7zStC/bwiu3MziBiqCYhcFF5gRCuLYA== X-Received: by 2002:aa7:d8d3:: with SMTP id k19mr12097701eds.64.1542610448782; Sun, 18 Nov 2018 22:54:08 -0800 (PST) Received: from gilgamesh.semihalf.com (31-172-191-173.noc.fibertech.net.pl. [31.172.191.173]) by smtp.gmail.com with ESMTPSA id i56sm4893236eda.37.2018.11.18.22.54.07 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Sun, 18 Nov 2018 22:54:07 -0800 (PST) From: Marcin Wojtas To: edk2-devel@lists.01.org Cc: leif.lindholm@linaro.org, ard.biesheuvel@linaro.org, hao.a.wu@intel.com, nadavh@marvell.com, mw@semihalf.com, jsd@semihalf.com, jaz@semihalf.com, kostap@marvell.com, tm@semihalf.com Date: Mon, 19 Nov 2018 07:53:32 +0100 Message-Id: <1542610420-9258-1-git-send-email-mw@semihalf.com> X-Mailer: git-send-email 2.7.4 Subject: [platforms: PATCH v6 0/8] Armada7k8k Xenon driver rework 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: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 X-List-Received-Date: Mon, 19 Nov 2018 06:54:11 -0000 Hi, The sixth version of the patchset applies minor correction in the last patch and also simplifies the BoardDesc library callbacks. There is no unnecessary allocation now. Patches are available in the github: https://github.com/MarvellEmbeddedProcessors/edk2-open-platform/commits/xenon-upstream-r20181119 Generic driver patches with fixes and extended SdMmcOverride protocol: https://github.com/MarvellEmbeddedProcessors/edk2-open-platform/commits/sdmmc-override-upstream-r20181109 I'm looking forward to the comments and remarks. Best regards, Marcin Changelog: v5->v6: * 3,4,5 - pass the pointer to the static table directly instead of the copy allocation * 6/8 - remove freeing SdMmc board description structure in NonDiscoverableDevicesDxe, as from now on the static table will be used * 8/8 - change SdMmcOverride protocol comment to '#PRODUCES' - s/Controler/Controller/ - remove freeing board description structure v4->v5: * 1,2,6/8 - add Ard's RB * 2,3,4,5 - change parameters to OUT - assign count only after allocation is successful * 7/8 & 8/8 - split driver rework patches into two steps v3->v4: * 1/7: - add Ard's RB * 2/7: - use local enum definition for SlotType in order not to include MdeModulePkg private header * 7/7: - rework capability handling, without using the structure defined in the MdeModulePkg header v2->v3 * 1/7: - rename NotifyPhase parameter to PhaseData * 7/7: - rename NotifyPhase parameter to PhaseData - update UHS_MODE_SEL only for HS200/HS400 in XenonSdMmcHcUhsSignaling - use local macros for standard SDHC registers in order not to include private MdeModulePkg header v1 -> v2 * 1/7 and 7/7 - adjust to modified SdMmcOverride NotifyPhase and Capability routines Marcin Wojtas (2): Silicon/SynQuacer/PlatformDxe: adjust to updated SdMmcOverride Marvell/Drivers: XenonDxe: Remove SdMmcPciHcDxe files Tomasz Michalec (6): Marvell/Library: ArmadaBoardDescLib: Extend SDMMC information SolidRun/Armada80x0McBin: Introduce board description library Marvell/Armada70x0Db: Introduce board description library Marvell/Armada80x0Db: Introduce board description library Marvell/Drivers: MvBoardDesc: Extend information for SdMmc Marvell/Drivers: XenonDxe: Switch to use generic SdMmcPciHcDxe Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc | 3 +- Platform/Marvell/Armada70x0Db/Armada70x0Db.dsc | 3 + Platform/Marvell/Armada80x0Db/Armada80x0Db.dsc | 3 + Platform/SolidRun/Armada80x0McBin/Armada80x0McBin.dsc | 3 + Silicon/Marvell/Armada7k8k/Armada7k8k.fdf | 3 +- Platform/Marvell/Armada70x0Db/Armada70x0DbBoardDescLib/Armada70x0DbBoardDescLib.inf | 34 + Platform/Marvell/Armada80x0Db/Armada80x0DbBoardDescLib/Armada80x0DbBoardDescLib.inf | 34 + Platform/SolidRun/Armada80x0McBin/Armada80x0McBinBoardDescLib/Armada80x0McBinBoardDescLib.inf | 34 + Silicon/Marvell/Drivers/BoardDesc/MvBoardDescDxe.inf | 1 + Silicon/Marvell/Drivers/SdMmc/XenonDxe/{SdMmcPciHcDxe.inf => XenonDxe.inf} | 33 +- Silicon/Marvell/Drivers/SdMmc/XenonDxe/SdMmcPciHcDxe.h | 791 -------- Silicon/Marvell/Drivers/SdMmc/XenonDxe/SdMmcPciHci.h | 550 ------ Silicon/Marvell/Drivers/SdMmc/XenonDxe/XenonPciHci.h | 151 ++ Silicon/Marvell/Drivers/SdMmc/XenonDxe/XenonSdMmcOverride.h | 53 + Silicon/Marvell/Drivers/SdMmc/XenonDxe/XenonSdhci.h | 131 +- Silicon/Marvell/Include/Library/ArmadaBoardDescLib.h | 21 +- Platform/Marvell/Armada70x0Db/Armada70x0DbBoardDescLib/Armada70x0DbBoardDescLib.c | 60 + Platform/Marvell/Armada80x0Db/Armada80x0DbBoardDescLib/Armada80x0DbBoardDescLib.c | 60 + Platform/SolidRun/Armada80x0McBin/Armada80x0McBinBoardDescLib/Armada80x0McBinBoardDescLib.c | 60 + Silicon/Marvell/Drivers/BoardDesc/MvBoardDescDxe.c | 24 +- Silicon/Marvell/Drivers/NonDiscoverableDxe/NonDiscoverableDxe.c | 1 - Silicon/Marvell/Drivers/SdMmc/XenonDxe/ComponentName.c | 211 --- Silicon/Marvell/Drivers/SdMmc/XenonDxe/EmmcDevice.c | 1164 ------------ Silicon/Marvell/Drivers/SdMmc/XenonDxe/SdDevice.c | 1190 ------------ Silicon/Marvell/Drivers/SdMmc/XenonDxe/SdMmcPciHcDxe.c | 1320 -------------- Silicon/Marvell/Drivers/SdMmc/XenonDxe/SdMmcPciHci.c | 1928 -------------------- Silicon/Marvell/Drivers/SdMmc/XenonDxe/XenonPciHci.c | 321 ++++ Silicon/Marvell/Drivers/SdMmc/XenonDxe/XenonSdMmcOverride.c | 429 +++++ Silicon/Marvell/Drivers/SdMmc/XenonDxe/XenonSdhci.c | 408 +++-- Silicon/Socionext/SynQuacer/Drivers/PlatformDxe/Emmc.c | 9 +- Silicon/Marvell/Drivers/SdMmc/XenonDxe/SdMmcPciHcDxe.uni | 23 - Silicon/Marvell/Drivers/SdMmc/XenonDxe/SdMmcPciHcDxeExtra.uni | 19 - 32 files changed, 1691 insertions(+), 7384 deletions(-) create mode 100644 Platform/Marvell/Armada70x0Db/Armada70x0DbBoardDescLib/Armada70x0DbBoardDescLib.inf create mode 100644 Platform/Marvell/Armada80x0Db/Armada80x0DbBoardDescLib/Armada80x0DbBoardDescLib.inf create mode 100644 Platform/SolidRun/Armada80x0McBin/Armada80x0McBinBoardDescLib/Armada80x0McBinBoardDescLib.inf rename Silicon/Marvell/Drivers/SdMmc/XenonDxe/{SdMmcPciHcDxe.inf => XenonDxe.inf} (65%) delete mode 100644 Silicon/Marvell/Drivers/SdMmc/XenonDxe/SdMmcPciHcDxe.h delete mode 100644 Silicon/Marvell/Drivers/SdMmc/XenonDxe/SdMmcPciHci.h create mode 100644 Silicon/Marvell/Drivers/SdMmc/XenonDxe/XenonPciHci.h create mode 100644 Silicon/Marvell/Drivers/SdMmc/XenonDxe/XenonSdMmcOverride.h create mode 100644 Platform/Marvell/Armada70x0Db/Armada70x0DbBoardDescLib/Armada70x0DbBoardDescLib.c create mode 100644 Platform/Marvell/Armada80x0Db/Armada80x0DbBoardDescLib/Armada80x0DbBoardDescLib.c create mode 100644 Platform/SolidRun/Armada80x0McBin/Armada80x0McBinBoardDescLib/Armada80x0McBinBoardDescLib.c delete mode 100644 Silicon/Marvell/Drivers/SdMmc/XenonDxe/ComponentName.c delete mode 100755 Silicon/Marvell/Drivers/SdMmc/XenonDxe/EmmcDevice.c delete mode 100644 Silicon/Marvell/Drivers/SdMmc/XenonDxe/SdDevice.c delete mode 100644 Silicon/Marvell/Drivers/SdMmc/XenonDxe/SdMmcPciHcDxe.c delete mode 100644 Silicon/Marvell/Drivers/SdMmc/XenonDxe/SdMmcPciHci.c create mode 100644 Silicon/Marvell/Drivers/SdMmc/XenonDxe/XenonPciHci.c create mode 100644 Silicon/Marvell/Drivers/SdMmc/XenonDxe/XenonSdMmcOverride.c delete mode 100644 Silicon/Marvell/Drivers/SdMmc/XenonDxe/SdMmcPciHcDxe.uni delete mode 100644 Silicon/Marvell/Drivers/SdMmc/XenonDxe/SdMmcPciHcDxeExtra.uni -- 2.7.4