[PATCH] rndis_host: set proper input size for OID_GEN_PHYSICAL_MEDIUM request
MSFT ActiveSync implementation requires that the size of the response for incoming query is to be provided in the request input length. Failure to set the input size proper results in failed request transfer, where the ActiveSync counterpart reports the NDIS_STATUS_INVALID_LENGTH (0xC0010014L) error. Set the input size for OID_GEN_PHYSICAL_MEDIUM query to the expected size of the response in order for the ActiveSync to properly respond to the request. Fixes: 039ee17d1baa ("rndis_host: Add RNDIS physical medium checking into generic_rndis_bind()") Signed-off-by: Andrey Zhizhikin --- drivers/net/usb/rndis_host.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/usb/rndis_host.c b/drivers/net/usb/rndis_host.c index 6609d21ef894..f813ca9dec53 100644 --- a/drivers/net/usb/rndis_host.c +++ b/drivers/net/usb/rndis_host.c @@ -387,7 +387,7 @@ generic_rndis_bind(struct usbnet *dev, struct usb_interface *intf, int flags) reply_len = sizeof *phym; retval = rndis_query(dev, intf, u.buf, RNDIS_OID_GEN_PHYSICAL_MEDIUM, -0, (void **) &phym, &reply_len); +reply_len, (void **)&phym, &reply_len); if (retval != 0 || !phym) { /* OID is optional so don't fail here. */ phym_unspec = cpu_to_le32(RNDIS_PHYSICAL_MEDIUM_UNSPECIFIED); -- 2.25.1
[PATCH] optee: extend normal memory check to also write-through
ARMv7 Architecture Reference Manual [1] section A3.5.5 details Normal memory type, together with cacheability attributes that could be applied to memory regions defined as "Normal memory". Section B2.1.2 of the Architecture Reference Manual [1] also provides details regarding the Memory attributes that could be assigned to particular memory regions, which includes the descrption of cacheability attributes and cache allocation hints. Memory type and cacheability attributes forms 2 separate definitions, where cacheability attributes defines a mechanism of coherency control rather than the type of memory itself. In other words: Normal memory type can be configured with several combination of cacheability attributes, namely: - Write-Through (WT) - Write-Back (WB) followed by cache allocation hint: - Write-Allocate - No Write-Allocate (also known as Read-Allocate) Those types are mapped in the kernel to corresponding macros: - Write-Through: L_PTE_MT_WRITETHROUGH - Write-Back Write-Allocate: L_PTE_MT_WRITEALLOC - Write-Back Read-Allocate: L_PTE_MT_WRITEBACK Current implementation of the op-tee driver takes in account only 2 last memory region types, while performing a check if the memory block is allocated as "Normal memory", leaving Write-Through allocations to be not considered. Extend verification mechanism to include also Normal memory regios, which are designated with Write-Through cacheability attributes. Link: [1]: https://developer.arm.com/documentation/ddi0406/cd Fixes: 853735e40424 ("optee: add writeback to valid memory type") Cc: sta...@vger.kernel.org Signed-off-by: Andrey Zhizhikin --- drivers/tee/optee/call.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/tee/optee/call.c b/drivers/tee/optee/call.c index c981757ba0d4..8da27d02a2d6 100644 --- a/drivers/tee/optee/call.c +++ b/drivers/tee/optee/call.c @@ -535,7 +535,8 @@ static bool is_normal_memory(pgprot_t p) { #if defined(CONFIG_ARM) return (((pgprot_val(p) & L_PTE_MT_MASK) == L_PTE_MT_WRITEALLOC) || - ((pgprot_val(p) & L_PTE_MT_MASK) == L_PTE_MT_WRITEBACK)); + ((pgprot_val(p) & L_PTE_MT_MASK) == L_PTE_MT_WRITEBACK) || + ((pgprot_val(p) & L_PTE_MT_MASK) == L_PTE_MT_WRITETHROUGH)); #elif defined(CONFIG_ARM64) return (pgprot_val(p) & PTE_ATTRINDX_MASK) == PTE_ATTRINDX(MT_NORMAL); #else -- 2.17.1
[PATCH v2 5/5] powerpc/configs: drop unused BACKLIGHT_GENERIC option
Commit 7ecdea4a0226 ("backlight: generic_bl: Remove this driver as it is unused") removed geenric_bl driver from the tree, together with corresponding config option. Remove BACKLIGHT_GENERIC config item from generic-64bit_defconfig. Fixes: 7ecdea4a0226 ("backlight: generic_bl: Remove this driver as it is unused") Cc: Sam Ravnborg Signed-off-by: Andrey Zhizhikin Reviewed-by: Krzysztof Kozlowski Acked-by: Daniel Thompson Acked-by: Sam Ravnborg Acked-by: Michael Ellerman --- arch/powerpc/configs/powernv_defconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/powerpc/configs/powernv_defconfig b/arch/powerpc/configs/powernv_defconfig index cf30fc24413b..60a30fffeda0 100644 --- a/arch/powerpc/configs/powernv_defconfig +++ b/arch/powerpc/configs/powernv_defconfig @@ -208,7 +208,6 @@ CONFIG_FB_MATROX_G=y CONFIG_FB_RADEON=m CONFIG_FB_IBM_GXT4500=m CONFIG_LCD_PLATFORM=m -CONFIG_BACKLIGHT_GENERIC=m # CONFIG_VGA_CONSOLE is not set CONFIG_LOGO=y CONFIG_HID_A4TECH=m -- 2.17.1
[PATCH v2 4/5] parisc: configs: drop unused BACKLIGHT_GENERIC option
Commit 7ecdea4a0226 ("backlight: generic_bl: Remove this driver as it is unused") removed geenric_bl driver from the tree, together with corresponding config option. Remove BACKLIGHT_GENERIC config item from generic-64bit_defconfig. Fixes: 7ecdea4a0226 ("backlight: generic_bl: Remove this driver as it is unused") Cc: Sam Ravnborg Signed-off-by: Andrey Zhizhikin Reviewed-by: Krzysztof Kozlowski Acked-by: Daniel Thompson Acked-by: Sam Ravnborg --- arch/parisc/configs/generic-64bit_defconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/parisc/configs/generic-64bit_defconfig b/arch/parisc/configs/generic-64bit_defconfig index 7e2d7026285e..8f81fcbf04c4 100644 --- a/arch/parisc/configs/generic-64bit_defconfig +++ b/arch/parisc/configs/generic-64bit_defconfig @@ -191,7 +191,6 @@ CONFIG_DRM=y CONFIG_DRM_RADEON=y CONFIG_FIRMWARE_EDID=y CONFIG_FB_MODE_HELPERS=y -# CONFIG_BACKLIGHT_GENERIC is not set CONFIG_FRAMEBUFFER_CONSOLE_ROTATION=y CONFIG_HIDRAW=y CONFIG_HID_PID=y -- 2.17.1
[PATCH v2 3/5] MIPS: configs: drop unused BACKLIGHT_GENERIC option
Commit 7ecdea4a0226 ("backlight: generic_bl: Remove this driver as it is unused") removed geenric_bl driver from the tree, together with corresponding config option. Remove BACKLIGHT_GENERIC config item from all MIPS configurations. Fixes: 7ecdea4a0226 ("backlight: generic_bl: Remove this driver as it is unused") Cc: Sam Ravnborg Signed-off-by: Andrey Zhizhikin Reviewed-by: Krzysztof Kozlowski Acked-by: Daniel Thompson Acked-by: Sam Ravnborg --- arch/mips/configs/gcw0_defconfig | 1 - arch/mips/configs/gpr_defconfig | 1 - arch/mips/configs/lemote2f_defconfig | 1 - arch/mips/configs/loongson3_defconfig | 1 - arch/mips/configs/mtx1_defconfig | 1 - arch/mips/configs/rs90_defconfig | 1 - 6 files changed, 6 deletions(-) diff --git a/arch/mips/configs/gcw0_defconfig b/arch/mips/configs/gcw0_defconfig index 7e28a4fe9d84..460683b52285 100644 --- a/arch/mips/configs/gcw0_defconfig +++ b/arch/mips/configs/gcw0_defconfig @@ -73,7 +73,6 @@ CONFIG_DRM_PANEL_NOVATEK_NT39016=y CONFIG_DRM_INGENIC=y CONFIG_DRM_ETNAVIV=y CONFIG_BACKLIGHT_CLASS_DEVICE=y -# CONFIG_BACKLIGHT_GENERIC is not set CONFIG_BACKLIGHT_PWM=y # CONFIG_VGA_CONSOLE is not set CONFIG_FRAMEBUFFER_CONSOLE=y diff --git a/arch/mips/configs/gpr_defconfig b/arch/mips/configs/gpr_defconfig index 9085f4d6c698..87e20f3391ed 100644 --- a/arch/mips/configs/gpr_defconfig +++ b/arch/mips/configs/gpr_defconfig @@ -251,7 +251,6 @@ CONFIG_SSB_DRIVER_PCICORE=y # CONFIG_VGA_ARB is not set # CONFIG_LCD_CLASS_DEVICE is not set CONFIG_BACKLIGHT_CLASS_DEVICE=y -# CONFIG_BACKLIGHT_GENERIC is not set # CONFIG_VGA_CONSOLE is not set CONFIG_USB_HID=m CONFIG_USB_HIDDEV=y diff --git a/arch/mips/configs/lemote2f_defconfig b/arch/mips/configs/lemote2f_defconfig index 3a9a453b1264..688c91918db2 100644 --- a/arch/mips/configs/lemote2f_defconfig +++ b/arch/mips/configs/lemote2f_defconfig @@ -145,7 +145,6 @@ CONFIG_FB_SIS_300=y CONFIG_FB_SIS_315=y # CONFIG_LCD_CLASS_DEVICE is not set CONFIG_BACKLIGHT_CLASS_DEVICE=y -CONFIG_BACKLIGHT_GENERIC=m # CONFIG_VGA_CONSOLE is not set CONFIG_FRAMEBUFFER_CONSOLE=y CONFIG_FRAMEBUFFER_CONSOLE_ROTATION=y diff --git a/arch/mips/configs/loongson3_defconfig b/arch/mips/configs/loongson3_defconfig index 38a817ead8e7..9c5fadef38cb 100644 --- a/arch/mips/configs/loongson3_defconfig +++ b/arch/mips/configs/loongson3_defconfig @@ -286,7 +286,6 @@ CONFIG_DRM_VIRTIO_GPU=y CONFIG_FB_RADEON=y CONFIG_LCD_CLASS_DEVICE=y CONFIG_LCD_PLATFORM=m -CONFIG_BACKLIGHT_GENERIC=m # CONFIG_VGA_CONSOLE is not set CONFIG_FRAMEBUFFER_CONSOLE=y CONFIG_FRAMEBUFFER_CONSOLE_ROTATION=y diff --git a/arch/mips/configs/mtx1_defconfig b/arch/mips/configs/mtx1_defconfig index 914af125a7fa..0ef2373404e5 100644 --- a/arch/mips/configs/mtx1_defconfig +++ b/arch/mips/configs/mtx1_defconfig @@ -450,7 +450,6 @@ CONFIG_WDT_MTX1=y # CONFIG_VGA_ARB is not set # CONFIG_LCD_CLASS_DEVICE is not set CONFIG_BACKLIGHT_CLASS_DEVICE=y -# CONFIG_BACKLIGHT_GENERIC is not set # CONFIG_VGA_CONSOLE is not set CONFIG_SOUND=m CONFIG_SND=m diff --git a/arch/mips/configs/rs90_defconfig b/arch/mips/configs/rs90_defconfig index dfbb9fed9a42..4f540bb94628 100644 --- a/arch/mips/configs/rs90_defconfig +++ b/arch/mips/configs/rs90_defconfig @@ -97,7 +97,6 @@ CONFIG_DRM_FBDEV_OVERALLOC=300 CONFIG_DRM_PANEL_SIMPLE=y CONFIG_DRM_INGENIC=y CONFIG_BACKLIGHT_CLASS_DEVICE=y -# CONFIG_BACKLIGHT_GENERIC is not set CONFIG_BACKLIGHT_PWM=y # CONFIG_VGA_CONSOLE is not set CONFIG_FRAMEBUFFER_CONSOLE=y -- 2.17.1
[PATCH v2 1/5] ARM: configs: drop unused BACKLIGHT_GENERIC option
Commit 7ecdea4a0226 ("backlight: generic_bl: Remove this driver as it is unused") removed geenric_bl driver from the tree, together with corresponding config option. Remove BACKLIGHT_GENERIC config item from all ARM configurations. Fixes: 7ecdea4a0226 ("backlight: generic_bl: Remove this driver as it is unused") Cc: Sam Ravnborg Signed-off-by: Andrey Zhizhikin Reviewed-by: Krzysztof Kozlowski Acked-by: Alexandre Belloni Acked-by: Daniel Thompson Acked-by: Sam Ravnborg --- arch/arm/configs/at91_dt_defconfig| 1 - arch/arm/configs/cm_x300_defconfig| 1 - arch/arm/configs/colibri_pxa300_defconfig | 1 - arch/arm/configs/jornada720_defconfig | 1 - arch/arm/configs/magician_defconfig | 1 - arch/arm/configs/mini2440_defconfig | 1 - arch/arm/configs/omap2plus_defconfig | 1 - arch/arm/configs/pxa3xx_defconfig | 1 - arch/arm/configs/qcom_defconfig | 1 - arch/arm/configs/sama5_defconfig | 1 - arch/arm/configs/sunxi_defconfig | 1 - arch/arm/configs/tegra_defconfig | 1 - arch/arm/configs/u8500_defconfig | 1 - 13 files changed, 13 deletions(-) diff --git a/arch/arm/configs/at91_dt_defconfig b/arch/arm/configs/at91_dt_defconfig index 4a0ba2ae1a25..6e52c9c965e6 100644 --- a/arch/arm/configs/at91_dt_defconfig +++ b/arch/arm/configs/at91_dt_defconfig @@ -132,7 +132,6 @@ CONFIG_DRM_ATMEL_HLCDC=y CONFIG_DRM_PANEL_SIMPLE=y CONFIG_FB_ATMEL=y CONFIG_BACKLIGHT_ATMEL_LCDC=y -# CONFIG_BACKLIGHT_GENERIC is not set CONFIG_BACKLIGHT_PWM=y CONFIG_FRAMEBUFFER_CONSOLE=y CONFIG_LOGO=y diff --git a/arch/arm/configs/cm_x300_defconfig b/arch/arm/configs/cm_x300_defconfig index 2f7acde2d921..502a9d870ca4 100644 --- a/arch/arm/configs/cm_x300_defconfig +++ b/arch/arm/configs/cm_x300_defconfig @@ -87,7 +87,6 @@ CONFIG_FB=y CONFIG_FB_PXA=y CONFIG_LCD_CLASS_DEVICE=y CONFIG_LCD_TDO24M=y -# CONFIG_BACKLIGHT_GENERIC is not set CONFIG_BACKLIGHT_DA903X=m CONFIG_FRAMEBUFFER_CONSOLE=y CONFIG_FRAMEBUFFER_CONSOLE_DETECT_PRIMARY=y diff --git a/arch/arm/configs/colibri_pxa300_defconfig b/arch/arm/configs/colibri_pxa300_defconfig index 0dae3b185284..26e5a67f8e2d 100644 --- a/arch/arm/configs/colibri_pxa300_defconfig +++ b/arch/arm/configs/colibri_pxa300_defconfig @@ -34,7 +34,6 @@ CONFIG_FB=y CONFIG_FB_PXA=y # CONFIG_LCD_CLASS_DEVICE is not set CONFIG_BACKLIGHT_CLASS_DEVICE=y -# CONFIG_BACKLIGHT_GENERIC is not set # CONFIG_VGA_CONSOLE is not set CONFIG_FRAMEBUFFER_CONSOLE=y CONFIG_LOGO=y diff --git a/arch/arm/configs/jornada720_defconfig b/arch/arm/configs/jornada720_defconfig index 9f079be2b84b..069f60ffdcd8 100644 --- a/arch/arm/configs/jornada720_defconfig +++ b/arch/arm/configs/jornada720_defconfig @@ -48,7 +48,6 @@ CONFIG_FB=y CONFIG_FB_S1D13XXX=y CONFIG_LCD_CLASS_DEVICE=y CONFIG_BACKLIGHT_CLASS_DEVICE=y -# CONFIG_BACKLIGHT_GENERIC is not set # CONFIG_VGA_CONSOLE is not set CONFIG_FRAMEBUFFER_CONSOLE=y CONFIG_FRAMEBUFFER_CONSOLE_DETECT_PRIMARY=y diff --git a/arch/arm/configs/magician_defconfig b/arch/arm/configs/magician_defconfig index d2e684f6565a..b4670d42f378 100644 --- a/arch/arm/configs/magician_defconfig +++ b/arch/arm/configs/magician_defconfig @@ -95,7 +95,6 @@ CONFIG_FB_PXA_OVERLAY=y CONFIG_FB_W100=y CONFIG_LCD_CLASS_DEVICE=y CONFIG_BACKLIGHT_CLASS_DEVICE=y -# CONFIG_BACKLIGHT_GENERIC is not set CONFIG_BACKLIGHT_PWM=y # CONFIG_VGA_CONSOLE is not set CONFIG_FRAMEBUFFER_CONSOLE=y diff --git a/arch/arm/configs/mini2440_defconfig b/arch/arm/configs/mini2440_defconfig index 301f29a1fcc3..898490aaa39e 100644 --- a/arch/arm/configs/mini2440_defconfig +++ b/arch/arm/configs/mini2440_defconfig @@ -158,7 +158,6 @@ CONFIG_FB_S3C2410=y CONFIG_LCD_CLASS_DEVICE=y CONFIG_LCD_PLATFORM=y CONFIG_BACKLIGHT_CLASS_DEVICE=y -# CONFIG_BACKLIGHT_GENERIC is not set CONFIG_BACKLIGHT_PWM=y CONFIG_FRAMEBUFFER_CONSOLE=y CONFIG_FRAMEBUFFER_CONSOLE_DETECT_PRIMARY=y diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig index de3b7813a1ce..7eae097a75d2 100644 --- a/arch/arm/configs/omap2plus_defconfig +++ b/arch/arm/configs/omap2plus_defconfig @@ -388,7 +388,6 @@ CONFIG_FB_TILEBLITTING=y CONFIG_LCD_CLASS_DEVICE=y CONFIG_LCD_PLATFORM=y CONFIG_BACKLIGHT_CLASS_DEVICE=y -CONFIG_BACKLIGHT_GENERIC=m CONFIG_BACKLIGHT_PWM=m CONFIG_BACKLIGHT_PANDORA=m CONFIG_BACKLIGHT_GPIO=m diff --git a/arch/arm/configs/pxa3xx_defconfig b/arch/arm/configs/pxa3xx_defconfig index 06bbc7a59b60..f0c34017f2aa 100644 --- a/arch/arm/configs/pxa3xx_defconfig +++ b/arch/arm/configs/pxa3xx_defconfig @@ -74,7 +74,6 @@ CONFIG_FB_PXA=y CONFIG_LCD_CLASS_DEVICE=y CONFIG_LCD_TDO24M=y CONFIG_BACKLIGHT_CLASS_DEVICE=y -# CONFIG_BACKLIGHT_GENERIC is not set CONFIG_BACKLIGHT_DA903X=y # CONFIG_VGA_CONSOLE is not set CONFIG_FRAMEBUFFER_CONSOLE=y diff --git a/arch/arm/configs/qcom_defconfig b/arch/arm/configs/qcom_defconfig index c882167e1496..d6733e745b80 100644 --- a/arch/arm/configs/qcom_defconfig ++
[PATCH v2 2/5] arm64: defconfig: drop unused BACKLIGHT_GENERIC option
Commit 7ecdea4a0226 ("backlight: generic_bl: Remove this driver as it is unused") removed geenric_bl driver from the tree, together with corresponding config option. Remove BACKLIGHT_GENERIC config item from arm64 configuration. Fixes: 7ecdea4a0226 ("backlight: generic_bl: Remove this driver as it is unused") Cc: Sam Ravnborg Signed-off-by: Andrey Zhizhikin Reviewed-by: Krzysztof Kozlowski Acked-by: Daniel Thompson Acked-by: Sam Ravnborg --- arch/arm64/configs/defconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig index 8e3f7ae71de5..280ed7404a1d 100644 --- a/arch/arm64/configs/defconfig +++ b/arch/arm64/configs/defconfig @@ -681,7 +681,6 @@ CONFIG_DRM_PANFROST=m CONFIG_FB=y CONFIG_FB_MODE_HELPERS=y CONFIG_FB_EFI=y -CONFIG_BACKLIGHT_GENERIC=m CONFIG_BACKLIGHT_PWM=m CONFIG_BACKLIGHT_LP855X=m CONFIG_LOGO=y -- 2.17.1
[PATCH v2 0/5] drop unused BACKLIGHT_GENERIC option
Since the removal of generic_bl driver from the source tree in commit 7ecdea4a0226 ("backlight: generic_bl: Remove this driver as it is unused") BACKLIGHT_GENERIC config option became obsolete as well and therefore subject to clean-up from all configuration files. This series introduces patches to address this removal, separated by architectures in the kernel tree. Changes in v2: - Collect all Acked-by: and Reviewed-by: tags - Include ARM SOC maintainer list to recipients Andrey Zhizhikin (5): ARM: configs: drop unused BACKLIGHT_GENERIC option arm64: defconfig: drop unused BACKLIGHT_GENERIC option MIPS: configs: drop unused BACKLIGHT_GENERIC option parisc: configs: drop unused BACKLIGHT_GENERIC option powerpc/configs: drop unused BACKLIGHT_GENERIC option arch/arm/configs/at91_dt_defconfig | 1 - arch/arm/configs/cm_x300_defconfig | 1 - arch/arm/configs/colibri_pxa300_defconfig | 1 - arch/arm/configs/jornada720_defconfig | 1 - arch/arm/configs/magician_defconfig | 1 - arch/arm/configs/mini2440_defconfig | 1 - arch/arm/configs/omap2plus_defconfig| 1 - arch/arm/configs/pxa3xx_defconfig | 1 - arch/arm/configs/qcom_defconfig | 1 - arch/arm/configs/sama5_defconfig| 1 - arch/arm/configs/sunxi_defconfig| 1 - arch/arm/configs/tegra_defconfig| 1 - arch/arm/configs/u8500_defconfig| 1 - arch/arm64/configs/defconfig| 1 - arch/mips/configs/gcw0_defconfig| 1 - arch/mips/configs/gpr_defconfig | 1 - arch/mips/configs/lemote2f_defconfig| 1 - arch/mips/configs/loongson3_defconfig | 1 - arch/mips/configs/mtx1_defconfig| 1 - arch/mips/configs/rs90_defconfig| 1 - arch/parisc/configs/generic-64bit_defconfig | 1 - arch/powerpc/configs/powernv_defconfig | 1 - 22 files changed, 22 deletions(-) base-commit: b65054597872ce3aefbc6a666385eabdf9e288da -- 2.17.1
[PATCH 5/5] powerpc/configs: drop unused BACKLIGHT_GENERIC option
Commit 7ecdea4a0226 ("backlight: generic_bl: Remove this driver as it is unused") removed geenric_bl driver from the tree, together with corresponding config option. Remove BACKLIGHT_GENERIC config item from generic-64bit_defconfig. Fixes: 7ecdea4a0226 ("backlight: generic_bl: Remove this driver as it is unused") Cc: Sam Ravnborg Signed-off-by: Andrey Zhizhikin --- arch/powerpc/configs/powernv_defconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/powerpc/configs/powernv_defconfig b/arch/powerpc/configs/powernv_defconfig index cf30fc24413b..60a30fffeda0 100644 --- a/arch/powerpc/configs/powernv_defconfig +++ b/arch/powerpc/configs/powernv_defconfig @@ -208,7 +208,6 @@ CONFIG_FB_MATROX_G=y CONFIG_FB_RADEON=m CONFIG_FB_IBM_GXT4500=m CONFIG_LCD_PLATFORM=m -CONFIG_BACKLIGHT_GENERIC=m # CONFIG_VGA_CONSOLE is not set CONFIG_LOGO=y CONFIG_HID_A4TECH=m -- 2.17.1
[PATCH 2/5] arm64: defconfig: drop unused BACKLIGHT_GENERIC option
Commit 7ecdea4a0226 ("backlight: generic_bl: Remove this driver as it is unused") removed geenric_bl driver from the tree, together with corresponding config option. Remove BACKLIGHT_GENERIC config item from arm64 configuration. Fixes: 7ecdea4a0226 ("backlight: generic_bl: Remove this driver as it is unused") Cc: Sam Ravnborg Signed-off-by: Andrey Zhizhikin --- arch/arm64/configs/defconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig index 8e3f7ae71de5..280ed7404a1d 100644 --- a/arch/arm64/configs/defconfig +++ b/arch/arm64/configs/defconfig @@ -681,7 +681,6 @@ CONFIG_DRM_PANFROST=m CONFIG_FB=y CONFIG_FB_MODE_HELPERS=y CONFIG_FB_EFI=y -CONFIG_BACKLIGHT_GENERIC=m CONFIG_BACKLIGHT_PWM=m CONFIG_BACKLIGHT_LP855X=m CONFIG_LOGO=y -- 2.17.1
[PATCH 4/5] parisc: configs: drop unused BACKLIGHT_GENERIC option
Commit 7ecdea4a0226 ("backlight: generic_bl: Remove this driver as it is unused") removed geenric_bl driver from the tree, together with corresponding config option. Remove BACKLIGHT_GENERIC config item from generic-64bit_defconfig. Fixes: 7ecdea4a0226 ("backlight: generic_bl: Remove this driver as it is unused") Cc: Sam Ravnborg Signed-off-by: Andrey Zhizhikin --- arch/parisc/configs/generic-64bit_defconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/parisc/configs/generic-64bit_defconfig b/arch/parisc/configs/generic-64bit_defconfig index 7e2d7026285e..8f81fcbf04c4 100644 --- a/arch/parisc/configs/generic-64bit_defconfig +++ b/arch/parisc/configs/generic-64bit_defconfig @@ -191,7 +191,6 @@ CONFIG_DRM=y CONFIG_DRM_RADEON=y CONFIG_FIRMWARE_EDID=y CONFIG_FB_MODE_HELPERS=y -# CONFIG_BACKLIGHT_GENERIC is not set CONFIG_FRAMEBUFFER_CONSOLE_ROTATION=y CONFIG_HIDRAW=y CONFIG_HID_PID=y -- 2.17.1
[PATCH 3/5] MIPS: configs: drop unused BACKLIGHT_GENERIC option
Commit 7ecdea4a0226 ("backlight: generic_bl: Remove this driver as it is unused") removed geenric_bl driver from the tree, together with corresponding config option. Remove BACKLIGHT_GENERIC config item from all MIPS configurations. Fixes: 7ecdea4a0226 ("backlight: generic_bl: Remove this driver as it is unused") Cc: Sam Ravnborg Signed-off-by: Andrey Zhizhikin --- arch/mips/configs/gcw0_defconfig | 1 - arch/mips/configs/gpr_defconfig | 1 - arch/mips/configs/lemote2f_defconfig | 1 - arch/mips/configs/loongson3_defconfig | 1 - arch/mips/configs/mtx1_defconfig | 1 - arch/mips/configs/rs90_defconfig | 1 - 6 files changed, 6 deletions(-) diff --git a/arch/mips/configs/gcw0_defconfig b/arch/mips/configs/gcw0_defconfig index 7e28a4fe9d84..460683b52285 100644 --- a/arch/mips/configs/gcw0_defconfig +++ b/arch/mips/configs/gcw0_defconfig @@ -73,7 +73,6 @@ CONFIG_DRM_PANEL_NOVATEK_NT39016=y CONFIG_DRM_INGENIC=y CONFIG_DRM_ETNAVIV=y CONFIG_BACKLIGHT_CLASS_DEVICE=y -# CONFIG_BACKLIGHT_GENERIC is not set CONFIG_BACKLIGHT_PWM=y # CONFIG_VGA_CONSOLE is not set CONFIG_FRAMEBUFFER_CONSOLE=y diff --git a/arch/mips/configs/gpr_defconfig b/arch/mips/configs/gpr_defconfig index 9085f4d6c698..87e20f3391ed 100644 --- a/arch/mips/configs/gpr_defconfig +++ b/arch/mips/configs/gpr_defconfig @@ -251,7 +251,6 @@ CONFIG_SSB_DRIVER_PCICORE=y # CONFIG_VGA_ARB is not set # CONFIG_LCD_CLASS_DEVICE is not set CONFIG_BACKLIGHT_CLASS_DEVICE=y -# CONFIG_BACKLIGHT_GENERIC is not set # CONFIG_VGA_CONSOLE is not set CONFIG_USB_HID=m CONFIG_USB_HIDDEV=y diff --git a/arch/mips/configs/lemote2f_defconfig b/arch/mips/configs/lemote2f_defconfig index 3a9a453b1264..688c91918db2 100644 --- a/arch/mips/configs/lemote2f_defconfig +++ b/arch/mips/configs/lemote2f_defconfig @@ -145,7 +145,6 @@ CONFIG_FB_SIS_300=y CONFIG_FB_SIS_315=y # CONFIG_LCD_CLASS_DEVICE is not set CONFIG_BACKLIGHT_CLASS_DEVICE=y -CONFIG_BACKLIGHT_GENERIC=m # CONFIG_VGA_CONSOLE is not set CONFIG_FRAMEBUFFER_CONSOLE=y CONFIG_FRAMEBUFFER_CONSOLE_ROTATION=y diff --git a/arch/mips/configs/loongson3_defconfig b/arch/mips/configs/loongson3_defconfig index 38a817ead8e7..9c5fadef38cb 100644 --- a/arch/mips/configs/loongson3_defconfig +++ b/arch/mips/configs/loongson3_defconfig @@ -286,7 +286,6 @@ CONFIG_DRM_VIRTIO_GPU=y CONFIG_FB_RADEON=y CONFIG_LCD_CLASS_DEVICE=y CONFIG_LCD_PLATFORM=m -CONFIG_BACKLIGHT_GENERIC=m # CONFIG_VGA_CONSOLE is not set CONFIG_FRAMEBUFFER_CONSOLE=y CONFIG_FRAMEBUFFER_CONSOLE_ROTATION=y diff --git a/arch/mips/configs/mtx1_defconfig b/arch/mips/configs/mtx1_defconfig index 914af125a7fa..0ef2373404e5 100644 --- a/arch/mips/configs/mtx1_defconfig +++ b/arch/mips/configs/mtx1_defconfig @@ -450,7 +450,6 @@ CONFIG_WDT_MTX1=y # CONFIG_VGA_ARB is not set # CONFIG_LCD_CLASS_DEVICE is not set CONFIG_BACKLIGHT_CLASS_DEVICE=y -# CONFIG_BACKLIGHT_GENERIC is not set # CONFIG_VGA_CONSOLE is not set CONFIG_SOUND=m CONFIG_SND=m diff --git a/arch/mips/configs/rs90_defconfig b/arch/mips/configs/rs90_defconfig index dfbb9fed9a42..4f540bb94628 100644 --- a/arch/mips/configs/rs90_defconfig +++ b/arch/mips/configs/rs90_defconfig @@ -97,7 +97,6 @@ CONFIG_DRM_FBDEV_OVERALLOC=300 CONFIG_DRM_PANEL_SIMPLE=y CONFIG_DRM_INGENIC=y CONFIG_BACKLIGHT_CLASS_DEVICE=y -# CONFIG_BACKLIGHT_GENERIC is not set CONFIG_BACKLIGHT_PWM=y # CONFIG_VGA_CONSOLE is not set CONFIG_FRAMEBUFFER_CONSOLE=y -- 2.17.1
[PATCH 1/5] ARM: configs: drop unused BACKLIGHT_GENERIC option
Commit 7ecdea4a0226 ("backlight: generic_bl: Remove this driver as it is unused") removed geenric_bl driver from the tree, together with corresponding config option. Remove BACKLIGHT_GENERIC config item from all ARM configurations. Fixes: 7ecdea4a0226 ("backlight: generic_bl: Remove this driver as it is unused") Cc: Sam Ravnborg Signed-off-by: Andrey Zhizhikin --- arch/arm/configs/at91_dt_defconfig| 1 - arch/arm/configs/cm_x300_defconfig| 1 - arch/arm/configs/colibri_pxa300_defconfig | 1 - arch/arm/configs/jornada720_defconfig | 1 - arch/arm/configs/magician_defconfig | 1 - arch/arm/configs/mini2440_defconfig | 1 - arch/arm/configs/omap2plus_defconfig | 1 - arch/arm/configs/pxa3xx_defconfig | 1 - arch/arm/configs/qcom_defconfig | 1 - arch/arm/configs/sama5_defconfig | 1 - arch/arm/configs/sunxi_defconfig | 1 - arch/arm/configs/tegra_defconfig | 1 - arch/arm/configs/u8500_defconfig | 1 - 13 files changed, 13 deletions(-) diff --git a/arch/arm/configs/at91_dt_defconfig b/arch/arm/configs/at91_dt_defconfig index 4a0ba2ae1a25..6e52c9c965e6 100644 --- a/arch/arm/configs/at91_dt_defconfig +++ b/arch/arm/configs/at91_dt_defconfig @@ -132,7 +132,6 @@ CONFIG_DRM_ATMEL_HLCDC=y CONFIG_DRM_PANEL_SIMPLE=y CONFIG_FB_ATMEL=y CONFIG_BACKLIGHT_ATMEL_LCDC=y -# CONFIG_BACKLIGHT_GENERIC is not set CONFIG_BACKLIGHT_PWM=y CONFIG_FRAMEBUFFER_CONSOLE=y CONFIG_LOGO=y diff --git a/arch/arm/configs/cm_x300_defconfig b/arch/arm/configs/cm_x300_defconfig index 2f7acde2d921..502a9d870ca4 100644 --- a/arch/arm/configs/cm_x300_defconfig +++ b/arch/arm/configs/cm_x300_defconfig @@ -87,7 +87,6 @@ CONFIG_FB=y CONFIG_FB_PXA=y CONFIG_LCD_CLASS_DEVICE=y CONFIG_LCD_TDO24M=y -# CONFIG_BACKLIGHT_GENERIC is not set CONFIG_BACKLIGHT_DA903X=m CONFIG_FRAMEBUFFER_CONSOLE=y CONFIG_FRAMEBUFFER_CONSOLE_DETECT_PRIMARY=y diff --git a/arch/arm/configs/colibri_pxa300_defconfig b/arch/arm/configs/colibri_pxa300_defconfig index 0dae3b185284..26e5a67f8e2d 100644 --- a/arch/arm/configs/colibri_pxa300_defconfig +++ b/arch/arm/configs/colibri_pxa300_defconfig @@ -34,7 +34,6 @@ CONFIG_FB=y CONFIG_FB_PXA=y # CONFIG_LCD_CLASS_DEVICE is not set CONFIG_BACKLIGHT_CLASS_DEVICE=y -# CONFIG_BACKLIGHT_GENERIC is not set # CONFIG_VGA_CONSOLE is not set CONFIG_FRAMEBUFFER_CONSOLE=y CONFIG_LOGO=y diff --git a/arch/arm/configs/jornada720_defconfig b/arch/arm/configs/jornada720_defconfig index 9f079be2b84b..069f60ffdcd8 100644 --- a/arch/arm/configs/jornada720_defconfig +++ b/arch/arm/configs/jornada720_defconfig @@ -48,7 +48,6 @@ CONFIG_FB=y CONFIG_FB_S1D13XXX=y CONFIG_LCD_CLASS_DEVICE=y CONFIG_BACKLIGHT_CLASS_DEVICE=y -# CONFIG_BACKLIGHT_GENERIC is not set # CONFIG_VGA_CONSOLE is not set CONFIG_FRAMEBUFFER_CONSOLE=y CONFIG_FRAMEBUFFER_CONSOLE_DETECT_PRIMARY=y diff --git a/arch/arm/configs/magician_defconfig b/arch/arm/configs/magician_defconfig index d2e684f6565a..b4670d42f378 100644 --- a/arch/arm/configs/magician_defconfig +++ b/arch/arm/configs/magician_defconfig @@ -95,7 +95,6 @@ CONFIG_FB_PXA_OVERLAY=y CONFIG_FB_W100=y CONFIG_LCD_CLASS_DEVICE=y CONFIG_BACKLIGHT_CLASS_DEVICE=y -# CONFIG_BACKLIGHT_GENERIC is not set CONFIG_BACKLIGHT_PWM=y # CONFIG_VGA_CONSOLE is not set CONFIG_FRAMEBUFFER_CONSOLE=y diff --git a/arch/arm/configs/mini2440_defconfig b/arch/arm/configs/mini2440_defconfig index 301f29a1fcc3..898490aaa39e 100644 --- a/arch/arm/configs/mini2440_defconfig +++ b/arch/arm/configs/mini2440_defconfig @@ -158,7 +158,6 @@ CONFIG_FB_S3C2410=y CONFIG_LCD_CLASS_DEVICE=y CONFIG_LCD_PLATFORM=y CONFIG_BACKLIGHT_CLASS_DEVICE=y -# CONFIG_BACKLIGHT_GENERIC is not set CONFIG_BACKLIGHT_PWM=y CONFIG_FRAMEBUFFER_CONSOLE=y CONFIG_FRAMEBUFFER_CONSOLE_DETECT_PRIMARY=y diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig index de3b7813a1ce..7eae097a75d2 100644 --- a/arch/arm/configs/omap2plus_defconfig +++ b/arch/arm/configs/omap2plus_defconfig @@ -388,7 +388,6 @@ CONFIG_FB_TILEBLITTING=y CONFIG_LCD_CLASS_DEVICE=y CONFIG_LCD_PLATFORM=y CONFIG_BACKLIGHT_CLASS_DEVICE=y -CONFIG_BACKLIGHT_GENERIC=m CONFIG_BACKLIGHT_PWM=m CONFIG_BACKLIGHT_PANDORA=m CONFIG_BACKLIGHT_GPIO=m diff --git a/arch/arm/configs/pxa3xx_defconfig b/arch/arm/configs/pxa3xx_defconfig index 06bbc7a59b60..f0c34017f2aa 100644 --- a/arch/arm/configs/pxa3xx_defconfig +++ b/arch/arm/configs/pxa3xx_defconfig @@ -74,7 +74,6 @@ CONFIG_FB_PXA=y CONFIG_LCD_CLASS_DEVICE=y CONFIG_LCD_TDO24M=y CONFIG_BACKLIGHT_CLASS_DEVICE=y -# CONFIG_BACKLIGHT_GENERIC is not set CONFIG_BACKLIGHT_DA903X=y # CONFIG_VGA_CONSOLE is not set CONFIG_FRAMEBUFFER_CONSOLE=y diff --git a/arch/arm/configs/qcom_defconfig b/arch/arm/configs/qcom_defconfig index c882167e1496..d6733e745b80 100644 --- a/arch/arm/configs/qcom_defconfig +++ b/arch/arm/configs/qcom_defconfig @@ -159,7 +159,6 @@ CONFIG_FB=y CONFIG_FRAMEBUFFER_CONSOLE=y # CONFIG_LCD
[PATCH 0/5] drop unused BACKLIGHT_GENERIC option
Since the removal of generic_bl driver from the source tree in commit 7ecdea4a0226 ("backlight: generic_bl: Remove this driver as it is unused") BACKLIGHT_GENERIC config option became obsolete as well and therefore subject to clean-up from all configuration files. This series introduces patches to address this removal, separated by architectures in the kernel tree. Andrey Zhizhikin (5): ARM: configs: drop unused BACKLIGHT_GENERIC option arm64: defconfig: drop unused BACKLIGHT_GENERIC option MIPS: configs: drop unused BACKLIGHT_GENERIC option parisc: configs: drop unused BACKLIGHT_GENERIC option powerpc/configs: drop unused BACKLIGHT_GENERIC option arch/arm/configs/at91_dt_defconfig | 1 - arch/arm/configs/cm_x300_defconfig | 1 - arch/arm/configs/colibri_pxa300_defconfig | 1 - arch/arm/configs/jornada720_defconfig | 1 - arch/arm/configs/magician_defconfig | 1 - arch/arm/configs/mini2440_defconfig | 1 - arch/arm/configs/omap2plus_defconfig| 1 - arch/arm/configs/pxa3xx_defconfig | 1 - arch/arm/configs/qcom_defconfig | 1 - arch/arm/configs/sama5_defconfig| 1 - arch/arm/configs/sunxi_defconfig| 1 - arch/arm/configs/tegra_defconfig| 1 - arch/arm/configs/u8500_defconfig| 1 - arch/arm64/configs/defconfig| 1 - arch/mips/configs/gcw0_defconfig| 1 - arch/mips/configs/gpr_defconfig | 1 - arch/mips/configs/lemote2f_defconfig| 1 - arch/mips/configs/loongson3_defconfig | 1 - arch/mips/configs/mtx1_defconfig| 1 - arch/mips/configs/rs90_defconfig| 1 - arch/parisc/configs/generic-64bit_defconfig | 1 - arch/powerpc/configs/powernv_defconfig | 1 - 22 files changed, 22 deletions(-) base-commit: b65054597872ce3aefbc6a666385eabdf9e288da prerequisite-patch-id: bfd382cf1dc021d20204f10ea9403319c1c32b12 prerequisite-patch-id: 5397c0c8648bb3e0b830207ea867138c11c6e644 prerequisite-patch-id: a3c284dff5fe6d02828918a886db6a8ed3197e20 -- 2.17.1
[PATCH] ARM: omap2plus_defconfig: drop unused POWER_AVS option
Commit 785b5bb41b0a ("PM: AVS: Drop the avs directory and the corresponding Kconfig") moved AVS code to SOC-specific folders, and removed corresponding Kconfig from drivers/power, leaving original POWER_AVS config option enabled in omap2plus_defconfig file. Remove the option, which has no references in the tree anymore. Fixes: 785b5bb41b0a ("PM: AVS: Drop the avs directory and the corresponding Kconfig") Cc: Nishanth Menon Cc: Ulf Hansson Signed-off-by: Andrey Zhizhikin --- arch/arm/configs/omap2plus_defconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig index 34793aabdb65..de3b7813a1ce 100644 --- a/arch/arm/configs/omap2plus_defconfig +++ b/arch/arm/configs/omap2plus_defconfig @@ -297,7 +297,6 @@ CONFIG_GPIO_TWL4030=y CONFIG_W1=m CONFIG_HDQ_MASTER_OMAP=m CONFIG_W1_SLAVE_DS250X=m -CONFIG_POWER_AVS=y CONFIG_POWER_RESET=y CONFIG_POWER_RESET_GPIO=y CONFIG_BATTERY_BQ27XXX=m -- 2.17.1
[PATCH 2/2] ARM: multi_v7_defconfig: drop unused POWER_AVS option
Commit 785b5bb41b0a ("PM: AVS: Drop the avs directory and the corresponding Kconfig") moved AVS code to SOC-specific folders, and removed corresponding Kconfig from drivers/power, leaving original POWER_AVS config option enabled in multi_v7_defconfig file. Remove the option, which has no references in the tree anymore. Fixes: 785b5bb41b0a ("PM: AVS: Drop the avs directory and the corresponding Kconfig") Cc: Nishanth Menon Cc: Ulf Hansson Signed-off-by: Andrey Zhizhikin --- arch/arm/configs/multi_v7_defconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig index a611b0c1e540..91752d918d34 100644 --- a/arch/arm/configs/multi_v7_defconfig +++ b/arch/arm/configs/multi_v7_defconfig @@ -466,7 +466,6 @@ CONFIG_GPIO_TPS6586X=y CONFIG_GPIO_TPS65910=y CONFIG_GPIO_TWL4030=y CONFIG_GPIO_MXC=y -CONFIG_POWER_AVS=y CONFIG_ROCKCHIP_IODOMAIN=y CONFIG_POWER_RESET_AS3722=y CONFIG_POWER_RESET_GPIO=y -- 2.17.1
[PATCH 1/2] arm64: defconfig: drop unused POWER_AVS option
Commit 785b5bb41b0a ("PM: AVS: Drop the avs directory and the corresponding Kconfig") moved AVS code to SOC-specific folders, and removed corresponding Kconfig from drivers/power, leaving original POWER_AVS config option enabled in defconfig file. Remove the option, which has no references in the tree anymore. Fixes: 785b5bb41b0a ("PM: AVS: Drop the avs directory and the corresponding Kconfig") Cc: Nishanth Menon Cc: Ulf Hansson Signed-off-by: Andrey Zhizhikin --- arch/arm64/configs/defconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig index 5cfe3cf6f2ac..8e3f7ae71de5 100644 --- a/arch/arm64/configs/defconfig +++ b/arch/arm64/configs/defconfig @@ -513,7 +513,6 @@ CONFIG_GPIO_PCA953X_IRQ=y CONFIG_GPIO_BD9571MWV=m CONFIG_GPIO_MAX77620=y CONFIG_GPIO_SL28CPLD=m -CONFIG_POWER_AVS=y CONFIG_QCOM_CPR=y CONFIG_ROCKCHIP_IODOMAIN=y CONFIG_POWER_RESET_MSM=y -- 2.17.1
[PATCH 0/2] drop unused POWER_AVS option for ARM/ARM64 config
This series cleans up kernel config files, removing the unused POWER_AVS option. It has been removed in commit 785b5bb41b0a ("PM: AVS: Drop the avs directory and the corresponding Kconfig"), from the tree and therefore should be removed frorm default config files delivered for ARM and ARM64 builds. Andrey Zhizhikin (2): arm64: defconfig: drop unused POWER_AVS option ARM: multi_v7_defconfig: drop unused POWER_AVS option arch/arm/configs/multi_v7_defconfig | 1 - arch/arm64/configs/defconfig| 1 - 2 files changed, 2 deletions(-) -- 2.17.1
Re: [PATCH v2] ARM: socfpga_defconfig: enable support for large block devices
On Thu, Mar 28, 2019 at 5:07 PM Dinh Nguyen wrote: > On 3/7/19 1:43 AM, Andrey Zhizhikin wrote: > > Hello Dinh, > > > > Just a short ping on this patch - do yo think you can accept this > > patch and have it merged? I'd like to know whether it is planned to be > > integrated, as it might be beneficial for a lot of socfpga users... > > > > Thanks a lot! > > > > Sorry I missed this. I've applied this. No problem, thanks a lot! > > Thanks, > Dinh -- Regards, Andrey.
Re: [PATCH v2] ARM: socfpga_defconfig: enable support for large block devices
Hello Dinh, Just a short ping on this patch - do yo think you can accept this patch and have it merged? I'd like to know whether it is planned to be integrated, as it might be beneficial for a lot of socfpga users... Thanks a lot! -- Regards, Andrey. On Wed, Feb 27, 2019 at 5:50 PM Andrey Zhizhikin wrote: > > Enable CONFIG_LBDAF, which is required by ext4 fs. This option could > handle both ext3 and ext4, with ext4 requires this option to be enabled, > otherwise the filesystem is mounted RO mode. > > Since the LBDAF is enabled by default for 32-bit systems, simply > removing the current "not set" entry enables the support. > > Signed-off-by: Andrey Zhizhikin > --- > arch/arm/configs/socfpga_defconfig | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/arch/arm/configs/socfpga_defconfig > b/arch/arm/configs/socfpga_defconfig > index 08d1b3e11d68..c96d93fb68c6 100644 > --- a/arch/arm/configs/socfpga_defconfig > +++ b/arch/arm/configs/socfpga_defconfig > @@ -21,7 +21,6 @@ CONFIG_NEON=y > CONFIG_OPROFILE=y > CONFIG_MODULES=y > CONFIG_MODULE_UNLOAD=y > -# CONFIG_LBDAF is not set > # CONFIG_BLK_DEV_BSG is not set > CONFIG_NET=y > CONFIG_PACKET=y > -- > 2.17.1 >
[PATCH v2] ARM: socfpga_defconfig: enable support for large block devices
Enable CONFIG_LBDAF, which is required by ext4 fs. This option could handle both ext3 and ext4, with ext4 requires this option to be enabled, otherwise the filesystem is mounted RO mode. Since the LBDAF is enabled by default for 32-bit systems, simply removing the current "not set" entry enables the support. Signed-off-by: Andrey Zhizhikin --- arch/arm/configs/socfpga_defconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/configs/socfpga_defconfig b/arch/arm/configs/socfpga_defconfig index 08d1b3e11d68..c96d93fb68c6 100644 --- a/arch/arm/configs/socfpga_defconfig +++ b/arch/arm/configs/socfpga_defconfig @@ -21,7 +21,6 @@ CONFIG_NEON=y CONFIG_OPROFILE=y CONFIG_MODULES=y CONFIG_MODULE_UNLOAD=y -# CONFIG_LBDAF is not set # CONFIG_BLK_DEV_BSG is not set CONFIG_NET=y CONFIG_PACKET=y -- 2.17.1
Re: [PATCH] ARM: socfpga_defconfig: enable support for large block devices
Hello Dinh, On Mon, Feb 25, 2019 at 5:35 PM Dinh Nguyen wrote: > > Hi Andrey, > > On 2/22/19 6:21 AM, Andrey Zhizhikin wrote: > > Enable CONFIG_LBDAF, which is required by ext4 fs. This option could > > hanle both ext3 and ext4, and ex4 requires this option to be enabled, > > otherwise the filesystem is mounted RO mode. > > > > Signed-off-by: Andrey Zhizhikin > > --- > > arch/arm/configs/socfpga_defconfig | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/arch/arm/configs/socfpga_defconfig > > b/arch/arm/configs/socfpga_defconfig > > index 08d1b3e11d68..b4b387486cd9 100644 > > --- a/arch/arm/configs/socfpga_defconfig > > +++ b/arch/arm/configs/socfpga_defconfig > > @@ -21,7 +21,7 @@ CONFIG_NEON=y > > CONFIG_OPROFILE=y > > CONFIG_MODULES=y > > CONFIG_MODULE_UNLOAD=y > > -# CONFIG_LBDAF is not set > > CONFIG_LDBAF is turned on by default for 32-bit. So all you have to do > is remove this line. Thanks a lot for clarifications here! I suspected that it was the case, just needed a confirmation here. Is it possible we would have this config option enabled for socfpga_defconfig? If you would agree, then I prepare a separate patch to have the "CONFIG_LBDAF is not set" removed. > > Thanks, > Dinh Thanks a lot! -- Andrey.
[PATCH] ARM: socfpga_defconfig: enable support for large block devices
Enable CONFIG_LBDAF, which is required by ext4 fs. This option could hanle both ext3 and ext4, and ex4 requires this option to be enabled, otherwise the filesystem is mounted RO mode. Signed-off-by: Andrey Zhizhikin --- arch/arm/configs/socfpga_defconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/configs/socfpga_defconfig b/arch/arm/configs/socfpga_defconfig index 08d1b3e11d68..b4b387486cd9 100644 --- a/arch/arm/configs/socfpga_defconfig +++ b/arch/arm/configs/socfpga_defconfig @@ -21,7 +21,7 @@ CONFIG_NEON=y CONFIG_OPROFILE=y CONFIG_MODULES=y CONFIG_MODULE_UNLOAD=y -# CONFIG_LBDAF is not set +CONFIG_LBDAF=y # CONFIG_BLK_DEV_BSG is not set CONFIG_NET=y CONFIG_PACKET=y -- 2.17.1
Re: [PATCH v2 2/2] uio: Introduce UIO driver dt-binding documentation
Hello Rob, Thanks for your reply here! > > No. See prior discussions: > > https://lkml.org/lkml/2016/5/18/457 > https://lkml.org/lkml/2017/10/8/238 Just for my clarify; to understand how to move on with the patch: Since UIO is not considered as a real HW and rather a aggregate, which could be used for to wrap virtually any HW block - does that mean no new dt-bindings would be accepted to it? I agree it is really hard (and practically impossible) to create compatible strings for all HW units potentially using UIO as a container, therefore I can see clearly arguments here. But if considered from a driver perspective: there is already DT awareness in the UIO driver (specifically the pdrv-genirq), and we have a "HW-like" functionality (irq handler, iomem and ioreg regions) which could be covered by the DT bindings. What if those "generic" properties that covers only those pieces of the UIO driver code could be assigned corresponding dt-bindings? I believe it is not the best and rather contradictory approach to not have any "compatible" string, but as long as those driver parameters are covered by DT properties, isn't it OK to have them? I can assume a lot of people are using UIO in exactly this way: defining a node in their DTS, assigning "compatible" via kernel command line and having devnode instantiated. Why can't a possibility be provided to them to have a generic description of how they can configure, tweak and use their UIO drivers in the correct and efficient way? This is especially true for people that are using FPGAs and enveloping their Soft-IPs with UIO to have a generic status/control capabilities. I cannot imagine how this tremendous amount of variations could be easily accommodated inside UIO compatible names... Or am I completely missing a point here? -- Regards, Andrey.
[PATCH v2 1/2] uio: Allow to take irq bottom-half into irq_handler with additional dt-binding
Certain Kernel preemption models are using threaded interrupt handlers, which is in general quite beneficial. However, threaded handlers introducing additional scheduler overhead, when the bottom-half thread should be woken up and scheduled for execution. This can result is additional latency, which in certain cases is not desired. UIO driver with Generic IRQ handler, that wraps a HW block might suffer a small degradation when it's bottom half is executed, since it needs its bottom half to be woken up by the scheduler every time INT is delivered. For high rate INT signals, this also bring additional undesired load on the scheduler itself. Since the actual ACK is performed in the top-half, and bottom-half of the UIO driver with Generic IRQ handler is relatively slick (only flag is set based on the INT reception), it might be beneficial to move this bottom-half to the irq_handler itself, rather than to have a separate thread to service it. This patch aims to address the task above by supplying IRQF_NO_THREAD to request_irq(), based on dt-binding which could be configured on a per-node basis. That means developers utilizing the UIO driver could decide which UIO instance is critical in terms of interrupt processing, and move their corresponding bottom-halves to the irq_handler to fight additional scheduling latency. When additional property is not found in corresponding dt-node, then instance behavior is not amended and overall system stays with default configuration for threaded IRQ (depending on how they are configured by Preemption model). Patch was originated on the ARM-based system with Kernel configuration CONFIG_PREEMPT_RT_FULL set, which effectively promotes all bottom-halves to threaded interrupt handlers. Once this patch has been enabled on 2 individual uio device nodes (out of around 20 registered in the system), no additional negative impact has been noted on the system overall. Having this patch enabled for individual UIO node allowed to have a latency reduction of around 20-30 usec from INT trigger to the user space IRQ handler. Those results can vary based on the platform and CPU architecture, but could be quite beneficial if above gain in comparable to the worst-case latency figures. This modification also brings several additional benefits: - It eliminates few re-scheduling operations, making INT ACK code more robust and relieves the pressure from the scheduler when HW interrupt for this IRQ is signaled at a high-enough frequency; - It makes top and bottom half to be executed back-to-back with IRQ OFF, making operation pseudo-atomic; - Above gain might be significant when average latency times for the systems are comparable Signed-off-by: Andrey Zhizhikin diff --git a/drivers/uio/uio_pdrv_genirq.c b/drivers/uio/uio_pdrv_genirq.c index f598ecd..86427a4 100644 --- a/drivers/uio/uio_pdrv_genirq.c +++ b/drivers/uio/uio_pdrv_genirq.c @@ -108,6 +108,7 @@ static int uio_pdrv_genirq_probe(struct platform_device *pdev) struct uio_pdrv_genirq_platdata *priv; struct uio_mem *uiomem; int ret = -EINVAL; + int no_threaded_irq = 0; int i; if (pdev->dev.of_node) { @@ -121,6 +122,14 @@ static int uio_pdrv_genirq_probe(struct platform_device *pdev) uioinfo->name = pdev->dev.of_node->name; uioinfo->version = "devicetree"; /* Multiple IRQs are not supported */ + + /* read additional property (if exists) and decide whether +* to have IRQ bottom half to be executed in a separate +* thread, or to have it executed in the irq_handler +* context +*/ + if (of_property_read_bool(pdev->dev.of_node, "no-threaded-irq")) + no_threaded_irq = 1; } if (!uioinfo || !uioinfo->name || !uioinfo->version) { @@ -134,6 +143,12 @@ static int uio_pdrv_genirq_probe(struct platform_device *pdev) return ret; } + /* execute BH in irq_handler if property set in FDT */ + if ((no_threaded_irq > 0) && !(uioinfo->irq_flags & IRQF_NO_THREAD)) { + dev_info(&pdev->dev, "promoting INT with IRQF_NO_THREAD\n"); + uioinfo->irq_flags |= IRQF_NO_THREAD; + } + priv = devm_kzalloc(&pdev->dev, sizeof(*priv), GFP_KERNEL); if (!priv) { dev_err(&pdev->dev, "unable to kmalloc\n"); -- 2.7.4
[PATCH v2 0/2] uio: Allow to take irq bottom-half into irq_handler with additional dt-binding
This patch series aimed to introduce additional feature for UIO driver with generic interrupt handler to allow IRQ bottom half to be executed in irq_handler context rather than as threaded IRQ. Andrey Zhizhikin (2): uio: Allow to take irq bottom-half into irq_handler with additional dt-binding uio: Introduce UIO driver dt-binding documentation .../devicetree/bindings/uio/uio-pdrv-genirq.txt| 46 ++ drivers/uio/uio_pdrv_genirq.c | 15 +++ 2 files changed, 61 insertions(+) create mode 100644 Documentation/devicetree/bindings/uio/uio-pdrv-genirq.txt -- 2.7.4
[PATCH v2 2/2] uio: Introduce UIO driver dt-binding documentation
Create Documentation portion of UIO driver with Generic Interrupt Handler. This patch creates a dt-binding documentation portion of the UIO Driver. In addition to definition of standard required properties, new optional property defined: - no-threaded-irq: when present, request_irq() is called with IRQF_NO_THREAD flag set, effectively skipping threaded interrupt handler and taking bottom-half into irq_handler Signed-off-by: Andrey Zhizhikin diff --git a/Documentation/devicetree/bindings/uio/uio-pdrv-genirq.txt b/Documentation/devicetree/bindings/uio/uio-pdrv-genirq.txt new file mode 100644 index 000..dfcc362 --- /dev/null +++ b/Documentation/devicetree/bindings/uio/uio-pdrv-genirq.txt @@ -0,0 +1,46 @@ +* Userspace I/O platform driver with generic IRQ handling code. + +Platform driver for User-space IO handling with generic IRQ code. +This driver is very similar to the regular UIO platform driver, but is +only suitable for devices that are connected to the interrupt +controller using unique interrupt lines. + +Required properties: +- compatible: Driver compatible string. For UIO device with generic IRQ + handling code this property is defined in device tree to any desired + name, and then set via module parameters passed to Kernel command line + in a form: + uio_pdrv_genirq.of_id= + +- reg: Physical base address and size for memory mapped region to be accessed + via UIO driver. + +- interrupts: Platform IRQ standard definition. For details on defining this + property, see interrupt-controller/interrupts.txt + +- interrupt-parent: Parent interrupt controller node. + For details, see interrupt-controller/interrupts.txt + +Optional properties: +- no-threaded-irq: when present, request_irq() is called with IRQF_NO_THREAD + flag set, effectively skipping threaded interrupt handler and taking + bottom-half into irq_handler + + +Examples: + +base-uio { + compatible = "platform-uio"; + reg = < 0xC000 0x0002 >; + interrupts = < 0 10 1 >; + interrupt-parent = <&intc>; +}; + +nothreaded-uio { + compatible = "platform-uio"; + reg = < 0xC004 0x0002 >; + interrupts = < 0 27 1 >; + interrupt-parent = <&intc>; + no-threaded-irq; +}; + -- 2.7.4
Re: [PATCH] uio: Allow to take irq bottom-half into irq_handler with additional dt-binding
On Thu, Dec 7, 2017 at 10:16 AM, Greg KH wrote: > > A: Because it messes up the order in which people normally read text. > Q: Why is top-posting such a bad thing? > A: Top-posting. > Q: What is the most annoying thing in e-mail? > > A: No. > Q: Should I include quotations after my reply? > > http://daringfireball.net/2007/07/on_top Understood, rules are golden! :) > > The uio-phys driver does use DT bindings, so perhaps look at how those > are defined. UIO in general uses standard bindings (like "compatible", "reg", "interrupt"), but the documentation part is completely missing. What I was able to find in Kernel tree is a HOW-TO from Hans-Juergen Koch dated 2006 and kobject reference on Embedding kobjects with UIO as an example. I've created a new binding document under Documentation/devicetree/bindings/uio/ and would submit it together with updated patch in series. If anyone happen to know which place I missed in the Kernel to look at for uio dt-binding Documentation part - please point me out. > > Ok, that all seems like a good thing to have the ability to do here, you > should mention it in the changelog text when you redo this patch. Would be done! I am planning to have all above corrected and re-submit the patch as series with Documentation part included. Cheers, Andrey.
Re: [PATCH] uio: Allow to take irq bottom-half into irq_handler with additional dt-binding
Hello Greg, Thanks a lot for your prompt reply! First of, this is my first patch submission to the Kernel, so thanks a lot for your additional guidelines here regarding missing pieces. Please don't judge me hard here. :) I would add new DT bindings to Documentation and contact DT maintainers to have a new binding discussed. However, I was not able to find any dt-binding documentation for uio drivers in the kernel, presumably I would have to create a new entry there... As for the win against latency and running the patch against the system which has all IRQ in threaded mode: I've actually originated this patch based on the PREEMPT_RT kernel configuration, where all IRQs are threaded. I have ARM-based system running around 20 genirq UIO instances, and was demoting 2 of those from threaded to non-threaded IRQ handlers without any issues recorded to all the IRQ handlers. This patch actually is aimed exactly with the logic that if new property is not found - then system behavior is not amended, and all IRQs stays threaded. If needed, then a developer can enable this property for it's node, but then he should be well-aware of what this property implications are. In average, using ftrace and kernelshark to analyze I observed the gain of 20-30 usec per chain: irq_handler_entry -> irq/XX-uio -> so I would say the gain is not very significant for average user-space task. However IMHO, there are several hidden benefits here with having this modification, namely: - It eliminates few re-scheduling operations, making INT ACK code more robust and relieves the pressure from the scheduler when HW interrupt for this IRQ is signaled at a high-enough frequency; - It makes top and bottom half to be executed back-to-back with IRQ OFF, making operation pseudo-atomic; - Above gain might be significant when average latency times for the systems are comparable. I do have a worst-case latency measured with cyclictest here at 50 usec, so as a developer I would consider to have above gain in my system. :) Please let me know what you think on those points, if they all make sense to you - otherwise I can drop this patch out. -- Regards, Andrey. On Wed, Dec 6, 2017 at 4:31 PM, Greg KH wrote: > On Wed, Dec 06, 2017 at 03:55:40PM +0100, Andrey Zhizhikin wrote: >> Certain Kernel preemption models are using threaded interrupt handlers, >> which is in general quite beneficial. However, threaded handlers >> introducing additional scheduler overhead, when the bottom-half thread >> should be woken up and scheduled for execution. This can result is >> additional latency, which in certain cases is not desired. >> >> UIO driver with Generic IRQ handler, that wraps a HW block might suffer >> a small degradation when it's bottom half is executed, since it needs >> its bottom half to be woken up by the scheduler every time INT is >> delivered. For high rate INT signals, this also bring additional >> undesired load on the scheduler itself. >> >> Since the actual ACK is performed in the top-half, and bottom-half of >> the UIO driver with Generic IRQ handler is relatively slick (only flag >> is set based on the INT reception), it might be beneficial to move this >> bottom-half to the irq_handler itself, rather than to have a separate >> thread to service it. >> >> This patch aims to address the task above, and in addition introduces >> a new dt-binding which could be configured on a per-node basis. That >> means developers utilizing the UIO driver could decide which UIO >> instance is critical in terms of interrupt processing, and move their >> corresponding bottom-halves to the irq_handler to fight additional >> scheduling latency. >> >> New DT binding: >> - no-threaded-irq: when present, request_irq() is called with >> IRQF_NO_THREAD flag set, effectively skipping threaded interrupt >> handler and taking bottom-half into irq_handler >> >> Signed-off-by: Andrey Zhizhikin > > For new DT bindings, don't you have to add them to the in-kernel > documentation and get an ack from the DT maintainers? Please do that > here. > > ALso, how much does this really save in latency/delay by not allowing a > threaded irq? What about systems that run all irqs in threaded mode? > Will that break something here? > > thanks, > > greg k-h
[PATCH] uio: Allow to take irq bottom-half into irq_handler with additional dt-binding
Certain Kernel preemption models are using threaded interrupt handlers, which is in general quite beneficial. However, threaded handlers introducing additional scheduler overhead, when the bottom-half thread should be woken up and scheduled for execution. This can result is additional latency, which in certain cases is not desired. UIO driver with Generic IRQ handler, that wraps a HW block might suffer a small degradation when it's bottom half is executed, since it needs its bottom half to be woken up by the scheduler every time INT is delivered. For high rate INT signals, this also bring additional undesired load on the scheduler itself. Since the actual ACK is performed in the top-half, and bottom-half of the UIO driver with Generic IRQ handler is relatively slick (only flag is set based on the INT reception), it might be beneficial to move this bottom-half to the irq_handler itself, rather than to have a separate thread to service it. This patch aims to address the task above, and in addition introduces a new dt-binding which could be configured on a per-node basis. That means developers utilizing the UIO driver could decide which UIO instance is critical in terms of interrupt processing, and move their corresponding bottom-halves to the irq_handler to fight additional scheduling latency. New DT binding: - no-threaded-irq: when present, request_irq() is called with IRQF_NO_THREAD flag set, effectively skipping threaded interrupt handler and taking bottom-half into irq_handler Signed-off-by: Andrey Zhizhikin diff --git a/drivers/uio/uio_pdrv_genirq.c b/drivers/uio/uio_pdrv_genirq.c index f598ecd..86427a4 100644 --- a/drivers/uio/uio_pdrv_genirq.c +++ b/drivers/uio/uio_pdrv_genirq.c @@ -108,6 +108,7 @@ static int uio_pdrv_genirq_probe(struct platform_device *pdev) struct uio_pdrv_genirq_platdata *priv; struct uio_mem *uiomem; int ret = -EINVAL; + int no_threaded_irq = 0; int i; if (pdev->dev.of_node) { @@ -121,6 +122,14 @@ static int uio_pdrv_genirq_probe(struct platform_device *pdev) uioinfo->name = pdev->dev.of_node->name; uioinfo->version = "devicetree"; /* Multiple IRQs are not supported */ + + /* read additional property (if exists) and decide whether +* to have IRQ bottom half to be executed in a separate +* thread, or to have it executed in the irq_handler +* context +*/ + if (of_property_read_bool(pdev->dev.of_node, "no-threaded-irq")) + no_threaded_irq = 1; } if (!uioinfo || !uioinfo->name || !uioinfo->version) { @@ -134,6 +143,12 @@ static int uio_pdrv_genirq_probe(struct platform_device *pdev) return ret; } + /* execute BH in irq_handler if property set in FDT */ + if ((no_threaded_irq > 0) && !(uioinfo->irq_flags & IRQF_NO_THREAD)) { + dev_info(&pdev->dev, "promoting INT with IRQF_NO_THREAD\n"); + uioinfo->irq_flags |= IRQF_NO_THREAD; + } + priv = devm_kzalloc(&pdev->dev, sizeof(*priv), GFP_KERNEL); if (!priv) { dev_err(&pdev->dev, "unable to kmalloc\n"); -- 2.7.4