Hi John,

On Thu, Mar 24, 2016 at 06:47:31AM +0100, John Crispin wrote:
> The patch has inconsistent ordering and indentation, see inline

Your suggestions are appreciated and the changes are pushed.

Luka
 
>       John
> 
> On 23/03/2016 23:31, openwrt-comm...@openwrt.org wrote:
> > Author: luka
> > Date: 2016-03-23 23:31:38 +0100 (Wed, 23 Mar 2016)
> > New Revision: 49082
> > 
> > Added:
> >    trunk/target/linux/ar71xx/files/arch/mips/ath79/mach-alfa-ap120c.c
> > Modified:
> >    trunk/target/linux/ar71xx/base-files/etc/board.d/02_network
> >    trunk/target/linux/ar71xx/base-files/lib/ar71xx.sh
> >    trunk/target/linux/ar71xx/config-4.1
> >    trunk/target/linux/ar71xx/config-4.4
> >    trunk/target/linux/ar71xx/files/arch/mips/ath79/Kconfig.openwrt
> >    trunk/target/linux/ar71xx/files/arch/mips/ath79/Makefile
> >    trunk/target/linux/ar71xx/files/arch/mips/ath79/machtypes.h
> >    trunk/target/linux/ar71xx/generic/profiles/alfa.mk
> >    trunk/target/linux/ar71xx/image/Makefile
> > Log:
> > ar71xx: add support for ap120c
> > 
> > Signed-off-by: Luka Perkov <l...@openwrt.org>
> > 
> > Modified: trunk/target/linux/ar71xx/base-files/etc/board.d/02_network
> > ===================================================================
> > --- trunk/target/linux/ar71xx/base-files/etc/board.d/02_network     
> > 2016-03-23 12:52:50 UTC (rev 49081)
> > +++ trunk/target/linux/ar71xx/base-files/etc/board.d/02_network     
> > 2016-03-23 22:31:38 UTC (rev 49082)
> > @@ -320,6 +320,7 @@
> >     ucidef_add_switch_attr "switch0" "enable" "false"
> >     ;;
> >  
> > +alfa-ap120c |\
> >  all0305 |\
> >  antminer-s1 |\
> >  antminer-s3 |\
> > 
> > Modified: trunk/target/linux/ar71xx/base-files/lib/ar71xx.sh
> > ===================================================================
> > --- trunk/target/linux/ar71xx/base-files/lib/ar71xx.sh      2016-03-23 
> > 12:52:50 UTC (rev 49081)
> > +++ trunk/target/linux/ar71xx/base-files/lib/ar71xx.sh      2016-03-23 
> > 22:31:38 UTC (rev 49082)
> > @@ -351,6 +351,9 @@
> >     *"ALFA Network AP96")
> >             name="alfa-ap96"
> >             ;;
> > +   *"ALFA Network AP120C")
> > +           name="alfa-ap120c"
> > +           ;;
> 
> here is is added after 96
> 
> >     *"ALFA Network N2/N5")
> >             name="alfa-nx"
> >             ;;
> > 
> > Modified: trunk/target/linux/ar71xx/config-4.1
> > ===================================================================
> > --- trunk/target/linux/ar71xx/config-4.1    2016-03-23 12:52:50 UTC (rev 
> > 49081)
> > +++ trunk/target/linux/ar71xx/config-4.1    2016-03-23 22:31:38 UTC (rev 
> > 49082)
> > @@ -29,6 +29,7 @@
> >  CONFIG_ATH79_DEV_SPI=y
> >  CONFIG_ATH79_DEV_USB=y
> >  CONFIG_ATH79_DEV_WMAC=y
> > +CONFIG_ATH79_MACH_ALFA_AP120C=y
> >  CONFIG_ATH79_MACH_ALFA_AP96=y
> 
> here is is added before 96
> 
> >  CONFIG_ATH79_MACH_ALFA_NX=y
> >  CONFIG_ATH79_MACH_ALL0258N=y
> > @@ -72,7 +73,6 @@
> >  CONFIG_ATH79_MACH_DLAN_PRO_1200_AC=y
> >  CONFIG_ATH79_MACH_DLAN_PRO_500_WP=y
> >  CONFIG_ATH79_MACH_DR344=y
> > -CONFIG_ATH79_MACH_GL_DOMINO=y
> >  CONFIG_ATH79_MACH_DRAGINO2=y
> >  CONFIG_ATH79_MACH_EAP300V2=y
> >  CONFIG_ATH79_MACH_EAP7660D=y
> > @@ -85,6 +85,7 @@
> >  CONFIG_ATH79_MACH_F9K1115V2=y
> >  CONFIG_ATH79_MACH_GL_AR150=y
> >  CONFIG_ATH79_MACH_GL_AR300=y
> > +CONFIG_ATH79_MACH_GL_DOMINO=y
> >  CONFIG_ATH79_MACH_GL_INET=y
> >  CONFIG_ATH79_MACH_GS_MINIBOX_V1=y
> >  CONFIG_ATH79_MACH_GS_OOLITE=y
> > 
> > Modified: trunk/target/linux/ar71xx/config-4.4
> > ===================================================================
> > --- trunk/target/linux/ar71xx/config-4.4    2016-03-23 12:52:50 UTC (rev 
> > 49081)
> > +++ trunk/target/linux/ar71xx/config-4.4    2016-03-23 22:31:38 UTC (rev 
> > 49082)
> > @@ -32,6 +32,7 @@
> >  CONFIG_ATH79_DEV_SPI=y
> >  CONFIG_ATH79_DEV_USB=y
> >  CONFIG_ATH79_DEV_WMAC=y
> > +CONFIG_ATH79_MACH_ALFA_AP120C=y
> >  CONFIG_ATH79_MACH_ALFA_AP96=y
> 
> here is is added before 96
> 
> >  CONFIG_ATH79_MACH_ALFA_NX=y
> >  CONFIG_ATH79_MACH_ALL0258N=y
> > 
> > Modified: trunk/target/linux/ar71xx/files/arch/mips/ath79/Kconfig.openwrt
> > ===================================================================
> > --- trunk/target/linux/ar71xx/files/arch/mips/ath79/Kconfig.openwrt 
> > 2016-03-23 12:52:50 UTC (rev 49081)
> > +++ trunk/target/linux/ar71xx/files/arch/mips/ath79/Kconfig.openwrt 
> > 2016-03-23 22:31:38 UTC (rev 49082)
> > @@ -6,6 +6,14 @@
> >     select ATH79_DEV_M25P80
> >     select ATH79_DEV_USB
> >  
> > +config ATH79_MACH_ALFA_AP120C
> > +   bool "ALFA Network AP120C board support"
> > +   select ATH79_DEV_AP9X_PCI if PCI
> > +   select ATH79_DEV_ETH
> > +   select ATH79_DEV_GPIO_BUTTONS
> > +   select ATH79_DEV_M25P80
> > +   select ATH79_DEV_WMAC
> 
> here is is added after 96
> 
> > +
> >  config ATH79_MACH_HORNET_UB
> >     bool "ALFA Network Hornet-UB board support"
> >     select SOC_AR933X
> > 
> > Modified: trunk/target/linux/ar71xx/files/arch/mips/ath79/Makefile
> > ===================================================================
> > --- trunk/target/linux/ar71xx/files/arch/mips/ath79/Makefile        
> > 2016-03-23 12:52:50 UTC (rev 49081)
> > +++ trunk/target/linux/ar71xx/files/arch/mips/ath79/Makefile        
> > 2016-03-23 22:31:38 UTC (rev 49082)
> > @@ -39,6 +39,7 @@
> >  # Machines
> >  #
> >  obj-$(CONFIG_ATH79_MACH_ALFA_AP96) += mach-alfa-ap96.o
> > +obj-$(CONFIG_ATH79_MACH_ALFA_AP120C)       += mach-alfa-ap120c.o
> 
> here is is added after 96
> 
> 
> >  obj-$(CONFIG_ATH79_MACH_ALFA_NX)   += mach-alfa-nx.o
> >  obj-$(CONFIG_ATH79_MACH_ALL0258N)  += mach-all0258n.o
> >  obj-$(CONFIG_ATH79_MACH_ALL0315N)  += mach-all0315n.o
> > 
> > Added: trunk/target/linux/ar71xx/files/arch/mips/ath79/mach-alfa-ap120c.c
> > ===================================================================
> > --- trunk/target/linux/ar71xx/files/arch/mips/ath79/mach-alfa-ap120c.c      
> >                         (rev 0)
> > +++ trunk/target/linux/ar71xx/files/arch/mips/ath79/mach-alfa-ap120c.c      
> > 2016-03-23 22:31:38 UTC (rev 49082)
> > @@ -0,0 +1,147 @@
> > +/*
> > + *  ALFA Network AP120C board support
> > + *
> > + *  Copyright (C) 2012 Gabor Juhos <juh...@openwrt.org>
> > + *  Copyright (C) 2016 Luka Perkov <l...@openwrt.org>
> > + *
> > + *  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/init.h>
> > +#include <linux/bitops.h>
> > +#include <linux/gpio.h>
> > +#include <linux/platform_device.h>
> > +#include <linux/spi/spi.h>
> > +#include <linux/ar8216_platform.h>
> > +#include <linux/ath9k_platform.h>
> > +
> > +#include <asm/mach-ath79/ath79.h>
> > +#include <asm/mach-ath79/ar71xx_regs.h>
> > +
> > +#include "common.h"
> > +#include "dev-eth.h"
> > +#include "dev-gpio-buttons.h"
> > +#include "dev-leds-gpio.h"
> > +#include "dev-m25p80.h"
> > +#include "dev-spi.h"
> > +#include "dev-ap9x-pci.h"
> > +#include "dev-wmac.h"
> > +#include "machtypes.h"
> > +
> > +#define ALFA_AP120C_GPIO_LED       0
> > +
> > +#define ALFA_AP120C_GPIO_BUTTON_WIFI       16
> > +
> > +#define ALFA_AP120C_GPIO_WATCH_DOG 20
> > +
> > +#define ALFA_AP120C_KEYS_POLL_INTERVAL             20      /* msecs */
> > +#define ALFA_AP120C_KEYS_DEBOUNCE_INTERVAL (3 * 
> > ALFA_AP120C_KEYS_POLL_INTERVAL)
> > +
> > +#define ALFA_AP120C_MAC_OFFSET             0x1002
> > +#define ALFA_AP120C_CAL0_OFFSET            0x1000
> > +
> > +static struct gpio_keys_button alfa_ap120c_gpio_keys[] __initdata = {
> > +   {
> > +           .desc           = "Wireless button",
> > +           .type           = EV_KEY,
> > +           .code           = KEY_RFKILL,
> > +           .debounce_interval = ALFA_AP120C_KEYS_DEBOUNCE_INTERVAL,
> > +           .gpio           = ALFA_AP120C_GPIO_BUTTON_WIFI,
> > +           .active_low     = 1,
> > +   }
> > +};
> > +
> > +static struct gpio_led alfa_ap120c_leds_gpio[] __initdata = {
> > +   {
> > +                   .name           = "ap120c:red:wlan",
> > +                   .gpio           = ALFA_AP120C_GPIO_LED,
> > +                   .active_low = 0,
> > +   }
> > +};
> > +
> > +static struct ar8327_pad_cfg ap120c_ar8327_pad0_cfg = {
> > +   .mode = AR8327_PAD_MAC_RGMII,
> > +   .txclk_delay_en = true,
> > +   .rxclk_delay_en = true,
> > +   .txclk_delay_sel = AR8327_CLK_DELAY_SEL1,
> > +   .rxclk_delay_sel = AR8327_CLK_DELAY_SEL2,
> > +};
> > +
> > +static struct ar8327_platform_data ap120c_ar8327_data = {
> > +   .pad0_cfg = &ap120c_ar8327_pad0_cfg,
> > +   .port0_cfg = {
> > +           .force_link = 1,
> > +           .speed = AR8327_PORT_SPEED_1000,
> > +           .duplex = 1,
> > +           .txpause = 1,
> > +           .rxpause = 1,
> > +   },
> > +};
> > +
> > +static struct mdio_board_info ap120c_mdio0_info[] = {
> > +   {
> > +           .bus_id = "ag71xx-mdio.0",
> > +           .phy_addr = 0,
> > +           .platform_data = &ap120c_ar8327_data,
> > +   },
> > +};
> > +
> > +static struct flash_platform_data flash __initdata = { NULL, NULL, 0 };
> > +
> > +#define ALFA_AP120C_LAN_PHYMASK            BIT(5)
> > +#define ALFA_AP120C_MDIO_PHYMASK   ALFA_AP120C_LAN_PHYMASK
> > +
> > +static void __init alfa_ap120c_init(void)
> > +{
> > +   u8 *art = (u8 *) KSEG1ADDR(0x1fff0000);
> > +   u8 mac[ETH_ALEN];
> > +
> > +   struct ath9k_platform_data *pdata;
> > +
> > +   ath79_register_leds_gpio(-1, ARRAY_SIZE(alfa_ap120c_leds_gpio),
> > +                                                           
> > alfa_ap120c_leds_gpio);
> 
> align with open parenthesis please
> 
> > +   ath79_register_gpio_keys_polled(-1, ALFA_AP120C_KEYS_POLL_INTERVAL,
> > +                                                                   
> > ARRAY_SIZE(alfa_ap120c_gpio_keys),
> > +                                                                   
> > alfa_ap120c_gpio_keys);
> > +
> 
> 
> align with open parenthesis please
> 
> > +   ath79_gpio_function_enable(AR71XX_GPIO_FUNC_SPI_CS1_EN |
> > +                                                           
> > AR71XX_GPIO_FUNC_SPI_CS2_EN);
> > +
> 
> align with open parenthesis please
> 
> > +   ath79_register_m25p80_multi(&flash);
> > +
> > +   ath79_init_mac(mac, art + ALFA_AP120C_MAC_OFFSET, 1);
> > +   ath79_register_wmac(art + ALFA_AP120C_CAL0_OFFSET, mac);
> > +
> > +   ath79_init_mac(mac, art + ALFA_AP120C_MAC_OFFSET, 2);
> > +   ap91_pci_init(NULL, mac);
> > +   pdata = ap9x_pci_get_wmac_data(0);
> > +   if (!pdata) {
> > +           pr_err("ap120c: unable to get address of wlan data\n");
> > +           return;
> > +   }
> > +   pdata->use_eeprom = true;
> > +
> > +   ath79_setup_ar934x_eth_cfg(AR934X_ETH_CFG_RGMII_GMAC0 |
> > +                                                           BIT(15) | 
> > BIT(17) | BIT(19) | BIT(21));
> > +
> 
> align with open parenthesis please
> 
> > +   ath79_register_mdio(0, 0x0);
> > +
> > +   ath79_init_mac(ath79_eth0_data.mac_addr, art + ALFA_AP120C_MAC_OFFSET, 
> > 0);
> > +
> > +   mdiobus_register_board_info(ap120c_mdio0_info, 
> > ARRAY_SIZE(ap120c_mdio0_info));
> > +
> > +   ath79_eth0_data.phy_if_mode = PHY_INTERFACE_MODE_RGMII;
> > +   ath79_eth0_data.phy_mask = ALFA_AP120C_LAN_PHYMASK;
> > +
> > +   ath79_eth0_pll_data.pll_1000 = 0x42000000;
> > +   ath79_eth0_pll_data.pll_10 = 0x00001313;
> > +
> > +   ath79_eth0_data.mii_bus_dev = &ath79_mdio0_device.dev;
> > +
> > +   ath79_register_eth(0);
> > +}
> > +
> > +MIPS_MACHINE(ATH79_MACH_ALFA_AP120C, "ALFA-AP120C", "ALFA Network AP120C",
> > +                   alfa_ap120c_init);
> > 
> align with open parenthesis please
> 
> 
> > Modified: trunk/target/linux/ar71xx/files/arch/mips/ath79/machtypes.h
> > ===================================================================
> > --- trunk/target/linux/ar71xx/files/arch/mips/ath79/machtypes.h     
> > 2016-03-23 12:52:50 UTC (rev 49081)
> > +++ trunk/target/linux/ar71xx/files/arch/mips/ath79/machtypes.h     
> > 2016-03-23 22:31:38 UTC (rev 49082)
> > @@ -18,6 +18,7 @@
> >     ATH79_MACH_GENERIC_OF = -1,     /* Device tree board */
> >     ATH79_MACH_GENERIC = 0,
> >     ATH79_MACH_ALFA_AP96,           /* ALFA Network AP96 board */
> > +   ATH79_MACH_ALFA_AP120C,         /* ALFA Network AP120 board */
> 
> here is is added after 96
> 
> >     ATH79_MACH_ALFA_NX,             /* ALFA Network N2/N5 board */
> >     ATH79_MACH_ALL0258N,            /* Allnet ALL0258N */
> >     ATH79_MACH_ALL0305,             /* Allnet ALL0305 */
> > 
> > Modified: trunk/target/linux/ar71xx/generic/profiles/alfa.mk
> > ===================================================================
> > --- trunk/target/linux/ar71xx/generic/profiles/alfa.mk      2016-03-23 
> > 12:52:50 UTC (rev 49081)
> > +++ trunk/target/linux/ar71xx/generic/profiles/alfa.mk      2016-03-23 
> > 22:31:38 UTC (rev 49082)
> > @@ -1,10 +1,22 @@
> >  #
> > -# Copyright (C) 2011-2013 OpenWrt.org
> > +# Copyright (C) 2011-2016 OpenWrt.org
> >  #
> >  # This is free software, licensed under the GNU General Public License v2.
> >  # See /LICENSE for more information.
> >  #
> >  
> > +define Profile/ALFAAP120C
> > +   NAME:=ALFA Network AP120C board
> > +   PACKAGES:=uboot-envtools
> > +endef
> > +
> > +define Profile/ALFAAP120C/Description
> > +   Package set optimized for the ALFA Network AP120C board
> > +endef
> > +
> > +$(eval $(call Profile,ALFAAP120C))
> > +
> > +
> 
> here is is added before 96
> 
> >  define Profile/ALFAAP96
> >     NAME:=ALFA Network AP96 board
> >     PACKAGES:=kmod-usb-core kmod-usb-ohci kmod-usb2 kmod-rtc-pcf2123
> > 
> > Modified: trunk/target/linux/ar71xx/image/Makefile
> > ===================================================================
> > --- trunk/target/linux/ar71xx/image/Makefile        2016-03-23 12:52:50 UTC 
> > (rev 49081)
> > +++ trunk/target/linux/ar71xx/image/Makefile        2016-03-23 22:31:38 UTC 
> > (rev 49082)
> > @@ -1557,6 +1557,7 @@
> >  endef
> >  
> >  
> > alfa_ap96_mtdlayout=mtdparts=spi0.0:256k(u-boot)ro,256k(u-boot-env)ro,13312k(rootfs),2048k(kernel),512k(caldata)ro,15360k@0x80000(firmware)
> > +alfa_ap120c_mtdlayout=mtdparts=spi0.0:256k(u-boot)ro,64k(u-boot-env)ro,13312k(rootfs),1536k(kernel),1152k(unknown)ro,64k(art)ro;spi0.1:-(unknown)
> 
> here is is added after 96
> 
> >  
> > alfa_mtdlayout_8M=mtdparts=spi0.0:256k(u-boot)ro,64k(u-boot-env)ro,6144k(rootfs),1600k(kernel),64k(nvram),64k(art)ro,7744k@0x50000(firmware)
> >  
> > alfa_mtdlayout_16M=mtdparts=spi0.0:256k(u-boot)ro,64k(u-boot-env)ro,15936k(firmware),64k(nvram),64k(art)ro
> >  
> > all0258n_mtdlayout=mtdparts=spi0.0:256k(u-boot),64k(u-boot-env),6272k(firmware),1536k(failsafe),64k(art)
> > @@ -2357,6 +2358,7 @@
> >  $(eval $(call 
> > SingleProfile,AthGzip,64k,WNDAP360,wndap360,WNDAP360,ttyS0,9600,$$(wndap360_mtdlayout),KRuImage))
> >  
> >  $(eval $(call 
> > SingleProfile,AthLzma,64k,ALFAAP96,alfa-ap96,ALFA-AP96,ttyS0,115200,$$(alfa_ap96_mtdlayout),RKuImage))
> > +$(eval $(call 
> > SingleProfile,AthLzma,64k,ALFAAP120C,alfa-ap120c,ALFA-AP120C,ttyS0,115200,$$(alfa_ap120c_mtdlayout),RKuImage))
> 
> here is is added after 96
> 
> >  $(eval $(call 
> > SingleProfile,AthLzma,64k,ALL0258N,all0258n,ALL0258N,ttyS0,115200,$$(all0258n_mtdlayout),KRuImage,65536))
> >  $(eval $(call 
> > SingleProfile,AthLzma,256k,ALL0315N,all0315n,ALL0315N,ttyS0,115200,$$(all0315n_mtdlayout),KRuImage,262144))
> >  $(eval $(call 
> > SingleProfile,AthLzma,64k,AP113,ap113,AP113,ttyS0,115200,$$(ap113_mtd_layout),RK))
> > _______________________________________________
> > openwrt-commits mailing list
> > openwrt-comm...@lists.openwrt.org
> > https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-commits
> > 
_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel

Reply via email to