[PATCH v3 1/2] mtd: nand: sunxi: Replace failsafe timing cfg with calculated value
Lucas Stach
l.stach at pengutronix.de
Thu Jun 11 08:11:39 PDT 2015
Am Donnerstag, den 11.06.2015, 16:58 +0200 schrieb Boris Brezillon:
> Hi Roy,
>
> On Thu, 11 Jun 2015 16:50:35 +0200
> Roy Spliet <r.spliet at ultimaker.com> wrote:
>
> > Hello Boris,
> >
> > Op 10-06-15 om 10:59 schreef Boris Brezillon:
> > > Hi Roy,
> > >
> > > On Wed, 10 Jun 2015 10:29:07 +0200
> > > Roy Spliet <r.spliet at ultimaker.com> wrote:
> > >
> > >> Calculates the timing cfg value once when initialising a chip, then sets
> > >> it on chip select. Register definition documented the A83 user manual.
> > > How about rewording the sentence this way:
> > >
> > > "
> > > The TIMING_CFG register was previously statically set to a magic value
> > > (extracted from Allwinner's BSP) when initializing the NAND controller.
> > > Now that we have more details about the TIMING_CFG register layout
> > > (extracted from the A83 user manual) we can dynamically calculate the
> > > appropriate value for each NAND chip and set it when selecting the
> > > chip.
> > > "
> > >
> > >> Signed-off-by: Roy Spliet <r.spliet at ultimaker.com>
> > >>
> > >> V2:
> > >> - Fix crippled comments
> > >>
> > >> V3:
> > >> - Warn for invalid timings
> > >> - Style
> > > Almost right: the changelog should be placed after the '---' line ;-).
> > Git (format-patch, send-email) doesn't let me do that to the best of my
> > knowledge. Other comments I will process, thanks.
>
> Use git format-patch to generate the patches and then add your
> changelog before sending the mails with git send-email.
>
> Or you could generate a cover-letter (pass --cover-letter to
> format-patch) and put your change log in there.
>
Just insert the '---' line after your S-o-B when you write the commit
log. This will do all the right things for you.
Regards,
Lucas
--
Pengutronix e.K. | Lucas Stach |
Industrial Linux Solutions | http://www.pengutronix.de/ |
More information about the linux-mtd
mailing list