From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 83278C32774 for ; Tue, 23 Aug 2022 07:16:10 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S240231AbiHWHQJ (ORCPT ); Tue, 23 Aug 2022 03:16:09 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49688 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S240387AbiHWHQG (ORCPT ); Tue, 23 Aug 2022 03:16:06 -0400 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 7DC2C59263 for ; Tue, 23 Aug 2022 00:16:05 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1661238964; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=Il0sasf1rmdUin35WzTgnPhTCMVfyK1Nq9+6f6LdDpI=; b=KbuY1U3xn9EXSneMz4eBRODIwJlNcGYRVfKP7RLgK6T4y53yjWeC/41xNeVnaQigKmqOqh xzlIF8+pzVwl++tATxTHBx8PfN+jTabMtc50xESXQvaZ2gi/PceKza9eePyrLiizTUnqsA BlnHnwQ4H+b8E/Fs9vISTySi1heiVVM= Received: from mail-wm1-f72.google.com (mail-wm1-f72.google.com [209.85.128.72]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_128_GCM_SHA256) id us-mta-388-2qonssK1PG6DPHNUg77aYA-1; Tue, 23 Aug 2022 03:15:55 -0400 X-MC-Unique: 2qonssK1PG6DPHNUg77aYA-1 Received: by mail-wm1-f72.google.com with SMTP id v64-20020a1cac43000000b003a4bea31b4dso9829995wme.3 for ; Tue, 23 Aug 2022 00:15:55 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=mime-version:message-id:date:references:in-reply-to:subject:cc:to :from:x-gm-message-state:from:to:cc; bh=Il0sasf1rmdUin35WzTgnPhTCMVfyK1Nq9+6f6LdDpI=; b=1frH+rgLOAaGi/bYuIPx9wv4/NuzmSOzaIzgKolJycCH4PxGsI7TZG8qiEL3XZESnI 1q2s2FJiXR5YgnC5I83K4NmzZD+BQpsZbGqCqyQJnQ12CmQ2vmYq7joPlfZ9hdM3qzhP ZZG3qmNG/y6gaWWF0RVd6l9WiR8+tlEqV+hPgbqFBXaHvRf8x+3gaiL73koCbCyXDWS2 j3Xq8d+3HbpnZMOmohQ/nuQzVQwPm5vcQ98Tejzt2eauLjkaFu76+dvd5WxhTnjE+CPR Y1fgIBuEZsTu8OVjpzK4PcfSgrLuHorNdzLR7nfYYyRo3MJSclVltskhYyqEmyc3yB/A 9Miw== X-Gm-Message-State: ACgBeo0nC/VK4lhKWWJSEMkayvOr0XPmbijB8SCKmy88sUnjCkoIZIza MZWPC0ebwpGJX8PSA6m0gyvUdugSNnk6i+b8kcdOrNj7OEt+x0+vLMWhcIpuQ5h4yhYbK/d5q9Y 7PieR7s7ZYLmVlx+vGuJ+TmiN X-Received: by 2002:a5d:69c3:0:b0:225:221f:25c with SMTP id s3-20020a5d69c3000000b00225221f025cmr11871597wrw.49.1661238954616; Tue, 23 Aug 2022 00:15:54 -0700 (PDT) X-Google-Smtp-Source: AA6agR5tPE6EDKqWGMRqYwPqG4ws37oKb0chXbAyuDzb6FstQmPHq7WGh3J3UD8GuwtvJu9GhpI8OA== X-Received: by 2002:a5d:69c3:0:b0:225:221f:25c with SMTP id s3-20020a5d69c3000000b00225221f025cmr11871575wrw.49.1661238954351; Tue, 23 Aug 2022 00:15:54 -0700 (PDT) Received: from fedora (nat-2.ign.cz. [91.219.240.2]) by smtp.gmail.com with ESMTPSA id j14-20020adfff8e000000b0021f0af83142sm13260423wrr.91.2022.08.23.00.15.53 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 23 Aug 2022 00:15:53 -0700 (PDT) From: Vitaly Kuznetsov To: "Michael Kelley (LINUX)" , "linux-hyperv@vger.kernel.org" Cc: "linux-kernel@vger.kernel.org" , "dri-devel@lists.freedesktop.org" , Wei Liu , Deepak Rawat , KY Srinivasan , Haiyang Zhang , Stephen Hemminger , Dexuan Cui Subject: RE: [PATCH v1 1/4] Drivers: hv: Move legacy Hyper-V PCI video device's ids to linux/hyperv.h In-Reply-To: References: <20220818142508.402273-1-vkuznets@redhat.com> <20220818142508.402273-2-vkuznets@redhat.com> Date: Tue, 23 Aug 2022 09:15:52 +0200 Message-ID: <8735dnxwev.fsf@redhat.com> MIME-Version: 1.0 Content-Type: text/plain Precedence: bulk List-ID: X-Mailing-List: linux-hyperv@vger.kernel.org "Michael Kelley (LINUX)" writes: > From: Vitaly Kuznetsov Sent: Thursday, August 18, 2022 7:25 AM >> >> There are already two places in kernel with PCI_VENDOR_ID_MICROSOFT/ >> PCI_DEVICE_ID_HYPERV_VIDEO and there's a need to use these from core >> Vmbus code. Move the defines to a common header. >> >> No functional change. >> >> Signed-off-by: Vitaly Kuznetsov >> --- >> drivers/gpu/drm/hyperv/hyperv_drm_drv.c | 3 --- >> drivers/video/fbdev/hyperv_fb.c | 4 ---- >> include/linux/hyperv.h | 4 ++++ >> 3 files changed, 4 insertions(+), 7 deletions(-) >> >> diff --git a/drivers/gpu/drm/hyperv/hyperv_drm_drv.c >> b/drivers/gpu/drm/hyperv/hyperv_drm_drv.c >> index 4a8941fa0815..46f6c454b820 100644 >> --- a/drivers/gpu/drm/hyperv/hyperv_drm_drv.c >> +++ b/drivers/gpu/drm/hyperv/hyperv_drm_drv.c >> @@ -23,9 +23,6 @@ >> #define DRIVER_MAJOR 1 >> #define DRIVER_MINOR 0 >> >> -#define PCI_VENDOR_ID_MICROSOFT 0x1414 >> -#define PCI_DEVICE_ID_HYPERV_VIDEO 0x5353 >> - >> DEFINE_DRM_GEM_FOPS(hv_fops); >> >> static struct drm_driver hyperv_driver = { >> diff --git a/drivers/video/fbdev/hyperv_fb.c b/drivers/video/fbdev/hyperv_fb.c >> index 886c564787f1..b58b445bb529 100644 >> --- a/drivers/video/fbdev/hyperv_fb.c >> +++ b/drivers/video/fbdev/hyperv_fb.c >> @@ -74,10 +74,6 @@ >> #define SYNTHVID_DEPTH_WIN8 32 >> #define SYNTHVID_FB_SIZE_WIN8 (8 * 1024 * 1024) >> >> -#define PCI_VENDOR_ID_MICROSOFT 0x1414 >> -#define PCI_DEVICE_ID_HYPERV_VIDEO 0x5353 >> - >> - >> enum pipe_msg_type { >> PIPE_MSG_INVALID, >> PIPE_MSG_DATA, >> diff --git a/include/linux/hyperv.h b/include/linux/hyperv.h >> index 3b42264333ef..4bb39a8f1af7 100644 >> --- a/include/linux/hyperv.h >> +++ b/include/linux/hyperv.h >> @@ -1516,6 +1516,10 @@ void vmbus_free_mmio(resource_size_t start, >> resource_size_t size); >> .guid = GUID_INIT(0xc376c1c3, 0xd276, 0x48d2, 0x90, 0xa9, \ >> 0xc0, 0x47, 0x48, 0x07, 0x2c, 0x60) >> >> +/* Legacy Hyper-V PCI video device */ >> +#define PCI_VENDOR_ID_MICROSOFT 0x1414 >> +#define PCI_DEVICE_ID_HYPERV_VIDEO 0x5353 > > I've never looked at this before, but shouldn't these move to > include/linux/pci_ids.h with all the others? And we've got > another #define of PCI_VENDOR_ID_MICROSOFT in > drivers/net/ethernet/microsoft/mana/gdma_main.c that > could be deleted. > Oh, true, include/linux/pci_ids.h is much better. Let me send an updated patch (detached from this series). -- Vitaly