Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Wed, 24 Aug 2022 09:17:44 -0700
From:   Dan Williams <dan.j.williams@intel.com>
To:     Davidlohr Bueso <dave@stgolabs.net>,
        Jonathan Cameron <Jonathan.Cameron@huawei.com>
CC:     Adam Manzanares <a.manzanares@samsung.com>,
        "alison.schofield@intel.com" <alison.schofield@intel.com>,
        "vishal.l.verma@intel.com" <vishal.l.verma@intel.com>,
        "ira.weiny@intel.com" <ira.weiny@intel.com>,
        "widawsk@kernel.org" <widawsk@kernel.org>,
        "dan.j.williams@intel.com" <dan.j.williams@intel.com>,
        "linux-cxl@vger.kernel.org" <linux-cxl@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] cxl: Replace HDM decoder granularity magic numbers
Message-ID: <63064f28720cf_1b3229490@dwillia2-xfh.jf.intel.com.notmuch>
References: <CGME20220822170552uscas1p1b1ee530bf38a14806010d65d1b593ab0@uscas1p1.samsung.com>
 <20220822170510.125082-1-a.manzanares@samsung.com>
 <20220822171703.sfizbmn6retpxggi@offworld>
 <20220824104401.00005cd4@huawei.com>
 <20220824153122.2bgths6lnsxlolos@offworld>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <20220824153122.2bgths6lnsxlolos@offworld>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?9iVqrlf9mDzUsGne1n3LFxW0g4KcCzaNF+G1jUodmorx54JbGuYiciKa4JR5?=
 =?us-ascii?Q?9XNAuJ8CgeLGepWUMMI8i+w7fpss0CQtuAwudMINwMUUINg0K+ZCJfvejL0c?=
 =?us-ascii?Q?P6XO2M9L/iojFGuYygOrBMLpmVkzdk4kpxgZwpidHRh9/RyBB9StbrPmGZSr?=
 =?us-ascii?Q?2zmvJ0d2FsXE0u4q058fL0UyBEiI1+CpwtKWPVGe+LK9e7now11pwIRApYaR?=
 =?us-ascii?Q?YDLohMbKEmBATbnVvBD5wIjs90UtUndDmWxUw/ax0Ke+8FDztLQIgbRyZBPg?=
 =?us-ascii?Q?ErP35UXd7Mya+OkALeMpym1HXJKEof1buSf7pM0WGBRrBNTfTwjbQB5Xhr9d?=
 =?us-ascii?Q?lJshjpKhCYMPsF1VJKbVfWZhex6gBahCLoc0G6TD34Hvzypd6cXnOh7nT0pk?=
 =?us-ascii?Q?ybPtG4wMjbQ7HMv11IGq3iBxYyjEJNhSrZKpP1qAJrD47ufrFoF6Ob4KJzEJ?=
 =?us-ascii?Q?bXjrCZFkFp3DjlyyZWX7GH1y/hSWo2FGw4JMy98fhWX5UVRDqoMPva7ZWTjf?=
 =?us-ascii?Q?R1AtFVkOPjo52yuvLnd4sN5MzMxb8qJcfb9iDuqq6pDxuOoNb8fEQhX43u6n?=
 =?us-ascii?Q?bVpgUi+mKynENVG1AJU9k6H+YECjBdOcw9eneryrvhZIZjXjSBX8Qn66A1ZV?=
 =?us-ascii?Q?f4Qweg3Cj4Ilf1BwOhwqFZLqyfpbf8iWu6D04FW5pOSeRhx7y6gCKADXpHos?=
 =?us-ascii?Q?ZCnORcWiTFI28mlHltCfIZ+d4PZ6FWqCVmPA+7TcjRCY4I8OiSy/ARlzBTDk?=
 =?us-ascii?Q?DMmhd+0U9FsuQJfK3OdhznrsYxZAo/43v2RfGjcXwMPlefo8OqN0BFcCcAwY?=
 =?us-ascii?Q?hbzv1oXpWiF/wzmAXQ6yqCSYAGk8gSmDOzulOefG+BnHzkRbBC8NTEZzCtde?=
 =?us-ascii?Q?TVPeclVG9ySoCQrPnZmBFgzEp3zSMQzgHiTORez9i/k1i8+Tr93G9Gj46b8H?=
 =?us-ascii?Q?JiNbqSFlkZQ5vhLQUo/AxC0STjMlv34SyF+5lBWzeJb3umAko8S4G/b1ODai?=
 =?us-ascii?Q?ga6sNNQ26AjhODCqFMHdGoUxUezDqvOsind37cdvefWTPLhglWnf4pXh0Y0x?=
 =?us-ascii?Q?Htp3LLo8NOjqYKNh2NFDh46/bulVIHrFAMyyWgUSn+Wm4Veqt1oeuvg8l5nG?=
 =?us-ascii?Q?xQHd6QP+xFU1ijN0dJJQahyjC8oMiOsNq/MMtrqoLhyYPWsfonyAUb8lRyLY?=
 =?us-ascii?Q?X6JRPuQEDoXdCOCaRluawPwNQ4kBMG+3ibSgoP8JNypR6vxFLcyUiFWbIHBh?=
 =?us-ascii?Q?fixqGpwvzt5R/0Kep3rh5BgeGD2R9+ZriTnv9w1mgGLwEg+arc3M8IcyA4eg?=
 =?us-ascii?Q?Y+CsK31VcutqiknWW28/reO9cVeQ5BOpWYoBXwQdvYtzfHbBPkNFbPARX1Yx?=
 =?us-ascii?Q?DSr2+KkYKiYGvO/wa18Ed9xDDXWGlcS18Eus/oJTv6OHtQpSyqRkXsA9vHJ7?=
 =?us-ascii?Q?swG6gpZI2aAJ4X/FHZeoM4mdMOYs4Fxj5crelOhpVZx1cGZDTqKCWyOb+1xL?=
 =?us-ascii?Q?GwijOUkkvh3l4vOWXrA+bVU8JRFUTy9cM+cbZgm9jtHOPZB2iTAZkgjpTVX1?=
 =?us-ascii?Q?U4VnIPmhwh0DtQkeTfsaYPWFEcflDcdtNHOOEqnGgegoPi9gE9IiK/6aIzIC?=
 =?us-ascii?Q?Ag=3D=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 0eaf3c64-f550-40e2-8b3b-08da85ec2e7a
