[OpenWrt-Devel] [PATCH] spi-ks8995: switch to upstream
Sergey Ryazanov
ryazanov.s.a at gmail.com
Wed Apr 1 05:09:59 EDT 2015
2015-04-01 12:02 GMT+03:00 John Crispin <blogic at openwrt.org>:
> Hi,
>
> did you runtime test this ?
>
No I did not. I do not have hw with this switch. Only compile tested.
spi_ks8995.c from 3.18.9 kernel differ from sources in spi-ks8995
package in several lines: register size made dynamic in upstream. All
other code are the same.
> John
>
> On 01/04/2015 01:50, Sergey Ryazanov wrote:
>> This driver is upstream since 2012 (commit: a8e510f682f), so switch to
>> it and remove own sources. Also place it under "Network Devices" menu
>> together with other phy drivers.
>>
>> Signed-off-by: Sergey Ryazanov <ryazanov.s.a at gmail.com>
>> ---
>> package/kernel/linux/modules/netdevices.mk | 17 ++
>> package/kernel/spi-ks8995/Makefile | 54 ----
>> package/kernel/spi-ks8995/src/Kconfig | 3 -
>> package/kernel/spi-ks8995/src/Makefile | 1 -
>> package/kernel/spi-ks8995/src/spi_ks8995.c | 419 -----------------------------
>> 5 files changed, 17 insertions(+), 477 deletions(-)
>> delete mode 100644 package/kernel/spi-ks8995/Makefile
>> delete mode 100644 package/kernel/spi-ks8995/src/Kconfig
>> delete mode 100644 package/kernel/spi-ks8995/src/Makefile
>> delete mode 100644 package/kernel/spi-ks8995/src/spi_ks8995.c
>>
>> diff --git a/package/kernel/linux/modules/netdevices.mk b/package/kernel/linux/modules/netdevices.mk
>> index dd8692a..4508998 100644
>> --- a/package/kernel/linux/modules/netdevices.mk
>> +++ b/package/kernel/linux/modules/netdevices.mk
>> @@ -810,3 +810,20 @@ define KernelPackage/vmxnet3/description
>> endef
>>
>> $(eval $(call KernelPackage,vmxnet3))
>> +
>> +
>> +define KernelPackage/spi-ks8995
>> + SUBMENU:=$(NETWORK_DEVICES_MENU)
>> + TITLE:=Micrel/Kendin KS8995 Ethernet switch control
>> + FILES:=$(LINUX_DIR)/drivers/net/phy/spi_ks8995.ko
>> + KCONFIG:=CONFIG_MICREL_KS8995MA \
>> + CONFIG_SPI=y \
>> + CONFIG_SPI_MASTER=y
>> + AUTOLOAD:=$(call AutoLoad,50,spi_ks8995)
>> +endef
>> +
>> +define KernelPackage/spi-ks8995/description
>> + Kernel module for Micrel/Kendin KS8995 ethernet switch
>> +endef
>> +
>> +$(eval $(call KernelPackage,spi-ks8995))
>> diff --git a/package/kernel/spi-ks8995/Makefile b/package/kernel/spi-ks8995/Makefile
>> deleted file mode 100644
>> index 0be9fe3..0000000
>> --- a/package/kernel/spi-ks8995/Makefile
>> +++ /dev/null
>> @@ -1,54 +0,0 @@
>> -#
>> -# Copyright (C) 2008 OpenWrt.org
>> -#
>> -# This is free software, licensed under the GNU General Public License v2.
>> -# See /LICENSE for more information.
>> -#
>> -
>> -include $(TOPDIR)/rules.mk
>> -include $(INCLUDE_DIR)/kernel.mk
>> -
>> -PKG_NAME:=spi-ks8995
>> -PKG_RELEASE:=1
>> -
>> -include $(INCLUDE_DIR)/package.mk
>> -
>> -define KernelPackage/spi-ks8995
>> - SUBMENU:=SPI Support
>> - TITLE:=Micrel/Kendin KS8995 Ethernet switch control
>> - FILES:=$(PKG_BUILD_DIR)/spi_ks8995.ko
>> - KCONFIG:=CONFIG_SPI=y \
>> - CONFIG_SPI_MASTER=y
>> - AUTOLOAD:=$(call AutoLoad,50,spi_ks8995)
>> -endef
>> -
>> -define KernelPackage/spi-ks8995/description
>> - Kernel module for Micrel/Kendin KS8995 ethernet switch
>> -endef
>> -
>> -EXTRA_KCONFIG:= \
>> - CONFIG_SPI_KS8995=m
>> -
>> -EXTRA_CFLAGS:= \
>> - $(patsubst CONFIG_%, -DCONFIG_%=1, $(patsubst %=m,%,$(filter %=m,$(EXTRA_KCONFIG)))) \
>> - $(patsubst CONFIG_%, -DCONFIG_%=1, $(patsubst %=y,%,$(filter %=y,$(EXTRA_KCONFIG)))) \
>> -
>> -MAKE_OPTS:= \
>> - ARCH="$(LINUX_KARCH)" \
>> - CROSS_COMPILE="$(TARGET_CROSS)" \
>> - SUBDIRS="$(PKG_BUILD_DIR)" \
>> - EXTRA_CFLAGS="$(EXTRA_CFLAGS)" \
>> - $(EXTRA_KCONFIG)
>> -
>> -define Build/Prepare
>> - mkdir -p $(PKG_BUILD_DIR)
>> - $(CP) ./src/* $(PKG_BUILD_DIR)/
>> -endef
>> -
>> -define Build/Compile
>> - $(MAKE) -C "$(LINUX_DIR)" \
>> - $(MAKE_OPTS) \
>> - modules
>> -endef
>> -
>> -$(eval $(call KernelPackage,spi-ks8995))
>> diff --git a/package/kernel/spi-ks8995/src/Kconfig b/package/kernel/spi-ks8995/src/Kconfig
>> deleted file mode 100644
>> index 7859be1..0000000
>> --- a/package/kernel/spi-ks8995/src/Kconfig
>> +++ /dev/null
>> @@ -1,3 +0,0 @@
>> -config SPI_KS8995
>> - tristate "Micrel/Kendin KS8995 Ethernet switch"
>> - depends on SPI
>> diff --git a/package/kernel/spi-ks8995/src/Makefile b/package/kernel/spi-ks8995/src/Makefile
>> deleted file mode 100644
>> index 810c3bd..0000000
>> --- a/package/kernel/spi-ks8995/src/Makefile
>> +++ /dev/null
>> @@ -1 +0,0 @@
>> -obj-${CONFIG_SPI_KS8995} += spi_ks8995.o
>> \ No newline at end of file
>> diff --git a/package/kernel/spi-ks8995/src/spi_ks8995.c b/package/kernel/spi-ks8995/src/spi_ks8995.c
>> deleted file mode 100644
>> index dc42b5d..0000000
>> --- a/package/kernel/spi-ks8995/src/spi_ks8995.c
>> +++ /dev/null
>> @@ -1,419 +0,0 @@
>> -/*
>> - * SPI driver for Micrel/Kendin KS8995M ethernet switch
>> - *
>> - * Copyright (C) 2008 Gabor Juhos <juhosg at openwrt.org>
>> - *
>> - * This file was based on: drivers/spi/at25.c
>> - * Copyright (C) 2006 David Brownell
>> - *
>> - * This program is free software; you can redistribute it and/or modify it
>> - * under the terms of the GNU General Public License version 2 as published
>> - * by the Free Software Foundation.
>> - */
>> -
>> -#include <linux/types.h>
>> -#include <linux/kernel.h>
>> -#include <linux/init.h>
>> -#include <linux/module.h>
>> -#include <linux/delay.h>
>> -#include <linux/device.h>
>> -
>> -#include <linux/spi/spi.h>
>> -
>> -#define DRV_VERSION "0.1.1"
>> -#define DRV_DESC "Micrel/Kendin KS8995 Ethernet switch SPI driver"
>> -
>> -/*-------------------------------------------------------------------------*/
>> -
>> -#define KS8995_REG_ID0 0x00 /* Chip ID0 */
>> -#define KS8995_REG_ID1 0x01 /* Chip ID1 */
>> -
>> -#define KS8995_REG_GC0 0x02 /* Global Control 0 */
>> -#define KS8995_REG_GC1 0x03 /* Global Control 1 */
>> -#define KS8995_REG_GC2 0x04 /* Global Control 2 */
>> -#define KS8995_REG_GC3 0x05 /* Global Control 3 */
>> -#define KS8995_REG_GC4 0x06 /* Global Control 4 */
>> -#define KS8995_REG_GC5 0x07 /* Global Control 5 */
>> -#define KS8995_REG_GC6 0x08 /* Global Control 6 */
>> -#define KS8995_REG_GC7 0x09 /* Global Control 7 */
>> -#define KS8995_REG_GC8 0x0a /* Global Control 8 */
>> -#define KS8995_REG_GC9 0x0b /* Global Control 9 */
>> -
>> -#define KS8995_REG_PC(p,r) ((0x10 * p) + r) /* Port Control */
>> -#define KS8995_REG_PS(p,r) ((0x10 * p) + r + 0xe) /* Port Status */
>> -
>> -#define KS8995_REG_TPC0 0x60 /* TOS Priority Control 0 */
>> -#define KS8995_REG_TPC1 0x61 /* TOS Priority Control 1 */
>> -#define KS8995_REG_TPC2 0x62 /* TOS Priority Control 2 */
>> -#define KS8995_REG_TPC3 0x63 /* TOS Priority Control 3 */
>> -#define KS8995_REG_TPC4 0x64 /* TOS Priority Control 4 */
>> -#define KS8995_REG_TPC5 0x65 /* TOS Priority Control 5 */
>> -#define KS8995_REG_TPC6 0x66 /* TOS Priority Control 6 */
>> -#define KS8995_REG_TPC7 0x67 /* TOS Priority Control 7 */
>> -
>> -#define KS8995_REG_MAC0 0x68 /* MAC address 0 */
>> -#define KS8995_REG_MAC1 0x69 /* MAC address 1 */
>> -#define KS8995_REG_MAC2 0x6a /* MAC address 2 */
>> -#define KS8995_REG_MAC3 0x6b /* MAC address 3 */
>> -#define KS8995_REG_MAC4 0x6c /* MAC address 4 */
>> -#define KS8995_REG_MAC5 0x6d /* MAC address 5 */
>> -
>> -#define KS8995_REG_IAC0 0x6e /* Indirect Access Control 0 */
>> -#define KS8995_REG_IAC1 0x6f /* Indirect Access Control 0 */
>> -
>> -#define KS8995_REG_IAD7 0x70 /* Indirect Access Data 7 */
>> -#define KS8995_REG_IAD6 0x71 /* Indirect Access Data 6 */
>> -#define KS8995_REG_IAD5 0x72 /* Indirect Access Data 5 */
>> -#define KS8995_REG_IAD4 0x73 /* Indirect Access Data 4 */
>> -#define KS8995_REG_IAD3 0x74 /* Indirect Access Data 3 */
>> -#define KS8995_REG_IAD2 0x75 /* Indirect Access Data 2 */
>> -#define KS8995_REG_IAD1 0x76 /* Indirect Access Data 1 */
>> -#define KS8995_REG_IAD0 0x77 /* Indirect Access Data 0 */
>> -
>> -#define KS8995_REGS_SIZE 0x80
>> -
>> -#define ID1_CHIPID_M 0xf
>> -#define ID1_CHIPID_S 4
>> -#define ID1_REVISION_M 0x7
>> -#define ID1_REVISION_S 1
>> -#define ID1_START_SW 1 /* start the switch */
>> -
>> -#define FAMILY_KS8995 0x95
>> -#define CHIPID_M 0
>> -
>> -#define KS8995_CMD_WRITE 0x02U
>> -#define KS8995_CMD_READ 0x03U
>> -
>> -#define KS8995_RESET_DELAY 10 /* usec */
>> -
>> -/*-------------------------------------------------------------------------*/
>> -
>> -struct ks8995_pdata {
>> - /* not yet implemented */
>> -};
>> -
>> -struct ks8995_switch {
>> - struct spi_device *spi;
>> - struct mutex lock;
>> - struct ks8995_pdata *pdata;
>> -};
>> -
>> -/*-------------------------------------------------------------------------*/
>> -
>> -static inline u8 get_chip_id(u8 val)
>> -{
>> - return ((val >> ID1_CHIPID_S) & ID1_CHIPID_M);
>> -}
>> -
>> -static inline u8 get_chip_rev(u8 val)
>> -{
>> - return ((val >> ID1_REVISION_S) & ID1_REVISION_M);
>> -}
>> -
>> -/*-------------------------------------------------------------------------*/
>> -
>> -static int ks8995_read(struct ks8995_switch *ks, char *buf,
>> - unsigned offset, size_t count)
>> -{
>> - u8 cmd[2];
>> - struct spi_transfer t[2];
>> - struct spi_message m;
>> - int err;
>> -
>> - spi_message_init(&m);
>> -
>> - memset(&t, 0, sizeof(t));
>> -
>> - t[0].tx_buf = cmd;
>> - t[0].len = sizeof(cmd);
>> - spi_message_add_tail(&t[0], &m);
>> -
>> - t[1].rx_buf = buf;
>> - t[1].len = count;
>> - spi_message_add_tail(&t[1], &m);
>> -
>> - cmd[0] = KS8995_CMD_READ;
>> - cmd[1] = offset;
>> -
>> - mutex_lock(&ks->lock);
>> - err = spi_sync(ks->spi, &m);
>> - mutex_unlock(&ks->lock);
>> -
>> - return err ? err : count;
>> -}
>> -
>> -
>> -static int ks8995_write(struct ks8995_switch *ks, char *buf,
>> - unsigned offset, size_t count)
>> -{
>> - u8 cmd[2];
>> - struct spi_transfer t[2];
>> - struct spi_message m;
>> - int err;
>> -
>> - spi_message_init(&m);
>> -
>> - memset(&t, 0, sizeof(t));
>> -
>> - t[0].tx_buf = cmd;
>> - t[0].len = sizeof(cmd);
>> - spi_message_add_tail(&t[0], &m);
>> -
>> - t[1].tx_buf = buf;
>> - t[1].len = count;
>> - spi_message_add_tail(&t[1], &m);
>> -
>> - cmd[0] = KS8995_CMD_WRITE;
>> - cmd[1] = offset;
>> -
>> - mutex_lock(&ks->lock);
>> - err = spi_sync(ks->spi, &m);
>> - mutex_unlock(&ks->lock);
>> -
>> - return err ? err : count;
>> -}
>> -
>> -static inline int ks8995_read_reg(struct ks8995_switch *ks, u8 addr, u8 *buf)
>> -{
>> - return (ks8995_read(ks, buf, addr, 1) != 1);
>> -}
>> -
>> -static inline int ks8995_write_reg(struct ks8995_switch *ks, u8 addr, u8 val)
>> -{
>> - char buf = val;
>> -
>> - return (ks8995_write(ks, &buf, addr, 1) != 1);
>> -}
>> -
>> -/*-------------------------------------------------------------------------*/
>> -
>> -#if 0
>> -static int ks8995_setup(struct spi_device *spi)
>> -{
>> - struct ks8995_switch *ks;
>> - u8 t;
>> -
>> - ks = dev_get_drvdata(&spi->dev);
>> -
>> - ks8995_write_reg(ks, KS8995_REG_GC0, 0x4c);
>> - ks8995_write_reg(ks, KS8995_REG_GC1, 0x05);
>> -
>> - ks8995_read_reg(ks, KS8995_REG_GC2, &t);
>> - ks8995_write_reg(ks, KS8995_REG_GC2, t | 1);
>> -
>> - ks8995_write_reg(ks, KS8995_REG_GC4, 0x20);
>> -
>> - ks8995_write_reg(ks, KS8995_REG_PC(1,0), 0x61);
>> - ks8995_write_reg(ks, KS8995_REG_PC(2,0), 0x61);
>> - ks8995_write_reg(ks, KS8995_REG_PC(3,0), 0x61);
>> - ks8995_write_reg(ks, KS8995_REG_PC(4,0), 0x61);
>> - ks8995_write_reg(ks, KS8995_REG_PC(5,0), 0x61);
>> -
>> - ks8995_write_reg(ks, KS8995_REG_PC(5,11), 0x18);
>> -
>> - ks8995_write_reg(ks, KS8995_REG_TPC0, 0xff);
>> - ks8995_write_reg(ks, KS8995_REG_TPC1, 0xff);
>> - ks8995_write_reg(ks, KS8995_REG_TPC2, 0xff);
>> - ks8995_write_reg(ks, KS8995_REG_TPC3, 0xff);
>> - ks8995_write_reg(ks, KS8995_REG_TPC4, 0xff);
>> - ks8995_write_reg(ks, KS8995_REG_TPC5, 0xff);
>> - ks8995_write_reg(ks, KS8995_REG_TPC6, 0xff);
>> - ks8995_write_reg(ks, KS8995_REG_TPC7, 0xfe);
>> -
>> - return 0;
>> -}
>> -#endif
>> -
>> -static int ks8995_stop(struct ks8995_switch *ks)
>> -{
>> - return ks8995_write_reg(ks, KS8995_REG_ID1, 0);
>> -}
>> -
>> -static int ks8995_start(struct ks8995_switch *ks)
>> -{
>> - return ks8995_write_reg(ks, KS8995_REG_ID1, 1);
>> -}
>> -
>> -static int ks8995_reset(struct ks8995_switch *ks)
>> -{
>> - int err;
>> -
>> - err = ks8995_stop(ks);
>> - if (err)
>> - return err;
>> -
>> - udelay(KS8995_RESET_DELAY);
>> -
>> - return ks8995_start(ks);
>> -}
>> -
>> -/*-------------------------------------------------------------------------*/
>> -
>> -static int ks8995_registers_read(struct kobject *kobj,
>> - struct bin_attribute *bin_attr, char *buf, loff_t off, size_t count)
>> -{
>> - struct device *dev;
>> - struct ks8995_switch *ks8995;
>> -
>> - dev = container_of(kobj, struct device, kobj);
>> - ks8995 = dev_get_drvdata(dev);
>> -
>> - if (unlikely(off > KS8995_REGS_SIZE))
>> - return 0;
>> -
>> - if ((off + count) > KS8995_REGS_SIZE)
>> - count = KS8995_REGS_SIZE - off;
>> -
>> - if (unlikely(!count))
>> - return count;
>> -
>> - return ks8995_read(ks8995, buf, off, count);
>> -}
>> -
>> -
>> -static int ks8995_registers_write(struct kobject *kobj,
>> - struct bin_attribute *bin_attr, char *buf, loff_t off, size_t count)
>> -{
>> - struct device *dev;
>> - struct ks8995_switch *ks8995;
>> -
>> - dev = container_of(kobj, struct device, kobj);
>> - ks8995 = dev_get_drvdata(dev);
>> -
>> - if (unlikely(off >= KS8995_REGS_SIZE))
>> - return -EFBIG;
>> -
>> - if ((off + count) > KS8995_REGS_SIZE)
>> - count = KS8995_REGS_SIZE - off;
>> -
>> - if (unlikely(!count))
>> - return count;
>> -
>> - return ks8995_write(ks8995, buf, off, count);
>> -}
>> -
>> -
>> -static struct bin_attribute ks8995_registers_attr = {
>> - .attr = {
>> - .name = "registers",
>> - .mode = S_IRUSR | S_IWUSR,
>> - },
>> - .size = KS8995_REGS_SIZE,
>> - .read = ks8995_registers_read,
>> - .write = ks8995_registers_write,
>> -};
>> -
>> -/*-------------------------------------------------------------------------*/
>> -
>> -static int ks8995_probe(struct spi_device *spi)
>> -{
>> - struct ks8995_switch *ks;
>> - struct ks8995_pdata *pdata;
>> - u8 ids[2];
>> - int err;
>> -
>> - /* Chip description */
>> - pdata = spi->dev.platform_data;
>> -
>> - ks = kzalloc(sizeof(*ks), GFP_KERNEL);
>> - if (!ks) {
>> - dev_err(&spi->dev, "no memory for private data\n");
>> - return-ENOMEM;
>> - }
>> -
>> - mutex_init(&ks->lock);
>> - ks->pdata = pdata;
>> - ks->spi = spi_dev_get(spi);
>> - dev_set_drvdata(&spi->dev, ks);
>> -
>> - spi->mode = SPI_MODE_0;
>> - spi->bits_per_word = 8;
>> - err = spi_setup(spi);
>> - if (err) {
>> - dev_err(&spi->dev, "spi_setup failed, err=%d \n", err);
>> - goto err_drvdata;
>> - }
>> -
>> - err = ks8995_read(ks, ids, KS8995_REG_ID0, sizeof(ids));
>> - if (err < 0) {
>> - dev_err(&spi->dev, "unable to read id registers, err=%d \n",
>> - err);
>> - goto err_drvdata;
>> - }
>> -
>> - switch (ids[0]) {
>> - case FAMILY_KS8995:
>> - break;
>> - default:
>> - dev_err(&spi->dev, "unknown family id:%02x\n", ids[0]);
>> - err = -ENODEV;
>> - goto err_drvdata;
>> - }
>> -
>> - err = ks8995_reset(ks);
>> - if (err)
>> - goto err_drvdata;
>> -
>> - err = sysfs_create_bin_file(&spi->dev.kobj, &ks8995_registers_attr);
>> - if (err) {
>> - dev_err(&spi->dev, "unable to create sysfs file, err=%d\n",
>> - err);
>> - goto err_drvdata;
>> - }
>> -
>> - dev_info(&spi->dev, "KS89%02X device found, Chip ID:%01x, "
>> - "Revision:%01x\n", ids[0],
>> - get_chip_id(ids[1]), get_chip_rev(ids[1]));
>> -
>> - return 0;
>> -
>> -err_drvdata:
>> - dev_set_drvdata(&spi->dev, NULL);
>> - kfree(ks);
>> - return err;
>> -}
>> -
>> -static int ks8995_remove(struct spi_device *spi)
>> -{
>> - struct ks8995_data *ks8995;
>> -
>> - ks8995 = dev_get_drvdata(&spi->dev);
>> - sysfs_remove_bin_file(&spi->dev.kobj, &ks8995_registers_attr);
>> -
>> - dev_set_drvdata(&spi->dev, NULL);
>> - kfree(ks8995);
>> -
>> - return 0;
>> -}
>> -
>> -/*-------------------------------------------------------------------------*/
>> -
>> -static struct spi_driver ks8995_driver = {
>> - .driver = {
>> - .name = "spi-ks8995",
>> - .bus = &spi_bus_type,
>> - .owner = THIS_MODULE,
>> - },
>> - .probe = ks8995_probe,
>> - .remove = ks8995_remove,
>> -};
>> -
>> -static int __init ks8995_init(void)
>> -{
>> - printk(KERN_INFO DRV_DESC " version " DRV_VERSION"\n");
>> -
>> - return spi_register_driver(&ks8995_driver);
>> -}
>> -module_init(ks8995_init);
>> -
>> -static void __exit ks8995_exit(void)
>> -{
>> - spi_unregister_driver(&ks8995_driver);
>> -}
>> -module_exit(ks8995_exit);
>> -
>> -MODULE_DESCRIPTION(DRV_DESC);
>> -MODULE_VERSION(DRV_VERSION);
>> -MODULE_AUTHOR("Gabor Juhos <juhosg at openwrt.org>");
>> -MODULE_LICENSE("GPL v2");
>> -
>>
> _______________________________________________
> openwrt-devel mailing list
> openwrt-devel at lists.openwrt.org
> https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
--
BR,
Sergey
С наилучшими пожеланиями
Рязанов Сергей
_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
More information about the openwrt-devel
mailing list