Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757041AbdCULzv (ORCPT ); Tue, 21 Mar 2017 07:55:51 -0400 Received: from edison.jonmasters.org ([173.255.233.168]:48854 "EHLO edison.jonmasters.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756509AbdCULzu (ORCPT ); Tue, 21 Mar 2017 07:55:50 -0400 To: David Daney , Vadim Lomovtsev References: <1485793552-12614-1-git-send-email-Vadim.Lomovtsev@caviumnetworks.com> <20170130211236.GW20550@bhelgaas-glaptop.roam.corp.google.com> <20170131102830.GA13854@localhost.localdomain> <20170131142525.GA9942@bhelgaas-glaptop.roam.corp.google.com> <20170131145720.GA14231@localhost.localdomain> <20170131203109.GA14127@bhelgaas-glaptop.roam.corp.google.com> <20170201125325.GA15805@localhost.localdomain> <20170201151807.GA15793@bhelgaas-glaptop.roam.corp.google.com> <376ae1df-da85-e576-75d6-3ae2ab0f1b41@jonmasters.org> <20170315113323.GA28027@localhost.localdomain> <251db823-591b-483b-9c55-13c911020fd1@jonmasters.org> <48b93116-cc0f-e59d-8aa3-704052ad6759@gmail.com> Cc: David.Daney@cavium.com, linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, stemerkhanov@CAVIUMNETWORKS.onmicrosoft.com, Bjorn Helgaas , tn@semihalf.com, bhelgaas@google.com, linux-arm-kernel@lists.infradead.org From: Jon Masters Organization: World Organi{s,z}ation Of Broken Dreams Message-ID: Date: Tue, 21 Mar 2017 07:38:07 -0400 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.0 MIME-Version: 1.0 In-Reply-To: <48b93116-cc0f-e59d-8aa3-704052ad6759@gmail.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-SA-Exim-Connect-IP: 74.92.29.237 X-SA-Exim-Mail-From: jcm@jonmasters.org Subject: Re: [PATCH] PCI: ACPI: Fix ThunderX PEM initialization X-SA-Exim-Version: 4.2.1 (built Sun, 08 Nov 2009 07:31:22 +0000) X-SA-Exim-Scanned: Yes (on edison.jonmasters.org) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1402 Lines: 27 On 03/16/2017 12:25 PM, David Daney wrote: > On 03/16/2017 07:32 AM, Jon Masters wrote: >>> Yes, it is now contains "CAVxxx" as _HID for device config object. >> >> Which is different from the version that was merged into upstream. That >> should never have happened. It will never happen again. I have spent some >> time over the past few days ensuring folks understand that I will not >> allow a repeat of this to occur the next time around. We will have >> platforms that are bulletproof and supported by upstream with any >> errata fixes in a very carefully controlled manner. There will >> under no circumstances ever be a situation like this again. > > We are still evaluating the merits of registering the values that appeared > in v4.10, and not changing them. We should know more in a couple of days. Thanks David. What was the verdict? (for the public record). If we need to get a change into upstream, let's get that teed up before 4.12 merge. And for other folks following along with this thread: I'm not just picking on Cavium here. I'll be doing the same with *every* ARM server SoC company as necessary over the coming months. We are going to have militantly compliant standards adherence in this industry and every ARM server SoC is going to "just work" with an upstream Linux kernel with an ACPI enabled platform. This will be so utterly clean and boring it'll be amazing. Jon.