From 3f365ac849969a313da9a4310f6f8ef13aaa3767 Mon Sep 17 00:00:00 2001 From: Denis 'GNUtoo' Carikli Date: Mon, 11 Nov 2024 22:07:55 +0100 Subject: [PATCH] blobs.list: coreboot: fam15h: remove minnowmax_{1,2}gb.absf. The file contains the following copyright header: // This file contains an 'Intel Peripheral Driver' and is // licensed for Intel CPUs and chipsets under the terms of your // license agreement with Intel or your vendor. [...] [...] // Copyright (c) 2010-2013 Intel Corporation. All rights reserved // This software and associated documentation (if any) is furnished // under a license and may only be used or copied in accordance // with the terms of the license. Except as permitted by such // license, no part of this software or documentation may be // reproduced, stored in a retrieval system, or transmitted in any // form or by any means without the express written consent of // Intel Corporation. While there is also many contradicting statements like this one in src/soc/intel/fsp_baytrail/Kconfig: ## This file is part of the coreboot project. ## ## Copyright (C) 2011 The ChromiumOS Authors. All rights reserved. ## Copyright (C) 2013-2014 Sage Electronic Engineering, LLC. ## ## This program is free software; you can redistribute it and/or modify ## it under the terms of the GNU General Public License as published by ## the Free Software Foundation; version 2 of the License. ## ## This program is distributed in the hope that it will be useful, ## but WITHOUT ANY WARRANTY; without even the implied warranty of ## MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the ## GNU General Public License for more details. The baytrail FSP was added in Coreboot by the commit 954f3882f1ea8512de9a5a6a38569c36bffae405 ("Add the Bay Trail FSP include & srx directories") by Martin Roth, proably not on behalf on Intel. The commit also contains an email address from Martin Roth with the se-eng.com domain (from Sage Electronic Engineering) and doesn't contain any email address related to Intel. This increase the probability that Intel wasn't involved in adding the Bay Trail FSP to Coreboot. Because of the (strong) doubts, the fact that the Bay Trail FSP was also removed upstream and that GNU Boot doesn't support computers with Intel Bay Trail, it's easier to just remove the nonfree software. Reported-by: Leah Rowe Signed-off-by: Denis 'GNUtoo' Carikli Acked-by: Adrien Bourmault --- resources/coreboot/fam15h_rdimm/blobs.list | 2 ++ resources/coreboot/fam15h_udimm/blobs.list | 2 ++ 2 files changed, 4 insertions(+) diff --git a/resources/coreboot/fam15h_rdimm/blobs.list b/resources/coreboot/fam15h_rdimm/blobs.list index f40c715..f3cc769 100644 --- a/resources/coreboot/fam15h_rdimm/blobs.list +++ b/resources/coreboot/fam15h_rdimm/blobs.list @@ -23,3 +23,5 @@ src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbInitKB/AlibSsdtKB.h src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbInitKB/GnbSamuPatchKB.h src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbInitKB/GnbSmuFirmwareKB.h src/vendorcode/cavium/bdk/libbdk-hal/if/bdk-if-phy-vetesse.c +src/vendorcode/intel/fsp1_0/baytrail/absf/minnowmax_1gb.absf +src/vendorcode/intel/fsp1_0/baytrail/absf/minnowmax_2gb.absf diff --git a/resources/coreboot/fam15h_udimm/blobs.list b/resources/coreboot/fam15h_udimm/blobs.list index f40c715..f3cc769 100644 --- a/resources/coreboot/fam15h_udimm/blobs.list +++ b/resources/coreboot/fam15h_udimm/blobs.list @@ -23,3 +23,5 @@ src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbInitKB/AlibSsdtKB.h src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbInitKB/GnbSamuPatchKB.h src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbInitKB/GnbSmuFirmwareKB.h src/vendorcode/cavium/bdk/libbdk-hal/if/bdk-if-phy-vetesse.c +src/vendorcode/intel/fsp1_0/baytrail/absf/minnowmax_1gb.absf +src/vendorcode/intel/fsp1_0/baytrail/absf/minnowmax_2gb.absf