On Thu, Jun 09, 2022 at 03:23:42PM +0200, Ansuel Smith wrote: > Reorder qcom_nand_host to save holes in the struct. You forgot to reorder other structs also as I requested :/ Thanks, Mani > > Signed-off-by: Ansuel Smith <ansuelsmth@xxxxxxxxx> > --- > drivers/mtd/nand/raw/qcom_nandc.c | 10 ++++++---- > 1 file changed, 6 insertions(+), 4 deletions(-) > > diff --git a/drivers/mtd/nand/raw/qcom_nandc.c b/drivers/mtd/nand/raw/qcom_nandc.c > index 1a77542c6d67..7fbbd3e7784c 100644 > --- a/drivers/mtd/nand/raw/qcom_nandc.c > +++ b/drivers/mtd/nand/raw/qcom_nandc.c > @@ -431,11 +431,12 @@ struct qcom_nand_controller { > * and reserved bytes > * @cw_data: the number of bytes within a codeword protected > * by ECC > + * @ecc_bytes_hw: ECC bytes used by controller hardware for this > + * chip > + * > * @use_ecc: request the controller to use ECC for the > * upcoming read/write > * @bch_enabled: flag to tell whether BCH ECC mode is used > - * @ecc_bytes_hw: ECC bytes used by controller hardware for this > - * chip > * @status: value to be returned if NAND_CMD_STATUS command > * is executed > * @last_command: keeps track of last command on this chip. used > @@ -452,11 +453,12 @@ struct qcom_nand_host { > int cs; > int cw_size; > int cw_data; > - bool use_ecc; > - bool bch_enabled; > int ecc_bytes_hw; > int spare_bytes; > int bbm_size; > + > + bool use_ecc; > + bool bch_enabled; > u8 status; > int last_command; > > -- > 2.36.1 > -- மணிவண்ணன் சதாசிவம்