[U-Boot] [UBOOT PATCH v2 0/2] microblaze: Moved U-boot headers to Kconfig

This seris of patch moved microblaze u-boot headers to the kconfig and enabled it in respective defconfig. These are the configs: -CONFIG_XILINX_GPIO -CONFIG_XILINX_TB_WATCHDOG
- Changes in v2: - Removed "Enable" from description - Added select HW_WATCHDOG - Branch: u-boot-microblaze/kconfig
Vipul Kumar (2): microblaze: Added Kconfig support for CONFIG_XILINX_GPIO microblaze: wdt: Added Kconfig support for CONFIG_XILINX_TB_WATCHDOG
configs/microblaze-generic_defconfig | 2 ++ drivers/gpio/Kconfig | 5 +++++ drivers/watchdog/Kconfig | 7 +++++++ include/configs/microblaze-generic.h | 2 -- 4 files changed, 14 insertions(+), 2 deletions(-)
-- 2.7.4
This email and any attachments are intended for the sole use of the named recipient(s) and contain(s) confidential information that may be proprietary, privileged or copyrighted under applicable law. If you are not the intended recipient, do not read, copy, or forward this email message or any attachments. Delete this email message and any attachments immediately.

This patch added Kconfig support for CONFIG_XILINX_TB_WATCHDOG and enabled it in respective defconfig.
Signed-off-by: Vipul Kumar vipulk@xilinx.com Signed-off-by: Siva Durga Prasad Paladugu sivadur@xilinx.com --- - Changes in v2: - Added select HW_WATCHDOG - Branch: u-boot-microblaze/kconfig --- configs/microblaze-generic_defconfig | 1 + drivers/watchdog/Kconfig | 7 +++++++ include/configs/microblaze-generic.h | 1 - 3 files changed, 8 insertions(+), 1 deletion(-)
diff --git a/configs/microblaze-generic_defconfig b/configs/microblaze-generic_defconfig index 699dc44..fba3820 100644 --- a/configs/microblaze-generic_defconfig +++ b/configs/microblaze-generic_defconfig @@ -63,3 +63,4 @@ CONFIG_XILINX_AXIEMAC=y CONFIG_XILINX_EMACLITE=y CONFIG_SYS_NS16550=y CONFIG_XILINX_UARTLITE=y +CONFIG_XILINX_TB_WATCHDOG=y diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig index fc46b67..a1e9432 100644 --- a/drivers/watchdog/Kconfig +++ b/drivers/watchdog/Kconfig @@ -78,4 +78,11 @@ config WDT_ORION Select this to enable Orion watchdog timer, which can be found on some Marvell Armada chips.
+config XILINX_TB_WATCHDOG + bool "Xilinx watchdog timer support" + depends on MICROBLAZE && !SPL_BUILD + select HW_WATCHDOG + help + Available for Xilinx Axi platforms to service timebase watchdog timer. + endmenu diff --git a/include/configs/microblaze-generic.h b/include/configs/microblaze-generic.h index 73c3c2a..391b26a 100644 --- a/include/configs/microblaze-generic.h +++ b/include/configs/microblaze-generic.h @@ -50,7 +50,6 @@ # define CONFIG_WATCHDOG_IRQ XILINX_WATCHDOG_IRQ # ifndef CONFIG_SPL_BUILD # define CONFIG_HW_WATCHDOG -# define CONFIG_XILINX_TB_WATCHDOG # endif #endif
-- 2.7.4
This email and any attachments are intended for the sole use of the named recipient(s) and contain(s) confidential information that may be proprietary, privileged or copyrighted under applicable law. If you are not the intended recipient, do not read, copy, or forward this email message or any attachments. Delete this email message and any attachments immediately.