X-MS-Exchange-CrossTenant-AuthSource: MWHPR1101MB2126.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Aug 2022 16:17:46.6467
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 7j59DCYzaLUwLv/5WCISJspbAJwR08a7M+VkM6L0ArzbwQRHITrdQi+2Hi4WD2TJiByq66Ql+xDdbemzDcQIutJ3CkXOj//4w4fIP4xrRRU=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR1101MB2311
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-7.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_HI,
        SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Davidlohr Bueso wrote:
> On Wed, 24 Aug 2022, Jonathan Cameron wrote:
> 
> >On Mon, 22 Aug 2022 10:17:03 -0700
> >Davidlohr Bueso <dave@stgolabs.net> wrote:
> 
> >> Actually the whole drivers/cxl/* could use updating the comments for 3.0.
> >
> >Interesting point.  What do we want to do on this?  Most similar
> >cases I've been involved on rely on referring to 'oldest' compatible spec.
> >(this is true for ACPI stuff for example).
> 
> I find it incredibly annoying to reference table or section numbers from old
> specs mention in the code. Unless dealing with specific version things, why
> use old specs at all?
> 
> The main drawback to this is obviously the constant need to update everything,
> so...

I think a happy medium is all new references being 3.0 and touching any
old references and refresh them up to 3.0.

That said if someone wants to do the work to refresh all of them, and
someone else wants to review those updates I would take the patch.