Commit e84ff3a2 authored by Sergio Paracuellos's avatar Sergio Paracuellos Committed by Greg Kroah-Hartman
Browse files

staging: mt7621-gpio: rename MTK_MAX_BANK into MTK_BANK_CNT 



There are 3 banks of gpios numbered '0' and '1' and '2'. So
the maximum bank number is "2". "3" is the count of banks.
In order to make the code looks and be correct on checking
max allowed gpio's id it makes sense to change the name of
this definition. Also there is another definitions which
start with the same prefix MKK_BANK_ of the new name so
having those with the same prefix makes all preprocessor
structure to be the same. This improves readability.

Signed-off-by: default avatarSergio Paracuellos <sergio.paracuellos@gmail.com>
Reviewed-by: default avatarNeilBrown <neil@brown.name>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent a5a5b283

drivers/staging/mt7621-gpio/gpio-mt7621.c

+5 −5
Original line number Diff line number Diff line
@@ -14,7 +14,7 @@ 
#include <linux/interrupt.h>

#include <linux/platform_device.h>



#define MTK_MAX_BANK		3

#define MTK_BANK_CNT		3

#define MTK_BANK_WIDTH		32



enum mediatek_gpio_reg {
@@ -43,7 +43,7 @@ struct mtk_data { 
	void __iomem *gpio_membase;

	int gpio_irq;

	struct irq_domain *gpio_irq_domain;

	struct mtk_gc *gc_map[MTK_MAX_BANK];

	struct mtk_gc *gc_map[MTK_BANK_CNT];

};



static inline struct mtk_gc
@@ -156,7 +156,7 @@ mediatek_gpio_bank_probe(struct platform_device *pdev, struct device_node *bank) 
	struct mtk_gc *rg;

	int ret;



	if (!id || be32_to_cpu(*id) >= MTK_MAX_BANK)

	if (!id || be32_to_cpu(*id) >= MTK_BANK_CNT)

		return -EINVAL;



	rg = devm_kzalloc(&pdev->dev, sizeof(struct mtk_gc), GFP_KERNEL);
@@ -202,7 +202,7 @@ mediatek_gpio_irq_handler(struct irq_desc *desc) 
	struct mtk_data *gpio_data = irq_desc_get_handler_data(desc);

	int i;



	for (i = 0; i < MTK_MAX_BANK; i++) {

	for (i = 0; i < MTK_BANK_CNT; i++) {

		struct mtk_gc *rg = gpio_data->gc_map[i];

		unsigned long pending;

		int bit;
@@ -345,7 +345,7 @@ mediatek_gpio_probe(struct platform_device *pdev) 
	gpio_data->gpio_irq = irq_of_parse_and_map(np, 0);

	if (gpio_data->gpio_irq) {

		gpio_data->gpio_irq_domain = irq_domain_add_linear(np,

			MTK_MAX_BANK * MTK_BANK_WIDTH,

			MTK_BANK_CNT * MTK_BANK_WIDTH,

			&irq_domain_ops, gpio_data);

		if (!gpio_data->gpio_irq_domain)

			dev_err(&pdev->dev, "irq_domain_add_linear failed\n");

CRA Git | Maintained and supported by SUSTech CRA and CCSE