On 28.2.2018 07:02, Vipul Kumar wrote:
This patch added Kconfig support for CONFIG_XILINX_TB_WATCHDOG and enabled it in respective defconfig.
Signed-off-by: Vipul Kumar vipulk@xilinx.com Signed-off-by: Siva Durga Prasad Paladugu sivadur@xilinx.com
- Changes in v2:
- Added select HW_WATCHDOG
- Branch: u-boot-microblaze/kconfig
configs/microblaze-generic_defconfig | 1 + drivers/watchdog/Kconfig | 7 +++++++ include/configs/microblaze-generic.h | 1 - 3 files changed, 8 insertions(+), 1 deletion(-)
diff --git a/configs/microblaze-generic_defconfig b/configs/microblaze-generic_defconfig index 699dc44..fba3820 100644 --- a/configs/microblaze-generic_defconfig +++ b/configs/microblaze-generic_defconfig @@ -63,3 +63,4 @@ CONFIG_XILINX_AXIEMAC=y CONFIG_XILINX_EMACLITE=y CONFIG_SYS_NS16550=y CONFIG_XILINX_UARTLITE=y +CONFIG_XILINX_TB_WATCHDOG=y diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig index fc46b67..a1e9432 100644 --- a/drivers/watchdog/Kconfig +++ b/drivers/watchdog/Kconfig @@ -78,4 +78,11 @@ config WDT_ORION Select this to enable Orion watchdog timer, which can be found on some Marvell Armada chips.
+config XILINX_TB_WATCHDOG
- bool "Xilinx watchdog timer support"
- depends on MICROBLAZE && !SPL_BUILD
Based on my chat with Tom this is not working and that needs to be done at the code/Makefile level.
- select HW_WATCHDOG
- help
Available for Xilinx Axi platforms to service timebase watchdog timer.
endmenu diff --git a/include/configs/microblaze-generic.h b/include/configs/microblaze-generic.h index 73c3c2a..391b26a 100644 --- a/include/configs/microblaze-generic.h +++ b/include/configs/microblaze-generic.h @@ -50,7 +50,6 @@ # define CONFIG_WATCHDOG_IRQ XILINX_WATCHDOG_IRQ # ifndef CONFIG_SPL_BUILD # define CONFIG_HW_WATCHDOG -# define CONFIG_XILINX_TB_WATCHDOG # endif #endif
When this is applied there is compilation warning. I think that will be just easier to convert this driver to DM and ged rid of all these configs.
Or don't select hw_watchdog via Kconfig even that dependency is there.
M
diff --git a/configs/microblaze-generic_defconfig b/configs/microblaze-generic_defconfig index 699dc447f0ff..fba382037a76 100644 --- a/configs/microblaze-generic_defconfig +++ b/configs/microblaze-generic_defconfig @@ -63,3 +63,4 @@ CONFIG_XILINX_AXIEMAC=y CONFIG_XILINX_EMACLITE=y CONFIG_SYS_NS16550=y CONFIG_XILINX_UARTLITE=y +CONFIG_XILINX_TB_WATCHDOG=y diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig index fc46b6774d57..3a55ae08abd8 100644 --- a/drivers/watchdog/Kconfig +++ b/drivers/watchdog/Kconfig @@ -78,4 +78,10 @@ config WDT_ORION Select this to enable Orion watchdog timer, which can be found on some Marvell Armada chips.
+config XILINX_TB_WATCHDOG + bool "Xilinx watchdog timer support" + depends on MICROBLAZE + help + Available for Xilinx Axi platforms to service timebase watchdog timer. + endmenu diff --git a/include/configs/microblaze-generic.h b/include/configs/microblaze-generic.h index 73c3c2ae485c..391b26ae82c5 100644 --- a/include/configs/microblaze-generic.h +++ b/include/configs/microblaze-generic.h @@ -50,7 +50,6 @@ # define CONFIG_WATCHDOG_IRQ XILINX_WATCHDOG_IRQ # ifndef CONFIG_SPL_BUILD # define CONFIG_HW_WATCHDOG -# define CONFIG_XILINX_TB_WATCHDOG # endif #endif

