mbox series

[GIT,PULL] SPEAr defconfig update

Message ID CACRpkdYRiWZbkduBRxQdPC_1hJjQJCYhX9NFB31t1Su1CSBNyw@mail.gmail.com
State New
Headers show
Series [GIT,PULL] SPEAr defconfig update | expand

Pull-request

git://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-integrator.git

Message

Linus Walleij Jan. 21, 2019, 12:30 p.m. UTC
Hi ARM SoC maintainers,

this:
1. updates the SPEAr 3xx defconfig
2. replaces the fbdev PL11x framebuffer driver with the DRM PL11x driver
   in the defconfig

This has the ACK of the SPEAr maintainer Viresh Kumar. I don't think
there even is a SPEAr git tree so I send this from my GPIO tree. If they
have other plans, they can say so now :D

Please pull it into ARM SoC in the appropriate place, next kernel is
fine.

Yours,
Linus Walleij

The following changes since commit bfeffd155283772bbe78c6a05dec7c0128ee500c:

  Linux 5.0-rc1 (2019-01-06 17:08:20 -0800)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-integrator.git
tags/pl111-spear-defconfig

for you to fetch changes up to 3e31d705fe0b42f2a7173c13a40ac3b88947a7d1:

  ARM: config: spear3xx: Activate PL111 DRM driver (2019-01-21 13:17:00 +0100)

----------------------------------------------------------------
This updates the SPEAr defconfig and then augments it to
use the DRM PL111 driver rather than the fbdev driver.

----------------------------------------------------------------
Linus Walleij (2):
      ARM: config: Update SPEAr 3xx defconfig
      ARM: config: spear3xx: Activate PL111 DRM driver

 arch/arm/configs/spear3xx_defconfig | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

Comments

Arnd Bergmann Jan. 30, 2019, 9:52 p.m. UTC | #1
On Mon, Jan 21, 2019 at 1:30 PM Linus Walleij <linus.walleij@linaro.org> wrote:
>
> Hi ARM SoC maintainers,
>
> this:
> 1. updates the SPEAr 3xx defconfig
> 2. replaces the fbdev PL11x framebuffer driver with the DRM PL11x driver
>    in the defconfig
>
> This has the ACK of the SPEAr maintainer Viresh Kumar. I don't think
> there even is a SPEAr git tree so I send this from my GPIO tree. If they
> have other plans, they can say so now :D
>
> Please pull it into ARM SoC in the appropriate place, next kernel is

I have a similar comment to the nomadik update, though there
is only one non-obvious option: why do you turn on CONFIG_PREEMPT=y?

       Arnd
Linus Walleij Jan. 31, 2019, 8:02 a.m. UTC | #2
On Wed, Jan 30, 2019 at 10:52 PM Arnd Bergmann <arnd@arndb.de> wrote:
> On Mon, Jan 21, 2019 at 1:30 PM Linus Walleij <linus.walleij@linaro.org> wrote:
> >
> > Hi ARM SoC maintainers,
> >
> > this:
> > 1. updates the SPEAr 3xx defconfig
> > 2. replaces the fbdev PL11x framebuffer driver with the DRM PL11x driver
> >    in the defconfig
> >
> > This has the ACK of the SPEAr maintainer Viresh Kumar. I don't think
> > there even is a SPEAr git tree so I send this from my GPIO tree. If they
> > have other plans, they can say so now :D
> >
> > Please pull it into ARM SoC in the appropriate place, next kernel is
>
> I have a similar comment to the nomadik update, though there
> is only one non-obvious option: why do you turn on CONFIG_PREEMPT=y?

Oh that's just something from my script, sorry.

It'd good that it comes up but I guess it should be a
separate patch.

These are not servers so any other configuration than
CONFIG_PREEMPT is probably just wrong, as is any
other setting than Tickless idle and HRTimers on.

I am thinking maybe we should actually have ARMv7 and
ARMv8 multi Kconfig simply select these things, I don't
really see why anyone would want to configure these
differently, it just seems vaguely cargoculty and overcautious.

Yours,
Linus Walleij
Arnd Bergmann Jan. 31, 2019, 10:48 a.m. UTC | #3
On Thu, Jan 31, 2019 at 9:03 AM Linus Walleij <linus.walleij@linaro.org> wrote:
>
> On Wed, Jan 30, 2019 at 10:52 PM Arnd Bergmann <arnd@arndb.de> wrote:
> > On Mon, Jan 21, 2019 at 1:30 PM Linus Walleij <linus.walleij@linaro.org> wrote:
> > >
> > > Hi ARM SoC maintainers,
> > >
> > > this:
> > > 1. updates the SPEAr 3xx defconfig
> > > 2. replaces the fbdev PL11x framebuffer driver with the DRM PL11x driver
> > >    in the defconfig
> > >
> > > This has the ACK of the SPEAr maintainer Viresh Kumar. I don't think
> > > there even is a SPEAr git tree so I send this from my GPIO tree. If they
> > > have other plans, they can say so now :D
> > >
> > > Please pull it into ARM SoC in the appropriate place, next kernel is
> >
> > I have a similar comment to the nomadik update, though there
> > is only one non-obvious option: why do you turn on CONFIG_PREEMPT=y?
>
> Oh that's just something from my script, sorry.
>
> It'd good that it comes up but I guess it should be a
> separate patch.
>
> These are not servers so any other configuration than
> CONFIG_PREEMPT is probably just wrong, as is any
> other setting than Tickless idle and HRTimers on.
>
> I am thinking maybe we should actually have ARMv7 and
> ARMv8 multi Kconfig simply select these things, I don't
> really see why anyone would want to configure these
> differently, it just seems vaguely cargoculty and overcautious.

Yes, I agree that turning these on is a good idea, but please
send that as a separate patch. One patch to turn it on in
all configurations you care about is fine.

      Arnd