From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751455AbcABXd0 (ORCPT ); Sat, 2 Jan 2016 18:33:26 -0500 Received: from domu-toccata.ens-lyon.fr ([140.77.166.138]:38006 "EHLO sonata.ens-lyon.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750972AbcABXdX (ORCPT ); Sat, 2 Jan 2016 18:33:23 -0500 X-Greylist: delayed 470 seconds by postgrey-1.27 at vger.kernel.org; Sat, 02 Jan 2016 18:33:23 EST Date: Sun, 3 Jan 2016 00:25:29 +0100 From: Samuel Thibault To: William Hubbs , Chris Brannon , Kirk Reiser , Greg Kroah-Hartman , speakup@linux-speakup.org, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: [PATCH] Staging: speakup: Fix getting port information Message-ID: <20160102232529.GC2860@var.home> Mail-Followup-To: Samuel Thibault , William Hubbs , Chris Brannon , Kirk Reiser , Greg Kroah-Hartman , speakup@linux-speakup.org, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.5.21+34 (58baf7c9f32f) (2010-12-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 5e6dc54 broke the port information in the speakup driver: SERIAL_PORT_DFNS only gets defined if asm/serial.h is included. Along the way, make sure that we do have information for the requested serial port number (index) Signed-off-by: Samuel Thibault --- a/drivers/staging/speakup/serialio.c +++ b/drivers/staging/speakup/serialio.c @@ -6,6 +6,9 @@ #include "spk_priv.h" #include "serialio.h" +#include +#include + #ifndef SERIAL_PORT_DFNS #define SERIAL_PORT_DFNS #endif @@ -26,6 +29,11 @@ const struct old_serial_port *spk_serial const struct old_serial_port *ser = rs_table + index; int err; + if (index > sizeof(rs_table) / sizeof(*rs_table)) { + pr_info("no port info for ttyS%d\n", index); + return NULL; + } + /* Divisor, bytesize and parity */ quot = ser->baud_base / baud; cval = cflag & (CSIZE | CSTOPB);