View Change
1 comment:
File it85spi.c:
Patch Set #10, Line 240: struct it85spi_data *data = (struct it85spi_data *) flash->mst->spi.data;
I strongly oppose to the use of casts to discard const qualifiers. […]
Angel, nice to meet you!
I am currently trying to understand spi_master.data, and why it is const (so we are on the same page on "re-think the situation thoroughly"). Original commit where spi_master.data was introduced does not have much info on why data should be const. Maybe only the pointer can be const, but the data can be modified.
To view, visit change 47655. To unsubscribe, or for help writing mail filters, visit settings.
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I389d34d62e753c012910aa5ff24a496b24a4753c
Gerrit-Change-Number: 47655
Gerrit-PatchSet: 10
Gerrit-Owner: Anastasia Klimchuk <aklm@chromium.org>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Daniel Kurtz <djkurtz@google.com>
Gerrit-Reviewer: Edward O'Callaghan <quasisec@chromium.org>
Gerrit-Reviewer: Eizan Miyamoto <eizan@chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Comment-Date: Tue, 22 Dec 2020 06:41:45 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Daniel Kurtz <djkurtz@google.com>
Comment-In-Reply-To: Angel Pons <th3fanbus@gmail.com>
Comment-In-Reply-To: Anastasia Klimchuk <aklm@chromium.org>
Gerrit-MessageType: comment