[PATCH] ACPI/PCI: pci_link: reduce verbosity when IRQ is enabled

[PATCH] ACPI/PCI: pci_link: reduce verbosity when IRQ is enabled

[Sinan Kaya]

Rafael,

On 1/16/2018 4:49 PM, Bjorn Helgaas wrote:
> On Tue, Jan 16, 2018 at 01:53:00PM -0500, Sinan Kaya wrote:
>> Correcting linux-pci email.
>>
>> On 1/16/2018 1:51 PM, Sinan Kaya wrote:
>>> When ACPI Link object is enabled, the message is printed with a warning
>>> prefix. Some test tools are capturing warning and test error types as
>>> errors. Let's reduce the verbosity of success case.
>>>
>>> Signed-off-by: Sinan Kaya <okaya@codeaurora.org>
> Acked-by: Bjorn Helgaas <bhelgaas@google.com>
> 
> Looks like this was a result of 4d9391557b68 ("ACPI: add missing KERN_*
> constants to printks"), which I think added the wrong level in this case.
> 

Any chance of merging this for 4.16?

Sinan

-- 
Sinan Kaya
Qualcomm Datacenter Technologies, Inc. as an affiliate of Qualcomm Technologies, Inc.
Qualcomm Technologies, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project.

[PATCH] ACPI/PCI: pci_link: reduce verbosity when IRQ is enabled

[Rafael J. Wysocki]

On Mon, Jan 29, 2018 at 8:01 PM, Sinan Kaya <okaya@codeaurora.org> wrote:
> Rafael,
>
> On 1/16/2018 4:49 PM, Bjorn Helgaas wrote:
>> On Tue, Jan 16, 2018 at 01:53:00PM -0500, Sinan Kaya wrote:
>>> Correcting linux-pci email.
>>>
>>> On 1/16/2018 1:51 PM, Sinan Kaya wrote:
>>>> When ACPI Link object is enabled, the message is printed with a warning
>>>> prefix. Some test tools are capturing warning and test error types as
>>>> errors. Let's reduce the verbosity of success case.
>>>>
>>>> Signed-off-by: Sinan Kaya <okaya@codeaurora.org>
>> Acked-by: Bjorn Helgaas <bhelgaas@google.com>
>>
>> Looks like this was a result of 4d9391557b68 ("ACPI: add missing KERN_*
>> constants to printks"), which I think added the wrong level in this case.
>>
>
> Any chance of merging this for 4.16?

There is.

I've just got back home from travels and will be looking at patches to
pick up for 4.16 shortly.

[PATCH] ACPI/PCI: pci_link: reduce verbosity when IRQ is enabled

[Rafael J. Wysocki]

On Thu, Feb 1, 2018 at 8:32 AM, Rafael J. Wysocki <rafael@kernel.org> wrote:
> On Mon, Jan 29, 2018 at 8:01 PM, Sinan Kaya <okaya@codeaurora.org> wrote:
>> Rafael,
>>
>> On 1/16/2018 4:49 PM, Bjorn Helgaas wrote:
>>> On Tue, Jan 16, 2018 at 01:53:00PM -0500, Sinan Kaya wrote:
>>>> Correcting linux-pci email.
>>>>
>>>> On 1/16/2018 1:51 PM, Sinan Kaya wrote:
>>>>> When ACPI Link object is enabled, the message is printed with a warning
>>>>> prefix. Some test tools are capturing warning and test error types as
>>>>> errors. Let's reduce the verbosity of success case.
>>>>>
>>>>> Signed-off-by: Sinan Kaya <okaya@codeaurora.org>
>>> Acked-by: Bjorn Helgaas <bhelgaas@google.com>
>>>
>>> Looks like this was a result of 4d9391557b68 ("ACPI: add missing KERN_*
>>> constants to printks"), which I think added the wrong level in this case.
>>>
>>
>> Any chance of merging this for 4.16?
>
> There is.

This one is already there in the Linus' tree AFAICS.

[PATCH] ACPI/PCI: pci_link: reduce verbosity when IRQ is enabled

[okaya at codeaurora.org]

On 2018-02-04 04:03, Rafael J. Wysocki wrote:
> On Thu, Feb 1, 2018 at 8:32 AM, Rafael J. Wysocki <rafael@kernel.org> 
> wrote:
>> On Mon, Jan 29, 2018 at 8:01 PM, Sinan Kaya <okaya@codeaurora.org> 
>> wrote:
>>> Rafael,
>>> 
>>> On 1/16/2018 4:49 PM, Bjorn Helgaas wrote:
>>>> On Tue, Jan 16, 2018 at 01:53:00PM -0500, Sinan Kaya wrote:
>>>>> Correcting linux-pci email.
>>>>> 
>>>>> On 1/16/2018 1:51 PM, Sinan Kaya wrote:
>>>>>> When ACPI Link object is enabled, the message is printed with a 
>>>>>> warning
>>>>>> prefix. Some test tools are capturing warning and test error types 
>>>>>> as
>>>>>> errors. Let's reduce the verbosity of success case.
>>>>>> 
>>>>>> Signed-off-by: Sinan Kaya <okaya@codeaurora.org>
>>>> Acked-by: Bjorn Helgaas <bhelgaas@google.com>
>>>> 
>>>> Looks like this was a result of 4d9391557b68 ("ACPI: add missing 
>>>> KERN_*
>>>> constants to printks"), which I think added the wrong level in this 
>>>> case.
>>>> 
>>> 
>>> Any chance of merging this for 4.16?
>> 
>> There is.
> 
> This one is already there in the Linus' tree AFAICS.

You are right. I see it there. I have not seen an applied email. I 
thought it was still outstanding.

> --
> To unsubscribe from this list: send the line "unsubscribe linux-acpi" 
> in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html