From mboxrd@z Thu Jan 1 00:00:00 1970 From: Philip Prindeville Subject: [PATCH v2 1/1] solos-pci: Fix regression introduced by newest firmware Date: Sun, 20 Mar 2011 18:52:15 -0700 Message-ID: <4D86AF4F.5010203@redfish-solutions.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: "linux-atm-general@lists.sourceforge.net" To: Netdev Return-path: Received: from mail.redfish-solutions.com ([66.232.79.143]:49099 "EHLO mail.redfish-solutions.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751793Ab1CUBwc (ORCPT ); Sun, 20 Mar 2011 21:52:32 -0400 Sender: netdev-owner@vger.kernel.org List-ID: The newest FPGA firmware on the Solos processors correctly signals carrier transitions, bitrate, etc. The driver previously ignored these messages, and the physical state was always ATM_PHY_SIG_UNKNOWN. Now that the board reports its state, we expose a bug whereby the transition from UNKNOWN to LOST causes us to release all VC's. We don't delete any VC's, but instead just send an indication of carrier change. Signed-off-by: Philip A Prindeville --- --- a/drivers/atm/solos-pci.c 2011-03-20 15:27:40.000000000 -0600 +++ b/drivers/atm/solos-pci.c 2011-03-20 16:32:11.000000000 -0600 @@ -382,8 +382,10 @@ static int process_status(struct solos_c /* Anything but 'Showtime' is down */ if (strcmp(state_str, "Showtime")) { atm_dev_signal_change(card->atmdev[port], ATM_PHY_SIG_LOST); +#if 0 atm_dev_release_vccs(card->atmdev[port]); +#endif dev_info(&card->dev->dev, "Port %d: %s\n", port, state_str); return 0; }