Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751591AbdGaUPs (ORCPT ); Mon, 31 Jul 2017 16:15:48 -0400 Received: from mail.kernel.org ([198.145.29.99]:35248 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751054AbdGaUPp (ORCPT ); Mon, 31 Jul 2017 16:15:45 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 34F0322B4B Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=helgaas@kernel.org Date: Mon, 31 Jul 2017 15:15:42 -0500 From: Bjorn Helgaas To: Logan Gunthorpe Cc: linux-ntb@googlegroups.com, linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, Jon Mason , Dave Jiang , Allen Hubbe , Bjorn Helgaas , Greg Kroah-Hartman , Kurt Schwemmer , Stephen Bates , Serge Semin Subject: Re: [PATCH v3 01/16] switchtec: move structure definitions into a common header Message-ID: <20170731201542.GA9303@bhelgaas-glaptop.roam.corp.google.com> References: <20170725205753.4735-1-logang@deltatee.com> <20170725205753.4735-2-logang@deltatee.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170725205753.4735-2-logang@deltatee.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 16744 Lines: 613 To follow the existing changelog subject pattern in drivers/ntb/hw, I would change yours like this: -switchtec: move structure definitions into a common header +NTB: switchtec: Move structure definitions into a common header I.e., add "NTB: " prefix and capitalize the first word of the description. If I were merging this, I would silently change this myself, but obviously this already has my ack and will go through Jon's tree. On Tue, Jul 25, 2017 at 02:57:38PM -0600, Logan Gunthorpe wrote: > Create the switchtec.h header in include/linux with hardware defines > and the switchtec_dev structure. Both moved directly from switchtec.c. > This is a prep patch for creating an NTB driver for Switchtec. > > Signed-off-by: Logan Gunthorpe > Reviewed-by: Stephen Bates > Reviewed-by: Kurt Schwemmer > Acked-by: Greg Kroah-Hartman > Acked-by: Bjorn Helgaas > --- > MAINTAINERS | 1 + > drivers/pci/switch/switchtec.c | 260 +------------------------------------- > include/linux/switchtec.h | 279 +++++++++++++++++++++++++++++++++++++++++ > 3 files changed, 281 insertions(+), 259 deletions(-) > create mode 100644 include/linux/switchtec.h > > diff --git a/MAINTAINERS b/MAINTAINERS > index 205d3977ac46..4ff2ad7c1c7b 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -10146,6 +10146,7 @@ F: Documentation/switchtec.txt > F: Documentation/ABI/testing/sysfs-class-switchtec > F: drivers/pci/switch/switchtec* > F: include/uapi/linux/switchtec_ioctl.h > +F: include/linux/switchtec.h > > PCI DRIVER FOR NVIDIA TEGRA > M: Thierry Reding > diff --git a/drivers/pci/switch/switchtec.c b/drivers/pci/switch/switchtec.c > index af81b2dec42e..5b75d3008ff8 100644 > --- a/drivers/pci/switch/switchtec.c > +++ b/drivers/pci/switch/switchtec.c > @@ -13,6 +13,7 @@ > * > */ > > +#include > #include > > #include > @@ -20,8 +21,6 @@ > #include > #include > #include > -#include > -#include > #include > > MODULE_DESCRIPTION("Microsemi Switchtec(tm) PCIe Management Driver"); > @@ -37,263 +36,6 @@ static dev_t switchtec_devt; > static struct class *switchtec_class; > static DEFINE_IDA(switchtec_minor_ida); > > -#define MICROSEMI_VENDOR_ID 0x11f8 > -#define MICROSEMI_NTB_CLASSCODE 0x068000 > -#define MICROSEMI_MGMT_CLASSCODE 0x058000 > - > -#define SWITCHTEC_MRPC_PAYLOAD_SIZE 1024 > -#define SWITCHTEC_MAX_PFF_CSR 48 > - > -#define SWITCHTEC_EVENT_OCCURRED BIT(0) > -#define SWITCHTEC_EVENT_CLEAR BIT(0) > -#define SWITCHTEC_EVENT_EN_LOG BIT(1) > -#define SWITCHTEC_EVENT_EN_CLI BIT(2) > -#define SWITCHTEC_EVENT_EN_IRQ BIT(3) > -#define SWITCHTEC_EVENT_FATAL BIT(4) > - > -enum { > - SWITCHTEC_GAS_MRPC_OFFSET = 0x0000, > - SWITCHTEC_GAS_TOP_CFG_OFFSET = 0x1000, > - SWITCHTEC_GAS_SW_EVENT_OFFSET = 0x1800, > - SWITCHTEC_GAS_SYS_INFO_OFFSET = 0x2000, > - SWITCHTEC_GAS_FLASH_INFO_OFFSET = 0x2200, > - SWITCHTEC_GAS_PART_CFG_OFFSET = 0x4000, > - SWITCHTEC_GAS_NTB_OFFSET = 0x10000, > - SWITCHTEC_GAS_PFF_CSR_OFFSET = 0x134000, > -}; > - > -struct mrpc_regs { > - u8 input_data[SWITCHTEC_MRPC_PAYLOAD_SIZE]; > - u8 output_data[SWITCHTEC_MRPC_PAYLOAD_SIZE]; > - u32 cmd; > - u32 status; > - u32 ret_value; > -} __packed; > - > -enum mrpc_status { > - SWITCHTEC_MRPC_STATUS_INPROGRESS = 1, > - SWITCHTEC_MRPC_STATUS_DONE = 2, > - SWITCHTEC_MRPC_STATUS_ERROR = 0xFF, > - SWITCHTEC_MRPC_STATUS_INTERRUPTED = 0x100, > -}; > - > -struct sw_event_regs { > - u64 event_report_ctrl; > - u64 reserved1; > - u64 part_event_bitmap; > - u64 reserved2; > - u32 global_summary; > - u32 reserved3[3]; > - u32 stack_error_event_hdr; > - u32 stack_error_event_data; > - u32 reserved4[4]; > - u32 ppu_error_event_hdr; > - u32 ppu_error_event_data; > - u32 reserved5[4]; > - u32 isp_error_event_hdr; > - u32 isp_error_event_data; > - u32 reserved6[4]; > - u32 sys_reset_event_hdr; > - u32 reserved7[5]; > - u32 fw_exception_hdr; > - u32 reserved8[5]; > - u32 fw_nmi_hdr; > - u32 reserved9[5]; > - u32 fw_non_fatal_hdr; > - u32 reserved10[5]; > - u32 fw_fatal_hdr; > - u32 reserved11[5]; > - u32 twi_mrpc_comp_hdr; > - u32 twi_mrpc_comp_data; > - u32 reserved12[4]; > - u32 twi_mrpc_comp_async_hdr; > - u32 twi_mrpc_comp_async_data; > - u32 reserved13[4]; > - u32 cli_mrpc_comp_hdr; > - u32 cli_mrpc_comp_data; > - u32 reserved14[4]; > - u32 cli_mrpc_comp_async_hdr; > - u32 cli_mrpc_comp_async_data; > - u32 reserved15[4]; > - u32 gpio_interrupt_hdr; > - u32 gpio_interrupt_data; > - u32 reserved16[4]; > -} __packed; > - > -enum { > - SWITCHTEC_CFG0_RUNNING = 0x04, > - SWITCHTEC_CFG1_RUNNING = 0x05, > - SWITCHTEC_IMG0_RUNNING = 0x03, > - SWITCHTEC_IMG1_RUNNING = 0x07, > -}; > - > -struct sys_info_regs { > - u32 device_id; > - u32 device_version; > - u32 firmware_version; > - u32 reserved1; > - u32 vendor_table_revision; > - u32 table_format_version; > - u32 partition_id; > - u32 cfg_file_fmt_version; > - u16 cfg_running; > - u16 img_running; > - u32 reserved2[57]; > - char vendor_id[8]; > - char product_id[16]; > - char product_revision[4]; > - char component_vendor[8]; > - u16 component_id; > - u8 component_revision; > -} __packed; > - > -struct flash_info_regs { > - u32 flash_part_map_upd_idx; > - > - struct active_partition_info { > - u32 address; > - u32 build_version; > - u32 build_string; > - } active_img; > - > - struct active_partition_info active_cfg; > - struct active_partition_info inactive_img; > - struct active_partition_info inactive_cfg; > - > - u32 flash_length; > - > - struct partition_info { > - u32 address; > - u32 length; > - } cfg0; > - > - struct partition_info cfg1; > - struct partition_info img0; > - struct partition_info img1; > - struct partition_info nvlog; > - struct partition_info vendor[8]; > -}; > - > -struct ntb_info_regs { > - u8 partition_count; > - u8 partition_id; > - u16 reserved1; > - u64 ep_map; > - u16 requester_id; > -} __packed; > - > -struct part_cfg_regs { > - u32 status; > - u32 state; > - u32 port_cnt; > - u32 usp_port_mode; > - u32 usp_pff_inst_id; > - u32 vep_pff_inst_id; > - u32 dsp_pff_inst_id[47]; > - u32 reserved1[11]; > - u16 vep_vector_number; > - u16 usp_vector_number; > - u32 port_event_bitmap; > - u32 reserved2[3]; > - u32 part_event_summary; > - u32 reserved3[3]; > - u32 part_reset_hdr; > - u32 part_reset_data[5]; > - u32 mrpc_comp_hdr; > - u32 mrpc_comp_data[5]; > - u32 mrpc_comp_async_hdr; > - u32 mrpc_comp_async_data[5]; > - u32 dyn_binding_hdr; > - u32 dyn_binding_data[5]; > - u32 reserved4[159]; > -} __packed; > - > -enum { > - SWITCHTEC_PART_CFG_EVENT_RESET = 1 << 0, > - SWITCHTEC_PART_CFG_EVENT_MRPC_CMP = 1 << 1, > - SWITCHTEC_PART_CFG_EVENT_MRPC_ASYNC_CMP = 1 << 2, > - SWITCHTEC_PART_CFG_EVENT_DYN_PART_CMP = 1 << 3, > -}; > - > -struct pff_csr_regs { > - u16 vendor_id; > - u16 device_id; > - u32 pci_cfg_header[15]; > - u32 pci_cap_region[48]; > - u32 pcie_cap_region[448]; > - u32 indirect_gas_window[128]; > - u32 indirect_gas_window_off; > - u32 reserved[127]; > - u32 pff_event_summary; > - u32 reserved2[3]; > - u32 aer_in_p2p_hdr; > - u32 aer_in_p2p_data[5]; > - u32 aer_in_vep_hdr; > - u32 aer_in_vep_data[5]; > - u32 dpc_hdr; > - u32 dpc_data[5]; > - u32 cts_hdr; > - u32 cts_data[5]; > - u32 reserved3[6]; > - u32 hotplug_hdr; > - u32 hotplug_data[5]; > - u32 ier_hdr; > - u32 ier_data[5]; > - u32 threshold_hdr; > - u32 threshold_data[5]; > - u32 power_mgmt_hdr; > - u32 power_mgmt_data[5]; > - u32 tlp_throttling_hdr; > - u32 tlp_throttling_data[5]; > - u32 force_speed_hdr; > - u32 force_speed_data[5]; > - u32 credit_timeout_hdr; > - u32 credit_timeout_data[5]; > - u32 link_state_hdr; > - u32 link_state_data[5]; > - u32 reserved4[174]; > -} __packed; > - > -struct switchtec_dev { > - struct pci_dev *pdev; > - struct device dev; > - struct cdev cdev; > - > - int partition; > - int partition_count; > - int pff_csr_count; > - char pff_local[SWITCHTEC_MAX_PFF_CSR]; > - > - void __iomem *mmio; > - struct mrpc_regs __iomem *mmio_mrpc; > - struct sw_event_regs __iomem *mmio_sw_event; > - struct sys_info_regs __iomem *mmio_sys_info; > - struct flash_info_regs __iomem *mmio_flash_info; > - struct ntb_info_regs __iomem *mmio_ntb; > - struct part_cfg_regs __iomem *mmio_part_cfg; > - struct part_cfg_regs __iomem *mmio_part_cfg_all; > - struct pff_csr_regs __iomem *mmio_pff_csr; > - > - /* > - * The mrpc mutex must be held when accessing the other > - * mrpc_ fields, alive flag and stuser->state field > - */ > - struct mutex mrpc_mutex; > - struct list_head mrpc_queue; > - int mrpc_busy; > - struct work_struct mrpc_work; > - struct delayed_work mrpc_timeout; > - bool alive; > - > - wait_queue_head_t event_wq; > - atomic_t event_cnt; > -}; > - > -static struct switchtec_dev *to_stdev(struct device *dev) > -{ > - return container_of(dev, struct switchtec_dev, dev); > -} > - > enum mrpc_state { > MRPC_IDLE = 0, > MRPC_QUEUED, > diff --git a/include/linux/switchtec.h b/include/linux/switchtec.h > new file mode 100644 > index 000000000000..1cbd0e63b0ab > --- /dev/null > +++ b/include/linux/switchtec.h > @@ -0,0 +1,279 @@ > +/* > + * Microsemi Switchtec PCIe Driver > + * Copyright (c) 2017, Microsemi Corporation > + * > + * This program is free software; you can redistribute it and/or modify it > + * under the terms and conditions of the GNU General Public License, > + * version 2, as published by the Free Software Foundation. > + * > + * This program is distributed in the hope it will be useful, but WITHOUT > + * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or > + * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for > + * more details. > + * > + */ > + > +#ifndef _SWITCHTEC_H > +#define _SWITCHTEC_H > + > +#include > +#include > + > +#define MICROSEMI_VENDOR_ID 0x11f8 > +#define MICROSEMI_NTB_CLASSCODE 0x068000 > +#define MICROSEMI_MGMT_CLASSCODE 0x058000 > + > +#define SWITCHTEC_MRPC_PAYLOAD_SIZE 1024 > +#define SWITCHTEC_MAX_PFF_CSR 48 > + > +#define SWITCHTEC_EVENT_OCCURRED BIT(0) > +#define SWITCHTEC_EVENT_CLEAR BIT(0) > +#define SWITCHTEC_EVENT_EN_LOG BIT(1) > +#define SWITCHTEC_EVENT_EN_CLI BIT(2) > +#define SWITCHTEC_EVENT_EN_IRQ BIT(3) > +#define SWITCHTEC_EVENT_FATAL BIT(4) > + > +enum { > + SWITCHTEC_GAS_MRPC_OFFSET = 0x0000, > + SWITCHTEC_GAS_TOP_CFG_OFFSET = 0x1000, > + SWITCHTEC_GAS_SW_EVENT_OFFSET = 0x1800, > + SWITCHTEC_GAS_SYS_INFO_OFFSET = 0x2000, > + SWITCHTEC_GAS_FLASH_INFO_OFFSET = 0x2200, > + SWITCHTEC_GAS_PART_CFG_OFFSET = 0x4000, > + SWITCHTEC_GAS_NTB_OFFSET = 0x10000, > + SWITCHTEC_GAS_PFF_CSR_OFFSET = 0x134000, > +}; > + > +struct mrpc_regs { > + u8 input_data[SWITCHTEC_MRPC_PAYLOAD_SIZE]; > + u8 output_data[SWITCHTEC_MRPC_PAYLOAD_SIZE]; > + u32 cmd; > + u32 status; > + u32 ret_value; > +} __packed; > + > +enum mrpc_status { > + SWITCHTEC_MRPC_STATUS_INPROGRESS = 1, > + SWITCHTEC_MRPC_STATUS_DONE = 2, > + SWITCHTEC_MRPC_STATUS_ERROR = 0xFF, > + SWITCHTEC_MRPC_STATUS_INTERRUPTED = 0x100, > +}; > + > +struct sw_event_regs { > + u64 event_report_ctrl; > + u64 reserved1; > + u64 part_event_bitmap; > + u64 reserved2; > + u32 global_summary; > + u32 reserved3[3]; > + u32 stack_error_event_hdr; > + u32 stack_error_event_data; > + u32 reserved4[4]; > + u32 ppu_error_event_hdr; > + u32 ppu_error_event_data; > + u32 reserved5[4]; > + u32 isp_error_event_hdr; > + u32 isp_error_event_data; > + u32 reserved6[4]; > + u32 sys_reset_event_hdr; > + u32 reserved7[5]; > + u32 fw_exception_hdr; > + u32 reserved8[5]; > + u32 fw_nmi_hdr; > + u32 reserved9[5]; > + u32 fw_non_fatal_hdr; > + u32 reserved10[5]; > + u32 fw_fatal_hdr; > + u32 reserved11[5]; > + u32 twi_mrpc_comp_hdr; > + u32 twi_mrpc_comp_data; > + u32 reserved12[4]; > + u32 twi_mrpc_comp_async_hdr; > + u32 twi_mrpc_comp_async_data; > + u32 reserved13[4]; > + u32 cli_mrpc_comp_hdr; > + u32 cli_mrpc_comp_data; > + u32 reserved14[4]; > + u32 cli_mrpc_comp_async_hdr; > + u32 cli_mrpc_comp_async_data; > + u32 reserved15[4]; > + u32 gpio_interrupt_hdr; > + u32 gpio_interrupt_data; > + u32 reserved16[4]; > +} __packed; > + > +enum { > + SWITCHTEC_CFG0_RUNNING = 0x04, > + SWITCHTEC_CFG1_RUNNING = 0x05, > + SWITCHTEC_IMG0_RUNNING = 0x03, > + SWITCHTEC_IMG1_RUNNING = 0x07, > +}; > + > +struct sys_info_regs { > + u32 device_id; > + u32 device_version; > + u32 firmware_version; > + u32 reserved1; > + u32 vendor_table_revision; > + u32 table_format_version; > + u32 partition_id; > + u32 cfg_file_fmt_version; > + u16 cfg_running; > + u16 img_running; > + u32 reserved2[57]; > + char vendor_id[8]; > + char product_id[16]; > + char product_revision[4]; > + char component_vendor[8]; > + u16 component_id; > + u8 component_revision; > +} __packed; > + > +struct flash_info_regs { > + u32 flash_part_map_upd_idx; > + > + struct active_partition_info { > + u32 address; > + u32 build_version; > + u32 build_string; > + } active_img; > + > + struct active_partition_info active_cfg; > + struct active_partition_info inactive_img; > + struct active_partition_info inactive_cfg; > + > + u32 flash_length; > + > + struct partition_info { > + u32 address; > + u32 length; > + } cfg0; > + > + struct partition_info cfg1; > + struct partition_info img0; > + struct partition_info img1; > + struct partition_info nvlog; > + struct partition_info vendor[8]; > +}; > + > +struct ntb_info_regs { > + u8 partition_count; > + u8 partition_id; > + u16 reserved1; > + u64 ep_map; > + u16 requester_id; > +} __packed; > + > +struct part_cfg_regs { > + u32 status; > + u32 state; > + u32 port_cnt; > + u32 usp_port_mode; > + u32 usp_pff_inst_id; > + u32 vep_pff_inst_id; > + u32 dsp_pff_inst_id[47]; > + u32 reserved1[11]; > + u16 vep_vector_number; > + u16 usp_vector_number; > + u32 port_event_bitmap; > + u32 reserved2[3]; > + u32 part_event_summary; > + u32 reserved3[3]; > + u32 part_reset_hdr; > + u32 part_reset_data[5]; > + u32 mrpc_comp_hdr; > + u32 mrpc_comp_data[5]; > + u32 mrpc_comp_async_hdr; > + u32 mrpc_comp_async_data[5]; > + u32 dyn_binding_hdr; > + u32 dyn_binding_data[5]; > + u32 reserved4[159]; > +} __packed; > + > +enum { > + SWITCHTEC_PART_CFG_EVENT_RESET = 1 << 0, > + SWITCHTEC_PART_CFG_EVENT_MRPC_CMP = 1 << 1, > + SWITCHTEC_PART_CFG_EVENT_MRPC_ASYNC_CMP = 1 << 2, > + SWITCHTEC_PART_CFG_EVENT_DYN_PART_CMP = 1 << 3, > +}; > + > +struct pff_csr_regs { > + u16 vendor_id; > + u16 device_id; > + u32 pci_cfg_header[15]; > + u32 pci_cap_region[48]; > + u32 pcie_cap_region[448]; > + u32 indirect_gas_window[128]; > + u32 indirect_gas_window_off; > + u32 reserved[127]; > + u32 pff_event_summary; > + u32 reserved2[3]; > + u32 aer_in_p2p_hdr; > + u32 aer_in_p2p_data[5]; > + u32 aer_in_vep_hdr; > + u32 aer_in_vep_data[5]; > + u32 dpc_hdr; > + u32 dpc_data[5]; > + u32 cts_hdr; > + u32 cts_data[5]; > + u32 reserved3[6]; > + u32 hotplug_hdr; > + u32 hotplug_data[5]; > + u32 ier_hdr; > + u32 ier_data[5]; > + u32 threshold_hdr; > + u32 threshold_data[5]; > + u32 power_mgmt_hdr; > + u32 power_mgmt_data[5]; > + u32 tlp_throttling_hdr; > + u32 tlp_throttling_data[5]; > + u32 force_speed_hdr; > + u32 force_speed_data[5]; > + u32 credit_timeout_hdr; > + u32 credit_timeout_data[5]; > + u32 link_state_hdr; > + u32 link_state_data[5]; > + u32 reserved4[174]; > +} __packed; > + > +struct switchtec_dev { > + struct pci_dev *pdev; > + struct device dev; > + struct cdev cdev; > + > + int partition; > + int partition_count; > + int pff_csr_count; > + char pff_local[SWITCHTEC_MAX_PFF_CSR]; > + > + void __iomem *mmio; > + struct mrpc_regs __iomem *mmio_mrpc; > + struct sw_event_regs __iomem *mmio_sw_event; > + struct sys_info_regs __iomem *mmio_sys_info; > + struct flash_info_regs __iomem *mmio_flash_info; > + struct ntb_info_regs __iomem *mmio_ntb; > + struct part_cfg_regs __iomem *mmio_part_cfg; > + struct part_cfg_regs __iomem *mmio_part_cfg_all; > + struct pff_csr_regs __iomem *mmio_pff_csr; > + > + /* > + * The mrpc mutex must be held when accessing the other > + * mrpc_ fields, alive flag and stuser->state field > + */ > + struct mutex mrpc_mutex; > + struct list_head mrpc_queue; > + int mrpc_busy; > + struct work_struct mrpc_work; > + struct delayed_work mrpc_timeout; > + bool alive; > + > + wait_queue_head_t event_wq; > + atomic_t event_cnt; > +}; > + > +static inline struct switchtec_dev *to_stdev(struct device *dev) > +{ > + return container_of(dev, struct switchtec_dev, dev); > +} > + > +#endif > -- > 2.11.0 >