Hi Geert, I love your patch! Yet something to improve: [auto build test ERROR on tty/tty-testing] [cannot apply to v5.4-rc1 next-20191001] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system. BTW, we also suggest to use '--base' option to specify the base tree in git format-patch, please see https://stackoverflow.com/a/37406982] url: https://github.com/0day-ci/linux/commits/Geert-Uytterhoeven/serial-sh-sci-Use-platform_get_irq_optional-for-optional-interrupts/20191002-171547 base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/tty.git tty-testing config: sparc64-allmodconfig (attached as .config) compiler: sparc64-linux-gcc (GCC) 7.4.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config to linux build tree GCC_VERSION=7.4.0 make.cross ARCH=sparc64 If you fix the issue, kindly add following tag Reported-by: kbuild test robot All errors (new ones prefixed by >>): drivers/tty/serial/sh-sci.c: In function 'sci_init_single': >> drivers/tty/serial/sh-sci.c:2899:24: error: implicit declaration of function 'platform_get_irq_optional'; did you mean 'platform_get_irq_byname'? [-Werror=implicit-function-declaration] sci_port->irqs[i] = platform_get_irq_optional(dev, i); ^~~~~~~~~~~~~~~~~~~~~~~~~ platform_get_irq_byname cc1: some warnings being treated as errors vim +2899 drivers/tty/serial/sh-sci.c 2874 2875 static int sci_init_single(struct platform_device *dev, 2876 struct sci_port *sci_port, unsigned int index, 2877 const struct plat_sci_port *p, bool early) 2878 { 2879 struct uart_port *port = &sci_port->port; 2880 const struct resource *res; 2881 unsigned int i; 2882 int ret; 2883 2884 sci_port->cfg = p; 2885 2886 port->ops = &sci_uart_ops; 2887 port->iotype = UPIO_MEM; 2888 port->line = index; 2889 2890 res = platform_get_resource(dev, IORESOURCE_MEM, 0); 2891 if (res == NULL) 2892 return -ENOMEM; 2893 2894 port->mapbase = res->start; 2895 sci_port->reg_size = resource_size(res); 2896 2897 for (i = 0; i < ARRAY_SIZE(sci_port->irqs); ++i) { 2898 if (i) > 2899 sci_port->irqs[i] = platform_get_irq_optional(dev, i); 2900 else 2901 sci_port->irqs[i] = platform_get_irq(dev, i); 2902 } 2903 2904 /* The SCI generates several interrupts. They can be muxed together or 2905 * connected to different interrupt lines. In the muxed case only one 2906 * interrupt resource is specified as there is only one interrupt ID. 2907 * In the non-muxed case, up to 6 interrupt signals might be generated 2908 * from the SCI, however those signals might have their own individual 2909 * interrupt ID numbers, or muxed together with another interrupt. 2910 */ 2911 if (sci_port->irqs[0] < 0) 2912 return -ENXIO; 2913 2914 if (sci_port->irqs[1] < 0) 2915 for (i = 1; i < ARRAY_SIZE(sci_port->irqs); i++) 2916 sci_port->irqs[i] = sci_port->irqs[0]; 2917 2918 sci_port->params = sci_probe_regmap(p); 2919 if (unlikely(sci_port->params == NULL)) 2920 return -EINVAL; 2921 2922 switch (p->type) { 2923 case PORT_SCIFB: 2924 sci_port->rx_trigger = 48; 2925 break; 2926 case PORT_HSCIF: 2927 sci_port->rx_trigger = 64; 2928 break; 2929 case PORT_SCIFA: 2930 sci_port->rx_trigger = 32; 2931 break; 2932 case PORT_SCIF: 2933 if (p->regtype == SCIx_SH7705_SCIF_REGTYPE) 2934 /* RX triggering not implemented for this IP */ 2935 sci_port->rx_trigger = 1; 2936 else 2937 sci_port->rx_trigger = 8; 2938 break; 2939 default: 2940 sci_port->rx_trigger = 1; 2941 break; 2942 } 2943 2944 sci_port->rx_fifo_timeout = 0; 2945 sci_port->hscif_tot = 0; 2946 2947 /* SCIFA on sh7723 and sh7724 need a custom sampling rate that doesn't 2948 * match the SoC datasheet, this should be investigated. Let platform 2949 * data override the sampling rate for now. 2950 */ 2951 sci_port->sampling_rate_mask = p->sampling_rate 2952 ? SCI_SR(p->sampling_rate) 2953 : sci_port->params->sampling_rate_mask; 2954 2955 if (!early) { 2956 ret = sci_init_clocks(sci_port, &dev->dev); 2957 if (ret < 0) 2958 return ret; 2959 2960 port->dev = &dev->dev; 2961 2962 pm_runtime_enable(&dev->dev); 2963 } 2964 2965 port->type = p->type; 2966 port->flags = UPF_FIXED_PORT | UPF_BOOT_AUTOCONF | p->flags; 2967 port->fifosize = sci_port->params->fifosize; 2968 2969 if (port->type == PORT_SCI) { 2970 if (sci_port->reg_size >= 0x20) 2971 port->regshift = 2; 2972 else 2973 port->regshift = 1; 2974 } 2975 2976 /* 2977 * The UART port needs an IRQ value, so we peg this to the RX IRQ 2978 * for the multi-IRQ ports, which is where we are primarily 2979 * concerned with the shutdown path synchronization. 2980 * 2981 * For the muxed case there's nothing more to do. 2982 */ 2983 port->irq = sci_port->irqs[SCIx_RXI_IRQ]; 2984 port->irqflags = 0; 2985 2986 port->serial_in = sci_serial_in; 2987 port->serial_out = sci_serial_out; 2988 2989 return 0; 2990 } 2991 --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation