From: Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>
To: Borislav Petkov <bp-Gina5bIWoIWzQB+pC5nmwQ@public.gmane.org>
Cc: Thor Thayer <tthayer-EIB2kfCEclfQT0dZR+AlfA@public.gmane.org>,
"robherring2-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org"
<robherring2-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
"dougthompson-aS9lmoZGLiVWk0Htik3J/w@public.gmane.org"
<dougthompson-aS9lmoZGLiVWk0Htik3J/w@public.gmane.org>,
"grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org"
<grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
Pawel Moll <Pawel.Moll-5wv7dgnIgG8@public.gmane.org>,
"ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org"
<ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org>,
"galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org"
<galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>,
"rob-VoJi6FS/r0vR7s880joybQ@public.gmane.org"
<rob-VoJi6FS/r0vR7s880joybQ@public.gmane.org>,
"linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org"
<linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org>,
"dinguyen-EIB2kfCEclfQT0dZR+AlfA@public.gmane.org"
<dinguyen-EIB2kfCEclfQT0dZR+AlfA@public.gmane.org>,
"devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
"linux-edac-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<linux-edac-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
"linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
Subject: Re: [PATCH 3/3] edac: altera: Add SDRAM EDAC support for CycloneV/ArriaV
Date: Tue, 8 Apr 2014 16:40:17 +0100 [thread overview]
Message-ID: <20140408154017.GF11876@e106331-lin.cambridge.arm.com> (raw)
In-Reply-To: <20140408152406.GI30077-fF5Pk5pvG8Y@public.gmane.org>
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.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: Mark Rutland <mark.rutland@arm.com>
To: Borislav Petkov <bp@alien8.de>
Cc: Thor Thayer <tthayer@altera.com>,
"robherring2@gmail.com" <robherring2@gmail.com>,
"dougthompson@xmission.com" <dougthompson@xmission.com>,
"grant.likely@linaro.org" <grant.likely@linaro.org>,
Pawel Moll <Pawel.Moll@arm.com>,
"ijc+devicetree@hellion.org.uk" <ijc+devicetree@hellion.org.uk>,
"galak@codeaurora.org" <galak@codeaurora.org>,
"rob@landley.net" <rob@landley.net>,
"linux@arm.linux.org.uk" <linux@arm.linux.org.uk>,
"dinguyen@altera.com" <dinguyen@altera.com>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"linux-edac@vger.kernel.org" <linux-edac@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 3/3] edac: altera: Add SDRAM EDAC support for CycloneV/ArriaV
Date: Tue, 8 Apr 2014 16:40:17 +0100 [thread overview]
Message-ID: <20140408154017.GF11876@e106331-lin.cambridge.arm.com> (raw)
In-Reply-To: <20140408152406.GI30077@pd.tnic>
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.
next prev parent reply other threads:[~2014-04-08 15:40 UTC|newest]
Thread overview: 55+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1396907649-20212-1-git-send-email-tthayer@altera.com>
2014-04-07 21:54 ` [PATCH 1/3] dts: socfpga: Add bindings for Altera SoC SDRAM controller tthayer at altera.com
2014-04-07 21:54 ` tthayer
2014-04-07 21:54 ` tthayer
2014-04-08 10:48 ` Mark Rutland
2014-04-08 10:48 ` Mark Rutland
2014-04-08 13:38 ` Steffen Trumtrar
2014-04-08 13:38 ` Steffen Trumtrar
2014-04-08 13:38 ` Steffen Trumtrar
2014-04-08 14:29 ` Thor Thayer
2014-04-08 14:29 ` Thor Thayer
2014-04-08 14:29 ` Thor Thayer
2014-04-08 14:33 ` Steffen Trumtrar
2014-04-08 14:33 ` Steffen Trumtrar
2014-04-08 14:33 ` Steffen Trumtrar
2014-04-08 16:02 ` delicious quinoa
2014-04-08 16:02 ` delicious quinoa
2014-04-08 18:52 ` Rob Herring
2014-04-08 18:52 ` Rob Herring
2014-04-11 14:21 ` Thor Thayer
2014-04-11 14:21 ` Thor Thayer
2014-04-11 14:21 ` Thor Thayer
2014-04-11 14:43 ` Thor Thayer
2014-04-11 14:43 ` Thor Thayer
2014-04-11 14:49 ` Thor Thayer
2014-04-11 14:49 ` Thor Thayer
2014-04-11 14:49 ` Thor Thayer
2014-07-10 21:02 ` Alan Tull
2014-07-10 21:02 ` Alan Tull
2014-04-07 21:54 ` [PATCH 2/3] dts: socfpga: Add bindings for Altera SoC SDRAM EDAC tthayer at altera.com
2014-04-07 21:54 ` tthayer
2014-04-07 21:54 ` tthayer
2014-04-08 10:51 ` Mark Rutland
2014-04-08 10:51 ` Mark Rutland
[not found] ` <1396907649-20212-1-git-send-email-tthayer-EIB2kfCEclfQT0dZR+AlfA@public.gmane.org>
2014-04-07 21:54 ` [PATCH 3/3] edac: altera: Add SDRAM EDAC support for CycloneV/ArriaV tthayer-EIB2kfCEclfQT0dZR+AlfA
2014-04-07 21:54 ` tthayer
[not found] ` <1396907649-20212-4-git-send-email-tthayer-EIB2kfCEclfQT0dZR+AlfA@public.gmane.org>
2014-04-08 10:08 ` Borislav Petkov
2014-04-08 10:08 ` Borislav Petkov
2014-04-08 13:57 ` Thor Thayer
2014-04-08 13:57 ` Thor Thayer
2014-04-08 15:24 ` Borislav Petkov
[not found] ` <20140408152406.GI30077-fF5Pk5pvG8Y@public.gmane.org>
2014-04-08 15:40 ` Mark Rutland [this message]
2014-04-08 15:40 ` Mark Rutland
2014-04-08 16:03 ` Borislav Petkov
[not found] ` <20140408160351.GK30077-fF5Pk5pvG8Y@public.gmane.org>
2014-04-08 16:10 ` Mark Rutland
2014-04-08 16:10 ` Mark Rutland
[not found] ` <20140408161054.GA26210-NuALmloUBlrZROr8t4l/smS4ubULX0JqMm0uRHvK7Nw@public.gmane.org>
2014-04-08 16:22 ` Borislav Petkov
2014-04-08 16:22 ` Borislav Petkov
[not found] ` <20140408162213.GL30077-fF5Pk5pvG8Y@public.gmane.org>
2014-04-08 21:15 ` Thor Thayer
2014-04-08 21:15 ` Thor Thayer
2014-04-08 10:45 ` Mark Rutland
2014-04-08 10:45 ` Mark Rutland
[not found] ` <20140408104525.GA11876-NuALmloUBlrZROr8t4l/smS4ubULX0JqMm0uRHvK7Nw@public.gmane.org>
2014-04-08 12:45 ` Steffen Trumtrar
2014-04-08 12:45 ` Steffen Trumtrar
[not found] ` <20140408124541.GA16054-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2014-04-08 14:00 ` Thor Thayer
2014-04-08 14:00 ` Thor Thayer
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20140408154017.GF11876@e106331-lin.cambridge.arm.com \
--to=mark.rutland-5wv7dgnigg8@public.gmane.org \
--cc=Pawel.Moll-5wv7dgnIgG8@public.gmane.org \
--cc=bp-Gina5bIWoIWzQB+pC5nmwQ@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=dinguyen-EIB2kfCEclfQT0dZR+AlfA@public.gmane.org \
--cc=dougthompson-aS9lmoZGLiVWk0Htik3J/w@public.gmane.org \
--cc=galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org \
--cc=grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
--cc=ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org \
--cc=linux-edac-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org \
--cc=rob-VoJi6FS/r0vR7s880joybQ@public.gmane.org \
--cc=robherring2-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=tthayer-EIB2kfCEclfQT0dZR+AlfA@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.