Re: [PATCH 0/3] Openrisc 4.10 fixes

From: Stafford Horne
Date: Sat Dec 31 2016 - 17:53:39 EST


Hi Jonas,

On Sat, Dec 31, 2016 at 03:10:37PM +0100, Jonas Bonn wrote:
> Hi Stafford,
>
> On 12/31/2016 12:52 PM, Stafford Horne wrote:
> > Hello,
> >
> > These are a few small changes for openrisc I would like to include in 4.10.
> > The main change is a fix for building with alldefconfig to properly setup
> > then _text symbol. Others are trivial.
> >
> > If no complaints I will push to 4.10 for inclusion and start working on
> > 4.11 changes.
>
> Post-merge window patches should be bug fixes, only. Patch 1/3 probably
> qualifies, but the others are far too trivial to be pushing now. Just put
> them on your 4.11 branch.

Thanks, for some reason I thought minor changes like this were ok too.
I will only push the one change in this case. The rest will go to my
4.11 queue, which I'll send for review in the next few days.

That brings up a question, Do you want to stay in the MAINTAINERS file?
I noticed sometimes you dont get around to review, but often when you do
the reviews are helpful like this one.

> > Let me know if these are too minor to even post for review.
> Nothing is too minor to post for review!

Good to know!

-Stafford

> >
> > Stafford Horne (3):
> > openrisc: Add _text symbol to fix ksym build error
> > openrisc: Add .gitignore
> > MAINTAINERS: Add the openrisc official repository
> >
> > MAINTAINERS | 1 +
> > arch/openrisc/kernel/.gitignore | 1 +
> > arch/openrisc/kernel/vmlinux.lds.S | 2 ++
> > 3 files changed, 4 insertions(+)
> > create mode 100644 arch/openrisc/kernel/.gitignore
> >
>