Re: [PATCH v2] ARM: configs: Remove useless UEVENT_HELPER_PATH

2019-07-05 Thread Olof Johansson 
On Fri, Jul 5, 2019 at 3:26 PM Olof Johansson via Linux.Kernel.Org
 wrote:

This didn't work as I anticipated. Please ignore, apologies for the spam.


-Olof

Re: [PATCH v2] ARM: configs: Remove useless UEVENT_HELPER_PATH

2019-06-19 Thread Olof Johansson 
On Tue, Jun 04, 2019 at 10:14:53AM +0200, Krzysztof Kozlowski wrote:
> Remove the CONFIG_UEVENT_HELPER_PATH because:
> 1. It is disabled since commit 1be01d4a5714 ("driver: base: Disable
>CONFIG_UEVENT_HELPER by default") as its dependency (UEVENT_HELPER) was
>made default to 'n',
> 2. It is not recommended (help message: "This should not be used today
>[...] creates a high system load") and was kept only for ancient
>userland,
> 3. Certain userland specifically requests it to be disabled (systemd
>README: "Legacy hotplug slows down the system and confuses udev").
> 
> Signed-off-by: Krzysztof Kozlowski 
> Acked-by: Geert Uytterhoeven 
> 
> ---
> 
> Changes since v2:
> 1. Remove unrelated files.
> 2. Add Geert's ack.

So your other patch added mini2440 in a follow-up patch, but this one doesn't?!

Applied with that fixup.


-Olof

Re: [PATCH v2] ARM: configs: Remove useless UEVENT_HELPER_PATH

2019-06-06 Thread Dinh Nguyen 



On 6/4/19 3:14 AM, Krzysztof Kozlowski wrote:
> Remove the CONFIG_UEVENT_HELPER_PATH because:
> 1. It is disabled since commit 1be01d4a5714 ("driver: base: Disable
>CONFIG_UEVENT_HELPER by default") as its dependency (UEVENT_HELPER) was
>made default to 'n',
> 2. It is not recommended (help message: "This should not be used today
>[...] creates a high system load") and was kept only for ancient
>userland,
> 3. Certain userland specifically requests it to be disabled (systemd
>README: "Legacy hotplug slows down the system and confuses udev").
> 
> Signed-off-by: Krzysztof Kozlowski 
> Acked-by: Geert Uytterhoeven 
> 

> diff --git a/arch/arm/configs/socfpga_defconfig 
> b/arch/arm/configs/socfpga_defconfig
> index 6701a975e785..fe2e1e82e233 100644
> --- a/arch/arm/configs/socfpga_defconfig
> +++ b/arch/arm/configs/socfpga_defconfig
> @@ -44,7 +44,6 @@ CONFIG_PCI=y
>  CONFIG_PCI_MSI=y
>  CONFIG_PCIE_ALTERA=y
>  CONFIG_PCIE_ALTERA_MSI=y
> -CONFIG_UEVENT_HELPER_PATH="/sbin/hotplug"
>  CONFIG_DEVTMPFS=y
>  CONFIG_DEVTMPFS_MOUNT=y
>  CONFIG_MTD=y

For (socfpga_defconfig):

Acked-by: Dinh Nguyen

Re: [PATCH v2] ARM: configs: Remove useless UEVENT_HELPER_PATH

2019-06-06 Thread Nicolas.Ferre 
On 04/06/2019 at 10:14, Krzysztof Kozlowski wrote:
> Remove the CONFIG_UEVENT_HELPER_PATH because:
> 1. It is disabled since commit 1be01d4a5714 ("driver: base: Disable
> CONFIG_UEVENT_HELPER by default") as its dependency (UEVENT_HELPER) was
> made default to 'n',
> 2. It is not recommended (help message: "This should not be used today
> [...] creates a high system load") and was kept only for ancient
> userland,
> 3. Certain userland specifically requests it to be disabled (systemd
> README: "Legacy hotplug slows down the system and confuses udev").
> 
> Signed-off-by: Krzysztof Kozlowski 
> Acked-by: Geert Uytterhoeven 
> 
> ---
> 
> Changes since v2:
> 1. Remove unrelated files.
> 2. Add Geert's ack.
> ---
>   arch/arm/configs/acs5k_defconfig  | 1 -
>   arch/arm/configs/acs5k_tiny_defconfig | 1 -
>   arch/arm/configs/am200epdkit_defconfig| 1 -
>   arch/arm/configs/aspeed_g4_defconfig  | 1 -
>   arch/arm/configs/aspeed_g5_defconfig  | 1 -
>   arch/arm/configs/at91_dt_defconfig| 1 -

[..]

>   arch/arm/configs/sama5_defconfig  | 1 -

for at91 (at91_dt_defconfig & sama5_defconfig):
Acked-by: Nicolas Ferre 

>   arch/arm/configs/socfpga_defconfig| 1 -
>   arch/arm/configs/spear13xx_defconfig  | 1 -
>   arch/arm/configs/spear3xx_defconfig   | 1 -
>   arch/arm/configs/spear6xx_defconfig   | 1 -
>   arch/arm/configs/spitz_defconfig  | 1 -
>   arch/arm/configs/tango4_defconfig | 1 -
>   arch/arm/configs/tct_hammer_defconfig | 1 -
>   arch/arm/configs/u300_defconfig   | 1 -
>   arch/arm/configs/u8500_defconfig  | 1 -
>   arch/arm/configs/vexpress_defconfig   | 1 -
>   arch/arm/configs/viper_defconfig  | 1 -
>   arch/arm/configs/xcep_defconfig   | 1 -
>   arch/arm/configs/zeus_defconfig   | 1 -
>   arch/arm/configs/zx_defconfig | 1 -
>   69 files changed, 69 deletions(-)
[..]
-- 
Nicolas Ferre

Re: [PATCH v2] ARM: configs: Remove useless UEVENT_HELPER_PATH

2019-06-05 Thread Shawn Guo 
On Tue, Jun 04, 2019 at 10:14:53AM +0200, Krzysztof Kozlowski wrote:
> Remove the CONFIG_UEVENT_HELPER_PATH because:
> 1. It is disabled since commit 1be01d4a5714 ("driver: base: Disable
>CONFIG_UEVENT_HELPER by default") as its dependency (UEVENT_HELPER) was
>made default to 'n',
> 2. It is not recommended (help message: "This should not be used today
>[...] creates a high system load") and was kept only for ancient
>userland,
> 3. Certain userland specifically requests it to be disabled (systemd
>README: "Legacy hotplug slows down the system and confuses udev").
> 
> Signed-off-by: Krzysztof Kozlowski 
> Acked-by: Geert Uytterhoeven 
> 
> ---
> 
> Changes since v2:
> 1. Remove unrelated files.
> 2. Add Geert's ack.
> ---
...
>  arch/arm/configs/imx_v4_v5_defconfig  | 1 -

Acked-by: Shawn Guo

Re: [PATCH v2] ARM: configs: Remove useless UEVENT_HELPER_PATH

2019-06-04 Thread Andrew Jeffery 



On Tue, 4 Jun 2019, at 17:45, Krzysztof Kozlowski wrote:
> Remove the CONFIG_UEVENT_HELPER_PATH because:
> 1. It is disabled since commit 1be01d4a5714 ("driver: base: Disable
>CONFIG_UEVENT_HELPER by default") as its dependency (UEVENT_HELPER) was
>made default to 'n',
> 2. It is not recommended (help message: "This should not be used today
>[...] creates a high system load") and was kept only for ancient
>userland,
> 3. Certain userland specifically requests it to be disabled (systemd
>README: "Legacy hotplug slows down the system and confuses udev").
> 
> Signed-off-by: Krzysztof Kozlowski 
> Acked-by: Geert Uytterhoeven 
> 
> ---
> 
> Changes since v2:
> 1. Remove unrelated files.
> 2. Add Geert's ack.
> ---
>  arch/arm/configs/acs5k_defconfig  | 1 -
>  arch/arm/configs/acs5k_tiny_defconfig | 1 -
>  arch/arm/configs/am200epdkit_defconfig| 1 -
>  arch/arm/configs/aspeed_g4_defconfig  | 1 -
>  arch/arm/configs/aspeed_g5_defconfig  | 1 -
>  arch/arm/configs/at91_dt_defconfig| 1 -
>  arch/arm/configs/axm55xx_defconfig| 1 -
>  arch/arm/configs/cm_x2xx_defconfig| 1 -
>  arch/arm/configs/cm_x300_defconfig| 1 -
>  arch/arm/configs/cns3420vb_defconfig  | 1 -
>  arch/arm/configs/colibri_pxa270_defconfig | 1 -
>  arch/arm/configs/colibri_pxa300_defconfig | 1 -
>  arch/arm/configs/corgi_defconfig  | 1 -
>  arch/arm/configs/dove_defconfig   | 1 -
>  arch/arm/configs/em_x270_defconfig| 1 -
>  arch/arm/configs/ep93xx_defconfig | 1 -
>  arch/arm/configs/eseries_pxa_defconfig| 1 -
>  arch/arm/configs/ezx_defconfig| 1 -
>  arch/arm/configs/gemini_defconfig | 1 -
>  arch/arm/configs/h3600_defconfig  | 1 -
>  arch/arm/configs/h5000_defconfig  | 1 -
>  arch/arm/configs/imote2_defconfig | 1 -
>  arch/arm/configs/imx_v4_v5_defconfig  | 1 -
>  arch/arm/configs/iop13xx_defconfig| 1 -
>  arch/arm/configs/iop32x_defconfig | 1 -
>  arch/arm/configs/iop33x_defconfig | 1 -
>  arch/arm/configs/ixp4xx_defconfig | 1 -
>  arch/arm/configs/jornada720_defconfig | 1 -
>  arch/arm/configs/keystone_defconfig   | 1 -
>  arch/arm/configs/ks8695_defconfig | 1 -
>  arch/arm/configs/lpc32xx_defconfig| 1 -
>  arch/arm/configs/magician_defconfig   | 1 -
>  arch/arm/configs/moxart_defconfig | 1 -
>  arch/arm/configs/multi_v5_defconfig   | 1 -
>  arch/arm/configs/mv78xx0_defconfig| 1 -
>  arch/arm/configs/mvebu_v5_defconfig   | 1 -
>  arch/arm/configs/mvebu_v7_defconfig   | 1 -
>  arch/arm/configs/nhk8815_defconfig| 1 -
>  arch/arm/configs/nuc910_defconfig | 1 -
>  arch/arm/configs/nuc950_defconfig | 1 -
>  arch/arm/configs/nuc960_defconfig | 1 -
>  arch/arm/configs/omap1_defconfig  | 1 -
>  arch/arm/configs/orion5x_defconfig| 1 -
>  arch/arm/configs/palmz72_defconfig| 1 -
>  arch/arm/configs/pcm027_defconfig | 1 -
>  arch/arm/configs/prima2_defconfig | 1 -
>  arch/arm/configs/pxa168_defconfig | 1 -
>  arch/arm/configs/pxa3xx_defconfig | 1 -
>  arch/arm/configs/pxa910_defconfig | 1 -
>  arch/arm/configs/pxa_defconfig| 1 -
>  arch/arm/configs/realview_defconfig   | 1 -
>  arch/arm/configs/s3c2410_defconfig| 1 -
>  arch/arm/configs/s3c6400_defconfig| 1 -
>  arch/arm/configs/s5pv210_defconfig| 1 -
>  arch/arm/configs/sama5_defconfig  | 1 -
>  arch/arm/configs/socfpga_defconfig| 1 -
>  arch/arm/configs/spear13xx_defconfig  | 1 -
>  arch/arm/configs/spear3xx_defconfig   | 1 -
>  arch/arm/configs/spear6xx_defconfig   | 1 -
>  arch/arm/configs/spitz_defconfig  | 1 -
>  arch/arm/configs/tango4_defconfig | 1 -
>  arch/arm/configs/tct_hammer_defconfig | 1 -
>  arch/arm/configs/u300_defconfig   | 1 -
>  arch/arm/configs/u8500_defconfig  | 1 -
>  arch/arm/configs/vexpress_defconfig   | 1 -
>  arch/arm/configs/viper_defconfig  | 1 -
>  arch/arm/configs/xcep_defconfig   | 1 -
>  arch/arm/configs/zeus_defconfig   | 1 -
>  arch/arm/configs/zx_defconfig | 1 -
>  69 files changed, 69 deletions(-)
> 
> diff --git a/arch/arm/configs/acs5k_defconfig 
> b/arch/arm/configs/acs5k_defconfig
> index d04ee19e5b75..bcb8bda09158 100644
> --- a/arch/arm/configs/acs5k_defconfig
> +++ b/arch/arm/configs/acs5k_defconfig
> @@ -30,7 +30,6 @@ CONFIG_INET=y
>  CONFIG_IP_PNP=y
>  CONFIG_IP_PNP_DHCP=y
>  # CONFIG_IPV6 is not set
> -CONFIG_UEVENT_HELPER_PATH="/sbin/hotplug"
>  CONFIG_MTD=y
>  CONFIG_MTD_BLOCK=y
>  CONFIG_MTD_CFI=y
> diff --git a/arch/arm/configs/acs5k_tiny_defconfig 
> b/arch/arm/configs/acs5k_tiny_defconfig
> index 25c593df41d1..e802cdebfd0b 100644
> --- a/arch/arm/configs/acs5k_tiny_defconfig
> +++ b/arch/arm/configs/acs5k_tiny_defconfig
> @@ -25,7 +25,6 @@ CONFIG_INET=y
>  #

Re: [PATCH v2] ARM: configs: Remove useless UEVENT_HELPER_PATH

2019-06-04 Thread Sudeep Holla 
On Tue, Jun 04, 2019 at 10:14:53AM +0200, Krzysztof Kozlowski wrote:
> Remove the CONFIG_UEVENT_HELPER_PATH because:
> 1. It is disabled since commit 1be01d4a5714 ("driver: base: Disable
>CONFIG_UEVENT_HELPER by default") as its dependency (UEVENT_HELPER) was
>made default to 'n',
> 2. It is not recommended (help message: "This should not be used today
>[...] creates a high system load") and was kept only for ancient
>userland,
> 3. Certain userland specifically requests it to be disabled (systemd
>README: "Legacy hotplug slows down the system and confuses udev").
> 
> Signed-off-by: Krzysztof Kozlowski 
> Acked-by: Geert Uytterhoeven 
> 
> ---
> 
> Changes since v2:
> 1. Remove unrelated files.
> 2. Add Geert's ack.
> ---

[...]

> diff --git a/arch/arm/configs/vexpress_defconfig 
> b/arch/arm/configs/vexpress_defconfig
> index d170da388389..25753552277a 100644
> --- a/arch/arm/configs/vexpress_defconfig
> +++ b/arch/arm/configs/vexpress_defconfig
> @@ -45,7 +45,6 @@ CONFIG_IP_PNP_BOOTP=y
>  # CONFIG_WIRELESS is not set
>  CONFIG_NET_9P=y
>  CONFIG_NET_9P_VIRTIO=y
> -CONFIG_UEVENT_HELPER_PATH="/sbin/hotplug"
>  CONFIG_DEVTMPFS=y
>  CONFIG_DMA_CMA=y
>  CONFIG_MTD=y

(for vexpress)

Acked-by: Sudeep Holla 

--
Regards,
Sudeep

[PATCH v2] ARM: configs: Remove useless UEVENT_HELPER_PATH

2019-06-04 Thread Krzysztof Kozlowski 
Remove the CONFIG_UEVENT_HELPER_PATH because:
1. It is disabled since commit 1be01d4a5714 ("driver: base: Disable
   CONFIG_UEVENT_HELPER by default") as its dependency (UEVENT_HELPER) was
   made default to 'n',
2. It is not recommended (help message: "This should not be used today
   [...] creates a high system load") and was kept only for ancient
   userland,
3. Certain userland specifically requests it to be disabled (systemd
   README: "Legacy hotplug slows down the system and confuses udev").

Signed-off-by: Krzysztof Kozlowski 
Acked-by: Geert Uytterhoeven 

---

Changes since v2:
1. Remove unrelated files.
2. Add Geert's ack.
---
 arch/arm/configs/acs5k_defconfig  | 1 -
 arch/arm/configs/acs5k_tiny_defconfig | 1 -
 arch/arm/configs/am200epdkit_defconfig| 1 -
 arch/arm/configs/aspeed_g4_defconfig  | 1 -
 arch/arm/configs/aspeed_g5_defconfig  | 1 -
 arch/arm/configs/at91_dt_defconfig| 1 -
 arch/arm/configs/axm55xx_defconfig| 1 -
 arch/arm/configs/cm_x2xx_defconfig| 1 -
 arch/arm/configs/cm_x300_defconfig| 1 -
 arch/arm/configs/cns3420vb_defconfig  | 1 -
 arch/arm/configs/colibri_pxa270_defconfig | 1 -
 arch/arm/configs/colibri_pxa300_defconfig | 1 -
 arch/arm/configs/corgi_defconfig  | 1 -
 arch/arm/configs/dove_defconfig   | 1 -
 arch/arm/configs/em_x270_defconfig| 1 -
 arch/arm/configs/ep93xx_defconfig | 1 -
 arch/arm/configs/eseries_pxa_defconfig| 1 -
 arch/arm/configs/ezx_defconfig| 1 -
 arch/arm/configs/gemini_defconfig | 1 -
 arch/arm/configs/h3600_defconfig  | 1 -
 arch/arm/configs/h5000_defconfig  | 1 -
 arch/arm/configs/imote2_defconfig | 1 -
 arch/arm/configs/imx_v4_v5_defconfig  | 1 -
 arch/arm/configs/iop13xx_defconfig| 1 -
 arch/arm/configs/iop32x_defconfig | 1 -
 arch/arm/configs/iop33x_defconfig | 1 -
 arch/arm/configs/ixp4xx_defconfig | 1 -
 arch/arm/configs/jornada720_defconfig | 1 -
 arch/arm/configs/keystone_defconfig   | 1 -
 arch/arm/configs/ks8695_defconfig | 1 -
 arch/arm/configs/lpc32xx_defconfig| 1 -
 arch/arm/configs/magician_defconfig   | 1 -
 arch/arm/configs/moxart_defconfig | 1 -
 arch/arm/configs/multi_v5_defconfig   | 1 -
 arch/arm/configs/mv78xx0_defconfig| 1 -
 arch/arm/configs/mvebu_v5_defconfig   | 1 -
 arch/arm/configs/mvebu_v7_defconfig   | 1 -
 arch/arm/configs/nhk8815_defconfig| 1 -
 arch/arm/configs/nuc910_defconfig | 1 -
 arch/arm/configs/nuc950_defconfig | 1 -
 arch/arm/configs/nuc960_defconfig | 1 -
 arch/arm/configs/omap1_defconfig  | 1 -
 arch/arm/configs/orion5x_defconfig| 1 -
 arch/arm/configs/palmz72_defconfig| 1 -
 arch/arm/configs/pcm027_defconfig | 1 -
 arch/arm/configs/prima2_defconfig | 1 -
 arch/arm/configs/pxa168_defconfig | 1 -
 arch/arm/configs/pxa3xx_defconfig | 1 -
 arch/arm/configs/pxa910_defconfig | 1 -
 arch/arm/configs/pxa_defconfig| 1 -
 arch/arm/configs/realview_defconfig   | 1 -
 arch/arm/configs/s3c2410_defconfig| 1 -
 arch/arm/configs/s3c6400_defconfig| 1 -
 arch/arm/configs/s5pv210_defconfig| 1 -
 arch/arm/configs/sama5_defconfig  | 1 -
 arch/arm/configs/socfpga_defconfig| 1 -
 arch/arm/configs/spear13xx_defconfig  | 1 -
 arch/arm/configs/spear3xx_defconfig   | 1 -
 arch/arm/configs/spear6xx_defconfig   | 1 -
 arch/arm/configs/spitz_defconfig  | 1 -
 arch/arm/configs/tango4_defconfig | 1 -
 arch/arm/configs/tct_hammer_defconfig | 1 -
 arch/arm/configs/u300_defconfig   | 1 -
 arch/arm/configs/u8500_defconfig  | 1 -
 arch/arm/configs/vexpress_defconfig   | 1 -
 arch/arm/configs/viper_defconfig  | 1 -
 arch/arm/configs/xcep_defconfig   | 1 -
 arch/arm/configs/zeus_defconfig   | 1 -
 arch/arm/configs/zx_defconfig | 1 -
 69 files changed, 69 deletions(-)

diff --git a/arch/arm/configs/acs5k_defconfig b/arch/arm/configs/acs5k_defconfig
index d04ee19e5b75..bcb8bda09158 100644
--- a/arch/arm/configs/acs5k_defconfig
+++ b/arch/arm/configs/acs5k_defconfig
@@ -30,7 +30,6 @@ CONFIG_INET=y
 CONFIG_IP_PNP=y
 CONFIG_IP_PNP_DHCP=y
 # CONFIG_IPV6 is not set
-CONFIG_UEVENT_HELPER_PATH="/sbin/hotplug"
 CONFIG_MTD=y
 CONFIG_MTD_BLOCK=y
 CONFIG_MTD_CFI=y
diff --git a/arch/arm/configs/acs5k_tiny_defconfig 
b/arch/arm/configs/acs5k_tiny_defconfig
index 25c593df41d1..e802cdebfd0b 100644
--- a/arch/arm/configs/acs5k_tiny_defconfig
+++ b/arch/arm/configs/acs5k_tiny_defconfig
@@ -25,7 +25,6 @@ CONFIG_INET=y
 # CONFIG_INET_XFRM_MODE_TUNNEL is not set
 # CONFIG_INET_XFRM_MODE_BEET is not set
 # CONFIG_IPV6 is not set
-CONFIG_UEVENT_HELPER_PATH="/sbin/hotplug"
 CONFIG_MTD=y
 CONFIG_MTD_BLOCK=y
 CONFIG_MTD_CFI=y
diff --git a/arch/arm/configs/am200epdkit_defconfig