Messages in this thread | | | Date | Wed, 4 Jan 2012 11:21:33 -0700 | From | Grant Likely <> | Subject | Re: [PATCHv3 1/3] gpio: add a driver for the Synopsys DesignWare APB GPIO block |
| |
On Mon, Jan 02, 2012 at 01:25:00PM +0000, Mark Brown wrote: > On Mon, Jan 02, 2012 at 12:53:16PM +0000, Jamie Iles wrote: > > The Synopsys DesignWare block is used in some ARM devices (picoxcell) > > and can be configured to provide multiple banks of GPIO pins. > > > > v3: - depend on rather than select IRQ_DOMAIN > > - split IRQ support into a separate patch > > v2: - use Rob Herring's irqdomain in generic irq chip patches > > - use reg property to indicate bank index > > - support irqs on both edges based on LinusW's u300 driver > > Put stuff like this after the ---, it shouldn't end up in git history.
I disagree. Keeping the revision history in the git commit is actually quite useful when trying to figure out which version of a patch actually got applied. I'm asking patch authors to keep it above the ---
g.
| |