From patchwork Sat Dec 9 21:03:00 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Helge Deller X-Patchwork-Id: 10103811 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 1448A602A7 for ; Sat, 9 Dec 2017 21:03:09 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 01FDA2870E for ; Sat, 9 Dec 2017 21:03:09 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id E754628841; Sat, 9 Dec 2017 21:03:08 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.9 required=2.0 tests=BAYES_00,FREEMAIL_FROM, RCVD_IN_DNSWL_HI,T_TVD_MIME_EPI autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 7FE312870E for ; Sat, 9 Dec 2017 21:03:08 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751263AbdLIVDI (ORCPT ); Sat, 9 Dec 2017 16:03:08 -0500 Received: from mout.gmx.net ([212.227.15.15]:49323 "EHLO mout.gmx.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751243AbdLIVDH (ORCPT ); Sat, 9 Dec 2017 16:03:07 -0500 Received: from [192.168.20.60] ([193.159.26.22]) by mail.gmx.com (mrgmx001 [212.227.17.190]) with ESMTPSA (Nemesis) id 0M4GRv-1fFSr82Ddj-00rncN; Sat, 09 Dec 2017 22:03:01 +0100 Subject: Re: Have my PA8800 back online... To: John David Anglin , Frank Scheiner Cc: debian-hppa@lists.debian.org References: <2ADB5C8A-DFEB-4CA5-92BA-96E459A3575E@bell.net> <8314a5d6-7df7-3282-0d91-a9b414a122e0@web.de> <526274E4-88D8-4DF8-8F74-5B775186BBEC@bell.net> <48320506-f7fa-822b-fb45-40eab1dbda02@bell.net> <17707adb-4f71-1d66-2a19-3cdfaff047f3@gmx.de> <53815372-58e8-70e2-bab4-1777e848cf5e@web.de> <79c110ec-2975-a827-4b9d-1351ab77779b@gmx.de> From: Helge Deller Message-ID: Date: Sat, 9 Dec 2017 22:03:00 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 In-Reply-To: Content-Language: en-US X-Provags-ID: V03:K0:sKWbfEBnlvmyI14bAO0TAunSfNASuGx0jCXTNW5pz6CMYnrXizv CGYzq76awIR02BfudteB4e95vPAlzwZjOFuE26kQoGcvaW/w8B2SOlDPcNJEH+divMFsNT3 8L8FTDoWuYuKb4AZ29AHAKU15diqKahBDAZkd+llwms06jy8bVFJE7scoJVjyXnq4S4VEBV mslsZokKCaoJ4i1EDqxNw== X-UI-Out-Filterresults: notjunk:1; V01:K0:XWEL1KXw2/I=:2PmJpkOteNYK/IualXNBkr wFCzXeQCblPWYDaQgaXdjqdPzlL1tVWVkT9F+kWjZ5a1lYkkMoWDvZeoNcisBHFiIQr8+SlGa 4TkUrOiF+f+i5Pw3vQpeCz987E0UQRo622iP/QvzXzpGoOhSz1PzeSdqykKs4AHlkkCmyo7M4 AOF2x5ps6Vc5KXTdWcmmnRTLIZs0hVJ59LpxCTRsWqsGqVs+4gWpIpDIpUNllirpYrgV/dF5e H4ENbntu5Ryd6nTtxTQPGikEjLnZT2Nd5kfz0K2j+C18CZlWCwJEdwcUF8JykrWf4VPfA1AgQ OITDQp+oVIEKwX41jlUVjuZkpYYB8TsoamqGzjruxjw3vBwJ8crvFGuIyeoJ5kB3h8unzwY3E 5lGczsig7YN+F7fMzKCgGPiU9Lkmdl1uVRAo+5yjwl3gTBUuQi6KBOnsyZRT20AOOgxgK/6aC 1FE1ff/lyzmJyIrBxhc0nAvMeY+VKFMtjlks3yrofhVgSRq2Efq7FtxxpedI5oXUkCa0Fz/Op xaatQOsqPBkOJOhDLs7VhrND0Agc0ND1yqqN/8MTfLRTBxRfIjMs482XzGg3dlh8P9oRPuxHa AzJ3bvsZM84rWqMsWao0jRsuVW8u0S22DLz9uCQHba9Rc+ZrIpzBh5lBZxjo9S7daQbs17+f2 xe/rEHZJ4qZR+MUqzRJOcXdflCAVWRqdRJdfG09c+Eo5OiEvrWmIeO4zUFdguYDLVs/0h9gky Dmoxi0C1qGSptjnd/kI/nYz41/MzMKlEiba8ZQh9gA4Ld9W84DNndbAPPRUswFfLMWiIpssl5 9JUqKVLa2IAWpyrEfW3djJRTtXIQw== Sender: linux-parisc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-parisc@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP On 08.12.2017 23:14, John David Anglin wrote: > On 2017-12-08 4:29 PM, Helge Deller wrote: >>> root@rp3440:~# lspci -nn >>> [...] >>> e0:01.0 Communication controller [0780]: Hewlett-Packard Company Auxiliary Diva Serial Port [103c:1290] (rev 01) >>> e0:01.1 Serial controller [0700]: Hewlett-Packard Company Diva Serial [GSP] Multiport UART [103c:1048] (rev 03) >>> >>> Interesting - actually the current situation looks like what [your patch from May 29th] should have accomplished. But as you dropped it in favor of a patch to palo, it seems to be unrelated. >>> >>> [your patch from May 29th]:https://patchwork.kernel.org/patch/9753613/ >> The patch had some other issues which is why I dropped it. >> I like the testing of the class id much more, as it's done by the current upstream code. >> E.g. this diva port reports PCI_CLASS_COMMUNICATION_OTHER as PCI CLASS ID, which differs to what the serial port driver should handle. >> > Yes, you previously noted that this port has no external connector or access via Ethernet.  I suspect it talks to some internal module. > I don't know if there's any reason to access the device it's connected to (if any).  It probably should be "ttyO0". > > Although the renumbering of the serial ports is somewhat annoying Yes, it's really annoying. Especially right now, where all kernels >= 4.14 need ttyS0, while older onese require ttyS1 as boot console. It's confusing. > maybe palo change should be reverted so default is always ttyS0 Yes, agreed. With the current situation we are not better with preselecting ttyS1 on rp34x0 machines. > I would like the enumeration of the RV100 port skipped as the implementation on the rp34xx machines is broken and can't be used. The RV100 port refers here to the built-in ATI Radeon 7000 graphics card which is part of Diva management card. It's not working on rp34x0 machines. > It's enumeration means one has to explicitly specify the console argument in the boot command line as there seems to no way to > prefer the serial console port. Can you please try attached patch which disables the serial MUX and ATI card? If it works for you and if we backport it to all kernels and if we revert palo to use ttyS0 for all machines we might be good. Helge diff --git a/drivers/parisc/lba_pci.c b/drivers/parisc/lba_pci.c index a25fed52f7e9..8e42827d3c27 100644 --- a/drivers/parisc/lba_pci.c +++ b/drivers/parisc/lba_pci.c @@ -1692,3 +1692,39 @@ void lba_set_iregs(struct parisc_device *lba, u32 ibase, u32 imask) iounmap(base_addr); } + +/* + * The design of the Diva management card in rp34x0 machines (rp3410, rp3440) + * seems rushed, so that many built-in components simply don't work. + * The following quirks disable the serial AUX port and the built-in ATI RV100 + * Radeon 7000 graphics card which both don't have any external connectors and + * thus are useless, and even worse, e.g. the AUX ports occupies ttyS0 and + * as such makes those machines the only PARISC machines on which we can't + * use ttyS0 as boot console. + */ +static void quirk_diva_ati_card(struct pci_dev *dev) +{ + /* subsystem IDs are from Diva */ + if (dev->subsystem_vendor != PCI_VENDOR_ID_HP || + dev->subsystem_device != 0x1292) + return; + + dev_info(dev, "Hiding Diva built-in ATI card."); + dev->device = 0; +} +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_ATI, PCI_DEVICE_ID_ATI_RADEON_QY, + quirk_diva_ati_card); + +static void quirk_diva_aux_disable(struct pci_dev *dev) +{ + /* subsystem IDs are from Diva */ + if (dev->subsystem_vendor != PCI_VENDOR_ID_HP || + dev->subsystem_device != 0x1291) + return; + + dev_info(dev, "Hiding Diva built-in AUX serial device"); + dev->device = 0; +} +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_HP, PCI_DEVICE_ID_HP_DIVA_AUX, + quirk_diva_aux_disable); +