cdavies at infradead.org said: + mtd->read_user_prot_reg = cfi_amdstd_secsi_read; + mtd->read_fact_prot_reg = cfi_amdstd_secsi_read; Can't we make this conditional on the flash type, and refrain from checking it at runtime? -- dwmw2