Commit 4c59714a authored by Lukas Bulwahn's avatar Lukas Bulwahn Committed by Bartosz Golaszewski
Browse files

gpio: remove the obsolete MX35 3DS BOARD MC9S08DZ60 GPIO functions 



Commit e1324ece ("ARM: imx: Remove i.MX35 board files") removes the
config MACH_MX35_3DS in arch/arm/mach-imx/Kconfig.

Hence, since then, the MX35 3DS BOARD MC9S08DZ60 GPIO functions are dead
code as its config GPIO_MC9S08DZ60 depends on the config MACH_MX35_3DS.

Luckily, ./scripts/checkkconfigsymbols.py warns on non-existing configs:

MACH_MX35_3DS
Referencing files: drivers/gpio/Kconfig

Remove the obsolete MX35 3DS BOARD MC9S08DZ60 GPIO functions.

Signed-off-by: default avatarLukas Bulwahn <lukas.bulwahn@gmail.com>
Reviewed-by: default avatarLinus Walleij <linus.walleij@linaro.org>
Reviewed-by: default avatarFabio Estevam <festevam@gmail.com>
Signed-off-by: default avatarBartosz Golaszewski <bgolaszewski@baylibre.com>
parent e5e26d80

drivers/gpio/Kconfig

+0 −6
Original line number Diff line number Diff line
@@ -1010,12 +1010,6 @@ config GPIO_MAX732X_IRQ 
	  Say yes here to enable the max732x to be used as an interrupt

	  controller. It requires the driver to be built in the kernel.



config GPIO_MC9S08DZ60

	bool "MX35 3DS BOARD MC9S08DZ60 GPIO functions"

	depends on I2C=y && MACH_MX35_3DS

	help

	  Select this to enable the MC9S08DZ60 GPIO driver



config GPIO_PCA953X

	tristate "PCA95[357]x, PCA9698, TCA64xx, and MAX7310 I/O ports"

	select REGMAP_I2C

drivers/gpio/Makefile

+0 −1
Original line number Diff line number Diff line
@@ -92,7 +92,6 @@ obj-$(CONFIG_GPIO_MAX77620) += gpio-max77620.o 
obj-$(CONFIG_GPIO_MAX77650)		+= gpio-max77650.o

obj-$(CONFIG_GPIO_MB86S7X)		+= gpio-mb86s7x.o

obj-$(CONFIG_GPIO_MC33880)		+= gpio-mc33880.o

obj-$(CONFIG_GPIO_MC9S08DZ60)		+= gpio-mc9s08dz60.o

obj-$(CONFIG_GPIO_MENZ127)		+= gpio-menz127.o

obj-$(CONFIG_GPIO_MERRIFIELD)		+= gpio-merrifield.o

obj-$(CONFIG_GPIO_ML_IOH)		+= gpio-ml-ioh.o

drivers/gpio/gpio-mc9s08dz60.c

deleted100644 → 0
+0 −112
Original line number Diff line number Diff line 
// SPDX-License-Identifier: GPL-2.0-or-later

/*

 * Copyright 2009-2012 Freescale Semiconductor, Inc. All Rights Reserved.

 *

 * Author: Wu Guoxing <b39297@freescale.com>

 */



#include <linux/kernel.h>

#include <linux/init.h>

#include <linux/slab.h>

#include <linux/i2c.h>

#include <linux/gpio/driver.h>



#define GPIO_GROUP_NUM 2

#define GPIO_NUM_PER_GROUP 8

#define GPIO_NUM (GPIO_GROUP_NUM*GPIO_NUM_PER_GROUP)



struct mc9s08dz60 {

	struct i2c_client *client;

	struct gpio_chip chip;

};



static void mc9s_gpio_to_reg_and_bit(int offset, u8 *reg, u8 *bit)

{

	*reg = 0x20 + offset / GPIO_NUM_PER_GROUP;

	*bit = offset % GPIO_NUM_PER_GROUP;

}



static int mc9s08dz60_get_value(struct gpio_chip *gc, unsigned offset)

{

	u8 reg, bit;

	s32 value;

	struct mc9s08dz60 *mc9s = gpiochip_get_data(gc);



	mc9s_gpio_to_reg_and_bit(offset, &reg, &bit);

	value = i2c_smbus_read_byte_data(mc9s->client, reg);



	return (value >= 0) ? (value >> bit) & 0x1 : 0;

}



static int mc9s08dz60_set(struct mc9s08dz60 *mc9s, unsigned offset, int val)

{

	u8 reg, bit;

	s32 value;



	mc9s_gpio_to_reg_and_bit(offset, &reg, &bit);

	value = i2c_smbus_read_byte_data(mc9s->client, reg);

	if (value >= 0) {

		if (val)

			value |= 1 << bit;

		else

			value &= ~(1 << bit);



		return i2c_smbus_write_byte_data(mc9s->client, reg, value);

	} else

		return value;



}





static void mc9s08dz60_set_value(struct gpio_chip *gc, unsigned offset, int val)

{

	struct mc9s08dz60 *mc9s = gpiochip_get_data(gc);



	mc9s08dz60_set(mc9s, offset, val);

}



static int mc9s08dz60_direction_output(struct gpio_chip *gc,

				       unsigned offset, int val)

{

	struct mc9s08dz60 *mc9s = gpiochip_get_data(gc);



	return mc9s08dz60_set(mc9s, offset, val);

}



static int mc9s08dz60_probe(struct i2c_client *client,

			    const struct i2c_device_id *id)

{

	struct mc9s08dz60 *mc9s;



	mc9s = devm_kzalloc(&client->dev, sizeof(*mc9s), GFP_KERNEL);

	if (!mc9s)

		return -ENOMEM;



	mc9s->chip.label = client->name;

	mc9s->chip.base = -1;

	mc9s->chip.parent = &client->dev;

	mc9s->chip.owner = THIS_MODULE;

	mc9s->chip.ngpio = GPIO_NUM;

	mc9s->chip.can_sleep = true;

	mc9s->chip.get = mc9s08dz60_get_value;

	mc9s->chip.set = mc9s08dz60_set_value;

	mc9s->chip.direction_output = mc9s08dz60_direction_output;

	mc9s->client = client;

	i2c_set_clientdata(client, mc9s);



	return devm_gpiochip_add_data(&client->dev, &mc9s->chip, mc9s);

}



static const struct i2c_device_id mc9s08dz60_id[] = {

	{"mc9s08dz60", 0},

	{},

};



static struct i2c_driver mc9s08dz60_i2c_driver = {

	.driver = {

		.name = "mc9s08dz60",

	},

	.probe = mc9s08dz60_probe,

	.id_table = mc9s08dz60_id,

};

builtin_i2c_driver(mc9s08dz60_i2c_driver);