Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1488033 > unrolled thread

Re: [PATCH v10 0/4] ACPI: parse the SPCR table

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH v10 0/4] ACPI: parse the SPCR table Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2016-09-21 12:40 +0200
    Re: [PATCH v10 0/4] ACPI: parse the SPCR table Timur Tabi <timur@codeaurora.org> - 2016-09-21 18:30 +0200
      Re: [PATCH v10 0/4] ACPI: parse the SPCR table Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2016-09-21 18:40 +0200
        Re: [PATCH v10 0/4] ACPI: parse the SPCR table Aleksey Makarov <aleksey.makarov@linaro.org> - 2016-09-21 19:30 +0200

#1488033 — Re: [PATCH v10 0/4] ACPI: parse the SPCR table

On Wed, Sep 21, 2016 at 01:27:30PM +0300, Aleksey Makarov wrote:
> 
> 
> On 09/05/2016 03:36 PM, Aleksey Makarov wrote:
> > 'ARM Server Base Boot Requirements' [1] mentions SPCR (Serial Port Console
> > Redirection Table) [2] as a mandatory ACPI table that specifies the
> > configuration of serial console.
> > 
> > Move "earlycon" early_param handling to earlycon.c to parse this option once
> > 
> > Parse SPCR table, setup earlycon and register specified console.
> > 
> > Enable parsing this table on ARM64.  Earlycon should be set up as early as
> > possible.  ACPI boot tables are mapped in
> > arch/arm64/kernel/acpi.c:acpi_boot_table_init() called from setup_arch() and
> > that's where we parse spcr.  So it has to be opted-in per-arch. When
> > ACPI_SPCR_TABLE is defined initialization of DT earlycon is deferred until the
> > DT/ACPI decision is done.
> > 
> > Implement console_match() for pl011.
> 
> Hi Greg,
> 
> Can you consider picking this series for 4.9 please?

I thought you asked Rafael to take them, they are not in my queue
anymore because of that.  Don't try to shop-around for maintainers
please, that's kind of rude...

greg k-h

[toc] | [next] | [standalone]

#1488251

On Wed, Sep 21, 2016 at 5:37 AM, Greg Kroah-Hartman
<gregkh@linuxfoundation.org> wrote:
>
> I thought you asked Rafael to take them, they are not in my queue
> anymore because of that.  Don't try to shop-around for maintainers
> please, that's kind of rude...

In Aleksey's defense, these patches have been floating around for far
too long, so I can understand his frustration.  But as you said, I
really hope Rafael picks these up for 4.9.  We need these patches for
ARM server.

-- 
Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project.

[toc] | [prev] | [next] | [standalone]

#1488257

On Wed, Sep 21, 2016 at 11:19:35AM -0500, Timur Tabi wrote:
> On Wed, Sep 21, 2016 at 5:37 AM, Greg Kroah-Hartman
> <gregkh@linuxfoundation.org> wrote:
> >
> > I thought you asked Rafael to take them, they are not in my queue
> > anymore because of that.  Don't try to shop-around for maintainers
> > please, that's kind of rude...
> 
> In Aleksey's defense, these patches have been floating around for far
> too long, so I can understand his frustration.

I understand the frustration as well, cross-maintainer messy patches
like this are hard to get merged at times at they fall between the
cracks a lot.

> But as you said, I really hope Rafael picks these up for 4.9.  We need
> these patches for ARM server.

I thought I saw an email saying he would do so, but it might have been
for some other patchset, sorry...

greg k-h

[toc] | [prev] | [next] | [standalone]

#1488296

On 09/21/2016 07:38 PM, Greg Kroah-Hartman wrote:
> On Wed, Sep 21, 2016 at 11:19:35AM -0500, Timur Tabi wrote:
>> On Wed, Sep 21, 2016 at 5:37 AM, Greg Kroah-Hartman
>> <gregkh@linuxfoundation.org> wrote:
>>>
>>> I thought you asked Rafael to take them, they are not in my queue
>>> anymore because of that.  Don't try to shop-around for maintainers
>>> please, that's kind of rude...
>>
>> In Aleksey's defense, these patches have been floating around for far
>> too long, so I can understand his frustration.
> 
> I understand the frustration as well, cross-maintainer messy patches
> like this are hard to get merged at times at they fall between the
> cracks a lot.
> 
>> But as you said, I really hope Rafael picks these up for 4.9.  We need
>> these patches for ARM server.
> 
> I thought I saw an email saying he would do so, but it might have been
> for some other patchset, sorry...

This can be for "ACPI: ARM64: support for ACPI_TABLE_UPGRADE":

	https://lkml.kernel.org/r/CAJZ5v0goXsQ2umcDXU0six+QtxcKGZq7mxhgxuvXTH2iZt6YNA@mail.gmail.com

As for this patchset, Rafael ACKed the ACPI part:

	https://lkml.kernel.org/r/CAJZ5v0hdoLTfjrD8+WxSoxM48dqbZK2KwY_h+63kHKHKgO=JFA@mail.gmail.com

but I can not find any acknowledge from him that he is ready to merge the series.
On the contrary, he expressed his doubt that he is "the right maintainer to send this to":

	https://lkml.kernel.org/r/CAJZ5v0gzjtFzig7nEumr83+J2dGb+OA8GNR2i45ZqznfV_hA-A@mail.gmail.com

I asked the ARM64 people, they said they prefer Rafael to do that:

	https://lkml.kernel.org/r/20160909151758.GA11418@arm.com

I interpreted the silence from Rafael as unwillingness to pull the series 
and thought it's OK to ask you.

I am sorry if this looks rude for Rafael, I did not mean that.

Thank you
Aleksey Makarov

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web