From: Akira Tsukamoto <akira.tsukamoto@gmail.com> To: Drew Fustini <drew@beagleboard.org> Cc: "Paul Walmsley" <paul.walmsley@sifive.com>, "Palmer Dabbelt" <palmer@dabbelt.com>, "Albert Ou" <aou@eecs.berkeley.edu>, "Atish Patra" <atish.patra@wdc.com>, "Anup Patel" <anup.patel@wdc.com>, "Emil Renner Berthing" <kernel@esmil.dk>, "Björn Töpel" <bjorn@kernel.org>, "Sagar Shrikant Kadam" <sagar.kadam@sifive.com>, linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/1] config: Enable jh7100 SoC Date: Thu, 24 Jun 2021 23:49:35 +0900 [thread overview] Message-ID: <d84539b7-e46d-f767-246b-85c20899ab94@gmail.com> (raw) In-Reply-To: <20210623190801.GB493015@x1> On 6/24/2021 4:08 AM, Drew Fustini wrote: > On Wed, Jun 23, 2021 at 09:46:54PM +0900, Akira Tsukamoto wrote: >> >> Signed-off-by: Akira Tsukamoto <akira.tsukamoto@gmail.com> >> --- >> arch/riscv/configs/defconfig | 105 +++++++++++++++++++++++++++++++++++ >> 1 file changed, 105 insertions(+) >> >> diff --git a/arch/riscv/configs/defconfig b/arch/riscv/configs/defconfig > > Thanks for submitting this so we can review and discuss. > >> index 1f2be234b11c..e07d26d2743c 100644 >> --- a/arch/riscv/configs/defconfig >> +++ b/arch/riscv/configs/defconfig >> @@ -138,3 +138,108 @@ CONFIG_DEBUG_BLOCK_EXT_DEVT=y >> CONFIG_MEMTEST=y >> # CONFIG_SYSFS_SYSCALL is not set >> CONFIG_EFI=y >> +CONFIG_FB_STARFIVE=y >> +CONFIG_FB_STARFIVE_HDMI_ADV7513=y >> +CONFIG_FB_STARFIVE_HDMI_TDA998X=y >> +CONFIG_FB_STARFIVE_SEEED5INCH=y >> +CONFIG_FB_STARFIVE_VIDEO=y > > These should be dropped as they are vendor drivers that will never be > upstream as fbdev is deprecated [1]. StarFive is working on DRM driver > but that is a couple months away from being ready. I will remove fbdev. And enable DRM driver only when them become ready. > >> +CONFIG_HW_RANDOM_STARFIVE_VIC=y >> +CONFIG_SOC_STARFIVE_VIC7100=y > > Before this gets merged upstream, we need to switch over to using JH7100 > instead of VIC. > > For those not familiar, VIC was the StarFive internal project name but > the product is named JH7100 [2]. I will wait to enable the configs above until the patches rename them and become upstream ready. > >> +CONFIG_FPGA_GMAC_FLUSH_DDR=y >> +CONFIG_MMC_DW_FLUSH_DDR=y >> +CONFIG_USB_CDNS3_HOST_FLUSH_DMA=y >> +CONFIG_SOC_STARFIVE_VIC7100_I2C_GPIO=y > > This config is being used in as a vendor hack in i2c_dw_configure_gpio() > drivers/i2c/busses/i2c-designware-master.c [3]. > > It's possible we may be able to eliminate this once I have completed > gpio and pinctrl drivers. Either way, this option will never be used in > an upstream kernel. Sure, I will remove them and try it since I think your drivers were in the Esmil branch. > >> +CONFIG_VIDEO_STARFIVE_VIN=y >> +CONFIG_VIDEO_STARFIVE_VIN_SENSOR_IMX219=y >> +CONFIG_VIDEO_STARFIVE_VIN_SENSOR_OV5640=y >> +CONFIG_VIDEO_STARFIVE_VIN_SENSOR_SC2235=y > > These should be dropped as they are vendor drivers that do not use V4L2 > and are not upstreamable [4]. StarFive is working on proper V4L2 drivers > but that is a couple months away from being ready. Thanks, I will also wait for the v4l2 being ready. Akira > >> +CONFIG_RCU_CPU_STALL_TIMEOUT=60 >> +CONFIG_LOG_CPU_MAX_BUF_SHIFT=15 >> +CONFIG_PRINTK_SAFE_LOG_BUF_SHIFT=12 >> +CONFIG_PM=y >> +CONFIG_PM_CLK=y >> +CONFIG_EFI_BOOTLOADER_CONTROL=y >> +CONFIG_BLK_PM=y >> +CONFIG_IOSCHED_BFQ=y >> +CONFIG_KSM=y >> +CONFIG_CMA=y >> +CONFIG_CMA_AREAS=7 >> +CONFIG_REGMAP_I2C=y >> +CONFIG_REGMAP_IRQ=y >> +CONFIG_MTD_OF_PARTS=y >> +CONFIG_MTD_BLKDEVS=y >> +CONFIG_MTD_BLOCK=y >> +CONFIG_MTD_PARTITIONED_MASTER=y >> +CONFIG_MTD_MAP_BANK_WIDTH_1=y >> +CONFIG_MTD_MAP_BANK_WIDTH_2=y >> +CONFIG_MTD_MAP_BANK_WIDTH_4=y >> +CONFIG_MTD_CFI_I1=y >> +CONFIG_MTD_CFI_I2=y >> +CONFIG_MTD_SPI_NOR=y >> +CONFIG_MTD_SPI_NOR_SWP_DISABLE_ON_VOLATILE=y >> +CONFIG_BLK_DEV_NBD=y >> +CONFIG_INPUT_LEDS=y >> +CONFIG_INPUT_EVDEV=y >> +CONFIG_SERIAL_8250_DMA=y >> +CONFIG_SERIAL_8250_DWLIB=y >> +CONFIG_SERIAL_8250_DW=y >> +CONFIG_HW_RANDOM_VIRTIO=y >> +CONFIG_I2C_CHARDEV=y >> +CONFIG_I2C_MUX=y >> +CONFIG_I2C_DESIGNWARE_CORE=y >> +CONFIG_I2C_DESIGNWARE_PLATFORM=y >> +CONFIG_SPI_MEM=y >> +CONFIG_SPI_CADENCE_QUADSPI=y >> +CONFIG_SPI_DESIGNWARE=y >> +CONFIG_SPI_DW_DMA=y >> +CONFIG_SPI_DW_MMIO=y >> +CONFIG_SPI_SPIDEV=y >> +CONFIG_GPIO_SYSFS=y >> +CONFIG_GPIO_STARFIVE_VIC=y > > I am working on upstreamable version of the GPIO driver and will submit > RFC soon. This will be called CONFIG_GPIO_STARFIVE_JH7100. > >> +CONFIG_GPIO_TPS65086=y >> +CONFIG_POWER_RESET_TPS65086=y >> +CONFIG_MFD_TPS65086=y >> +CONFIG_USB_HID=y >> +CONFIG_USB_CDNS_SUPPORT=y >> +CONFIG_USB_CDNS_HOST=y >> +CONFIG_USB_CDNS3=y >> +CONFIG_USB_CDNS3_HOST=y >> +CONFIG_USB_ROLE_SWITCH=y >> +CONFIG_SDIO_UART=y >> +CONFIG_MMC_DW=y >> +CONFIG_MMC_DW_PLTFM=y >> +CONFIG_NEW_LEDS=y >> +CONFIG_LEDS_CLASS=y >> +CONFIG_LEDS_GPIO=y >> +CONFIG_LEDS_TRIGGERS=y >> +CONFIG_LEDS_TRIGGER_HEARTBEAT=y >> +CONFIG_RTC_DRV_EFI=y >> +CONFIG_DMADEVICES=y >> +CONFIG_DMA_ENGINE=y >> +CONFIG_DMA_VIRTUAL_CHANNELS=y >> +CONFIG_DMA_OF=y >> +CONFIG_DW_AXI_DMAC=y >> +CONFIG_DW_AXI_DMAC_STARFIVE=y >> +CONFIG_COMMON_CLK_SI544=y >> +CONFIG_COMMON_CLK_PWM=y >> +CONFIG_SIFIVE_L2=y >> +CONFIG_SIFIVE_L2_FLUSH=y >> +CONFIG_SIFIVE_L2_FLUSH_START=0x80000000 >> +CONFIG_SIFIVE_L2_FLUSH_SIZE=0x800000000 >> +CONFIG_SIFIVE_L2_IRQ_DISABLE=y >> +CONFIG_PWM=y >> +CONFIG_PWM_SYSFS=y >> +CONFIG_PWM_SIFIVE_PTC=y >> +CONFIG_RESET_CONTROLLER=y >> +CONFIG_PROC_KCORE=y >> +CONFIG_EFIVAR_FS=y >> +CONFIG_ZLIB_DEFLATE=y >> +CONFIG_DMA_CMA=y >> +CONFIG_CMA_SIZE_MBYTES=640 >> +CONFIG_CMA_SIZE_SEL_MBYTES=y >> +CONFIG_CMA_ALIGNMENT=8 >> +CONFIG_NET_VENDOR_STMICRO=y >> +CONFIG_STMMAC_ETH=y >> +CONFIG_STMMAC_PLATFORM=y >> +CONFIG_DWMAC_GENERIC=y >> +CONFIG_MICREL_PHY=y >> -- >> 2.17.1 >> >> > > Thanks, > Drew > > [1] https://github.com/starfive-tech/linux/tree/beaglev/drivers/video/fbdev/starfive > [2] https://github.com/starfive-tech/beaglev_doc/ > [3] https://github.com/starfive-tech/linux/blob/beaglev/drivers/i2c/busses/i2c-designware-master.c#L170 > [4] https://github.com/starfive-tech/linux/tree/beaglev/drivers/media/platform/starfive >
WARNING: multiple messages have this Message-ID (diff)
From: Akira Tsukamoto <akira.tsukamoto@gmail.com> To: Drew Fustini <drew@beagleboard.org> Cc: "Paul Walmsley" <paul.walmsley@sifive.com>, "Palmer Dabbelt" <palmer@dabbelt.com>, "Albert Ou" <aou@eecs.berkeley.edu>, "Atish Patra" <atish.patra@wdc.com>, "Anup Patel" <anup.patel@wdc.com>, "Emil Renner Berthing" <kernel@esmil.dk>, "Björn Töpel" <bjorn@kernel.org>, "Sagar Shrikant Kadam" <sagar.kadam@sifive.com>, linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/1] config: Enable jh7100 SoC Date: Thu, 24 Jun 2021 23:49:35 +0900 [thread overview] Message-ID: <d84539b7-e46d-f767-246b-85c20899ab94@gmail.com> (raw) In-Reply-To: <20210623190801.GB493015@x1> On 6/24/2021 4:08 AM, Drew Fustini wrote: > On Wed, Jun 23, 2021 at 09:46:54PM +0900, Akira Tsukamoto wrote: >> >> Signed-off-by: Akira Tsukamoto <akira.tsukamoto@gmail.com> >> --- >> arch/riscv/configs/defconfig | 105 +++++++++++++++++++++++++++++++++++ >> 1 file changed, 105 insertions(+) >> >> diff --git a/arch/riscv/configs/defconfig b/arch/riscv/configs/defconfig > > Thanks for submitting this so we can review and discuss. > >> index 1f2be234b11c..e07d26d2743c 100644 >> --- a/arch/riscv/configs/defconfig >> +++ b/arch/riscv/configs/defconfig >> @@ -138,3 +138,108 @@ CONFIG_DEBUG_BLOCK_EXT_DEVT=y >> CONFIG_MEMTEST=y >> # CONFIG_SYSFS_SYSCALL is not set >> CONFIG_EFI=y >> +CONFIG_FB_STARFIVE=y >> +CONFIG_FB_STARFIVE_HDMI_ADV7513=y >> +CONFIG_FB_STARFIVE_HDMI_TDA998X=y >> +CONFIG_FB_STARFIVE_SEEED5INCH=y >> +CONFIG_FB_STARFIVE_VIDEO=y > > These should be dropped as they are vendor drivers that will never be > upstream as fbdev is deprecated [1]. StarFive is working on DRM driver > but that is a couple months away from being ready. I will remove fbdev. And enable DRM driver only when them become ready. > >> +CONFIG_HW_RANDOM_STARFIVE_VIC=y >> +CONFIG_SOC_STARFIVE_VIC7100=y > > Before this gets merged upstream, we need to switch over to using JH7100 > instead of VIC. > > For those not familiar, VIC was the StarFive internal project name but > the product is named JH7100 [2]. I will wait to enable the configs above until the patches rename them and become upstream ready. > >> +CONFIG_FPGA_GMAC_FLUSH_DDR=y >> +CONFIG_MMC_DW_FLUSH_DDR=y >> +CONFIG_USB_CDNS3_HOST_FLUSH_DMA=y >> +CONFIG_SOC_STARFIVE_VIC7100_I2C_GPIO=y > > This config is being used in as a vendor hack in i2c_dw_configure_gpio() > drivers/i2c/busses/i2c-designware-master.c [3]. > > It's possible we may be able to eliminate this once I have completed > gpio and pinctrl drivers. Either way, this option will never be used in > an upstream kernel. Sure, I will remove them and try it since I think your drivers were in the Esmil branch. > >> +CONFIG_VIDEO_STARFIVE_VIN=y >> +CONFIG_VIDEO_STARFIVE_VIN_SENSOR_IMX219=y >> +CONFIG_VIDEO_STARFIVE_VIN_SENSOR_OV5640=y >> +CONFIG_VIDEO_STARFIVE_VIN_SENSOR_SC2235=y > > These should be dropped as they are vendor drivers that do not use V4L2 > and are not upstreamable [4]. StarFive is working on proper V4L2 drivers > but that is a couple months away from being ready. Thanks, I will also wait for the v4l2 being ready. Akira > >> +CONFIG_RCU_CPU_STALL_TIMEOUT=60 >> +CONFIG_LOG_CPU_MAX_BUF_SHIFT=15 >> +CONFIG_PRINTK_SAFE_LOG_BUF_SHIFT=12 >> +CONFIG_PM=y >> +CONFIG_PM_CLK=y >> +CONFIG_EFI_BOOTLOADER_CONTROL=y >> +CONFIG_BLK_PM=y >> +CONFIG_IOSCHED_BFQ=y >> +CONFIG_KSM=y >> +CONFIG_CMA=y >> +CONFIG_CMA_AREAS=7 >> +CONFIG_REGMAP_I2C=y >> +CONFIG_REGMAP_IRQ=y >> +CONFIG_MTD_OF_PARTS=y >> +CONFIG_MTD_BLKDEVS=y >> +CONFIG_MTD_BLOCK=y >> +CONFIG_MTD_PARTITIONED_MASTER=y >> +CONFIG_MTD_MAP_BANK_WIDTH_1=y >> +CONFIG_MTD_MAP_BANK_WIDTH_2=y >> +CONFIG_MTD_MAP_BANK_WIDTH_4=y >> +CONFIG_MTD_CFI_I1=y >> +CONFIG_MTD_CFI_I2=y >> +CONFIG_MTD_SPI_NOR=y >> +CONFIG_MTD_SPI_NOR_SWP_DISABLE_ON_VOLATILE=y >> +CONFIG_BLK_DEV_NBD=y >> +CONFIG_INPUT_LEDS=y >> +CONFIG_INPUT_EVDEV=y >> +CONFIG_SERIAL_8250_DMA=y >> +CONFIG_SERIAL_8250_DWLIB=y >> +CONFIG_SERIAL_8250_DW=y >> +CONFIG_HW_RANDOM_VIRTIO=y >> +CONFIG_I2C_CHARDEV=y >> +CONFIG_I2C_MUX=y >> +CONFIG_I2C_DESIGNWARE_CORE=y >> +CONFIG_I2C_DESIGNWARE_PLATFORM=y >> +CONFIG_SPI_MEM=y >> +CONFIG_SPI_CADENCE_QUADSPI=y >> +CONFIG_SPI_DESIGNWARE=y >> +CONFIG_SPI_DW_DMA=y >> +CONFIG_SPI_DW_MMIO=y >> +CONFIG_SPI_SPIDEV=y >> +CONFIG_GPIO_SYSFS=y >> +CONFIG_GPIO_STARFIVE_VIC=y > > I am working on upstreamable version of the GPIO driver and will submit > RFC soon. This will be called CONFIG_GPIO_STARFIVE_JH7100. > >> +CONFIG_GPIO_TPS65086=y >> +CONFIG_POWER_RESET_TPS65086=y >> +CONFIG_MFD_TPS65086=y >> +CONFIG_USB_HID=y >> +CONFIG_USB_CDNS_SUPPORT=y >> +CONFIG_USB_CDNS_HOST=y >> +CONFIG_USB_CDNS3=y >> +CONFIG_USB_CDNS3_HOST=y >> +CONFIG_USB_ROLE_SWITCH=y >> +CONFIG_SDIO_UART=y >> +CONFIG_MMC_DW=y >> +CONFIG_MMC_DW_PLTFM=y >> +CONFIG_NEW_LEDS=y >> +CONFIG_LEDS_CLASS=y >> +CONFIG_LEDS_GPIO=y >> +CONFIG_LEDS_TRIGGERS=y >> +CONFIG_LEDS_TRIGGER_HEARTBEAT=y >> +CONFIG_RTC_DRV_EFI=y >> +CONFIG_DMADEVICES=y >> +CONFIG_DMA_ENGINE=y >> +CONFIG_DMA_VIRTUAL_CHANNELS=y >> +CONFIG_DMA_OF=y >> +CONFIG_DW_AXI_DMAC=y >> +CONFIG_DW_AXI_DMAC_STARFIVE=y >> +CONFIG_COMMON_CLK_SI544=y >> +CONFIG_COMMON_CLK_PWM=y >> +CONFIG_SIFIVE_L2=y >> +CONFIG_SIFIVE_L2_FLUSH=y >> +CONFIG_SIFIVE_L2_FLUSH_START=0x80000000 >> +CONFIG_SIFIVE_L2_FLUSH_SIZE=0x800000000 >> +CONFIG_SIFIVE_L2_IRQ_DISABLE=y >> +CONFIG_PWM=y >> +CONFIG_PWM_SYSFS=y >> +CONFIG_PWM_SIFIVE_PTC=y >> +CONFIG_RESET_CONTROLLER=y >> +CONFIG_PROC_KCORE=y >> +CONFIG_EFIVAR_FS=y >> +CONFIG_ZLIB_DEFLATE=y >> +CONFIG_DMA_CMA=y >> +CONFIG_CMA_SIZE_MBYTES=640 >> +CONFIG_CMA_SIZE_SEL_MBYTES=y >> +CONFIG_CMA_ALIGNMENT=8 >> +CONFIG_NET_VENDOR_STMICRO=y >> +CONFIG_STMMAC_ETH=y >> +CONFIG_STMMAC_PLATFORM=y >> +CONFIG_DWMAC_GENERIC=y >> +CONFIG_MICREL_PHY=y >> -- >> 2.17.1 >> >> > > Thanks, > Drew > > [1] https://github.com/starfive-tech/linux/tree/beaglev/drivers/video/fbdev/starfive > [2] https://github.com/starfive-tech/beaglev_doc/ > [3] https://github.com/starfive-tech/linux/blob/beaglev/drivers/i2c/busses/i2c-designware-master.c#L170 > [4] https://github.com/starfive-tech/linux/tree/beaglev/drivers/media/platform/starfive > _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv
next prev parent reply other threads:[~2021-06-24 14:49 UTC|newest] Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-06-23 12:45 [RFC PATCH 0/1] Adding jh7100 SoC to defconfig Akira Tsukamoto 2021-06-23 12:45 ` Akira Tsukamoto 2021-06-23 12:46 ` [PATCH 1/1] config: Enable jh7100 SoC Akira Tsukamoto 2021-06-23 12:46 ` Akira Tsukamoto 2021-06-23 13:21 ` Jisheng Zhang 2021-06-23 13:21 ` Jisheng Zhang 2021-06-24 14:35 ` Akira Tsukamoto 2021-06-24 14:35 ` Akira Tsukamoto 2021-06-23 19:08 ` Drew Fustini 2021-06-23 19:08 ` Drew Fustini 2021-06-24 14:49 ` Akira Tsukamoto [this message] 2021-06-24 14:49 ` Akira Tsukamoto 2021-06-23 18:50 ` [RFC PATCH 0/1] Adding jh7100 SoC to defconfig Drew Fustini 2021-06-23 18:50 ` Drew Fustini 2021-06-24 14:42 ` Akira Tsukamoto 2021-06-24 14:42 ` Akira Tsukamoto
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=d84539b7-e46d-f767-246b-85c20899ab94@gmail.com \ --to=akira.tsukamoto@gmail.com \ --cc=anup.patel@wdc.com \ --cc=aou@eecs.berkeley.edu \ --cc=atish.patra@wdc.com \ --cc=bjorn@kernel.org \ --cc=drew@beagleboard.org \ --cc=kernel@esmil.dk \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-riscv@lists.infradead.org \ --cc=palmer@dabbelt.com \ --cc=paul.walmsley@sifive.com \ --cc=sagar.kadam@sifive.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.