This patch added Kconfig support for CONFIG_XILINX_GPIO and enabled it in respective defconfig.
Signed-off-by: Vipul Kumar vipulk@xilinx.com Signed-off-by: Siva Durga Prasad Paladugu sivadur@xilinx.com --- - Changes in v2: - Removed "Enable" from description - Branch: u-boot-microblaze/kconfig --- configs/microblaze-generic_defconfig | 1 + drivers/gpio/Kconfig | 5 +++++ include/configs/microblaze-generic.h | 1 - 3 files changed, 6 insertions(+), 1 deletion(-)
diff --git a/configs/microblaze-generic_defconfig b/configs/microblaze-generic_defconfig index ac49404..699dc44 100644 --- a/configs/microblaze-generic_defconfig +++ b/configs/microblaze-generic_defconfig @@ -46,6 +46,7 @@ CONFIG_BOOTP_BOOTPATH=y CONFIG_BOOTP_GATEWAY=y CONFIG_BOOTP_HOSTNAME=y CONFIG_SPL_DM=y +CONFIG_XILINX_GPIO=y CONFIG_MTD_NOR_FLASH=y CONFIG_PHY_ATHEROS=y CONFIG_PHY_BROADCOM=y diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig index 34d4409..cc75aec 100644 --- a/drivers/gpio/Kconfig +++ b/drivers/gpio/Kconfig @@ -183,6 +183,11 @@ config SANDBOX_GPIO_COUNT of 'anonymous' GPIOs that do not belong to any device or bank. Select a suitable value depending on your needs.
+config XILINX_GPIO + bool "Xilinx GPIO driver" + help + This config enable the Xilinx GPIO driver for Microblaze. + config CMD_TCA642X bool "tca642x - Command to access tca642x state" help diff --git a/include/configs/microblaze-generic.h b/include/configs/microblaze-generic.h index fe9aaae..73c3c2a 100644 --- a/include/configs/microblaze-generic.h +++ b/include/configs/microblaze-generic.h @@ -41,7 +41,6 @@
/* gpio */ #ifdef XILINX_GPIO_BASEADDR -# define CONFIG_XILINX_GPIO # define CONFIG_SYS_GPIO_0_ADDR XILINX_GPIO_BASEADDR #endif
-- 2.7.4
This email and any attachments are intended for the sole use of the named recipient(s) and contain(s) confidential information that may be proprietary, privileged or copyrighted under applicable law. If you are not the intended recipient, do not read, copy, or forward this email message or any attachments. Delete this email message and any attachments immediately.

On 28.2.2018 07:02, Vipul Kumar wrote:
This patch added Kconfig support for CONFIG_XILINX_GPIO and enabled it in respective defconfig.
Signed-off-by: Vipul Kumar vipulk@xilinx.com Signed-off-by: Siva Durga Prasad Paladugu sivadur@xilinx.com
- Changes in v2:
- Removed "Enable" from description
- Branch: u-boot-microblaze/kconfig
configs/microblaze-generic_defconfig | 1 + drivers/gpio/Kconfig | 5 +++++ include/configs/microblaze-generic.h | 1 - 3 files changed, 6 insertions(+), 1 deletion(-)
diff --git a/configs/microblaze-generic_defconfig b/configs/microblaze-generic_defconfig index ac49404..699dc44 100644 --- a/configs/microblaze-generic_defconfig +++ b/configs/microblaze-generic_defconfig @@ -46,6 +46,7 @@ CONFIG_BOOTP_BOOTPATH=y CONFIG_BOOTP_GATEWAY=y CONFIG_BOOTP_HOSTNAME=y CONFIG_SPL_DM=y +CONFIG_XILINX_GPIO=y CONFIG_MTD_NOR_FLASH=y CONFIG_PHY_ATHEROS=y CONFIG_PHY_BROADCOM=y diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig index 34d4409..cc75aec 100644 --- a/drivers/gpio/Kconfig +++ b/drivers/gpio/Kconfig @@ -183,6 +183,11 @@ config SANDBOX_GPIO_COUNT of 'anonymous' GPIOs that do not belong to any device or bank. Select a suitable value depending on your needs.
+config XILINX_GPIO
- bool "Xilinx GPIO driver"
- help
This config enable the Xilinx GPIO driver for Microblaze.
config CMD_TCA642X bool "tca642x - Command to access tca642x state" help diff --git a/include/configs/microblaze-generic.h b/include/configs/microblaze-generic.h index fe9aaae..73c3c2a 100644 --- a/include/configs/microblaze-generic.h +++ b/include/configs/microblaze-generic.h @@ -41,7 +41,6 @@
/* gpio */ #ifdef XILINX_GPIO_BASEADDR -# define CONFIG_XILINX_GPIO # define CONFIG_SYS_GPIO_0_ADDR XILINX_GPIO_BASEADDR #endif
Applied this one.
M
participants (2)
-
Michal Simek
-
Vipul Kumar