March 2018 - flashrom-gerrit - mail.coreboot.org

Change in flashrom[master]: Enable 4BA mode for Spansion 25FL256S
by Nico Huber (Code Review) 14 Mar '18

14 Mar '18

Hello Paul Menzel, build bot (Jenkins), I'd like you to reexamine a change. Please visit https://review.coreboot.org/25133 to look at the new patch set (#4). Change subject: Enable 4BA mode for Spansion 25FL256S ...................................................................... Enable 4BA mode for Spansion 25FL256S 4BA mode is entered by setting bit 7 for the extended address register. Change-Id: I807bf55d65763a9f48a6a3377f14f4e5288a7a4c Signed-off-by: Nico Huber <nico.h(a)gmx.de> Tested-by: Michael Fuckner <michael(a)fuckner.net> --- M flash.h M flashchips.c M flashrom.c M spi25.c 4 files changed, 11 insertions(+), 7 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/33/25133/4 -- To view, visit https://review.coreboot.org/25133 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I807bf55d65763a9f48a6a3377f14f4e5288a7a4c Gerrit-Change-Number: 25133 Gerrit-PatchSet: 4 Gerrit-Owner: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

Change in flashrom[master]: flashchips: Add Spansion 25FL256S......0
by Nico Huber (Code Review) 14 Mar '18

14 Mar '18

Hello Paul Menzel, build bot (Jenkins), I'd like you to reexamine a change. Please visit https://review.coreboot.org/25132 to look at the new patch set (#3). Change subject: flashchips: Add Spansion 25FL256S......0 ...................................................................... flashchips: Add Spansion 25FL256S......0 The Spansion 25SFL256S supports 4BA through an extended address register, a 4BA mode set by bit 7 of that register, or native 4BA instructions. Enable the former only for now. Unfortunately the S25SF256S uses another instruction to write the exten- ded address register. So we add an override for the instruction byte. Change-Id: I0a95a81dfe86434f049215ebd8477392391b9efc Signed-off-by: Nico Huber <nico.h(a)gmx.de> Tested-by: Michael Fuckner <michael(a)fuckner.net> --- M flash.h M flashchips.c M spi25.c 3 files changed, 47 insertions(+), 1 deletion(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/32/25132/3 -- To view, visit https://review.coreboot.org/25132 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I0a95a81dfe86434f049215ebd8477392391b9efc Gerrit-Change-Number: 25132 Gerrit-PatchSet: 3 Gerrit-Owner: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

Change in flashrom[master]: Enable native 4BA instructions for Spansion 25FL256S
by build bot (Jenkins) (Code Review) 14 Mar '18

14 Mar '18

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/25134 ) Change subject: Enable native 4BA instructions for Spansion 25FL256S ...................................................................... Patch Set 4: Verified+1 Build Successful https://qa.coreboot.org/job/flashrom-customrules/1205/ : SUCCESS https://qa.coreboot.org/job/flashrom_gerrit/1008/ : SUCCESS -- To view, visit https://review.coreboot.org/25134 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I0ffc816ca714ecce5b89b1eaadb5e73ccb38d9ab Gerrit-Change-Number: 25134 Gerrit-PatchSet: 4 Gerrit-Owner: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 14 Mar 2018 10:44:41 +0000 Gerrit-HasComments: No Gerrit-HasLabels: Yes

1 0

Change in flashrom[master]: Enable native 4BA instructions for Spansion 25FL256S
by Nico Huber (Code Review) 14 Mar '18

14 Mar '18

Hello Paul Menzel, build bot (Jenkins), I'd like you to reexamine a change. Please visit https://review.coreboot.org/25134 to look at the new patch set (#4). Change subject: Enable native 4BA instructions for Spansion 25FL256S ...................................................................... Enable native 4BA instructions for Spansion 25FL256S Change-Id: I0ffc816ca714ecce5b89b1eaadb5e73ccb38d9ab Signed-off-by: Nico Huber <nico.h(a)gmx.de> --- M flashchips.c 1 file changed, 4 insertions(+), 1 deletion(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/34/25134/4 -- To view, visit https://review.coreboot.org/25134 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I0ffc816ca714ecce5b89b1eaadb5e73ccb38d9ab Gerrit-Change-Number: 25134 Gerrit-PatchSet: 4 Gerrit-Owner: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

Change in flashrom[master]: flashchips: Add Spansion 25FL256S......0
by build bot (Jenkins) (Code Review) 13 Mar '18

13 Mar '18

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/25132 ) Change subject: flashchips: Add Spansion 25FL256S......0 ...................................................................... Patch Set 2: Verified+1 Build Successful https://qa.coreboot.org/job/flashrom-customrules/1202/ : SUCCESS https://qa.coreboot.org/job/flashrom_gerrit/1005/ : SUCCESS -- To view, visit https://review.coreboot.org/25132 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I0a95a81dfe86434f049215ebd8477392391b9efc Gerrit-Change-Number: 25132 Gerrit-PatchSet: 2 Gerrit-Owner: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Tue, 13 Mar 2018 17:43:10 +0000 Gerrit-HasComments: No Gerrit-HasLabels: Yes

