Messages in this thread | | | Date | Tue, 8 Apr 2014 16:40:17 +0100 | From | Mark Rutland <> | Subject | Re: [PATCH 3/3] edac: altera: Add SDRAM EDAC support for CycloneV/ArriaV |
| |
On Tue, Apr 08, 2014 at 04:24:06PM +0100, Borislav Petkov wrote: > On Tue, Apr 08, 2014 at 08:57:39AM -0500, Thor Thayer wrote: > > Yes, Altera has a group specifically supporting Linux drivers on the > > Altera SoCs. > > Then please add MAINTAINERS file entry for this EDAC driver so that > people can send issues/reports to that group. > > > I was told that the device tree additions should be separate patches > > in a series since the device tree additions go to a separate group for > > approval. > > Well, the EDAC driver depends on the devicetree stuff, right? If so, > they should go together.
The patches should be in the same series, but for review purposes it's nicer if the bindings are separate patches from the code within that series.
I usually look at the drivers implementing bindings and prefer to be Cc'd on the whole series, with both the binding and driver.
Cheers, Mark.
| |