GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

util/spd_tools/spd_gen/lp5: Encode Bank Architecture 

ADL supports 8B Bank Architecture, whereas Sabrina supports either BG or
16B Bank Architectures depending on the speed. This influences SDRAM
Density and Banks, SDRAM Addressing bytes in SPD. Encode them as per the
individual SoC advisories.

BUG=b:211510456
TEST=Generate SPDs for Sabrina.

Signed-off-by: Karthikeyan Ramasubramanian <kramasub@google.com>
Change-Id: Ic854ccccb2b301e75d0f28cd36daf87fd41e07e7
Reviewed-on: https://review.coreboot.org/c/coreboot/+/61948
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nick Vaccaro <nvaccaro@google.com>
This commit is contained in:
Karthikeyan Ramasubramanian 2022-02-14 23:34:24 -07:00 committed by Felix Held
parent 3248db0e5a
commit e1d6f5b80d
4 changed files with 109 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
spd/lp5/set-1/spd-1.hex
View File

@ -1,4 +1,4 @@
23 11 13 0E 15 1A 95 18 00 40 00 00 02 02 00 00
23 11 13 0E 85 19 95 18 00 40 00 00 02 02 00 00
00 00 0A 00 00 00 00 00 AA 00 90 A8 90 90 06 C0
03 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00

2
spd/lp5/set-1/spd-2.hex
View File

@ -1,4 +1,4 @@
23 11 13 0E 15 1A B5 18 00 40 00 00 0A 02 00 00
23 11 13 0E 85 19 B5 18 00 40 00 00 0A 02 00 00
00 00 0A 00 00 00 00 00 AA 00 90 A8 90 90 06 C0
03 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00

2
spd/lp5/set-1/spd-3.hex
View File

@ -1,4 +1,4 @@
23 11 13 0E 16 22 95 18 00 40 00 00 02 02 00 00
23 11 13 0E 86 21 95 18 00 40 00 00 02 02 00 00
00 00 0A 00 00 00 00 00 AA 00 90 A8 90 C0 08 60
04 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00

125
util/spd_tools/src/spd_gen/lp5.go
View File

