[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