[PATCH v3 5/5] mtd: rawnand: denali: remove hard-coded DENALI_DEFAULT_OOB_SKIP_BYTES
miquel.raynal at bootlin.com
Tue Jan 14 09:05:34 PST 2020
On Fri, 2019-12-20 at 11:31:55 UTC, Masahiro Yamada wrote:
> As commit 0d55c668b218 (mtd: rawnand: denali: set SPARE_AREA_SKIP_BYTES
> register to 8 if unset") says, there were three solutions discussed:
>  Add a DT property to specify the skipped bytes in OOB
>  Associate the preferred value with compatible
>  Hard-code the default value in the driver
> At that time,  was chosen because I did not have enough information
> about the other platforms than UniPhier.
> That commit also says "The preferred value may vary by platform. If so,
> please trade up to a different solution." My intention was to replace
>  with , not keep both  and .
> Now that we have switched to  for SOCFPGA's SPARE_AREA_SKIP_BYTES=2,
>  should be removed. This should be OK because denali_pci.c just
> gets back to the original behavior.
> Signed-off-by: Masahiro Yamada <yamada.masahiro at socionext.com>
Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git nand/next, thanks.
More information about the linux-mtd