From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-13.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id D0ABEC4338F for ; Wed, 28 Jul 2021 12:54:45 +0000 (UTC) Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 3738160E09 for ; Wed, 28 Jul 2021 12:54:45 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 3738160E09 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=wapice.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=lists.denx.de Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id F0BD082C95; Wed, 28 Jul 2021 14:54:34 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=wapice.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (1024-bit key; unprotected) header.d=wapice0.onmicrosoft.com header.i=@wapice0.onmicrosoft.com header.b="n+0sqA9i"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 50AFE8348D; Tue, 27 Jul 2021 15:14:51 +0200 (CEST) Received: from EUR01-HE1-obe.outbound.protection.outlook.com (mail-he1eur01on0626.outbound.protection.outlook.com [IPv6:2a01:111:f400:fe1e::626]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 137C68317A for ; Tue, 27 Jul 2021 15:14:47 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=wapice.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=pauli.oikkonen@wapice.com ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=gR+1c8qONSqPuJJHdBuRJ4ikD+3wxA/iOTUPa1/0AVMM3FKOdJtVu17RrWUo4yoNZ2WmyMxrqe80sv0a9dxpafmZx/4YGWvlHZ4rnBI78nsCfoS40eB+HS/Evu749ORnO29Ao/bbq1f1P3FobLRhIl6ikAoAqFNPyhG0NjmGneTqqSqrJdWAaXUELUKNqyB8pGxF+2KZVSA8UBb7CmE6sUsLJKT+davii5Tv6N5NORxWBfUtMw6oQ2FaUyrk87E8cK47ey/MfNAPOlR65EMZg3iUVydVqYPWTx34SNkP6b5P+jHBN3/f/KIEB3XeedYdOumPGyTn1b+WiH8R/Y0Vvw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=expbe8PZJmtBL1GIhFCvl4qkZ/iuzzzfZihr43WZscE=; b=HrCjwN5U5yLZPTyTYQfUpBGHrNRE/NiwxEQUELIE84tWfbeFd131v9b6J0K8YQO6TuK5a9L+p8Y68MtETJguPKXXGH4onPOa1ZWhnlZJF5Q+pm2gC8aQAlvVvOtbXeE7gFV4os41iAIeglHz/8NExeS+JY/xU9C1SAJ47/858OOnoarTK0+Ta9KHblQPxZ+zRRwOXUhtvipNDUOTz6OSHS8kvtFWMaFT8NIGCxzUv6WzGihXwXYi0lRUGxA5XrwaQYbWANtwwSkWFOe7ZnpQ6v9MboWld9ev07mBHhnQ80BYnBzQLRqWeItsI6YGUA3CGevo6dvnyEfh+HHme++64A== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=wapice.com; dmarc=pass action=none header.from=wapice.com; dkim=pass header.d=wapice.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=wapice0.onmicrosoft.com; s=selector2-wapice0-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=expbe8PZJmtBL1GIhFCvl4qkZ/iuzzzfZihr43WZscE=; b=n+0sqA9i3opJnudAQY8MjucEEP7idgzgTljhU2EX0G5XiAhRVOFKSShijLs9tSvM0gzSQAa4vTjttZPT/kiqZAU7kkYzNlctklMKNHztVGs2OAyXyZnD8pNCMVPEvXjequDzi0vJpopGAbFQiETlrJHktdShxl1FtjOAk/cftVQ= Received: from AM6PR03MB3829.eurprd03.prod.outlook.com (2603:10a6:20b:21::16) by AM5PR0301MB2434.eurprd03.prod.outlook.com (2603:10a6:203:12::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4373.17; Tue, 27 Jul 2021 13:14:45 +0000 Received: from AM6PR03MB3829.eurprd03.prod.outlook.com ([fe80::4dcf:d6a1:b987:7882]) by AM6PR03MB3829.eurprd03.prod.outlook.com ([fe80::4dcf:d6a1:b987:7882%6]) with mapi id 15.20.4352.031; Tue, 27 Jul 2021 13:14:45 +0000 From: Pauli Oikkonen To: "u-boot@lists.denx.de" CC: "jteki@openedev.com" Subject: [PATCH] spi: fix xilinx-spi lockup when fifo-size undefined in dtree Thread-Topic: [PATCH] spi: fix xilinx-spi lockup when fifo-size undefined in dtree Thread-Index: AQHXguke4iGeB0Ts/U2U1EWsyt7GmA== Date: Tue, 27 Jul 2021 13:14:44 +0000 Message-ID: Accept-Language: fi-FI, en-US Content-Language: fi-FI X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: lists.denx.de; dkim=none (message not signed) header.d=none;lists.denx.de; dmarc=none action=none header.from=wapice.com; x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 575cced4-d45d-4a77-4d4e-08d9510080b0 x-ms-traffictypediagnostic: AM5PR0301MB2434: x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:6790; x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: gS09ysYrh9sNpUiHrEK1x1d5FsT3MV4sGWFPgNiM47oM3PcP9lL/8DhsFV/vAEfSXLW/TGrWNeWXK7vjTuMeAWtysLaVEgyTfmVxgozRAD6o+TU+9NumNfOfAaVyZAPYV2e+9RBlALI2ljPOBlTVPzI6kMQruq1GOWuoRdcBFMdSPhUbo+Rj0M80g+MgUB4d7Hqj4S/n5adFcE0Xsl3JbsF3dM1XNofcx3iKXvb0i022vxMwH34FKOUt4OHFJG62d4zfsLWblkRh/4ou0laXdMbkaZ5NbDtZZm/VX+bu3BU/SLwazev0vFQ7rBaveTRS+1qliX1gpzKlyxKpr+v3xxh7A12yrM5bYJxTCFYFZPSC5vvKIPdm6iy8fRgyTF5IptqRfThlcgaKcq80qmhTXxm42hgGsSk+dT1hmfTx6h59babDNo9B/9T95reK0h+VzfGCWzJfKASBC1LCiyJxceWSoDfHtn23YbuBiysvvIkW3MD5Kc8+0IILkYHB2D9xhE4W/P5S8k/DcIoZEP7dJWE5zPwLl1ym4+mcUbZX5fxeK6tQ0yCpJWEc2oMhSSFmbrLDvs4OAWKp/MtfGdZ0pmgya3AVwHDwkWRCvkGjU5dAhRuTAwHADUhtlDLE9i8D3NIXE7+WRnZ3D4BCRczy83AudS1dfBvq6SMRDQLsMVrRxE3ri/wbE0RDIrHMaudMCWQL5OgcH6I8p4W04Liv/A== x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:AM6PR03MB3829.eurprd03.prod.outlook.com; PTR:; CAT:NONE; SFS:(39830400003)(346002)(136003)(396003)(376002)(366004)(52536014)(9686003)(86362001)(4326008)(55016002)(8676002)(33656002)(6506007)(38100700002)(122000001)(6916009)(44832011)(2906002)(66476007)(66556008)(316002)(66946007)(71200400001)(478600001)(5660300002)(91956017)(64756008)(66446008)(8936002)(7696005)(83380400001)(26005)(76116006)(186003)(38070700004); DIR:OUT; SFP:1101; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?iso-8859-1?Q?zajuY9RJuQ5MvvQNNZ0Wmp9Mkxsnw1b/VFMo7Sab/mYlJaPRLiCud6JYrB?= =?iso-8859-1?Q?1Wyc5f+iRjjI+syyRVUXDXk1g6RjuWsNDHwlHGLoADo1muHz0mH0ghKQnM?= =?iso-8859-1?Q?OvChFkbKx0J+wxyyQgZZg8Ldad+A8HS3MvWq3Oq9WUagmvKFdi10XPUuzs?= =?iso-8859-1?Q?NkBBsE8VK/AnYD5OV7Auax69NowoBTC7ca7SJo2LBkp/11suEnY2kUhSzu?= =?iso-8859-1?Q?9NkbhiD7M2ItYg2nDP7IhUk4QVMai0Af3/6F4rMOV02Z/dbMujVox+bO3s?= =?iso-8859-1?Q?6H/sNg9ktDjKtHYB+lCqeF4v5zy3Vpod4KB6lRjZJXOhKZci0bEMHBcXTH?= =?iso-8859-1?Q?mCQQ7vZxI1yfpfex2vmBC4qJqRTvII7zXGHOemeKl4XaggkPZxxI+cM4hV?= =?iso-8859-1?Q?Z7rXyTiW69pzMXs89gftQ0Ex4svn7/XZZQwoBb6lG08oFgFdAesMT5hDcz?= =?iso-8859-1?Q?cZOzsnXY0JRSphiSzQzoFEjomhaMQWkGLFT8RSwtwvL7tA3yYNzew3NIpS?= =?iso-8859-1?Q?K+WU9OYpiGzu48bKdHf3H0ZCSZi3+X+3b4nKxywG91MtJPh9KPSmVj+QI/?= =?iso-8859-1?Q?AJlfAca3a2LdqPvmo8orfzvyrjoCoQLgB0zROS0rU4V5d/5aMeORoalkIx?= =?iso-8859-1?Q?JjKlDOek2oRbSiZtJ/DPAmTcB7g4QKAEnF9PAX3jQQqH6KTNmTGYx/M+rj?= =?iso-8859-1?Q?Wiw9bBsxEU1a9LEfSEXyS4PBeHD6JQMbL4fETlWKfO+wM9s89m2vq5gri9?= =?iso-8859-1?Q?mOdpvFNd1fJF00qSRO/yNUtV1XgD46ssoX9BeAr4C5xB+Q7fqUl+C+4wJO?= =?iso-8859-1?Q?sYtVbvIqCNUVoZ6RDh/wqUVLy3PCFjDn6bT1Uz1d/ZUpkqdDtogugfIj1f?= =?iso-8859-1?Q?wj0Aft/mOnw2smRg5O99dz3Gg/KZEP1xr1JO1HLXg++Z1lGBwh0Hc6h4Qb?= =?iso-8859-1?Q?0ktLjx7fWb5O3kPAPitYSxIrHEQxIWqimrsJXNE+8TRqncOUgpVrDUZ4Hm?= =?iso-8859-1?Q?LQDjlgnMh63E8BxrT/+GiL4sUB1EeT7WLl6+gypuSDBhLA7QFqUBu6fP3H?= =?iso-8859-1?Q?ZMEllfP/iCAMQDiviKf6oLif9XVklvvUpkEfiWtu9V79KMOJEP+WlsuXFg?= =?iso-8859-1?Q?iJdOdkeEjgq8f7jg5cCR4RR5Luo2Uhi74scg0fJIV+UI0DJA99OXO81Jyl?= =?iso-8859-1?Q?yBeBqIhsm25jn1kXVm08z2oFM8BKUDUJSuFY9j9ohhS6nzXJW4em/Z5vj8?= =?iso-8859-1?Q?K/N5Hbu34+Zx0Iyt6LX7EhYqtzfrv/giR/BYxdKyDmuCM/H0OoLp2a6yoh?= =?iso-8859-1?Q?urwF3+KVjC3gG9fgArLVDz1ImRSsCruK+lBRxK3yAvtY1V4=3D?= x-ms-exchange-transport-forked: True Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: wapice.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: AM6PR03MB3829.eurprd03.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 575cced4-d45d-4a77-4d4e-08d9510080b0 X-MS-Exchange-CrossTenant-originalarrivaltime: 27 Jul 2021 13:14:44.9713 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: e93ae9a2-b103-404e-aa22-611d62f84bcb X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: N0+YnM33natlzvLg1fl1wMil9KxFNoTuPRJmhLUbv46kToDMiQH+rl8clyJc+NZ0veGIE1OqVLs+d79C7Xn5RA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM5PR0301MB2434 X-Mailman-Approved-At: Wed, 28 Jul 2021 14:54:29 +0200 X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.34 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.2 at phobos.denx.de X-Virus-Status: Clean If fifo_depth is 0, the driver will lock up.=0A= =0A= If fifo-size is not defined in device tree, the driver would use 0 as=0A= a default value. This however will cause an infinite loop and a lockup=0A= in any read or write. Use 1 as a default FIFO size instead, no FIFO=0A= should be zero-length anyway.=0A= =0A= Signed-off-by: Pauli Oikkonen =0A= CC: Jagan Teki =0A= ---=0A= drivers/spi/xilinx_spi.c | 7 ++++++-=0A= 1 file changed, 6 insertions(+), 1 deletion(-)=0A= =0A= diff --git a/drivers/spi/xilinx_spi.c b/drivers/spi/xilinx_spi.c=0A= index b892cdae9b..da0f8b3bb4 100644=0A= --- a/drivers/spi/xilinx_spi.c=0A= +++ b/drivers/spi/xilinx_spi.c=0A= @@ -110,12 +110,17 @@ struct xilinx_spi_priv {=0A= =0A= static int xilinx_spi_probe(struct udevice *bus)=0A= {=0A= + int success;=0A= struct xilinx_spi_priv *priv =3D dev_get_priv(bus);=0A= struct xilinx_spi_regs *regs =3D priv->regs;=0A= =0A= priv->regs =3D (struct xilinx_spi_regs *)dev_read_addr(bus);=0A= =0A= - priv->fifo_depth =3D dev_read_u32_default(bus, "fifo-size", 0);=0A= + success =3D dev_read_u32(bus, "fifo-size", &priv->fifo_depth);=0A= + if (success !=3D 0) {=0A= + debug("%s: no fifo-size defined in dtree, using 1\n", __func__);=0A= + priv->fifo_depth =3D 1;=0A= + }=0A= =0A= writel(SPISSR_RESET_VALUE, ®s->srr);=0A= =0A= -- =0A= 2.25.1=0A= =0A=