[{"id":1755859,"web_url":"http://patchwork.ozlabs.org/comment/1755859/","msgid":"<2faa9545-3440-9d37-1b4f-56595d704b93@au1.ibm.com>","date":"2017-08-24T04:45:37","subject":"Re: [PATCH kernel 3/5] powerpc/eeh: Remove unnecessary pointer to\n\tphb from eeh_dev","submitter":{"id":66955,"url":"http://patchwork.ozlabs.org/api/people/66955/","name":"Andrew Donnellan","email":"andrew.donnellan@au1.ibm.com"},"content":"On 23/08/17 20:18, Alexey Kardashevskiy wrote:\n> The eeh_dev struct already holds a pointer to pci_dn which it does not\n> exist without and pci_dn itself holds the very same pointer so just\n> use it.\n> \n> Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>\n\nReviewed-by: Andrew Donnellan <andrew.donnellan@au1.ibm.com>\n\n> ---\n>   arch/powerpc/include/asm/eeh.h               |  1 -\n>   arch/powerpc/kernel/eeh.c                    |  7 +++----\n>   arch/powerpc/kernel/eeh_dev.c                |  2 --\n>   arch/powerpc/kernel/eeh_driver.c             |  2 +-\n>   arch/powerpc/kernel/eeh_pe.c                 | 24 +++++++++++++-----------\n>   arch/powerpc/platforms/powernv/eeh-powernv.c |  5 ++---\n>   6 files changed, 19 insertions(+), 22 deletions(-)\n> \n> diff --git a/arch/powerpc/include/asm/eeh.h b/arch/powerpc/include/asm/eeh.h\n> index 26a6a43f8799..777d37aa0a7f 100644\n> --- a/arch/powerpc/include/asm/eeh.h\n> +++ b/arch/powerpc/include/asm/eeh.h\n> @@ -141,7 +141,6 @@ struct eeh_dev {\n>   \tstruct eeh_pe *pe;\t\t/* Associated PE\t\t*/\n>   \tstruct list_head list;\t\t/* Form link list in the PE\t*/\n>   \tstruct list_head rmv_list;\t/* Record the removed edevs\t*/\n> -\tstruct pci_controller *phb;\t/* Associated PHB\t\t*/\n>   \tstruct pci_dn *pdn;\t\t/* Associated PCI device node\t*/\n>   \tstruct pci_dev *pdev;\t\t/* Associated PCI device\t*/\n>   \tbool in_error;\t\t\t/* Error flag for edev\t\t*/\n> diff --git a/arch/powerpc/kernel/eeh.c b/arch/powerpc/kernel/eeh.c\n> index 63992b2d8e15..c877014d11ce 100644\n> --- a/arch/powerpc/kernel/eeh.c\n> +++ b/arch/powerpc/kernel/eeh.c\n> @@ -169,10 +169,10 @@ static size_t eeh_dump_dev_log(struct eeh_dev *edev, char *buf, size_t len)\n>   \tchar buffer[128];\n>   \n>   \tn += scnprintf(buf+n, len-n, \"%04x:%02x:%02x.%01x\\n\",\n> -\t\t       edev->phb->global_number, pdn->busno,\n> +\t\t       pdn->phb->global_number, pdn->busno,\n>   \t\t       PCI_SLOT(pdn->devfn), PCI_FUNC(pdn->devfn));\n>   \tpr_warn(\"EEH: of node=%04x:%02x:%02x.%01x\\n\",\n> -\t\tedev->phb->global_number, pdn->busno,\n> +\t\tpdn->phb->global_number, pdn->busno,\n>   \t\tPCI_SLOT(pdn->devfn), PCI_FUNC(pdn->devfn));\n>   \n>   \teeh_ops->read_config(pdn, PCI_VENDOR_ID, 4, &cfg);\n> @@ -1064,7 +1064,7 @@ core_initcall_sync(eeh_init);\n>    */\n>   void eeh_add_device_early(struct pci_dn *pdn)\n>   {\n> -\tstruct pci_controller *phb;\n> +\tstruct pci_controller *phb = pdn ? pdn->phb : NULL;\n>   \tstruct eeh_dev *edev = pdn_to_eeh_dev(pdn);\n>   \n>   \tif (!edev)\n> @@ -1074,7 +1074,6 @@ void eeh_add_device_early(struct pci_dn *pdn)\n>   \t\treturn;\n>   \n>   \t/* USB Bus children of PCI devices will not have BUID's */\n> -\tphb = edev->phb;\n>   \tif (NULL == phb ||\n>   \t    (eeh_has_flag(EEH_PROBE_MODE_DEVTREE) && 0 == phb->buid))\n>   \t\treturn;\n> diff --git a/arch/powerpc/kernel/eeh_dev.c b/arch/powerpc/kernel/eeh_dev.c\n> index d6b2ca70d14d..bdf4a3698a35 100644\n> --- a/arch/powerpc/kernel/eeh_dev.c\n> +++ b/arch/powerpc/kernel/eeh_dev.c\n> @@ -50,7 +50,6 @@\n>    */\n>   struct eeh_dev *eeh_dev_init(struct pci_dn *pdn)\n>   {\n> -\tstruct pci_controller *phb = pdn->phb;\n>   \tstruct eeh_dev *edev;\n>   \n>   \t/* Allocate EEH device */\n> @@ -64,7 +63,6 @@ struct eeh_dev *eeh_dev_init(struct pci_dn *pdn)\n>   \t/* Associate EEH device with OF node */\n>   \tpdn->edev = edev;\n>   \tedev->pdn = pdn;\n> -\tedev->phb = phb;\n>   \tINIT_LIST_HEAD(&edev->list);\n>   \tINIT_LIST_HEAD(&edev->rmv_list);\n>   \n> diff --git a/arch/powerpc/kernel/eeh_driver.c b/arch/powerpc/kernel/eeh_driver.c\n> index c405c79e50cd..8b840191df59 100644\n> --- a/arch/powerpc/kernel/eeh_driver.c\n> +++ b/arch/powerpc/kernel/eeh_driver.c\n> @@ -428,7 +428,7 @@ static void *eeh_add_virt_device(void *data, void *userdata)\n>   \n>   \tif (!(edev->physfn)) {\n>   \t\tpr_warn(\"%s: EEH dev %04x:%02x:%02x.%01x not for VF\\n\",\n> -\t\t\t__func__, edev->phb->global_number, pdn->busno,\n> +\t\t\t__func__, pdn->phb->global_number, pdn->busno,\n>   \t\t\tPCI_SLOT(pdn->devfn), PCI_FUNC(pdn->devfn));\n>   \t\treturn NULL;\n>   \t}\n> diff --git a/arch/powerpc/kernel/eeh_pe.c b/arch/powerpc/kernel/eeh_pe.c\n> index 84d79f3da7d6..419c3f07afd5 100644\n> --- a/arch/powerpc/kernel/eeh_pe.c\n> +++ b/arch/powerpc/kernel/eeh_pe.c\n> @@ -339,11 +339,12 @@ static struct eeh_pe *eeh_pe_get_parent(struct eeh_dev *edev)\n>   int eeh_add_to_parent_pe(struct eeh_dev *edev)\n>   {\n>   \tstruct eeh_pe *pe, *parent;\n> +\tstruct pci_dn *pdn = eeh_dev_to_pdn(edev);\n>   \n>   \t/* Check if the PE number is valid */\n>   \tif (!eeh_has_flag(EEH_VALID_PE_ZERO) && !edev->pe_config_addr) {\n>   \t\tpr_err(\"%s: Invalid PE#0 for edev 0x%x on PHB#%x\\n\",\n> -\t\t       __func__, edev->config_addr, edev->phb->global_number);\n> +\t\t       __func__, edev->config_addr, pdn->phb->global_number);\n>   \t\treturn -EINVAL;\n>   \t}\n>   \n> @@ -353,7 +354,7 @@ int eeh_add_to_parent_pe(struct eeh_dev *edev)\n>   \t * PE should be composed of PCI bus and its subordinate\n>   \t * components.\n>   \t */\n> -\tpe = eeh_pe_get(edev->pdn->phb, edev->pe_config_addr,\n> +\tpe = eeh_pe_get(pdn->phb, edev->pe_config_addr,\n>   \t\t\tedev->config_addr);\n>   \tif (pe && !(pe->type & EEH_PE_INVALID)) {\n>   \t\t/* Mark the PE as type of PCI bus */\n> @@ -363,7 +364,7 @@ int eeh_add_to_parent_pe(struct eeh_dev *edev)\n>   \t\t/* Put the edev to PE */\n>   \t\tlist_add_tail(&edev->list, &pe->edevs);\n>   \t\tpr_debug(\"EEH: Add %04x:%02x:%02x.%01x to Bus PE#%x\\n\",\n> -\t\t\tedev->phb->global_number,\n> +\t\t\t pdn->phb->global_number,\n>   \t\t\tedev->config_addr >> 8,\n>   \t\t\tPCI_SLOT(edev->config_addr & 0xFF),\n>   \t\t\tPCI_FUNC(edev->config_addr & 0xFF),\n> @@ -386,7 +387,7 @@ int eeh_add_to_parent_pe(struct eeh_dev *edev)\n>   \n>   \t\tpr_debug(\"EEH: Add %04x:%02x:%02x.%01x to Device \"\n>   \t\t\t \"PE#%x, Parent PE#%x\\n\",\n> -\t\t\tedev->phb->global_number,\n> +\t\t\t pdn->phb->global_number,\n>   \t\t\tedev->config_addr >> 8,\n>                           PCI_SLOT(edev->config_addr & 0xFF),\n>                           PCI_FUNC(edev->config_addr & 0xFF),\n> @@ -396,9 +397,9 @@ int eeh_add_to_parent_pe(struct eeh_dev *edev)\n>   \n>   \t/* Create a new EEH PE */\n>   \tif (edev->physfn)\n> -\t\tpe = eeh_pe_alloc(edev->phb, EEH_PE_VF);\n> +\t\tpe = eeh_pe_alloc(pdn->phb, EEH_PE_VF);\n>   \telse\n> -\t\tpe = eeh_pe_alloc(edev->phb, EEH_PE_DEVICE);\n> +\t\tpe = eeh_pe_alloc(pdn->phb, EEH_PE_DEVICE);\n>   \tif (!pe) {\n>   \t\tpr_err(\"%s: out of memory!\\n\", __func__);\n>   \t\treturn -ENOMEM;\n> @@ -414,10 +415,10 @@ int eeh_add_to_parent_pe(struct eeh_dev *edev)\n>   \t */\n>   \tparent = eeh_pe_get_parent(edev);\n>   \tif (!parent) {\n> -\t\tparent = eeh_phb_pe_get(edev->phb);\n> +\t\tparent = eeh_phb_pe_get(pdn->phb);\n>   \t\tif (!parent) {\n>   \t\t\tpr_err(\"%s: No PHB PE is found (PHB Domain=%d)\\n\",\n> -\t\t\t\t__func__, edev->phb->global_number);\n> +\t\t\t\t__func__, pdn->phb->global_number);\n>   \t\t\tedev->pe = NULL;\n>   \t\t\tkfree(pe);\n>   \t\t\treturn -EEXIST;\n> @@ -434,7 +435,7 @@ int eeh_add_to_parent_pe(struct eeh_dev *edev)\n>   \tedev->pe = pe;\n>   \tpr_debug(\"EEH: Add %04x:%02x:%02x.%01x to \"\n>   \t\t \"Device PE#%x, Parent PE#%x\\n\",\n> -\t\t edev->phb->global_number,\n> +\t\t pdn->phb->global_number,\n>   \t\t edev->config_addr >> 8,\n>   \t\t PCI_SLOT(edev->config_addr & 0xFF),\n>   \t\t PCI_FUNC(edev->config_addr & 0xFF),\n> @@ -456,10 +457,11 @@ int eeh_rmv_from_parent_pe(struct eeh_dev *edev)\n>   {\n>   \tstruct eeh_pe *pe, *parent, *child;\n>   \tint cnt;\n> +\tstruct pci_dn *pdn = eeh_dev_to_pdn(edev);\n>   \n>   \tif (!edev->pe) {\n>   \t\tpr_debug(\"%s: No PE found for device %04x:%02x:%02x.%01x\\n\",\n> -\t\t\t __func__,  edev->phb->global_number,\n> +\t\t\t __func__,  pdn->phb->global_number,\n>   \t\t\t edev->config_addr >> 8,\n>   \t\t\t PCI_SLOT(edev->config_addr & 0xFF),\n>   \t\t\t PCI_FUNC(edev->config_addr & 0xFF));\n> @@ -722,7 +724,7 @@ static void eeh_bridge_check_link(struct eeh_dev *edev)\n>   \t\treturn;\n>   \n>   \tpr_debug(\"%s: Check PCIe link for %04x:%02x:%02x.%01x ...\\n\",\n> -\t\t __func__, edev->phb->global_number,\n> +\t\t __func__, pdn->phb->global_number,\n>   \t\t edev->config_addr >> 8,\n>   \t\t PCI_SLOT(edev->config_addr & 0xFF),\n>   \t\t PCI_FUNC(edev->config_addr & 0xFF));\n> diff --git a/arch/powerpc/platforms/powernv/eeh-powernv.c b/arch/powerpc/platforms/powernv/eeh-powernv.c\n> index ac8c01cd251c..552b0cd4e8ba 100644\n> --- a/arch/powerpc/platforms/powernv/eeh-powernv.c\n> +++ b/arch/powerpc/platforms/powernv/eeh-powernv.c\n> @@ -926,7 +926,6 @@ void pnv_pci_reset_secondary_bus(struct pci_dev *dev)\n>   static void pnv_eeh_wait_for_pending(struct pci_dn *pdn, const char *type,\n>   \t\t\t\t     int pos, u16 mask)\n>   {\n> -\tstruct eeh_dev *edev = pdn_to_eeh_dev(pdn);\n>   \tint i, status = 0;\n>   \n>   \t/* Wait for Transaction Pending bit to be cleared */\n> @@ -940,7 +939,7 @@ static void pnv_eeh_wait_for_pending(struct pci_dn *pdn, const char *type,\n>   \n>   \tpr_warn(\"%s: Pending transaction while issuing %sFLR to %04x:%02x:%02x.%01x\\n\",\n>   \t\t__func__, type,\n> -\t\tedev->phb->global_number, pdn->busno,\n> +\t\tpdn->phb->global_number, pdn->busno,\n>   \t\tPCI_SLOT(pdn->devfn), PCI_FUNC(pdn->devfn));\n>   }\n>   \n> @@ -1714,7 +1713,7 @@ static int pnv_eeh_restore_config(struct pci_dn *pdn)\n>   \tif (edev->physfn) {\n>   \t\tret = pnv_eeh_restore_vf_config(pdn);\n>   \t} else {\n> -\t\tphb = edev->phb->private_data;\n> +\t\tphb = pdn->phb->private_data;\n>   \t\tret = opal_pci_reinit(phb->opal_id,\n>   \t\t\t\t      OPAL_REINIT_PCI_DEV, edev->config_addr);\n>   \t}\n>","headers":{"Return-Path":"<linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org>","X-Original-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Delivered-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Received":["from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\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 3xdBbY39p3z9sPk\n\tfor <patchwork-incoming@ozlabs.org>;\n\tThu, 24 Aug 2017 14:47:05 +1000 (AEST)","from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\tby lists.ozlabs.org (Postfix) with ESMTP id 3xdBbY2M7tzDrRy\n\tfor <patchwork-incoming@ozlabs.org>;\n\tThu, 24 Aug 2017 14:47:05 +1000 (AEST)","from mx0a-001b2d01.pphosted.com (mx0a-001b2d01.pphosted.com\n\t[148.163.156.1])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby lists.ozlabs.org (Postfix) with ESMTPS id 3xdBZL2BGKzDqmf\n\tfor <linuxppc-dev@lists.ozlabs.org>;\n\tThu, 24 Aug 2017 14:46:01 +1000 (AEST)","from pps.filterd (m0098396.ppops.net [127.0.0.1])\n\tby mx0a-001b2d01.pphosted.com (8.16.0.21/8.16.0.21) with SMTP id\n\tv7O4jtKV099027\n\tfor <linuxppc-dev@lists.ozlabs.org>; Thu, 24 Aug 2017 00:45:59 -0400","from e23smtp07.au.ibm.com (e23smtp07.au.ibm.com [202.81.31.140])\n\tby mx0a-001b2d01.pphosted.com with ESMTP id 2chgypgs40-1\n\t(version=TLSv1.2 cipher=AES256-SHA bits=256 verify=NOT)\n\tfor <linuxppc-dev@lists.ozlabs.org>; Thu, 24 Aug 2017 00:45:57 -0400","from localhost\n\tby e23smtp07.au.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use\n\tOnly! Violators will be prosecuted\n\tfor <linuxppc-dev@lists.ozlabs.org> from\n\t<andrew.donnellan@au1.ibm.com>; Thu, 24 Aug 2017 14:45:41 +1000","from d23relay10.au.ibm.com (202.81.31.229)\n\tby e23smtp07.au.ibm.com (202.81.31.204) with IBM ESMTP SMTP Gateway:\n\tAuthorized Use Only! Violators will be prosecuted; \n\tThu, 24 Aug 2017 14:45:39 +1000","from d23av03.au.ibm.com (d23av03.au.ibm.com [9.190.234.97])\n\tby d23relay10.au.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id\n\tv7O4jc3v37552380\n\tfor <linuxppc-dev@lists.ozlabs.org>; Thu, 24 Aug 2017 14:45:38 +1000","from d23av03.au.ibm.com (localhost [127.0.0.1])\n\tby d23av03.au.ibm.com (8.14.4/8.14.4/NCO v10.0 AVout) with ESMTP id\n\tv7O4jUfe009555\n\tfor <linuxppc-dev@lists.ozlabs.org>; Thu, 24 Aug 2017 14:45:30 +1000","from ozlabs.au.ibm.com (ozlabs.au.ibm.com [9.192.253.14])\n\tby d23av03.au.ibm.com (8.14.4/8.14.4/NCO v10.0 AVin) with ESMTP id\n\tv7O4jUtK009547; Thu, 24 Aug 2017 14:45:30 +1000","from [10.61.2.125] (haven.au.ibm.com [9.192.254.114])\n\t(using TLSv1.2 with cipher DHE-RSA-AES128-SHA (128/128 bits))\n\t(No client certificate requested)\n\tby ozlabs.au.ibm.com (Postfix) with ESMTPSA id E31EFA008C;\n\tThu, 24 Aug 2017 14:45:37 +1000 (AEST)"],"Subject":"Re: [PATCH kernel 3/5] powerpc/eeh: Remove unnecessary pointer to\n\tphb from eeh_dev","To":"Alexey Kardashevskiy <aik@ozlabs.ru>, linuxppc-dev@lists.ozlabs.org","References":"<20170823101901.33961-1-aik@ozlabs.ru>\n\t<20170823101901.33961-4-aik@ozlabs.ru>","From":"Andrew Donnellan <andrew.donnellan@au1.ibm.com>","Date":"Thu, 24 Aug 2017 14:45:37 +1000","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.2.1","MIME-Version":"1.0","In-Reply-To":"<20170823101901.33961-4-aik@ozlabs.ru>","Content-Type":"text/plain; charset=utf-8; format=flowed","Content-Language":"en-AU","Content-Transfer-Encoding":"7bit","X-TM-AS-MML":"disable","x-cbid":"17082404-0044-0000-0000-00000283E03D","X-IBM-AV-DETECTION":"SAVI=unused REMOTE=unused XFE=unused","x-cbparentid":"17082404-0045-0000-0000-0000071741BE","Message-Id":"<2faa9545-3440-9d37-1b4f-56595d704b93@au1.ibm.com>","X-Proofpoint-Virus-Version":"vendor=fsecure engine=2.50.10432:, ,\n\tdefinitions=2017-08-24_01:, , signatures=0","X-Proofpoint-Spam-Details":"rule=outbound_notspam policy=outbound score=0\n\tspamscore=0 suspectscore=18\n\tmalwarescore=0 phishscore=0 adultscore=0 bulkscore=0 classifier=spam\n\tadjust=0 reason=mlx scancount=1 engine=8.0.1-1707230000\n\tdefinitions=main-1708240076","X-BeenThere":"linuxppc-dev@lists.ozlabs.org","X-Mailman-Version":"2.1.23","Precedence":"list","List-Id":"Linux on PowerPC Developers Mail List\n\t<linuxppc-dev.lists.ozlabs.org>","List-Unsubscribe":"<https://lists.ozlabs.org/options/linuxppc-dev>,\n\t<mailto:linuxppc-dev-request@lists.ozlabs.org?subject=unsubscribe>","List-Archive":"<http://lists.ozlabs.org/pipermail/linuxppc-dev/>","List-Post":"<mailto:linuxppc-dev@lists.ozlabs.org>","List-Help":"<mailto:linuxppc-dev-request@lists.ozlabs.org?subject=help>","List-Subscribe":"<https://lists.ozlabs.org/listinfo/linuxppc-dev>,\n\t<mailto:linuxppc-dev-request@lists.ozlabs.org?subject=subscribe>","Cc":"Alistair Popple <alistair@popple.id.au>,\n\tDavid Gibson <david@gibson.dropbear.id.au>","Errors-To":"linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org","Sender":"\"Linuxppc-dev\"\n\t<linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org>"}},{"id":1758386,"web_url":"http://patchwork.ozlabs.org/comment/1758386/","msgid":"<87efrwryyi.fsf@concordia.ellerman.id.au>","date":"2017-08-28T06:25:09","subject":"Re: [PATCH kernel 3/5] powerpc/eeh: Remove unnecessary pointer to\n\tphb from eeh_dev","submitter":{"id":46580,"url":"http://patchwork.ozlabs.org/api/people/46580/","name":"Michael Ellerman","email":"mpe@ellerman.id.au"},"content":"Alexey Kardashevskiy <aik@ozlabs.ru> writes:\n\n> The eeh_dev struct already holds a pointer to pci_dn which it does not\n> exist without and pci_dn itself holds the very same pointer so just\n> use it.\n>\n> Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>\n> ---\n>  arch/powerpc/include/asm/eeh.h               |  1 -\n>  arch/powerpc/kernel/eeh.c                    |  7 +++----\n>  arch/powerpc/kernel/eeh_dev.c                |  2 --\n>  arch/powerpc/kernel/eeh_driver.c             |  2 +-\n>  arch/powerpc/kernel/eeh_pe.c                 | 24 +++++++++++++-----------\n>  arch/powerpc/platforms/powernv/eeh-powernv.c |  5 ++---\n\nDoesn't build ?\n\n  arch/powerpc/platforms/pseries/eeh_pseries.c: In function ‘pseries_eeh_probe’:\n  arch/powerpc/platforms/pseries/eeh_pseries.c:250:15: error: ‘struct eeh_dev’ has no member named ‘phb’\n  pe.phb = edev->phb;\n               ^~\n\ncheers","headers":{"Return-Path":"<linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org>","X-Original-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Delivered-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@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 3xghcZ5PCXz9sNc\n\tfor <patchwork-incoming@ozlabs.org>;\n\tMon, 28 Aug 2017 16:26:38 +1000 (AEST)","from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\tby lists.ozlabs.org (Postfix) with ESMTP id 3xghcZ4VyjzDqJr\n\tfor <patchwork-incoming@ozlabs.org>;\n\tMon, 28 Aug 2017 16:26:38 +1000 (AEST)","from ozlabs.org (ozlabs.org [IPv6:2401:3900:2:1::2])\n\t(using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby lists.ozlabs.org (Postfix) with ESMTPS id 3xghZt5xd0zDq5m\n\tfor <linuxppc-dev@lists.ozlabs.org>;\n\tMon, 28 Aug 2017 16:25:10 +1000 (AEST)","from authenticated.ozlabs.org (localhost [127.0.0.1])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPSA id 3xghZt2jvwz9sNc;\n\tMon, 28 Aug 2017 16:25:10 +1000 (AEST)"],"From":"Michael Ellerman <mpe@ellerman.id.au>","To":"Alexey Kardashevskiy <aik@ozlabs.ru>, linuxppc-dev@lists.ozlabs.org","Subject":"Re: [PATCH kernel 3/5] powerpc/eeh: Remove unnecessary pointer to\n\tphb from eeh_dev","In-Reply-To":"<20170823101901.33961-4-aik@ozlabs.ru>","References":"<20170823101901.33961-1-aik@ozlabs.ru>\n\t<20170823101901.33961-4-aik@ozlabs.ru>","User-Agent":"Notmuch/0.21 (https://notmuchmail.org)","Date":"Mon, 28 Aug 2017 16:25:09 +1000","Message-ID":"<87efrwryyi.fsf@concordia.ellerman.id.au>","MIME-Version":"1.0","Content-Type":"text/plain; charset=utf-8","Content-Transfer-Encoding":"quoted-printable","X-BeenThere":"linuxppc-dev@lists.ozlabs.org","X-Mailman-Version":"2.1.23","Precedence":"list","List-Id":"Linux on PowerPC Developers Mail List\n\t<linuxppc-dev.lists.ozlabs.org>","List-Unsubscribe":"<https://lists.ozlabs.org/options/linuxppc-dev>,\n\t<mailto:linuxppc-dev-request@lists.ozlabs.org?subject=unsubscribe>","List-Archive":"<http://lists.ozlabs.org/pipermail/linuxppc-dev/>","List-Post":"<mailto:linuxppc-dev@lists.ozlabs.org>","List-Help":"<mailto:linuxppc-dev-request@lists.ozlabs.org?subject=help>","List-Subscribe":"<https://lists.ozlabs.org/listinfo/linuxppc-dev>,\n\t<mailto:linuxppc-dev-request@lists.ozlabs.org?subject=subscribe>","Cc":"Alexey Kardashevskiy <aik@ozlabs.ru>,\n\tAlistair Popple <alistair@popple.id.au>,\n\tDavid Gibson <david@gibson.dropbear.id.au>","Errors-To":"linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org","Sender":"\"Linuxppc-dev\"\n\t<linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org>"}}]