| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <1474963347.2494.8.camel@pengutronix.de>
Date: Tue, 27 Sep 2016 10:02:27 +0200
From: Philipp Zabel <p.zabel@...gutronix.de>
To: Peter Griffin <peter.griffin@...aro.org>
Cc: linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
kernel@...inux.com, patrice.chotard@...com,
devicetree@...r.kernel.org, lee.jones@...aro.org
Subject: Re: [PATCH 15/19] reset: sti: Remove STiH415/6 reset support
Hi Peter,
Am Mittwoch, den 14.09.2016, 14:27 +0100 schrieb Peter Griffin:
> Support for STiH415/6 SoCs is being removed from the
> kernel because the platforms are obsolete. This patch removes
> the reset drivers for these SoC's.
>
> Signed-off-by: Peter Griffin <peter.griffin@...aro.org>
> Cc: <p.zabel@...gutronix.de>
> ---
> arch/arm/mach-sti/Kconfig | 2 -
> drivers/reset/sti/Kconfig | 8 ---
> drivers/reset/sti/Makefile | 2 -
> drivers/reset/sti/reset-stih415.c | 112 -----------------------------
> drivers/reset/sti/reset-stih416.c | 143 --------------------------------------
> 5 files changed, 267 deletions(-)
> delete mode 100644 drivers/reset/sti/reset-stih415.c
> delete mode 100644 drivers/reset/sti/reset-stih416.c
>
> diff --git a/arch/arm/mach-sti/Kconfig b/arch/arm/mach-sti/Kconfig
> index 119e110..f8eeeff 100644
> --- a/arch/arm/mach-sti/Kconfig
> +++ b/arch/arm/mach-sti/Kconfig
> @@ -28,7 +28,6 @@ if ARCH_STI
> config SOC_STIH415
> bool "STiH415 STMicroelectronics Consumer Electronics family"
> default y
> - select STIH415_RESET
> help
> This enables support for STMicroelectronics Digital Consumer
> Electronics family StiH415 parts, primarily targeted at set-top-box
> @@ -38,7 +37,6 @@ config SOC_STIH415
> config SOC_STIH416
> bool "STiH416 STMicroelectronics Consumer Electronics family"
> default y
> - select STIH416_RESET
> help
> This enables support for STMicroelectronics Digital Consumer
> Electronics family StiH416 parts, primarily targeted at set-top-box
> diff --git a/drivers/reset/sti/Kconfig b/drivers/reset/sti/Kconfig
> index 6131785..71592b5 100644
> --- a/drivers/reset/sti/Kconfig
> +++ b/drivers/reset/sti/Kconfig
> @@ -3,14 +3,6 @@ if ARCH_STI
> config STI_RESET_SYSCFG
> bool
>
> -config STIH415_RESET
> - bool
> - select STI_RESET_SYSCFG
> -
> -config STIH416_RESET
> - bool
> - select STI_RESET_SYSCFG
> -
> config STIH407_RESET
> bool
> select STI_RESET_SYSCFG
> diff --git a/drivers/reset/sti/Makefile b/drivers/reset/sti/Makefile
> index dc85dfb..f9d8241 100644
> --- a/drivers/reset/sti/Makefile
> +++ b/drivers/reset/sti/Makefile
> @@ -1,5 +1,3 @@
> obj-$(CONFIG_STI_RESET_SYSCFG) += reset-syscfg.o
>
> -obj-$(CONFIG_STIH415_RESET) += reset-stih415.o
> -obj-$(CONFIG_STIH416_RESET) += reset-stih416.o
> obj-$(CONFIG_STIH407_RESET) += reset-stih407.o
> diff --git a/drivers/reset/sti/reset-stih415.c b/drivers/reset/sti/reset-stih415.c
> deleted file mode 100644
> index 6f220cd..0000000
> --- a/drivers/reset/sti/reset-stih415.c
> +++ /dev/null
> @@ -1,112 +0,0 @@
> -/*
> - * Copyright (C) 2013 STMicroelectronics (R&D) Limited
> - * Author: Stephen Gallimore <stephen.gallimore@...com>
> - * Author: Srinivas Kandagatla <srinivas.kandagatla@...com>
> - *
> - * This program is free software; you can redistribute it and/or modify
> - * it under the terms of the GNU General Public License as published by
> - * the Free Software Foundation; either version 2 of the License, or
> - * (at your option) any later version.
> - */
> -#include <linux/module.h>
> -#include <linux/of.h>
> -#include <linux/of_platform.h>
> -#include <linux/platform_device.h>
> -
> -#include <dt-bindings/reset/stih415-resets.h>
> -
> -#include "reset-syscfg.h"
> -
> -/*
> - * STiH415 Peripheral powerdown definitions.
> - */
> -static const char stih415_front[] = "st,stih415-front-syscfg";
> -static const char stih415_rear[] = "st,stih415-rear-syscfg";
> -static const char stih415_sbc[] = "st,stih415-sbc-syscfg";
> -static const char stih415_lpm[] = "st,stih415-lpm-syscfg";
> -
> -#define STIH415_PDN_FRONT(_bit) \
> - _SYSCFG_RST_CH(stih415_front, SYSCFG_114, _bit, SYSSTAT_187, _bit)
> -
> -#define STIH415_PDN_REAR(_cntl, _stat) \
> - _SYSCFG_RST_CH(stih415_rear, SYSCFG_336, _cntl, SYSSTAT_384, _stat)
> -
> -#define STIH415_SRST_REAR(_reg, _bit) \
> - _SYSCFG_RST_CH_NO_ACK(stih415_rear, _reg, _bit)
> -
> -#define STIH415_SRST_SBC(_reg, _bit) \
> - _SYSCFG_RST_CH_NO_ACK(stih415_sbc, _reg, _bit)
> -
> -#define STIH415_SRST_FRONT(_reg, _bit) \
> - _SYSCFG_RST_CH_NO_ACK(stih415_front, _reg, _bit)
> -
> -#define STIH415_SRST_LPM(_reg, _bit) \
> - _SYSCFG_RST_CH_NO_ACK(stih415_lpm, _reg, _bit)
> -
> -#define SYSCFG_114 0x38 /* Powerdown request EMI/NAND/Keyscan */
> -#define SYSSTAT_187 0x15c /* Powerdown status EMI/NAND/Keyscan */
> -
> -#define SYSCFG_336 0x90 /* Powerdown request USB/SATA/PCIe */
> -#define SYSSTAT_384 0x150 /* Powerdown status USB/SATA/PCIe */
> -
> -#define SYSCFG_376 0x130 /* Reset generator 0 control 0 */
> -#define SYSCFG_166 0x108 /* Softreset Ethernet 0 */
> -#define SYSCFG_31 0x7c /* Softreset Ethernet 1 */
> -#define LPM_SYSCFG_1 0x4 /* Softreset IRB */
> -
> -static const struct syscfg_reset_channel_data stih415_powerdowns[] = {
> - [STIH415_EMISS_POWERDOWN] = STIH415_PDN_FRONT(0),
> - [STIH415_NAND_POWERDOWN] = STIH415_PDN_FRONT(1),
> - [STIH415_KEYSCAN_POWERDOWN] = STIH415_PDN_FRONT(2),
> - [STIH415_USB0_POWERDOWN] = STIH415_PDN_REAR(0, 0),
> - [STIH415_USB1_POWERDOWN] = STIH415_PDN_REAR(1, 1),
> - [STIH415_USB2_POWERDOWN] = STIH415_PDN_REAR(2, 2),
> - [STIH415_SATA0_POWERDOWN] = STIH415_PDN_REAR(3, 3),
> - [STIH415_SATA1_POWERDOWN] = STIH415_PDN_REAR(4, 4),
> - [STIH415_PCIE_POWERDOWN] = STIH415_PDN_REAR(5, 8),
> -};
> -
> -static const struct syscfg_reset_channel_data stih415_softresets[] = {
> - [STIH415_ETH0_SOFTRESET] = STIH415_SRST_FRONT(SYSCFG_166, 0),
> - [STIH415_ETH1_SOFTRESET] = STIH415_SRST_SBC(SYSCFG_31, 0),
> - [STIH415_IRB_SOFTRESET] = STIH415_SRST_LPM(LPM_SYSCFG_1, 6),
> - [STIH415_USB0_SOFTRESET] = STIH415_SRST_REAR(SYSCFG_376, 9),
> - [STIH415_USB1_SOFTRESET] = STIH415_SRST_REAR(SYSCFG_376, 10),
> - [STIH415_USB2_SOFTRESET] = STIH415_SRST_REAR(SYSCFG_376, 11),
> - [STIH415_KEYSCAN_SOFTRESET] = STIH415_SRST_LPM(LPM_SYSCFG_1, 8),
> -};
> -
> -static struct syscfg_reset_controller_data stih415_powerdown_controller = {
> - .wait_for_ack = true,
> - .nr_channels = ARRAY_SIZE(stih415_powerdowns),
> - .channels = stih415_powerdowns,
> -};
> -
> -static struct syscfg_reset_controller_data stih415_softreset_controller = {
> - .wait_for_ack = false,
> - .active_low = true,
> - .nr_channels = ARRAY_SIZE(stih415_softresets),
> - .channels = stih415_softresets,
> -};
> -
> -static const struct of_device_id stih415_reset_match[] = {
> - { .compatible = "st,stih415-powerdown",
> - .data = &stih415_powerdown_controller, },
> - { .compatible = "st,stih415-softreset",
> - .data = &stih415_softreset_controller, },
> - {},
> -};
> -
> -static struct platform_driver stih415_reset_driver = {
> - .probe = syscfg_reset_probe,
> - .driver = {
> - .name = "reset-stih415",
> - .of_match_table = stih415_reset_match,
> - },
> -};
> -
> -static int __init stih415_reset_init(void)
> -{
> - return platform_driver_register(&stih415_reset_driver);
> -}
> -arch_initcall(stih415_reset_init);
> diff --git a/drivers/reset/sti/reset-stih416.c b/drivers/reset/sti/reset-stih416.c
> deleted file mode 100644
> index c581d606e..0000000
> --- a/drivers/reset/sti/reset-stih416.c
> +++ /dev/null
> @@ -1,143 +0,0 @@
> -/*
> - * Copyright (C) 2013 STMicroelectronics (R&D) Limited
> - * Author: Stephen Gallimore <stephen.gallimore@...com>
> - * Author: Srinivas Kandagatla <srinivas.kandagatla@...com>
> - *
> - * This program is free software; you can redistribute it and/or modify
> - * it under the terms of the GNU General Public License as published by
> - * the Free Software Foundation; either version 2 of the License, or
> - * (at your option) any later version.
> - */
> -#include <linux/module.h>
> -#include <linux/of.h>
> -#include <linux/of_platform.h>
> -#include <linux/platform_device.h>
> -
> -#include <dt-bindings/reset/stih416-resets.h>
> -
> -#include "reset-syscfg.h"
> -
> -/*
> - * STiH416 Peripheral powerdown definitions.
> - */
> -static const char stih416_front[] = "st,stih416-front-syscfg";
> -static const char stih416_rear[] = "st,stih416-rear-syscfg";
> -static const char stih416_sbc[] = "st,stih416-sbc-syscfg";
> -static const char stih416_lpm[] = "st,stih416-lpm-syscfg";
> -static const char stih416_cpu[] = "st,stih416-cpu-syscfg";
> -
> -#define STIH416_PDN_FRONT(_bit) \
> - _SYSCFG_RST_CH(stih416_front, SYSCFG_1500, _bit, SYSSTAT_1578, _bit)
> -
> -#define STIH416_PDN_REAR(_cntl, _stat) \
> - _SYSCFG_RST_CH(stih416_rear, SYSCFG_2525, _cntl, SYSSTAT_2583, _stat)
> -
> -#define SYSCFG_1500 0x7d0 /* Powerdown request EMI/NAND/Keyscan */
> -#define SYSSTAT_1578 0x908 /* Powerdown status EMI/NAND/Keyscan */
> -
> -#define SYSCFG_2525 0x834 /* Powerdown request USB/SATA/PCIe */
> -#define SYSSTAT_2583 0x91c /* Powerdown status USB/SATA/PCIe */
> -
> -#define SYSCFG_2552 0x8A0 /* Reset Generator control 0 */
> -#define SYSCFG_1539 0x86c /* Softreset Ethernet 0 */
> -#define SYSCFG_510 0x7f8 /* Softreset Ethernet 1 */
> -#define LPM_SYSCFG_1 0x4 /* Softreset IRB */
> -#define SYSCFG_2553 0x8a4 /* Softreset SATA0/1, PCIE0/1 */
> -#define SYSCFG_7563 0x8cc /* MPE softresets 0 */
> -#define SYSCFG_7564 0x8d0 /* MPE softresets 1 */
> -
> -#define STIH416_SRST_CPU(_reg, _bit) \
> - _SYSCFG_RST_CH_NO_ACK(stih416_cpu, _reg, _bit)
> -
> -#define STIH416_SRST_FRONT(_reg, _bit) \
> - _SYSCFG_RST_CH_NO_ACK(stih416_front, _reg, _bit)
> -
> -#define STIH416_SRST_REAR(_reg, _bit) \
> - _SYSCFG_RST_CH_NO_ACK(stih416_rear, _reg, _bit)
> -
> -#define STIH416_SRST_LPM(_reg, _bit) \
> - _SYSCFG_RST_CH_NO_ACK(stih416_lpm, _reg, _bit)
> -
> -#define STIH416_SRST_SBC(_reg, _bit) \
> - _SYSCFG_RST_CH_NO_ACK(stih416_sbc, _reg, _bit)
> -
> -static const struct syscfg_reset_channel_data stih416_powerdowns[] = {
> - [STIH416_EMISS_POWERDOWN] = STIH416_PDN_FRONT(0),
> - [STIH416_NAND_POWERDOWN] = STIH416_PDN_FRONT(1),
> - [STIH416_KEYSCAN_POWERDOWN] = STIH416_PDN_FRONT(2),
> - [STIH416_USB0_POWERDOWN] = STIH416_PDN_REAR(0, 0),
> - [STIH416_USB1_POWERDOWN] = STIH416_PDN_REAR(1, 1),
> - [STIH416_USB2_POWERDOWN] = STIH416_PDN_REAR(2, 2),
> - [STIH416_USB3_POWERDOWN] = STIH416_PDN_REAR(6, 5),
> - [STIH416_SATA0_POWERDOWN] = STIH416_PDN_REAR(3, 3),
> - [STIH416_SATA1_POWERDOWN] = STIH416_PDN_REAR(4, 4),
> - [STIH416_PCIE0_POWERDOWN] = STIH416_PDN_REAR(7, 9),
> - [STIH416_PCIE1_POWERDOWN] = STIH416_PDN_REAR(5, 8),
> -};
> -
> -static const struct syscfg_reset_channel_data stih416_softresets[] = {
> - [STIH416_ETH0_SOFTRESET] = STIH416_SRST_FRONT(SYSCFG_1539, 0),
> - [STIH416_ETH1_SOFTRESET] = STIH416_SRST_SBC(SYSCFG_510, 0),
> - [STIH416_IRB_SOFTRESET] = STIH416_SRST_LPM(LPM_SYSCFG_1, 6),
> - [STIH416_USB0_SOFTRESET] = STIH416_SRST_REAR(SYSCFG_2552, 9),
> - [STIH416_USB1_SOFTRESET] = STIH416_SRST_REAR(SYSCFG_2552, 10),
> - [STIH416_USB2_SOFTRESET] = STIH416_SRST_REAR(SYSCFG_2552, 11),
> - [STIH416_USB3_SOFTRESET] = STIH416_SRST_REAR(SYSCFG_2552, 28),
> - [STIH416_SATA0_SOFTRESET] = STIH416_SRST_REAR(SYSCFG_2553, 7),
> - [STIH416_SATA1_SOFTRESET] = STIH416_SRST_REAR(SYSCFG_2553, 3),
> - [STIH416_PCIE0_SOFTRESET] = STIH416_SRST_REAR(SYSCFG_2553, 15),
> - [STIH416_PCIE1_SOFTRESET] = STIH416_SRST_REAR(SYSCFG_2553, 2),
> - [STIH416_AUD_DAC_SOFTRESET] = STIH416_SRST_REAR(SYSCFG_2553, 14),
> - [STIH416_HDTVOUT_SOFTRESET] = STIH416_SRST_REAR(SYSCFG_2552, 5),
> - [STIH416_VTAC_M_RX_SOFTRESET] = STIH416_SRST_REAR(SYSCFG_2552, 25),
> - [STIH416_VTAC_A_RX_SOFTRESET] = STIH416_SRST_REAR(SYSCFG_2552, 26),
> - [STIH416_SYNC_HD_SOFTRESET] = STIH416_SRST_REAR(SYSCFG_2553, 5),
> - [STIH416_SYNC_SD_SOFTRESET] = STIH416_SRST_REAR(SYSCFG_2553, 6),
> - [STIH416_BLITTER_SOFTRESET] = STIH416_SRST_CPU(SYSCFG_7563, 10),
> - [STIH416_GPU_SOFTRESET] = STIH416_SRST_CPU(SYSCFG_7563, 11),
> - [STIH416_VTAC_M_TX_SOFTRESET] = STIH416_SRST_CPU(SYSCFG_7563, 18),
> - [STIH416_VTAC_A_TX_SOFTRESET] = STIH416_SRST_CPU(SYSCFG_7563, 19),
> - [STIH416_VTG_AUX_SOFTRESET] = STIH416_SRST_CPU(SYSCFG_7563, 21),
> - [STIH416_JPEG_DEC_SOFTRESET] = STIH416_SRST_CPU(SYSCFG_7563, 23),
> - [STIH416_HVA_SOFTRESET] = STIH416_SRST_CPU(SYSCFG_7564, 2),
> - [STIH416_COMPO_M_SOFTRESET] = STIH416_SRST_CPU(SYSCFG_7564, 3),
> - [STIH416_COMPO_A_SOFTRESET] = STIH416_SRST_CPU(SYSCFG_7564, 4),
> - [STIH416_VP8_DEC_SOFTRESET] = STIH416_SRST_CPU(SYSCFG_7564, 10),
> - [STIH416_VTG_MAIN_SOFTRESET] = STIH416_SRST_CPU(SYSCFG_7564, 16),
> - [STIH416_KEYSCAN_SOFTRESET] = STIH416_SRST_LPM(LPM_SYSCFG_1, 8),
> -};
> -
> -static struct syscfg_reset_controller_data stih416_powerdown_controller = {
> - .wait_for_ack = true,
> - .nr_channels = ARRAY_SIZE(stih416_powerdowns),
> - .channels = stih416_powerdowns,
> -};
> -
> -static struct syscfg_reset_controller_data stih416_softreset_controller = {
> - .wait_for_ack = false,
> - .active_low = true,
> - .nr_channels = ARRAY_SIZE(stih416_softresets),
> - .channels = stih416_softresets,
> -};
> -
> -static const struct of_device_id stih416_reset_match[] = {
> - { .compatible = "st,stih416-powerdown",
> - .data = &stih416_powerdown_controller, },
> - { .compatible = "st,stih416-softreset",
> - .data = &stih416_softreset_controller, },
> - {},
> -};
> -
> -static struct platform_driver stih416_reset_driver = {
> - .probe = syscfg_reset_probe,
> - .driver = {
> - .name = "reset-stih416",
> - .of_match_table = stih416_reset_match,
> - },
> -};
> -
> -static int __init stih416_reset_init(void)
> -{
> - return platform_driver_register(&stih416_reset_driver);
> -}
> -arch_initcall(stih416_reset_init);
Can I pick up patches 15 and 19, or are there dependencies in the
series?
In the latter case,
Acked-by: Philipp Zabel <p.zabel@...gutronix.de>
to merge both together with the other patches. Currently there is no
conflict with changes queued from the reset tree.
regards
Philipp
Powered by blists - more mailing lists