From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: None (no SPF record) identity=mailfrom; client-ip=2a00:1450:4010:c07::243; helo=mail-lf0-x243.google.com; envelope-from=mw@semihalf.com; receiver=edk2-devel@lists.01.org Received: from mail-lf0-x243.google.com (mail-lf0-x243.google.com [IPv6:2a00:1450:4010:c07::243]) (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 8EDCE2034A794 for ; Tue, 28 Nov 2017 00:15:08 -0800 (PST) Received: by mail-lf0-x243.google.com with SMTP id k66so35935643lfg.3 for ; Tue, 28 Nov 2017 00:19:31 -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=Px8dhb8eZ8buAEoFDi/dAvemNXYaszRhJAuFKAJ77xo=; b=X67ABigKkAfF1kjzSHfByUPE2dppV0cnQGEgwjVGkZUBIGUp68AZMpzDmFdnwiZww3 wHiTj4VRLTA2SGpuEnGdrQF2Uas6QrGAAxMDGAOlpRK2m1QGj7cvwKZxFE4H7IBGtB+8 vapdaMzOnHn31+SvpA3BO9jzSiYnfCPb/sAHBkpt3EvioJpMwz1vrIbvFmJnGew/w3k1 nL5A9PwzUjw1irg9vcpBDwxabwfFnQ338DNo29xG1PCUjPXlUdTfhcPUSCfgMBEdhj9/ i4BNQHpVi+WNr3juvOXwVtlFbO5DqqqCqb9NQ2Mty1D3a91L4lQbN5DYplQXZuzYYSgt Qphw== 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=Px8dhb8eZ8buAEoFDi/dAvemNXYaszRhJAuFKAJ77xo=; b=QexIts2LZ/8rMktHdi0eLlo2vulwZkNA62OdLT5CSHV+/CsP/kJVprVchy2mmCC6xM XiihGl6xgPcy1OvneQYUXcK9VFJ50Dm0z41s9J9gB4Cw1OZwOEA3x45WlguQgyK0TXUc dE6uxqktjdSwXOIGuAMwUx5RE3FLj70dh/NxiIyr3GcDv0K90Ag5ZJgoThBawF9sqn8m SAOEq2gJSSuwBOt+r5Eg92UGl9zlQwA2J2xOts1zpHbNdCmTz4L7DR6EI1Lx9U6H7raF QtpHuTSQP/j05M4WjhFRkavOxxltIXx2HS9CRIwY9sZobSt74vLKqtFiVDlk9NY/Oyy7 o5CA== X-Gm-Message-State: AJaThX7Uk4C77/LAqWQIGm6osS3bqV/r0Tq4+LymN2flAhjnwxyl/xJU KvS9hORkFWA3lav0jAVukRXkQsXt5Tw= X-Google-Smtp-Source: AGs4zMYJ+BtDySeKH4LyT2cgxUZLoUnQXZ5FJjjlBE1t1QYTDjDG1Fwb+Om5APZH314WtUfUzq4j3g== X-Received: by 10.25.195.211 with SMTP id t202mr13576170lff.53.1511857168533; Tue, 28 Nov 2017 00:19:28 -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 j85sm79732lfh.53.2017.11.28.00.19.27 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Tue, 28 Nov 2017 00:19:27 -0800 (PST) From: Marcin Wojtas To: edk2-devel@lists.01.org Cc: leif.lindholm@linaro.org, ard.biesheuvel@linaro.org, nadavh@marvell.com, neta@marvell.com, kostap@marvell.com, jinghua@marvell.com, mw@semihalf.com, jsd@semihalf.com Date: Tue, 28 Nov 2017 09:19:09 +0100 Message-Id: <1511857153-9266-1-git-send-email-mw@semihalf.com> X-Mailer: git-send-email 2.7.4 Subject: [platforms: PATCH v3 0/4] Armada 7k/8k variable support X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 28 Nov 2017 08:15:08 -0000 Hi, I submit v3 of the Armada variable support with the MvFvbDxe driver improved according to the recent review. The biggest changes are a proper implementation of Set/GetAttributes protocol callbacks. Details can be found in the changelog below. Patches are available in the github: https://github.com/MarvellEmbeddedProcessors/edk2-open-platform/commits/var-upstream-r20171128 I'm looking forward to the comments or remarks. Best regards, Marcin Changelog: v2 -> v3 * 2/4 - 4/4 - Add RBs * 1/4 - Fix line-breaking in MvFvbValidateFvHeader - Check actual WriteDisabled attributes MvFvbEraseBlocks - Rework MvFvbGetAttributes/MvFvbSetAttributes to properly support those features v1 -> v2: * Change patches order in order to safely apply related Depex modifications * 1/4: - Fix style (line breaking, openining brackets, alignment, typos) - Rework MvFvbSetAttributes - because it can return only EFI_SUCCESS / EFI_INVALID_PARAMETER and should return FVB attributes, add debug information, that operation is not supported and return default values via MvFvbGetAttributes. - Return EFI_SUCCESS, in case NumBytes == 0 in MvFvbRead. * 2/4: - Move Depex configuration from 4/4 * 3/4: - Move Depex configuration from 4/4 - Reword and break commit message. * 4/4: - Remove Depex configuration Marcin Wojtas (4): Platform/Marvell: Introduce MvFvbDxe variable support driver Marvell/Drivers: MvSpiFlash: Enable using driver in RT Marvell/Drivers: MvSpiDxe: Enable using driver in RT Marvell/Armada: Enable variables support Platform/Marvell/Armada/Armada.dsc.inc | 25 +- Platform/Marvell/Armada/Armada70x0.fdf | 6 +- Platform/Marvell/Drivers/Spi/Devices/MvSpiFlash.c | 58 +- Platform/Marvell/Drivers/Spi/Devices/MvSpiFlash.h | 1 + Platform/Marvell/Drivers/Spi/Devices/MvSpiFlash.inf | 11 +- Platform/Marvell/Drivers/Spi/MvSpiDxe.c | 50 +- Platform/Marvell/Drivers/Spi/MvSpiDxe.h | 2 + Platform/Marvell/Drivers/Spi/MvSpiDxe.inf | 9 +- Platform/Marvell/Drivers/Spi/Variables/MvFvbDxe.c | 1138 ++++++++++++++++++++ Platform/Marvell/Drivers/Spi/Variables/MvFvbDxe.h | 128 +++ Platform/Marvell/Drivers/Spi/Variables/MvFvbDxe.inf | 91 ++ Platform/Marvell/Include/Protocol/Spi.h | 7 + Platform/Marvell/Marvell.dec | 4 + 13 files changed, 1516 insertions(+), 14 deletions(-) create mode 100644 Platform/Marvell/Drivers/Spi/Variables/MvFvbDxe.c create mode 100644 Platform/Marvell/Drivers/Spi/Variables/MvFvbDxe.h create mode 100644 Platform/Marvell/Drivers/Spi/Variables/MvFvbDxe.inf -- 2.7.4