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 X-Spam-Level: X-Spam-Status: No, score=-3.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 353F0C432BE for ; Tue, 10 Aug 2021 17:03:49 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 1B03160E9B for ; Tue, 10 Aug 2021 17:03:49 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230250AbhHJREG (ORCPT ); Tue, 10 Aug 2021 13:04:06 -0400 Received: from mga09.intel.com ([134.134.136.24]:36149 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229474AbhHJREF (ORCPT ); Tue, 10 Aug 2021 13:04:05 -0400 X-IronPort-AV: E=McAfee;i="6200,9189,10072"; a="214932763" X-IronPort-AV: E=Sophos;i="5.84,310,1620716400"; d="scan'208";a="214932763" Received: from orsmga005.jf.intel.com ([10.7.209.41]) by orsmga102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 Aug 2021 10:03:42 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.84,310,1620716400"; d="scan'208";a="638866988" Received: from linux.intel.com ([10.54.29.200]) by orsmga005.jf.intel.com with ESMTP; 10 Aug 2021 10:03:41 -0700 Received: from debox1-desk1.jf.intel.com (debox1-desk1.jf.intel.com [10.54.75.53]) by linux.intel.com (Postfix) with ESMTP id AE3A6580922; Tue, 10 Aug 2021 10:03:41 -0700 (PDT) Message-ID: <9a142d180bffa7524bb36ccdba1c866f8b084720.camel@linux.intel.com> Subject: Re: [PATCH 00/20] Move Intel platform drivers to intel directory to improve readability. From: "David E. Box" Reply-To: david.e.box@linux.intel.com To: Kate Hsuan , Hans de Goede , Mark Gross , Alex Hung , Sujith Thomas , Rajneesh Bhardwaj , David E Box , Zha Qipeng , Mika Westerberg , Srinivas Pandruvada , AceLan Kao , Jithu Joseph , Maurice Ma , Andy Shevchenko , Dan Carpenter , Daniel Scally , linux-kernel@vger.kernel.org, Dell.Client.Kernel@dell.com Cc: platform-driver-x86@vger.kernel.org Date: Tue, 10 Aug 2021 10:03:41 -0700 In-Reply-To: <20210810095832.4234-1-hpa@redhat.com> References: <20210810095832.4234-1-hpa@redhat.com> Organization: David E. Box Content-Type: text/plain; charset="UTF-8" User-Agent: Evolution 3.38.4 (3.38.4-1.fc33) MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, On Tue, 2021-08-10 at 17:58 +0800, Kate Hsuan wrote: > All the intel platform specific drivers are moved to intel/. > It makes more clear file structure to improve the readability. > A lot of subfolders for single files. Is this necessary? I'm submitting a single driver soon, unrelated to anything else. I was just going to drop it in /intel. David > Kate Hsuan (20): >   Move Intel hid of pdx86 to intel directory to improve readability. >   Move Intel WMI driver of pdx86 to intel/ directory to improve >     readability. >   Move Intel bxtwc driver of pdx86 to intel/ directory to improve >     readability. >   Move Intel chtdc_ti driver of pdx86 to improve readability. >   Move MRFLD power button driver of pdx86 to intel directory to > improve >     readability. >   Move Intel PMC core of pdx86 to intel/ directory to improve >     readability. >   Move Intel PMT driver of pdx86 to intel/ to improve readability. >   Move Intel P-Unit of pdx86 to intel/ directory to improve > readability. >   Move Intel SCU IPC of pdx86 to intel directory to increase >     readability. >   Move Intel SoC telemetry driver to intel directory to improve >     readability. >   Move Intel IPS driver of pdx86 to improve readability. >   Move Intel RST driver of pdx86 to intel directory to improve >     readability. >   Move Intel smartconnect driver of pdx86 to intel/ directory to > improve >     readability. >   Move Intel SST driver to intel/ directory to improve readability. >   Move Intel turbo max 3 driver to intel/ directory to improve >     readability. >   Move Intel uncore freq driver to intel/ directory to improve >     readability. >   Move Intel int0002 vgpio driver to intel/ directory to inprove >     readability. >   Move Intel thermal driver for menlow platform driver to intel/ >     directory to improve readability. >   Move OakTrail driver to the intel/ directory to improve > readability. >   Move Intel virtual botton driver to intel/ directory to improve >     readability. > >  drivers/platform/x86/Kconfig                  | 297 ---------------- > -- >  drivers/platform/x86/Makefile                 |  34 +- >  drivers/platform/x86/intel/Kconfig            |  21 ++ >  drivers/platform/x86/intel/Makefile           |  31 ++ >  drivers/platform/x86/intel/bxtwc/Kconfig      |  15 + >  drivers/platform/x86/intel/bxtwc/Makefile     |   6 + >  .../x86/{ => intel/bxtwc}/intel_bxtwc_tmu.c   |   0 >  drivers/platform/x86/intel/chtdc_ti/Kconfig   |  16 + >  drivers/platform/x86/intel/chtdc_ti/Makefile  |   7 + >  .../chtdc_ti}/intel_chtdc_ti_pwrbtn.c         |   0 >  drivers/platform/x86/intel/hid/Kconfig        |  17 + >  drivers/platform/x86/intel/hid/Makefile       |   7 + >  .../platform/x86/{ => intel/hid}/intel-hid.c  |   0 >  drivers/platform/x86/intel/int0002/Kconfig    |  23 ++ >  drivers/platform/x86/intel/int0002/Makefile   |   6 + >  .../{ => intel/int0002}/intel_int0002_vgpio.c |   0 >  .../{ => intel}/intel_speed_select_if/Kconfig |   0 >  .../intel_speed_select_if/Makefile            |   0 >  .../intel_speed_select_if/isst_if_common.c    |   0 >  .../intel_speed_select_if/isst_if_common.h    |   0 >  .../intel_speed_select_if/isst_if_mbox_msr.c  |   0 >  .../intel_speed_select_if/isst_if_mbox_pci.c  |   0 >  .../intel_speed_select_if/isst_if_mmio.c      |   0 >  drivers/platform/x86/intel/ips/Kconfig        |  14 + >  drivers/platform/x86/intel/ips/Makefile       |   6 + >  .../platform/x86/{ => intel/ips}/intel_ips.c  |   0 >  .../platform/x86/{ => intel/ips}/intel_ips.h  |   0 >  drivers/platform/x86/intel/menlow/Kconfig     |  14 + >  drivers/platform/x86/intel/menlow/Makefile    |   6 + >  .../x86/{ => intel/menlow}/intel_menlow.c     |   0 >  drivers/platform/x86/intel/mrfld/Kconfig      |  17 + >  drivers/platform/x86/intel/mrfld/Makefile     |   6 + >  .../{ => intel/mrfld}/intel_mrfld_pwrbtn.c    |   0 >  drivers/platform/x86/intel/oaktrail/Kconfig   |  15 + >  drivers/platform/x86/intel/oaktrail/Makefile  |   6 + >  .../x86/{ => intel/oaktrail}/intel_oaktrail.c |   0 >  drivers/platform/x86/intel/pmc_core/Kconfig   |  26 ++ >  drivers/platform/x86/intel/pmc_core/Makefile  |   6 + >  .../x86/{ => intel/pmc_core}/intel_pmc_core.c |   0 >  .../x86/{ => intel/pmc_core}/intel_pmc_core.h |   0 >  .../pmc_core}/intel_pmc_core_pltdrv.c         |   0 >  drivers/platform/x86/intel/pmt/Kconfig        |  41 +++ >  drivers/platform/x86/intel/pmt/Makefile       |   9 + >  .../x86/{ => intel/pmt}/intel_pmt_class.c     |   0 >  .../x86/{ => intel/pmt}/intel_pmt_class.h     |   0 >  .../x86/{ => intel/pmt}/intel_pmt_crashlog.c  |   0 >  .../x86/{ => intel/pmt}/intel_pmt_telemetry.c |   0 >  drivers/platform/x86/intel/punit/Kconfig      |  10 + >  drivers/platform/x86/intel/punit/Makefile     |   6 + >  .../x86/{ => intel/punit}/intel_punit_ipc.c   |   0 >  drivers/platform/x86/intel/rst/Kconfig        |  16 + >  drivers/platform/x86/intel/rst/Makefile       |   6 + >  .../platform/x86/{ => intel/rst}/intel-rst.c  |   0 >  drivers/platform/x86/intel/scu/Kconfig        |  52 +++ >  drivers/platform/x86/intel/scu/Makefile       |  11 + >  .../x86/{ => intel/scu}/intel_scu_ipc.c       |   0 >  .../x86/{ => intel/scu}/intel_scu_ipcutil.c   |   0 >  .../x86/{ => intel/scu}/intel_scu_pcidrv.c    |   0 >  .../x86/{ => intel/scu}/intel_scu_pltdrv.c    |   0 >  .../x86/{ => intel/scu}/intel_scu_wdt.c       |   0 >  .../platform/x86/intel/smartconnect/Kconfig   |  18 ++ >  .../platform/x86/intel/smartconnect/Makefile  |   6 + >  .../smartconnect}/intel-smartconnect.c        |   0 >  drivers/platform/x86/intel/telemetry/Kconfig  |  16 + >  drivers/platform/x86/intel/telemetry/Makefile |   9 + >  .../telemetry}/intel_telemetry_core.c         |   0 >  .../telemetry}/intel_telemetry_debugfs.c      |   0 >  .../telemetry}/intel_telemetry_pltdrv.c       |   0 >  .../platform/x86/intel/turbo_max_3/Kconfig    |  14 + >  .../platform/x86/intel/turbo_max_3/Makefile   |   6 + >  .../turbo_max_3}/intel_turbo_max_3.c          |   0 >  .../platform/x86/intel/uncore_freq/Kconfig    |  15 + >  .../platform/x86/intel/uncore_freq/Makefile   |   6 + >  .../uncore_freq}/intel-uncore-frequency.c     |   0 >  drivers/platform/x86/intel/vbtn/Kconfig       |  16 + >  drivers/platform/x86/intel/vbtn/Makefile      |   6 + >  .../x86/{ => intel/vbtn}/intel-vbtn.c         |   0 >  drivers/platform/x86/intel/wmi/Kconfig        |  26 ++ >  drivers/platform/x86/intel/wmi/Makefile       |   7 + >  .../{ => intel/wmi}/intel-wmi-sbl-fw-update.c |   0 >  .../{ => intel/wmi}/intel-wmi-thunderbolt.c   |   0 >  81 files changed, 562 insertions(+), 330 deletions(-) >  create mode 100644 drivers/platform/x86/intel/bxtwc/Kconfig >  create mode 100644 drivers/platform/x86/intel/bxtwc/Makefile >  rename drivers/platform/x86/{ => intel/bxtwc}/intel_bxtwc_tmu.c > (100%) >  create mode 100644 drivers/platform/x86/intel/chtdc_ti/Kconfig >  create mode 100644 drivers/platform/x86/intel/chtdc_ti/Makefile >  rename drivers/platform/x86/{ => > intel/chtdc_ti}/intel_chtdc_ti_pwrbtn.c (100%) >  create mode 100644 drivers/platform/x86/intel/hid/Kconfig >  create mode 100644 drivers/platform/x86/intel/hid/Makefile >  rename drivers/platform/x86/{ => intel/hid}/intel-hid.c (100%) >  create mode 100644 drivers/platform/x86/intel/int0002/Kconfig >  create mode 100644 drivers/platform/x86/intel/int0002/Makefile >  rename drivers/platform/x86/{ => > intel/int0002}/intel_int0002_vgpio.c (100%) >  rename drivers/platform/x86/{ => > intel}/intel_speed_select_if/Kconfig (100%) >  rename drivers/platform/x86/{ => > intel}/intel_speed_select_if/Makefile (100%) >  rename drivers/platform/x86/{ => > intel}/intel_speed_select_if/isst_if_common.c (100%) >  rename drivers/platform/x86/{ => > intel}/intel_speed_select_if/isst_if_common.h (100%) >  rename drivers/platform/x86/{ => > intel}/intel_speed_select_if/isst_if_mbox_msr.c (100%) >  rename drivers/platform/x86/{ => > intel}/intel_speed_select_if/isst_if_mbox_pci.c (100%) >  rename drivers/platform/x86/{ => > intel}/intel_speed_select_if/isst_if_mmio.c (100%) >  create mode 100644 drivers/platform/x86/intel/ips/Kconfig >  create mode 100644 drivers/platform/x86/intel/ips/Makefile >  rename drivers/platform/x86/{ => intel/ips}/intel_ips.c (100%) >  rename drivers/platform/x86/{ => intel/ips}/intel_ips.h (100%) >  create mode 100644 drivers/platform/x86/intel/menlow/Kconfig >  create mode 100644 drivers/platform/x86/intel/menlow/Makefile >  rename drivers/platform/x86/{ => intel/menlow}/intel_menlow.c (100%) >  create mode 100644 drivers/platform/x86/intel/mrfld/Kconfig >  create mode 100644 drivers/platform/x86/intel/mrfld/Makefile >  rename drivers/platform/x86/{ => intel/mrfld}/intel_mrfld_pwrbtn.c > (100%) >  create mode 100644 drivers/platform/x86/intel/oaktrail/Kconfig >  create mode 100644 drivers/platform/x86/intel/oaktrail/Makefile >  rename drivers/platform/x86/{ => intel/oaktrail}/intel_oaktrail.c > (100%) >  create mode 100644 drivers/platform/x86/intel/pmc_core/Kconfig >  create mode 100644 drivers/platform/x86/intel/pmc_core/Makefile >  rename drivers/platform/x86/{ => intel/pmc_core}/intel_pmc_core.c > (100%) >  rename drivers/platform/x86/{ => intel/pmc_core}/intel_pmc_core.h > (100%) >  rename drivers/platform/x86/{ => > intel/pmc_core}/intel_pmc_core_pltdrv.c (100%) >  create mode 100644 drivers/platform/x86/intel/pmt/Kconfig >  create mode 100644 drivers/platform/x86/intel/pmt/Makefile >  rename drivers/platform/x86/{ => intel/pmt}/intel_pmt_class.c (100%) >  rename drivers/platform/x86/{ => intel/pmt}/intel_pmt_class.h (100%) >  rename drivers/platform/x86/{ => intel/pmt}/intel_pmt_crashlog.c > (100%) >  rename drivers/platform/x86/{ => intel/pmt}/intel_pmt_telemetry.c > (100%) >  create mode 100644 drivers/platform/x86/intel/punit/Kconfig >  create mode 100644 drivers/platform/x86/intel/punit/Makefile >  rename drivers/platform/x86/{ => intel/punit}/intel_punit_ipc.c > (100%) >  create mode 100644 drivers/platform/x86/intel/rst/Kconfig >  create mode 100644 drivers/platform/x86/intel/rst/Makefile >  rename drivers/platform/x86/{ => intel/rst}/intel-rst.c (100%) >  create mode 100644 drivers/platform/x86/intel/scu/Kconfig >  create mode 100644 drivers/platform/x86/intel/scu/Makefile >  rename drivers/platform/x86/{ => intel/scu}/intel_scu_ipc.c (100%) >  rename drivers/platform/x86/{ => intel/scu}/intel_scu_ipcutil.c > (100%) >  rename drivers/platform/x86/{ => intel/scu}/intel_scu_pcidrv.c > (100%) >  rename drivers/platform/x86/{ => intel/scu}/intel_scu_pltdrv.c > (100%) >  rename drivers/platform/x86/{ => intel/scu}/intel_scu_wdt.c (100%) >  create mode 100644 drivers/platform/x86/intel/smartconnect/Kconfig >  create mode 100644 drivers/platform/x86/intel/smartconnect/Makefile >  rename drivers/platform/x86/{ => intel/smartconnect}/intel- > smartconnect.c (100%) >  create mode 100644 drivers/platform/x86/intel/telemetry/Kconfig >  create mode 100644 drivers/platform/x86/intel/telemetry/Makefile >  rename drivers/platform/x86/{ => > intel/telemetry}/intel_telemetry_core.c (100%) >  rename drivers/platform/x86/{ => > intel/telemetry}/intel_telemetry_debugfs.c (100%) >  rename drivers/platform/x86/{ => > intel/telemetry}/intel_telemetry_pltdrv.c (100%) >  create mode 100644 drivers/platform/x86/intel/turbo_max_3/Kconfig >  create mode 100644 drivers/platform/x86/intel/turbo_max_3/Makefile >  rename drivers/platform/x86/{ => > intel/turbo_max_3}/intel_turbo_max_3.c (100%) >  create mode 100644 drivers/platform/x86/intel/uncore_freq/Kconfig >  create mode 100644 drivers/platform/x86/intel/uncore_freq/Makefile >  rename drivers/platform/x86/{ => intel/uncore_freq}/intel-uncore- > frequency.c (100%) >  create mode 100644 drivers/platform/x86/intel/vbtn/Kconfig >  create mode 100644 drivers/platform/x86/intel/vbtn/Makefile >  rename drivers/platform/x86/{ => intel/vbtn}/intel-vbtn.c (100%) >  create mode 100644 drivers/platform/x86/intel/wmi/Kconfig >  create mode 100644 drivers/platform/x86/intel/wmi/Makefile >  rename drivers/platform/x86/{ => intel/wmi}/intel-wmi-sbl-fw- > update.c (100%) >  rename drivers/platform/x86/{ => intel/wmi}/intel-wmi-thunderbolt.c > (100%) >