On Fri, Jan 29, 2021 at 4:32 PM Jacob Keller <jacob.e.kel...@intel.com> wrote: > > > > On 1/29/2021 1:04 PM, Willem de Bruijn wrote: > > On Fri, Jan 29, 2021 at 4:01 PM Willem de Bruijn > > <willemdebruijn.ker...@gmail.com> wrote: > >> > >> On Thu, Jan 28, 2021 at 7:46 PM Tony Nguyen <anthony.l.ngu...@intel.com> > >> wrote: > >>> > >>> From: Jacob Keller <jacob.e.kel...@intel.com> > >>> > >>> The ice flash contains two copies of each of the NVM, Option ROM, and > >>> Netlist modules. Each bank has a pointer word and a size word. In order > >>> to correctly read from the active flash bank, the driver must calculate > >>> the offset manually. > >>> > >>> During NVM initialization, read the Shadow RAM control word and > >>> determine which bank is active for each NVM module. Additionally, cache > >>> the size and pointer values for use in calculating the correct offset. > >>> > >>> Signed-off-by: Jacob Keller <jacob.e.kel...@intel.com> > >>> Tested-by: Tony Brelinski <tonyx.brelin...@intel.com> > >>> Signed-off-by: Tony Nguyen <anthony.l.ngu...@intel.com> > >>> --- > >>> drivers/net/ethernet/intel/ice/ice_nvm.c | 151 ++++++++++++++++++++++ > >>> drivers/net/ethernet/intel/ice/ice_type.h | 37 ++++++ > >>> 2 files changed, 188 insertions(+) > >>> > >>> diff --git a/drivers/net/ethernet/intel/ice/ice_nvm.c > >>> b/drivers/net/ethernet/intel/ice/ice_nvm.c > >>> index b0f0b4fc266b..308344045397 100644 > >>> --- a/drivers/net/ethernet/intel/ice/ice_nvm.c > >>> +++ b/drivers/net/ethernet/intel/ice/ice_nvm.c > >>> @@ -603,6 +603,151 @@ static enum ice_status > >>> ice_discover_flash_size(struct ice_hw *hw) > >>> return status; > >>> } > >>> > >>> +/** > >>> + * ice_read_sr_pointer - Read the value of a Shadow RAM pointer word > >>> + * @hw: pointer to the HW structure > >>> + * @offset: the word offset of the Shadow RAM word to read > >>> + * @pointer: pointer value read from Shadow RAM > >>> + * > >>> + * Read the given Shadow RAM word, and convert it to a pointer value > >>> specified > >>> + * in bytes. This function assumes the specified offset is a valid > >>> pointer > >>> + * word. > >>> + * > >>> + * Each pointer word specifies whether it is stored in word size or 4KB > >>> + * sector size by using the highest bit. The reported pointer value will > >>> be in > >>> + * bytes, intended for flat NVM reads. > >>> + */ > >>> +static enum ice_status > >>> +ice_read_sr_pointer(struct ice_hw *hw, u16 offset, u32 *pointer) > >>> +{ > >>> + enum ice_status status; > >>> + u16 value; > >>> + > >>> + status = ice_read_sr_word(hw, offset, &value); > >>> + if (status) > >>> + return status; > >>> + > >>> + /* Determine if the pointer is in 4KB or word units */ > >>> + if (value & ICE_SR_NVM_PTR_4KB_UNITS) > >>> + *pointer = (value & ~ICE_SR_NVM_PTR_4KB_UNITS) * 4 * 1024; > >>> + else > >>> + *pointer = value * 2; > >> > >> Should this be << 2, for 4B words? > > > > Never mind, sorry. I gather from patch 3 that wordsize is 16b. > > > > > Ah, yes that could have been explained a bit better. In this context, a > word is indeed 2 bytes. > > Perhaps we could have used "<< 1" and "<< 12" or similar instead of the > multiplication, but I felt this was a bit more clear.
Thanks. That doesn't matter (for me). I just wrongly assumed wordsize to be 4B.