@ -48,6 +48,12 @@ type LP5SpeedParams struct {
MaxCASLatency int
}
type LP5BankArchParams struct {
NumBanks int
BankGroups int
BurstAddressBits int
}
type LP5SPDAttribFunc func(*LP5MemAttributes) byte
type LP5SPDAttribTableEntry struct {
@ -55,8 +61,11 @@ type LP5SPDAttribTableEntry struct {
getVal LP5SPDAttribFunc
}
type LP5SetFunc func(*LP5MemAttributes) int
type LP5Set struct {
SPDRevision byte
getBankArch LP5SetFunc
optionalFeatures byte
otherOptionalFeatures byte
busWidthEncoding byte
@ -146,24 +155,15 @@ const (
)
const (
/*
* LPDDR5 has a flexible bank architecture with three programmable bank modes: BG, 8B, 16B.
* ADL will use 8B mode for all parts.
*
* From JEDEC spec:
* 7:6 (Bank Group Bits) = 00 (no bank groups)
* 5:4 (Bank Address Bits) = 01 (8 banks)
* Set bits 7:4 to 0b0001.
*/
LP5BankGroupsBanks = 0x1
// The column addresses are the same for x8 & x16 and for all Bank Architectures.
LP5ColAddressBits = 6
)
/*
* Tables 8 and 9 from JESD209-5B.
* ADL uses 8B mode for all parts. The column addresses are the same for x8 and x16.
* Effective column address bits = column address bits + burst address bits = 6 + 5 = 11.
* As per JESD 21-C, this is encoded as 0b010.
*/
LP5ColumnAddressBits = 0x2
const (
// LPDDR5 has a flexible bank architecture with three programmable bank modes: BG, 8B, 16B.
LP5BGBankArch = iota
LP58BBankArch
LP516BBankArch
)
/* ------------------------------------------------------------------------------------------ */
@ -178,6 +178,7 @@ var LP5PlatformSetMap = map[int][]int{
var LP5SetInfo = map[int]LP5Set{
0: {
SPDRevision: LP5SPDValueRevision1_0,
getBankArch: LP5GetBankArchSet0,
/*
* From JEDEC spec:
* 5:4 (Maximum Activate Window) = 00 (8192 * tREFI)
@ -196,6 +197,7 @@ var LP5SetInfo = map[int]LP5Set{
},
1: {
SPDRevision: LP5SPDValueRevision1_1,
getBankArch: LP5GetBankArchSet1,
/*
* For Sabrina (as per advisory b/211510456):
* 5:4 (Maximum Activate Window) = 01 (4096 * tREFI)
@ -285,6 +287,24 @@ var LP5DensityGbToSPDEncoding = map[int]LP5DensityParams{
},
}
/*
* Maps the number of banks to the SPD encoding as per JESD 21-C.
*/
var LP5NumBanksEncoding = map[int]byte{
4: 0x0,
8: 0x1,
16: 0x2,
}
/*
* Maps the Bank Group bits to the SPD encoding as per JESD 21-C.
*/
var LP5BankGroupsEncoding = map[int]byte{
1: 0x0,
2: 0x1,
4: 0x2,
}
/*
* Maps the number of row address bits to the SPD encoding as per JESD 21-C.
*/
@ -296,6 +316,34 @@ var LP5RowAddressBitsEncoding = map[int]byte{
18: 0x6,
}
/*
* Maps the number of column address bits to the SPD encoding as per JESD 21-C.
*/
var LP5ColAddressBitsEncoding = map[int]byte{
9: 0x0,
10: 0x1,
11: 0x2,
12: 0x3,
}
var LP5BankArchToSPDEncoding = map[int]LP5BankArchParams{
LP5BGBankArch: {
NumBanks: 4,
BankGroups: 4,
BurstAddressBits: 4,
},
LP58BBankArch: {
NumBanks: 8,
BankGroups: 1,
BurstAddressBits: 5,
},
LP516BBankArch: {
NumBanks: 16,
BankGroups: 1,
BurstAddressBits: 4,
},
}
/*
* TCKMinPs:
* LPDDR5 has two clocks: the command/address clock (CK) and the data clock (WCK). They are
@ -361,6 +409,40 @@ func LP5EncodeSPDRevision(memAttribs *LP5MemAttributes) byte {
return f.SPDRevision
}
func LP5GetBankArchSet0(memAttribs *LP5MemAttributes) int {
// ADL will use 8B mode for all parts.
return LP58BBankArch
}
func LP5GetBankArchSet1(memAttribs *LP5MemAttributes) int {
/*
* Sabrina does not support 8B. It uses 16B Bank Architecture for speed <= 3200 Mbps.
* It uses BG Bank Architecture for speed > 3200 Mbps.
*/
if memAttribs.SpeedMbps <= 3200 {
return LP516BBankArch
}
return LP5BGBankArch
}
func LP5GetBankArch(memAttribs *LP5MemAttributes) int {
f, ok := LP5SetInfo[LP5CurrSet]
if ok == false || f.getBankArch == nil {
return LP5BGBankArch
}
return f.getBankArch(memAttribs)
}
func LP5GetNumBanks(memAttribs *LP5MemAttributes) int {
return LP5BankArchToSPDEncoding[LP5GetBankArch(memAttribs)].NumBanks
}
func LP5GetBankGroups(memAttribs *LP5MemAttributes) int {
return LP5BankArchToSPDEncoding[LP5GetBankArch(memAttribs)].BankGroups
}
func LP5EncodeDensityBanks(memAttribs *LP5MemAttributes) byte {
var b byte
@ -368,17 +450,22 @@ func LP5EncodeDensityBanks(memAttribs *LP5MemAttributes) byte {
b = LP5DensityGbToSPDEncoding[memAttribs.DensityPerDieGb].DensityEncoding
// 5:4 Bank address bits.
b |= LP5NumBanksEncoding[LP5GetNumBanks(memAttribs)] << 4
// 7:6 Bank group bits.
b |= LP5BankGroupsBanks << 4
b |= LP5BankGroupsEncoding[LP5GetBankGroups(memAttribs)] << 6
return b
}
func LP5GetBurstAddressBits(memAttribs *LP5MemAttributes) int {
return LP5BankArchToSPDEncoding[LP5GetBankArch(memAttribs)].BurstAddressBits
}
func LP5EncodeSdramAddressing(memAttribs *LP5MemAttributes) byte {
var b byte
// 2:0 Column address bits.
b = LP5ColumnAddressBits
b = LP5ColAddressBitsEncoding[LP5ColAddressBits + LP5GetBurstAddressBits(memAttribs)]
// 5:3 Row address bits.
density := memAttribs.DensityPerDieGb