1 0

Change in flashrom[master]: Enable 4BA mode for Spansion 25FL256S
by build bot (Jenkins) (Code Review) 13 Mar '18

13 Mar '18

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/25133 ) Change subject: Enable 4BA mode for Spansion 25FL256S ...................................................................... Patch Set 3: Verified+1 Build Successful https://qa.coreboot.org/job/flashrom-customrules/1204/ : SUCCESS https://qa.coreboot.org/job/flashrom_gerrit/1007/ : SUCCESS -- To view, visit https://review.coreboot.org/25133 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I807bf55d65763a9f48a6a3377f14f4e5288a7a4c Gerrit-Change-Number: 25133 Gerrit-PatchSet: 3 Gerrit-Owner: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Tue, 13 Mar 2018 17:43:07 +0000 Gerrit-HasComments: No Gerrit-HasLabels: Yes

1 0

Change in flashrom[master]: Enable native 4BA instructions for Spansion 25FL256S
by build bot (Jenkins) (Code Review) 13 Mar '18

13 Mar '18

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/25134 ) Change subject: Enable native 4BA instructions for Spansion 25FL256S ...................................................................... Patch Set 3: Verified+1 Build Successful https://qa.coreboot.org/job/flashrom-customrules/1203/ : SUCCESS https://qa.coreboot.org/job/flashrom_gerrit/1006/ : SUCCESS -- To view, visit https://review.coreboot.org/25134 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I0ffc816ca714ecce5b89b1eaadb5e73ccb38d9ab Gerrit-Change-Number: 25134 Gerrit-PatchSet: 3 Gerrit-Owner: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Tue, 13 Mar 2018 17:42:58 +0000 Gerrit-HasComments: No Gerrit-HasLabels: Yes

1 0

Change in flashrom[master]: flashchips: Add Spansion 25FL256S......0
by Nico Huber (Code Review) 13 Mar '18

13 Mar '18

Hello build bot (Jenkins), I'd like you to reexamine a change. Please visit https://review.coreboot.org/25132 to look at the new patch set (#2). Change subject: flashchips: Add Spansion 25FL256S......0 ...................................................................... flashchips: Add Spansion 25FL256S......0 The Spansion 25SFL256S supports 4BA through an extended address register, a 4BA mode set by bit 7 of that register, or native 4BA instructions. Enable the former only for now. Unfortunately the S25SF256S uses another instruction to write the exten- ded address register. So we add an override for the instruction byte. Change-Id: I0a95a81dfe86434f049215ebd8477392391b9efc Signed-off-by: Nico Huber <nico.h(a)gmx.de> --- M flash.h M flashchips.c M spi25.c 3 files changed, 47 insertions(+), 1 deletion(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/32/25132/2 -- To view, visit https://review.coreboot.org/25132 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I0a95a81dfe86434f049215ebd8477392391b9efc Gerrit-Change-Number: 25132 Gerrit-PatchSet: 2 Gerrit-Owner: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

Change in flashrom[master]: Enable 4BA mode for Spansion 25FL256S
by build bot (Jenkins) (Code Review) 13 Mar '18

13 Mar '18

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/25133 ) Change subject: Enable 4BA mode for Spansion 25FL256S ...................................................................... Patch Set 2: Verified+1 Build Successful https://qa.coreboot.org/job/flashrom-customrules/1200/ : SUCCESS https://qa.coreboot.org/job/flashrom_gerrit/1003/ : SUCCESS -- To view, visit https://review.coreboot.org/25133 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I807bf55d65763a9f48a6a3377f14f4e5288a7a4c Gerrit-Change-Number: 25133 Gerrit-PatchSet: 2 Gerrit-Owner: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Tue, 13 Mar 2018 17:39:52 +0000 Gerrit-HasComments: No Gerrit-HasLabels: Yes

1 0

Change in flashrom[master]: Enable native 4BA instructions for Spansion 25FL256S
by build bot (Jenkins) (Code Review) 13 Mar '18

13 Mar '18

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/25134 ) Change subject: Enable native 4BA instructions for Spansion 25FL256S ...................................................................... Patch Set 2: Verified+1 Build Successful https://qa.coreboot.org/job/flashrom-customrules/1201/ : SUCCESS https://qa.coreboot.org/job/flashrom_gerrit/1004/ : SUCCESS -- To view, visit https://review.coreboot.org/25134 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I0ffc816ca714ecce5b89b1eaadb5e73ccb38d9ab Gerrit-Change-Number: 25134 Gerrit-PatchSet: 2 Gerrit-Owner: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Tue, 13 Mar 2018 17:39:50 +0000 Gerrit-HasComments: No Gerrit-HasLabels: Yes

1 0

2024

2023

2022

2021

2020

2019

2018

2017

flashrom-gerrit March 2018