| Message ID | 20250116-vop2-pt2-v1-10-c9d646b53930@riseup.net |
|---|---|
| State | Superseded |
| Delegated to: | Kever Yang |
| Headers | show |
| Series | Rockchip VOP2 support | expand |
Hi Dang, On 2025-01-16 10:15, Dang Huynh wrote: > Now that we have VOP2 support, let's enable it and support the > built in USB keyboard. > > Signed-off-by: Dang Huynh <danct12@riseup.net> > --- > configs/pinetab2-rk3566_defconfig | 15 +++++++++++++++ > 1 file changed, 15 insertions(+) > > diff --git a/configs/pinetab2-rk3566_defconfig b/configs/pinetab2-rk3566_defconfig > index 45e63b42d192dc172f24e071b09c8203675965d8..b250d8d12e9d5e045e28db4d455850f94514321f 100644 > --- a/configs/pinetab2-rk3566_defconfig > +++ b/configs/pinetab2-rk3566_defconfig > @@ -23,7 +23,10 @@ CONFIG_FIT_VERBOSE=y > CONFIG_SPL_FIT_SIGNATURE=y > CONFIG_SPL_LOAD_FIT=y > CONFIG_LEGACY_IMAGE_FORMAT=y > +CONFIG_USE_PREBOOT=y Please try to avoid adding USE_PREBOOT because it will slow down boot by several seconds thanks to it force running "usb start". For normal use starting usb is typically not needed. We should try and delay usb start until it is really needed. > CONFIG_DEFAULT_FDT_FILE="rockchip/rk3566-pinetab2-v2.0.dtb" > +CONFIG_PRE_CONSOLE_BUFFER=y > +CONFIG_SYS_CONSOLE_ENV_OVERWRITE=y > # CONFIG_DISPLAY_CPUINFO is not set > CONFIG_DISPLAY_BOARDINFO_LATE=y > CONFIG_SPL_MAX_SIZE=0x40000 > @@ -57,6 +60,8 @@ CONFIG_BUTTON=y > CONFIG_BUTTON_ADC=y > CONFIG_BUTTON_GPIO=y > CONFIG_SPL_CLK=y > +CONFIG_CLK_CCF=y > +CONFIG_CLK_COMPOSITE_CCF=y Is CCF and COMPOSITE_CCF needed? > # CONFIG_USB_FUNCTION_FASTBOOT is not set > CONFIG_ROCKCHIP_GPIO=y > CONFIG_SYS_I2C_ROCKCHIP=y > @@ -69,6 +74,7 @@ CONFIG_MMC_SDHCI_SDMA=y > CONFIG_MMC_SDHCI_ROCKCHIP=y > CONFIG_SPI_FLASH_SFDP_SUPPORT=y > CONFIG_SPI_FLASH_SILICONKAISER=y > +CONFIG_PHY_ROCKCHIP_INNO_DSIDPHY=y > CONFIG_PHY_ROCKCHIP_INNO_USB2=y > CONFIG_PHY_ROCKCHIP_NANENG_COMBOPHY=y > CONFIG_SPL_PINCTRL=y > @@ -94,6 +100,7 @@ CONFIG_USB_OHCI_HCD=y > CONFIG_USB_OHCI_GENERIC=y > CONFIG_USB_DWC3=y > CONFIG_USB_DWC3_GENERIC=y > +CONFIG_USB_KEYBOARD=y > CONFIG_USB_HOST_ETHER=y > CONFIG_USB_ETHER_ASIX=y > CONFIG_USB_ETHER_ASIX88179=y > @@ -101,4 +108,12 @@ CONFIG_USB_ETHER_RTL8152=y > CONFIG_USB_GADGET=y > CONFIG_USB_GADGET_DOWNLOAD=y > CONFIG_USB_FUNCTION_ROCKUSB=y > +CONFIG_VIDEO=y > +CONFIG_CONSOLE_ROTATION=y > +CONFIG_VIDEO_LCD_BOE_TH101MB31IG002_28A=y > +CONFIG_DISPLAY=y > +CONFIG_VIDEO_ROCKCHIP=y > +CONFIG_DISPLAY_ROCKCHIP_HDMI=y > +CONFIG_DISPLAY_ROCKCHIP_DW_MIPI=y > +CONFIG_VIDEO_DT_SIMPLEFB=y I do not see a simple-framebuffer compatible, is VIDEO_DT_SIMPLEFB needed? Regards, Jonas > CONFIG_ERRNO_STR=y >
Hi Jonas, On Sunday, January 19, 2025 2:08:37 PM UTC Jonas Karlman wrote: > Hi Dang, > > On 2025-01-16 10:15, Dang Huynh wrote: > > +CONFIG_CLK_CCF=y > > +CONFIG_CLK_COMPOSITE_CCF=y > > Is CCF and COMPOSITE_CCF needed? Yes, it's required to turn on and set video port clock. COMPOSITE_CCF shouldn't be needed. > > > +CONFIG_VIDEO_DT_SIMPLEFB=y > > I do not see a simple-framebuffer compatible, is VIDEO_DT_SIMPLEFB > needed? It shoudln't be needed, whoops. > > Regards, > Jonas >
diff --git a/configs/pinetab2-rk3566_defconfig b/configs/pinetab2-rk3566_defconfig index 45e63b42d192dc172f24e071b09c8203675965d8..b250d8d12e9d5e045e28db4d455850f94514321f 100644 --- a/configs/pinetab2-rk3566_defconfig +++ b/configs/pinetab2-rk3566_defconfig @@ -23,7 +23,10 @@ CONFIG_FIT_VERBOSE=y CONFIG_SPL_FIT_SIGNATURE=y CONFIG_SPL_LOAD_FIT=y CONFIG_LEGACY_IMAGE_FORMAT=y +CONFIG_USE_PREBOOT=y CONFIG_DEFAULT_FDT_FILE="rockchip/rk3566-pinetab2-v2.0.dtb" +CONFIG_PRE_CONSOLE_BUFFER=y +CONFIG_SYS_CONSOLE_ENV_OVERWRITE=y # CONFIG_DISPLAY_CPUINFO is not set CONFIG_DISPLAY_BOARDINFO_LATE=y CONFIG_SPL_MAX_SIZE=0x40000 @@ -57,6 +60,8 @@ CONFIG_BUTTON=y CONFIG_BUTTON_ADC=y CONFIG_BUTTON_GPIO=y CONFIG_SPL_CLK=y +CONFIG_CLK_CCF=y +CONFIG_CLK_COMPOSITE_CCF=y # CONFIG_USB_FUNCTION_FASTBOOT is not set CONFIG_ROCKCHIP_GPIO=y CONFIG_SYS_I2C_ROCKCHIP=y @@ -69,6 +74,7 @@ CONFIG_MMC_SDHCI_SDMA=y CONFIG_MMC_SDHCI_ROCKCHIP=y CONFIG_SPI_FLASH_SFDP_SUPPORT=y CONFIG_SPI_FLASH_SILICONKAISER=y +CONFIG_PHY_ROCKCHIP_INNO_DSIDPHY=y CONFIG_PHY_ROCKCHIP_INNO_USB2=y CONFIG_PHY_ROCKCHIP_NANENG_COMBOPHY=y CONFIG_SPL_PINCTRL=y @@ -94,6 +100,7 @@ CONFIG_USB_OHCI_HCD=y CONFIG_USB_OHCI_GENERIC=y CONFIG_USB_DWC3=y CONFIG_USB_DWC3_GENERIC=y +CONFIG_USB_KEYBOARD=y CONFIG_USB_HOST_ETHER=y CONFIG_USB_ETHER_ASIX=y CONFIG_USB_ETHER_ASIX88179=y @@ -101,4 +108,12 @@ CONFIG_USB_ETHER_RTL8152=y CONFIG_USB_GADGET=y CONFIG_USB_GADGET_DOWNLOAD=y CONFIG_USB_FUNCTION_ROCKUSB=y +CONFIG_VIDEO=y +CONFIG_CONSOLE_ROTATION=y +CONFIG_VIDEO_LCD_BOE_TH101MB31IG002_28A=y +CONFIG_DISPLAY=y +CONFIG_VIDEO_ROCKCHIP=y +CONFIG_DISPLAY_ROCKCHIP_HDMI=y +CONFIG_DISPLAY_ROCKCHIP_DW_MIPI=y +CONFIG_VIDEO_DT_SIMPLEFB=y CONFIG_ERRNO_STR=y
Now that we have VOP2 support, let's enable it and support the built in USB keyboard. Signed-off-by: Dang Huynh <danct12@riseup.net> --- configs/pinetab2-rk3566_defconfig | 15 +++++++++++++++ 1 file changed, 15 insertions(+)