From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ot1-f52.google.com (mail-ot1-f52.google.com [209.85.210.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 12D026BFCE for ; Fri, 20 Jun 2025 18:46:52 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.52 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1750445214; cv=none; b=FL9vplw+58y4XPb8U6OMAJrttC+eQpmFrWMQAwD7aWOkh9GV9iz211mmcvYyFbIEoSYp28wva54ZjZi7w6UapoFd9GzrNvwOJqTNVYfVVPythoJhH7Ppbkf/u27VWxqsleB+r86T/NqbtTE6qRafZwF8L8nXsCA94tgCYMJ9pkY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1750445214; c=relaxed/simple; bh=d/GlD2KP1uZD+1yoFhPJcFL2TMCqvOPZQ9p8ZBOmw+k=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=HWQ8moOI+HmyuoKB+d7ThEaxUzJ4bwKul5uFq+7p61xtxh71v/50tK+RGcxvMNQUuUzQ1JgaBHkf+Jqc5tAdyLMXadQLVRc2UeprWbvsBZ8a2ACY1ROFc3yl/+uvgqWSMm4pfqDaJxQ70xp4gBxZISMt0UFOPF4TmY9nzAd1dpU= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=minyard.net; spf=none smtp.mailfrom=minyard.net; dkim=pass (2048-bit key) header.d=minyard-net.20230601.gappssmtp.com header.i=@minyard-net.20230601.gappssmtp.com header.b=3OcFBS5V; arc=none smtp.client-ip=209.85.210.52 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=minyard.net Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=minyard.net Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=minyard-net.20230601.gappssmtp.com header.i=@minyard-net.20230601.gappssmtp.com header.b="3OcFBS5V" Received: by mail-ot1-f52.google.com with SMTP id 46e09a7af769-735a9e65471so1421938a34.1 for ; Fri, 20 Jun 2025 11:46:52 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=minyard-net.20230601.gappssmtp.com; s=20230601; t=1750445212; x=1751050012; darn=lists.linux.dev; h=in-reply-to:content-disposition:mime-version:references:reply-to :message-id:subject:cc:to:from:date:from:to:cc:subject:date :message-id:reply-to; bh=s8NHYIJCrgO6+UfKO/xAGNidF0eDJbqLbRizmGJ3qXI=; b=3OcFBS5VFx5r0pgfATVemG1ghrOMYqNnbfdUBdBbQJzwjZbcx5gSEJnd7IGOktYts+ fRF+7ZhBL+vpXnMDbaYz/wGT83ChdKXrpRslPFp9xvYQK4TUp97aVk33xcElQ4Ixx27G lVYaOHL8Pge5Q8X48gUNR+KHT+5+BXvO9xTBaP/7oQJ2WUJAMEKJqY7SDz4NwalqGxrq 4iYb8T0mBAtiz62FyZ4CNFBqZ2UrWY12JQThvEZRg2Fu+GDKdsQ+FKKCqKi4FeWK2DhG HNpnQt8fVFzRIhDALr9COPkwK1gx3YzJPMA965UQpZu1/gf9Nx5a6XusSenpFsQF9yAe alPw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1750445212; x=1751050012; h=in-reply-to:content-disposition:mime-version:references:reply-to :message-id:subject:cc:to:from:date:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=s8NHYIJCrgO6+UfKO/xAGNidF0eDJbqLbRizmGJ3qXI=; b=kfRRX4rlogZH+2hOvkbHD62gFqTk3jwy6xpnIM39lAEbcNyB8sfpNdXAn2ceVwPK0Z qfy9nzDN6H32+p7NYJh4KCHXlEoBDeBDQGsAj8fv9sD90VZ9i0yb5z9T51HO9ewEraaP AB/0/WRcAFq/Vk6KgRuk+60Ev3lTHYOWh4ACg3PvibaY4L3ct9DDiVdjBypXJhtsRfOd SHAv/T0+J/CmDNAl51JoVz1tr1YsG0M3HhAzMqoz7++ZI+PFBIJ10I+lQ4VtvkKFNKRa AGeXh5m9rulvyWX6tFa1CB+JdU+T/RTKsjvSL7vINxFU5kKe0zYhnoI6dTGupSwthtDM IAyg== X-Forwarded-Encrypted: i=1; AJvYcCW5RoemIZXFzd+s5byPms68TOL9qdd6x+g15I0TJK8ijDgUZz2WVg6jb2gmZnQAFwKnuQ4IVgrXvwc=@lists.linux.dev X-Gm-Message-State: AOJu0YxDu2pmx45B0zW8bNwvLqQu1nM7yYEJctAwgLYYezqA7bCPvGBv mtdd/3fj6LG148COWQgn8cl5FHnTXipX9KwZ8wS09MrRYmccYdwepAllQvQvYPkUYe0= X-Gm-Gg: ASbGncsj8/eEFJxY76wb/QcLwjxiRlTWQcPRGB0DvFGyhJdUzXDugM+dI2cOGnr1qsf InlV3SWPytHeoRGc03a/ZBNf+Vj06fR/bM4a4NN629wcPnQTwP0V/7LN+2Nx03b++OzcR0UA0h9 +22Qrju7raBOKqTHcWgcuJ7s0fAwybzYUHlq6AvgySkZ6B/0r1P42xrEIdh9jtlT9hYzjSURjZn jXa0RBHj8TCX3vdrD5TADbqMHVLMkb9RGPBqUfsqu2KhuuZrIgZ0UbyNJ96nUR+MQTGXhVf9zPS 5K41JbIQhJD1AM8GI5g2twgKw2foQUVoE8gL03AWfMOXCcOpT5AMxjwc2OzuF9bvb94IaSiBYto = X-Google-Smtp-Source: AGHT+IGvXifjETCnN4g1olyDwWNAcDk+oFv5m2B4XET98iABiZega1zQLzY80woia5ktb/6ap2w1Cg== X-Received: by 2002:a05:6870:d288:b0:2ea:73fb:7850 with SMTP id 586e51a60fabf-2eeee4cb6b9mr2820332fac.24.1750445211862; Fri, 20 Jun 2025 11:46:51 -0700 (PDT) Received: from mail.minyard.net ([2001:470:b8f6:1b:680:2e07:b628:1be4]) by smtp.gmail.com with ESMTPSA id 586e51a60fabf-2ee6655a2c7sm491389fac.17.2025.06.20.11.46.49 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 20 Jun 2025 11:46:51 -0700 (PDT) Date: Fri, 20 Jun 2025 13:46:46 -0500 From: Corey Minyard To: Binbin Zhou Cc: Binbin Zhou , Huacai Chen , Lee Jones , Corey Minyard , Huacai Chen , Xuerui Wang , loongarch@lists.linux.dev, linux-kernel@vger.kernel.org, openipmi-developer@lists.sourceforge.net, jeffbai@aosc.io, kexybiscuit@aosc.io, wangyao@lemote.com, Chong Qiao Subject: Re: [PATCH v5 1/3] mfd: ls2kbmc: Introduce Loongson-2K BMC core driver Message-ID: Reply-To: corey@minyard.net References: <82cbc8558f15981e0953ab229d2afcc5501f982c.1750301674.git.zhoubinbin@loongson.cn> Precedence: bulk X-Mailing-List: loongarch@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <82cbc8558f15981e0953ab229d2afcc5501f982c.1750301674.git.zhoubinbin@loongson.cn> On Fri, Jun 20, 2025 at 10:06:27AM +0800, Binbin Zhou wrote: > The Loongson-2K Board Management Controller provides an PCIe interface > to the host to access the feature implemented in the BMC. > > The BMC is assembled on a server similar to the server machine with > Loongson-3 CPU. It supports multiple sub-devices like DRM and IPMI. > > Reviewed-by: Huacai Chen > Co-developed-by: Chong Qiao > Signed-off-by: Chong Qiao > Signed-off-by: Binbin Zhou > --- > drivers/mfd/Kconfig | 11 +++ > drivers/mfd/Makefile | 2 + > drivers/mfd/ls2k-bmc-core.c | 156 ++++++++++++++++++++++++++++++++++++ > 3 files changed, 169 insertions(+) > create mode 100644 drivers/mfd/ls2k-bmc-core.c > > diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig > index 96992af22565..a1081c4211b0 100644 > --- a/drivers/mfd/Kconfig > +++ b/drivers/mfd/Kconfig > @@ -2389,6 +2389,18 @@ config MFD_INTEL_M10_BMC_PMCI > additional drivers must be enabled in order to use the functionality > of the device. > > +config MFD_LS2K_BMC_CORE > + bool "Loongson-2K Board Management Controller Support" > + select MFD_CORE Also, from the recent bug report, this is missing a depends on PCI. > + help > + Say yes here to add support for the Loongson-2K BMC which is a Board > + Management Controller connected to the PCIe bus. The device supports > + multiple sub-devices like display and IPMI. This driver provides common > + support for accessing the devices. > + > + The display is enabled by default in the driver, while the IPMI interface > + is enabled independently through the IPMI_LS2K option in the IPMI section. > + > config MFD_QNAP_MCU > tristate "QNAP microcontroller unit core driver" > depends on SERIAL_DEV_BUS > diff --git a/drivers/mfd/Makefile b/drivers/mfd/Makefile > index 5e5cc279af60..6bad54edca34 100644 > --- a/drivers/mfd/Makefile > +++ b/drivers/mfd/Makefile > @@ -282,6 +282,8 @@ obj-$(CONFIG_MFD_INTEL_M10_BMC_CORE) += intel-m10-bmc-core.o > obj-$(CONFIG_MFD_INTEL_M10_BMC_SPI) += intel-m10-bmc-spi.o > obj-$(CONFIG_MFD_INTEL_M10_BMC_PMCI) += intel-m10-bmc-pmci.o > > +obj-$(CONFIG_MFD_LS2K_BMC_CORE) += ls2k-bmc-core.o > + > obj-$(CONFIG_MFD_ATC260X) += atc260x-core.o > obj-$(CONFIG_MFD_ATC260X_I2C) += atc260x-i2c.o > > diff --git a/drivers/mfd/ls2k-bmc-core.c b/drivers/mfd/ls2k-bmc-core.c > new file mode 100644 > index 000000000000..9ee1edf286e7 > --- /dev/null > +++ b/drivers/mfd/ls2k-bmc-core.c > @@ -0,0 +1,156 @@ > +// SPDX-License-Identifier: GPL-2.0-only > +/* > + * Loongson-2K Board Management Controller (BMC) Core Driver. > + * > + * Copyright (C) 2024-2025 Loongson Technology Corporation Limited. > + * > + * Authors: > + * Chong Qiao > + * Binbin Zhou > + */ > + > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > + > +/* LS2K BMC resources */ > +#define LS2K_DISPLAY_RES_START (SZ_16M + SZ_2M) > +#define LS2K_IPMI_RES_SIZE 0x1C > +#define LS2K_IPMI0_RES_START (SZ_16M + 0xF00000) > +#define LS2K_IPMI1_RES_START (LS2K_IPMI0_RES_START + LS2K_IPMI_RES_SIZE) > +#define LS2K_IPMI2_RES_START (LS2K_IPMI1_RES_START + LS2K_IPMI_RES_SIZE) > +#define LS2K_IPMI3_RES_START (LS2K_IPMI2_RES_START + LS2K_IPMI_RES_SIZE) > +#define LS2K_IPMI4_RES_START (LS2K_IPMI3_RES_START + LS2K_IPMI_RES_SIZE) > + > +static struct resource ls2k_display_resources[] = { > + DEFINE_RES_MEM_NAMED(LS2K_DISPLAY_RES_START, SZ_4M, "simpledrm-res"), > +}; > + > +static struct resource ls2k_ipmi0_resources[] = { > + DEFINE_RES_MEM_NAMED(LS2K_IPMI0_RES_START, LS2K_IPMI_RES_SIZE, "ipmi0-res"), > +}; > + > +static struct resource ls2k_ipmi1_resources[] = { > + DEFINE_RES_MEM_NAMED(LS2K_IPMI1_RES_START, LS2K_IPMI_RES_SIZE, "ipmi1-res"), > +}; > + > +static struct resource ls2k_ipmi2_resources[] = { > + DEFINE_RES_MEM_NAMED(LS2K_IPMI2_RES_START, LS2K_IPMI_RES_SIZE, "ipmi2-res"), > +}; > + > +static struct resource ls2k_ipmi3_resources[] = { > + DEFINE_RES_MEM_NAMED(LS2K_IPMI3_RES_START, LS2K_IPMI_RES_SIZE, "ipmi3-res"), > +}; > + > +static struct resource ls2k_ipmi4_resources[] = { > + DEFINE_RES_MEM_NAMED(LS2K_IPMI4_RES_START, LS2K_IPMI_RES_SIZE, "ipmi4-res"), > +}; > + > +static struct mfd_cell ls2k_bmc_cells[] = { > + MFD_CELL_RES("simple-framebuffer", ls2k_display_resources), > + MFD_CELL_RES("ls2k-ipmi-si", ls2k_ipmi0_resources), > + MFD_CELL_RES("ls2k-ipmi-si", ls2k_ipmi1_resources), > + MFD_CELL_RES("ls2k-ipmi-si", ls2k_ipmi2_resources), > + MFD_CELL_RES("ls2k-ipmi-si", ls2k_ipmi3_resources), > + MFD_CELL_RES("ls2k-ipmi-si", ls2k_ipmi4_resources), > +}; > + > +/* > + * Currently the Loongson-2K BMC hardware does not have an I2C interface to adapt to the > + * resolution. We set the resolution by presetting "video=1280x1024-16@2M" to the BMC memory. > + */ > +static int ls2k_bmc_parse_mode(struct pci_dev *pdev, struct simplefb_platform_data *pd) > +{ > + char *mode; > + int depth, ret; > + > + /* The last 16M of PCI BAR0 is used to store the resolution string. */ > + mode = devm_ioremap(&pdev->dev, pci_resource_start(pdev, 0) + SZ_16M, SZ_16M); > + if (!mode) > + return -ENOMEM; > + > + /* The resolution field starts with the flag "video=". */ > + if (!strncmp(mode, "video=", 6)) > + mode = mode + 6; > + > + ret = kstrtoint(strsep(&mode, "x"), 10, &pd->width); > + if (ret) > + return ret; > + > + ret = kstrtoint(strsep(&mode, "-"), 10, &pd->height); > + if (ret) > + return ret; > + > + ret = kstrtoint(strsep(&mode, "@"), 10, &depth); > + if (ret) > + return ret; > + > + pd->stride = pd->width * depth / 8; > + pd->format = depth == 32 ? "a8r8g8b8" : "r5g6b5"; > + > + return 0; > +} > + > +static int ls2k_bmc_probe(struct pci_dev *dev, const struct pci_device_id *id) > +{ > + struct simplefb_platform_data pd; > + resource_size_t base; > + int ret; > + > + ret = pci_enable_device(dev); > + if (ret) > + return ret; > + > + ret = ls2k_bmc_parse_mode(dev, &pd); > + if (ret) > + goto disable_pci; > + > + ls2k_bmc_cells[0].platform_data = &pd; > + ls2k_bmc_cells[0].pdata_size = sizeof(pd); > + base = dev->resource[0].start + LS2K_DISPLAY_RES_START; > + > + /* Remove conflicting efifb device */ > + ret = aperture_remove_conflicting_devices(base, SZ_4M, "simple-framebuffer"); > + if (ret) { > + dev_err(&dev->dev, "Failed to removed firmware framebuffers: %d\n", ret); > + goto disable_pci; > + } > + > + return devm_mfd_add_devices(&dev->dev, PLATFORM_DEVID_AUTO, > + ls2k_bmc_cells, ARRAY_SIZE(ls2k_bmc_cells), > + &dev->resource[0], 0, NULL); > + > +disable_pci: > + pci_disable_device(dev); > + return ret; > +} > + > +static void ls2k_bmc_remove(struct pci_dev *dev) > +{ > + pci_disable_device(dev); > +} > + > +static struct pci_device_id ls2k_bmc_devices[] = { > + { PCI_DEVICE(PCI_VENDOR_ID_LOONGSON, 0x1a05) }, > + { } > +}; > +MODULE_DEVICE_TABLE(pci, ls2k_bmc_devices); > + > +static struct pci_driver ls2k_bmc_driver = { > + .name = "ls2k-bmc", > + .id_table = ls2k_bmc_devices, > + .probe = ls2k_bmc_probe, > + .remove = ls2k_bmc_remove, > +}; > +module_pci_driver(ls2k_bmc_driver); > + > +MODULE_DESCRIPTION("Loongson-2K BMC Core driver"); > +MODULE_AUTHOR("Loongson Technology Corporation Limited"); > +MODULE_LICENSE("GPL"); > -- > 2.47.1 >