[{"id":1761459,"web_url":"http://patchwork.ozlabs.org/comment/1761459/","msgid":"<CAOSf1CGgjNYr=YKVx8g=jJ=PV7fZxgB_5RQaf2zshDvf8-M-=g@mail.gmail.com>","list_archive_url":null,"date":"2017-09-01T04:07:48","subject":"Re: [Skiboot] [PATCH v2 3/5] hdata/i2c: update the list of known\n\ti2c devs as of version 2","submitter":{"id":68108,"url":"http://patchwork.ozlabs.org/api/people/68108/","name":"Oliver O'Halloran","email":"oohall@gmail.com"},"content":"On Thu, Aug 31, 2017 at 5:12 PM, Claudio Carvalho\n<cclaudio@linux.vnet.ibm.com> wrote:\n> Some i2c devices are not properly identified during the HDAT parsing.\n>\n> This fixes that by updating the list of known i2c devices as of the i2c\n> array version 2.\n>\n> Signed-off-by: Claudio Carvalho <cclaudio@linux.vnet.ibm.com>\n> ---\n>  hdata/i2c.c | 13 +++++++++++--\n>  1 file changed, 11 insertions(+), 2 deletions(-)\n>\n> diff --git a/hdata/i2c.c b/hdata/i2c.c\n> index bfb5b48..14650b6 100644\n> --- a/hdata/i2c.c\n> +++ b/hdata/i2c.c\n> @@ -91,8 +91,14 @@ struct hdat_i2c_type {\n>  };\n>\n>  struct hdat_i2c_type hdat_i2c_devs[] = {\n> +       { 0x1, \"gpio\", \"nxp,pca9551\" },\n>         /* XXX: Please verify that all VPD EEPROMs are of this type */\n> -       { 0x2, \"eeprom\", \"atmel,24c128\" }\n> +       { 0x2, \"eeprom\", \"atmel,24c128\" },\n> +       { 0x3, \"tpm\", \"nuvoton,npct650\" },\n> +       { 0x6, \"gpio\", \"nxp,pca9552\" },\n> +       { 0x7, \"gpio\", \"nxp,pca9553\" },\n> +       { 0x8, \"gpio\", \"nxp,pca9554\" },\n> +       { 0x9, \"gpio\", \"nxp,pca9555\" }\n>  };\n>\n>  struct hdat_i2c_label {\n> @@ -103,11 +109,14 @@ struct hdat_i2c_label {\n>  struct hdat_i2c_label hdat_i2c_labels[] = {\n>         { 0x1, \"9551-led-controller\" },\n>         { 0x2, \"seeprom\" },\n> +       { 0x4, \"tpm\" },\n>         { 0x5, \"module-vpd\" },\n>         { 0x6, \"dimm SPD\" },\n>         { 0x7, \"proc-vpd\" },\n>         { 0x8, \"sbe-eeprom\" },\n> -       { 0x9, \"planar-vpd\" }\n> +       { 0x9, \"planar-vpd\" },\n> +       { 0xd, \"secure-window-open-gpios\" },\n> +       { 0xe, \"physical-presence-gpios\" }\n>  };\n>\n>  /*\n> --\n> 2.7.4\n>\n\nReviewed-by: Oliver O'Halloran <oohall@gmail.com>","headers":{"Return-Path":"<skiboot-bounces+incoming=patchwork.ozlabs.org@lists.ozlabs.org>","X-Original-To":["incoming@patchwork.ozlabs.org","skiboot@lists.ozlabs.org"],"Delivered-To":["patchwork-incoming@bilbo.ozlabs.org","skiboot@lists.ozlabs.org"],"Received":["from lists.ozlabs.org (lists.ozlabs.org [103.22.144.68])\n\t(using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xk5Ll3dRpz9sPt\n\tfor <incoming@patchwork.ozlabs.org>;\n\tFri,  1 Sep 2017 14:07:59 +1000 (AEST)","from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\tby lists.ozlabs.org (Postfix) with ESMTP id 3xk5Ll26pfzDqjC\n\tfor <incoming@patchwork.ozlabs.org>;\n\tFri,  1 Sep 2017 14:07:59 +1000 (AEST)","from mail-qk0-x241.google.com (mail-qk0-x241.google.com\n\t[IPv6:2607:f8b0:400d:c09::241])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128\n\tbits)) (No client certificate requested)\n\tby lists.ozlabs.org (Postfix) with ESMTPS id 3xk5LZ6wYjzDqY8\n\tfor <skiboot@lists.ozlabs.org>; Fri,  1 Sep 2017 14:07:50 +1000 (AEST)","by mail-qk0-x241.google.com with SMTP id o63so1105531qkb.5\n\tfor <skiboot@lists.ozlabs.org>; Thu, 31 Aug 2017 21:07:50 -0700 (PDT)","by 10.140.21.10 with HTTP; Thu, 31 Aug 2017 21:07:48 -0700 (PDT)"],"Authentication-Results":["ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"NPGrFi1W\"; dkim-atps=neutral","lists.ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"NPGrFi1W\"; dkim-atps=neutral","lists.ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"NPGrFi1W\"; dkim-atps=neutral"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=mime-version:in-reply-to:references:from:date:message-id:subject:to\n\t:cc; bh=a3gnxq11CAtpCPe42wdQH9oLj5Ns7I0/mhy4mHqyISo=;\n\tb=NPGrFi1WH9+COODTkSyL8w0UMnJZbKHX6BBIQj9sbO79VJUEnkI1N2aYnfKzn/y98b\n\tcKhZzoNuVni+DAAXV3kfW1pCgpEm4u1TXQ3G17n7F6p9foEqGusLmVZaa42Msilydoge\n\t2VL1MYcUearS91eGR4vyRpHZRw4ADdWziZ2Yts4URveAHQby10tZn8FEf2lCO3TTtshb\n\tfAyDXZw9/ssQGdpL927nsdt4IVlHMoOR2w6OlINcTx5T9gmOafnnbFb76kYgIpVq8QQD\n\tN/TMWo+OiVLkkkXrU0fUfYx2uawm8+mhP0tvVhFDPRqavAmmRK3SBOw7Ef1xiOP5JSfU\n\tTyXw==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:in-reply-to:references:from:date\n\t:message-id:subject:to:cc;\n\tbh=a3gnxq11CAtpCPe42wdQH9oLj5Ns7I0/mhy4mHqyISo=;\n\tb=Ygze9nw1yn1JbHStIesGc3jt69mRXi/KcXqhVsMC7fQaCkbjhmFpVZ48wqttlMgqCo\n\tiDot+/i6ANWYzJn/5CTxaoFot4hiqOJ8PjbRACywD9xHoCUXuFGQOEQRN1E9pGs5I6g2\n\ti1uAn/vtUULfNdAFZkhL57QhYlufAHQXaPaFKY8r1kuB6OWR3vKhzrJ0+FWxdyqohoVz\n\tc1ZwLYgDfihyC0ns7idzs+eZVtMn1+5ZGPvq0zOf/jHXb3epBKED46jbdm7KD60Y1brX\n\twETYVqs4V1JU/vE3thkKQcmooKk7GdSQdXxRaUKVwHNOWNU+lITlwyJopl8JvI+ugg+b\n\tueAw==","X-Gm-Message-State":"AHPjjUgOuU2bBUwyjHWvN7YaTjal28aaduzQRyJ6vdfEeRto5bPVlsMq\n\tr34GQB2AuZZ6Bf/QoQTe81/HRJah/w==","X-Google-Smtp-Source":"ADKCNb41KavZkpshKw3nOro5Q1bg8nyWet657s8JZBqzKgNX324fTBGXEirDd9S2ywWFTvEjmpi6yPeJ2LAmQFnab7Q=","X-Received":"by 10.55.165.21 with SMTP id o21mr816297qke.338.1504238869070;\n\tThu, 31 Aug 2017 21:07:49 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<1504163557-14432-4-git-send-email-cclaudio@linux.vnet.ibm.com>","References":"<1504163557-14432-1-git-send-email-cclaudio@linux.vnet.ibm.com>\n\t<1504163557-14432-4-git-send-email-cclaudio@linux.vnet.ibm.com>","From":"Oliver <oohall@gmail.com>","Date":"Fri, 1 Sep 2017 14:07:48 +1000","Message-ID":"<CAOSf1CGgjNYr=YKVx8g=jJ=PV7fZxgB_5RQaf2zshDvf8-M-=g@mail.gmail.com>","To":"Claudio Carvalho <cclaudio@linux.vnet.ibm.com>","Subject":"Re: [Skiboot] [PATCH v2 3/5] hdata/i2c: update the list of known\n\ti2c devs as of version 2","X-BeenThere":"skiboot@lists.ozlabs.org","X-Mailman-Version":"2.1.23","Precedence":"list","List-Id":"Mailing list for skiboot development <skiboot.lists.ozlabs.org>","List-Unsubscribe":"<https://lists.ozlabs.org/options/skiboot>,\n\t<mailto:skiboot-request@lists.ozlabs.org?subject=unsubscribe>","List-Archive":"<http://lists.ozlabs.org/pipermail/skiboot/>","List-Post":"<mailto:skiboot@lists.ozlabs.org>","List-Help":"<mailto:skiboot-request@lists.ozlabs.org?subject=help>","List-Subscribe":"<https://lists.ozlabs.org/listinfo/skiboot>,\n\t<mailto:skiboot-request@lists.ozlabs.org?subject=subscribe>","Cc":"skiboot list <skiboot@lists.ozlabs.org>","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"skiboot-bounces+incoming=patchwork.ozlabs.org@lists.ozlabs.org","Sender":"\"Skiboot\"\n\t<skiboot-bounces+incoming=patchwork.ozlabs.org@lists.ozlabs.org>"}}]