coreboot-kgpe-d16/util/scripts/decode_spd.sh
Patrick Georgi 6b5bc77c9b treewide: Remove "this file is part of" lines
Stefan thinks they don't add value.

Command used:
sed -i -e '/file is part of /d' $(git grep "file is part of " |egrep ":( */\*.*\*/\$|#|;#|-- | *\* )" | cut -d: -f1 |grep -v crossgcc |grep -v gcov | grep -v /elf.h |grep -v nvramtool)

The exceptions are for:
 - crossgcc (patch file)
 - gcov (imported from gcc)
 - elf.h (imported from GNU's libc)
 - nvramtool (more complicated header)

The removed lines are:
-       fmt.Fprintln(f, "/* This file is part of the coreboot project. */")
-# This file is part of a set of unofficial pre-commit hooks available
-/* This file is part of coreboot */
-# This file is part of msrtool.
-/* This file is part of msrtool. */
- * This file is part of ncurses, designed to be appended after curses.h.in
-/* This file is part of pgtblgen. */
- * This file is part of the coreboot project.
- /* This file is part of the coreboot project. */
-#  This file is part of the coreboot project.
-# This file is part of the coreboot project.
-## This file is part of the coreboot project.
--- This file is part of the coreboot project.
-/* This file is part of the coreboot project */
-/* This file is part of the coreboot project. */
-;## This file is part of the coreboot project.
-# This file is part of the coreboot project. It originated in the
- * This file is part of the coreinfo project.
-## This file is part of the coreinfo project.
- * This file is part of the depthcharge project.
-/* This file is part of the depthcharge project. */
-/* This file is part of the ectool project. */
- * This file is part of the GNU C Library.
- * This file is part of the libpayload project.
-## This file is part of the libpayload project.
-/* This file is part of the Linux kernel. */
-## This file is part of the superiotool project.
-/* This file is part of the superiotool project */
-/* This file is part of uio_usbdebug */

Change-Id: I82d872b3b337388c93d5f5bf704e9ee9e53ab3a9
Signed-off-by: Patrick Georgi <pgeorgi@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/41194
Reviewed-by: HAOUAS Elyes <ehaouas@noos.fr>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
2020-05-11 17:11:40 +00:00

67 lines
1.7 KiB
Bash
Executable file

#!/bin/bash
#
#
# SPDX-License-Identifier: GPL-2.0-only
# Parses spd hex files and outputs the contents in various formats
#
#
# Outputs csv, set, and json in same folder as SPD_HEX_FILE
#
# Example:
# decode_spd.sh ../../src/mainboard/google/zork/spd/micron-MT40A512M16TB-062E-J.spd.hex
#
# Outputs ../../src/mainboard/google/zork/spd/micron-MT40A512M16TB-062E-J.spd.{json|csv|set}
#
# TODO: This script assumes bincfg binary is at ../bincfg/bincfg (which is the
# result of running the bincfg make), and the specs are at
# ../bincfg/*.spec. This dependency should be made more resilliant and
# configurable.
set -e
function read8 () {
echo $(( 16#$(xxd -s "${2}" -l 1 -p "${1}") ))
}
for file in "$@"
do
bintmp=$(mktemp)
outfile="${file%.hex}.set"
echo "Decoding ${file}, outputting to ${outfile}"
grep -v '^#' "${file}" | xxd -r -p - "${bintmp}"
dram_type=$(read8 "${bintmp}" 2)
if [ ! "${dram_type}" -eq 12 ]
then
#TODO: Handle other dram types
printf "Error: Expecting dram4 (12), got %d\n" "${dram_type}"
continue
fi
revision=$(read8 "${bintmp}" 1)
if [ ! "${revision}" -eq $((0x13)) ]
then
printf "Warning: Expecting revision 0x13, got 0x%x.\n" "${revision}"
fi
module_type=$(read8 "${bintmp}" 3)
case "${module_type}" in
1) # RDIMM
spec="../bincfg/ddr4_registered_spd_512.spec"
;;
2 | 3) #UDIMM | SO-DIMM
spec="../bincfg/ddr4_unbuffered_spd_512.spec"
;;
* )
printf "Error: Unhandled module type %d.\n" "${module_type}"
;;
esac
../bincfg/bincfg -d "${spec}" "${bintmp}" "${outfile}"
grep -v '^#' "${outfile}" | sed -e 's/ = \([^,]\+\)/: "\1"/g' \
> "${file%.hex}.json"
grep -v -e '^#' -e '^{' -e '^}' "${outfile}" | sed -e 's/=/,/g' \
> "${file%.hex}.csv"
done