From patchwork Mon Feb 19 16:32:02 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?b?TWljaGHFgiBCYXJuYcWb?= X-Patchwork-Id: 1901045 X-Patchwork-Delegate: trini@ti.com Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@legolas.ozlabs.org Authentication-Results: legolas.ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=google.com header.i=@google.com header.a=rsa-sha256 header.s=20230601 header.b=w5HlGgkZ; dkim-atps=neutral Authentication-Results: legolas.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=lists.denx.de (client-ip=85.214.62.61; helo=phobos.denx.de; envelope-from=u-boot-bounces@lists.denx.de; receiver=patchwork.ozlabs.org) Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (secp384r1) server-digest SHA384) (No client certificate requested) by legolas.ozlabs.org (Postfix) with ESMTPS id 4Tdp8F6znXz20Qg for ; Tue, 20 Feb 2024 03:36:29 +1100 (AEDT) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 6A5BB87F51; Mon, 19 Feb 2024 17:35:50 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (2048-bit key; unprotected) header.d=google.com header.i=@google.com header.b="w5HlGgkZ"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id CBA8387F22; Mon, 19 Feb 2024 17:32:45 +0100 (CET) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on phobos.denx.de X-Spam-Level: X-Spam-Status: No, score=-9.6 required=5.0 tests=BAYES_00,DKIMWL_WL_MED, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_NONE, SPF_PASS,T_SCC_BODY_TEXT_LINE,USER_IN_DEF_DKIM_WL autolearn=ham autolearn_force=no version=3.4.2 Received: from mail-yw1-x114a.google.com (mail-yw1-x114a.google.com [IPv6:2607:f8b0:4864:20::114a]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 7B5F583CF3 for ; Mon, 19 Feb 2024 17:32:33 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=3n4LTZQYKBigFEVREWKSSKPI.GSQY-FSSXPMWXW.HIRb.HI@flex--barnas.bounces.google.com Received: by mail-yw1-x114a.google.com with SMTP id 00721157ae682-607cd6c11d7so63995157b3.0 for ; Mon, 19 Feb 2024 08:32:33 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1708360352; x=1708965152; darn=lists.denx.de; h=content-transfer-encoding:cc:to:from:subject:message-id:references :mime-version:in-reply-to:date:from:to:cc:subject:date:message-id :reply-to; bh=O7PUmy+xQ3/8cEPxwAI006xHbvDymz/RHTPdFuXk89Y=; b=w5HlGgkZdcmMcTtmJD8e+0aVONfvPXz56NfDuJu7uTj9cHE6U6vDM2NZgXH3EaNABJ j51eiqFmt7nOl6ekkKKmSlKd57gH6x0cIa0jNymQXHUHyajrFo5Yd+0jDwWr/padl8GW JpbOtdCM0dA6x6LQx5Hf/EyPgKlBkvojMnNYALIO8BIoE+Ma1xAIcs/jProD+Xh72wXL dkIEsZhvEHhltbUtcuz4Bt5RRpCZmGsecmd9HJ5pD8bJQKnOQ64voAb4gR+YQT23im1L /vxR42dl0U+w6kGvGOmDmqzRvOG6pBjGKp5118YIDbUzzye5jAF+bSBEq+r8sIOFbgwc +lgQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1708360352; x=1708965152; h=content-transfer-encoding:cc:to:from:subject:message-id:references :mime-version:in-reply-to:date:x-gm-message-state:from:to:cc:subject :date:message-id:reply-to; bh=O7PUmy+xQ3/8cEPxwAI006xHbvDymz/RHTPdFuXk89Y=; b=aBbhunlHOeA0lSFvtNHdg28gszdA6Fb/iLIbSWnbMusNbK9nnNnlZkjV8fpaxpZ6cC UgcgMP6KCx2DYBZAfaVhydPAoXjdqXSEcbwq8zgALx2IPeFDn9+D/ICTUHwxCmi4UzD+ 7rXMAAw/EklXPUc93DC7Gx+02ThYfUY7rTGuGu3BensiISU6MMcunR1V9b4FYdGG+XWn TWuZC5dDWpO3xoMyaQcrHz9kTGqumFkzCBi1qzm88kcGJVi/4IINW5GdFx2ivafydKfo JV+oZAW4UazQQL+rYgGT7sTqpWKaX4QlqsQxqdCwUQ4N0W+rQmVkJ0WVSxQogkN/tyMf wT9w== X-Gm-Message-State: AOJu0YxF7yDab1jrTRzY/J+AsTWD42+PUNt/v8qa4QaW9v5I7LnoeR2I DpXs0JuxTcEP+BCPZAwIl3Dkkmgxy4GtlXLU1VL9IcxbV/gQIYyhxk3UpKTtpMbXsb2BATsghmb LZomKnwwliaQqkq4O5ifMpt6UZiHQcPBY+V4agdQP77jxQHvdjkdCnfmd1LMR1R6S8CMifuqHpX m0qXZgWpDmgH02yrsklvuDeR3VxkihTQ== X-Google-Smtp-Source: AGHT+IEt+z9kj5a4v9d/6e6UNWRvUxfUuFlHDxY95YyEKOp6HVhYYxKgMmAFR7VGC8i7rV6wLWPO8Ov9IVA= X-Received: from zombine.c.googlers.com ([fda3:e722:ac3:cc00:68:949d:c0a8:7f3e]) (user=barnas job=sendgmr) by 2002:a05:6902:1001:b0:dcc:79ab:e522 with SMTP id w1-20020a056902100100b00dcc79abe522mr515037ybt.11.1708360351892; Mon, 19 Feb 2024 08:32:31 -0800 (PST) Date: Mon, 19 Feb 2024 16:32:02 +0000 In-Reply-To: <20240219163213.855667-1-barnas@google.com> Mime-Version: 1.0 References: <20240219163213.855667-1-barnas@google.com> X-Mailer: git-send-email 2.44.0.rc0.258.g7320e95886-goog Message-ID: <20240219163213.855667-2-barnas@google.com> Subject: [PATCH 1/3] misc: atsha204a: remove broken for loop From: " =?utf-8?b?TWljaGHFgiBCYXJuYcWb?= " To: u-boot@lists.denx.de Cc: " =?utf-8?b?TWljaGHFgiBCYXJuYcWb?= " , Tom Rini X-Mailman-Approved-At: Mon, 19 Feb 2024 17:35:44 +0100 X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 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.8 at phobos.denx.de X-Virus-Status: Clean Some previous commit changed the continue statement to return, making the for loop used to retry waking up the chip to always return after one iteration. This commit removes the loop, cleaning the code a little. Signed-off-by: Michał Barnaś --- drivers/misc/atsha204a-i2c.c | 45 +++++++++++++++--------------------- 1 file changed, 19 insertions(+), 26 deletions(-) diff --git a/drivers/misc/atsha204a-i2c.c b/drivers/misc/atsha204a-i2c.c index d3c515828f..ab83bbc3e9 100644 --- a/drivers/misc/atsha204a-i2c.c +++ b/drivers/misc/atsha204a-i2c.c @@ -96,40 +96,33 @@ int atsha204a_wakeup(struct udevice *dev) { u8 req[4]; struct atsha204a_resp resp; - int try, res; + int res; debug("Waking up ATSHA204A\n"); - for (try = 1; try <= 10; ++try) { - debug("Try %i... ", try); - - /* - * The device ignores any levels or transitions on the SCL pin - * when the device is idle, asleep or during waking up. - * Don't check for error when waking up the device. - */ - memset(req, 0, 4); - atsha204a_send(dev, req, 4); + /* + * The device ignores any levels or transitions on the SCL pin + * when the device is idle, asleep or during waking up. + * Don't check for error when waking up the device. + */ + memset(req, 0, 4); + atsha204a_send(dev, req, 4); - udelay(ATSHA204A_TWLO_US + ATSHA204A_TWHI_US); + udelay(ATSHA204A_TWLO_US + ATSHA204A_TWHI_US); - res = atsha204a_recv_resp(dev, &resp); - if (res) { - debug("failed on receiving response, ending\n"); - return res; - } - - if (resp.code != ATSHA204A_STATUS_AFTER_WAKE) { - debug ("failed (responce code = %02x), ending\n", - resp.code); - return -EBADMSG; - } + res = atsha204a_recv_resp(dev, &resp); + if (res) { + debug("failed on receiving response, ending\n"); + return res; + } - debug("success\n"); - return 0; + if (resp.code != ATSHA204A_STATUS_AFTER_WAKE) { + debug("failed (response code = %02x), ending\n", resp.code); + return -EBADMSG; } - return -ETIMEDOUT; + debug("success\n"); + return 0; } int atsha204a_idle(struct udevice *dev) From patchwork Mon Feb 19 16:32:03 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?b?TWljaGHFgiBCYXJuYcWb?= X-Patchwork-Id: 1901046 X-Patchwork-Delegate: trini@ti.com Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@legolas.ozlabs.org Authentication-Results: legolas.ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=google.com header.i=@google.com header.a=rsa-sha256 header.s=20230601 header.b=mIaAfVKf; dkim-atps=neutral Authentication-Results: legolas.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=lists.denx.de (client-ip=2a01:238:438b:c500:173d:9f52:ddab:ee01; helo=phobos.denx.de; envelope-from=u-boot-bounces@lists.denx.de; receiver=patchwork.ozlabs.org) Received: from phobos.denx.de (phobos.denx.de [IPv6:2a01:238:438b:c500:173d:9f52:ddab:ee01]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (secp384r1)) (No client certificate requested) by legolas.ozlabs.org (Postfix) with ESMTPS id 4Tdp902wdQz20Qg for ; Tue, 20 Feb 2024 03:37:08 +1100 (AEDT) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id B65AB87F57; Mon, 19 Feb 2024 17:35:51 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (2048-bit key; unprotected) header.d=google.com header.i=@google.com header.b="mIaAfVKf"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id A060783CF3; Mon, 19 Feb 2024 17:32:46 +0100 (CET) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on phobos.denx.de X-Spam-Level: X-Spam-Status: No, score=-9.6 required=5.0 tests=BAYES_00,DKIMWL_WL_MED, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_NONE, SPF_PASS,T_SCC_BODY_TEXT_LINE,USER_IN_DEF_DKIM_WL autolearn=ham autolearn_force=no version=3.4.2 Received: from mail-yw1-x114a.google.com (mail-yw1-x114a.google.com [IPv6:2607:f8b0:4864:20::114a]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id B9E01874B1 for ; Mon, 19 Feb 2024 17:32:39 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=3poLTZQYKBi8MLcYLdRZZRWP.NZXf-MZZeWTded.OPYi.OP@flex--barnas.bounces.google.com Received: by mail-yw1-x114a.google.com with SMTP id 00721157ae682-60761bdbd4cso82089727b3.3 for ; Mon, 19 Feb 2024 08:32:39 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1708360358; x=1708965158; darn=lists.denx.de; h=content-transfer-encoding:cc:to:from:subject:message-id:references :mime-version:in-reply-to:date:from:to:cc:subject:date:message-id :reply-to; bh=/qu+M7VTGiGaeSLYLihyK8S/nXk79S+1zY7hDGlK8Ik=; b=mIaAfVKfTqxf7o336WOvlOPoipPfoo4uOxd9nnWKVEY1C0NxUD5yxwuo3EeORsSreb p4Eu+4O1/veMXCceb/uRUxRYytjy7Pm56F2F8HgZomKOE3jY6dseLWjdXWo8wj9NJZfU PEN4cWgybzvS6IpZNxuqu1vuTquuNTZu9EXnDiKRc3fRp4xY+pg9Afqq/5Fasu2AAKwA lIqOyA5uQavTh7SeoIYZnm9Es2y/G+dYKL5QYHydo/yT8a+W7tgwCR95j7qR6Q+jrxdW LCz+ZFJmajfHeWfhaaXkDp3ldu7Ix0oF7Dag9DVDoqxN0+6VHQKTaIBv1ibw1OBhDLoi A7OQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1708360358; x=1708965158; h=content-transfer-encoding:cc:to:from:subject:message-id:references :mime-version:in-reply-to:date:x-gm-message-state:from:to:cc:subject :date:message-id:reply-to; bh=/qu+M7VTGiGaeSLYLihyK8S/nXk79S+1zY7hDGlK8Ik=; b=TeyLPIEG2RuQWfPHbPMkVjatEgEdL8UclmhL0i5pVZ6VE6sUnYnfdQqm4lVmsi9c7B L+zUk5IkSWZHruj+X2kdGhuGteeamrBhF+hTw4k+5GkvS6oRr9P+w5WczU4r7dB/bDn0 aIWRDDfMorZoKsriXJQwYR+9D8CuKJgVVIHA2+xnEF2gI56FuGrydu/okXTJAE2JjHcg NQUzXaxgSZTkpW1DXJhGKTGGVcJnUuGJ03WkoMCUfd0tDQHvsUWEYg3YbYZH6bn5i5jQ jgWtZXJ0taiK1HQg3gRQVA7K2KdvAAwGSQDOo6Logn9LWhq2x0B/Dl9yMtCWStP31Ojo Ngkg== X-Gm-Message-State: AOJu0YxAhQZvyaPUEB+6S13zNK0sHNcpcjYsFPZ6Ov/d3EcqqqwSsSCM egVK12Jd+g2ydbWsEOcoG81DE7mFKX6oz7+DMbppNfrB+YojTpR7TscZ0APf3uf7GXt+EblNWdS OmTvqRTaxiUK61dt4duD+l/fatj9l3yuR5zdjpTmIllrnwLgt3Q34O7UI5IvTmPE/tc5xQAOrUR 9c/A31htYV91wSi34ZMzs5pyY4eN0/ZA== X-Google-Smtp-Source: AGHT+IETLJsqQX5no/SBOLqadEe91uq0pqnMRcEkBvxQGGY1FtPZwJ/spnBt30lOnWLed6qUEbpijuEKsE8= X-Received: from zombine.c.googlers.com ([fda3:e722:ac3:cc00:68:949d:c0a8:7f3e]) (user=barnas job=sendgmr) by 2002:a05:6902:100b:b0:dcc:c57c:8873 with SMTP id w11-20020a056902100b00b00dccc57c8873mr3738004ybt.9.1708360358492; Mon, 19 Feb 2024 08:32:38 -0800 (PST) Date: Mon, 19 Feb 2024 16:32:03 +0000 In-Reply-To: <20240219163213.855667-1-barnas@google.com> Mime-Version: 1.0 References: <20240219163213.855667-1-barnas@google.com> X-Mailer: git-send-email 2.44.0.rc0.258.g7320e95886-goog Message-ID: <20240219163213.855667-3-barnas@google.com> Subject: [PATCH 2/3] misc: atsha204a: fix sleep function From: " =?utf-8?b?TWljaGHFgiBCYXJuYcWb?= " To: u-boot@lists.denx.de Cc: " =?utf-8?b?TWljaGHFgiBCYXJuYcWb?= " , Tom Rini X-Mailman-Approved-At: Mon, 19 Feb 2024 17:35:44 +0100 X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 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.8 at phobos.denx.de X-Virus-Status: Clean Fix the sleep function to issue the sleep command instead of idle one. Signed-off-by: Michał Barnaś --- drivers/misc/atsha204a-i2c.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/misc/atsha204a-i2c.c b/drivers/misc/atsha204a-i2c.c index ab83bbc3e9..29daefb2a5 100644 --- a/drivers/misc/atsha204a-i2c.c +++ b/drivers/misc/atsha204a-i2c.c @@ -139,7 +139,7 @@ int atsha204a_idle(struct udevice *dev) int atsha204a_sleep(struct udevice *dev) { int res; - u8 req = ATSHA204A_FUNC_IDLE; + u8 req = ATSHA204A_FUNC_SLEEP; res = atsha204a_send(dev, &req, 1); if (res) From patchwork Mon Feb 19 16:32:04 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?b?TWljaGHFgiBCYXJuYcWb?= X-Patchwork-Id: 1901047 X-Patchwork-Delegate: trini@ti.com Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@legolas.ozlabs.org Authentication-Results: legolas.ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=google.com header.i=@google.com header.a=rsa-sha256 header.s=20230601 header.b=F0knDODp; dkim-atps=neutral Authentication-Results: legolas.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=lists.denx.de (client-ip=2a01:238:438b:c500:173d:9f52:ddab:ee01; helo=phobos.denx.de; envelope-from=u-boot-bounces@lists.denx.de; receiver=patchwork.ozlabs.org) Received: from phobos.denx.de (phobos.denx.de [IPv6:2a01:238:438b:c500:173d:9f52:ddab:ee01]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (secp384r1)) (No client certificate requested) by legolas.ozlabs.org (Postfix) with ESMTPS id 4Tdp9D5ndqz20Qg for ; Tue, 20 Feb 2024 03:37:20 +1100 (AEDT) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id D0B4087F4F; Mon, 19 Feb 2024 17:36:01 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (2048-bit key; unprotected) header.d=google.com header.i=@google.com header.b="F0knDODp"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id BAA7D87F3F; Mon, 19 Feb 2024 17:33:11 +0100 (CET) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on phobos.denx.de X-Spam-Level: X-Spam-Status: No, score=-9.6 required=5.0 tests=BAYES_00,DKIMWL_WL_MED, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_NONE, SPF_PASS,T_SCC_BODY_TEXT_LINE,USER_IN_DEF_DKIM_WL autolearn=ham autolearn_force=no version=3.4.2 Received: from mail-yb1-xb4a.google.com (mail-yb1-xb4a.google.com [IPv6:2607:f8b0:4864:20::b4a]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id DFCC487DE9 for ; Mon, 19 Feb 2024 17:32:49 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=3sILTZQYKBjkWVmiVnbjjbgZ.Xjhp-Wjjogdnon.YZis.YZ@flex--barnas.bounces.google.com Received: by mail-yb1-xb4a.google.com with SMTP id 3f1490d57ef6-dc6b269686aso6776915276.1 for ; Mon, 19 Feb 2024 08:32:49 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1708360368; x=1708965168; darn=lists.denx.de; h=content-transfer-encoding:cc:to:from:subject:message-id:references :mime-version:in-reply-to:date:from:to:cc:subject:date:message-id :reply-to; bh=+Io/XkVO0aE6tyFm8S2xv8fYf7/3UUb5MxQd7q2MHD4=; b=F0knDODpyhYS8gNQYMGx+3EuO27ccgxR3L4FToXJbeB878Kyz2vSH17nsoT0M3w9cK SpWWuFb+rdubV0z9NB3byVhfmkIthMr6q+kYxeUUXeC7CY2lCeCWYxNxHuE7AwU/8ezc 9dGxrxsBI2K9xuGB0+aub06frJMv+nFHEseN4arha2ondtof7dAR7MT4T50XqkNyJBka uNpAaVHJLH/QSIU4TqQ4CQKTZTV9dkqzEpj0RPq62ErXusXJHK1TfNRD5ZuxayEWe3oa 0q9uzlQTv+RTAX6nkaMlK3mHtbH20THh8Z6eZ0cQIOd7Rz7JQ5WR14A/qTGdocc/uZMO C15g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1708360368; x=1708965168; h=content-transfer-encoding:cc:to:from:subject:message-id:references :mime-version:in-reply-to:date:x-gm-message-state:from:to:cc:subject :date:message-id:reply-to; bh=+Io/XkVO0aE6tyFm8S2xv8fYf7/3UUb5MxQd7q2MHD4=; b=FiANYTgFWbJj53ATNVGOD5CwYGRgQU1PSm1lUYAEZJKzuU77NOk384O6AFARFBJcis oHwR0kCX4at3/0w8wFu/DX1O370RIZhS/H5lkQUIBc5O2f9v/sLvboZ1aevDtq/I2M6b ngi6UZnek3HoRhrv1Tmg60804gEfn2Wy7OYBYS1JykGGvSRWhRkG2ka+ypQeNTarRBUE 6GKUENzDeEcKmOgTZAQ0IXu2TfjB3jFo/OZjAN4ukTR3Im5p3qde71D+d2DimLoVzZyE Unf81UEgjGiMm9MN8UXrfhl7QtR2s+50+6wNi44wwWRmPtq47Y/ftaK97HmHsuLp9Fe3 MWfg== X-Gm-Message-State: AOJu0Yxpr6JXtEfr1Tu5aU65hKNxAF+GC/tns/r7KzNXx4UCP0pe56gq lA9lPl2VOWk6ILDfDhqXlIsW2g8QRf5Tt7Yui3d5BjG2d034ELXoxaV7HCvbb0gZVhbWF0gAwG9 TjTGtQKJXplsOOM/SyR7taerFW+ienaUD4IHkr6aj+LJ/G2uVPDYdZrKOSs16Ma0IqAKiz+2eL3 MVdPrX/uRz7u68/gz5PBU+e2lWnuW6jw== X-Google-Smtp-Source: AGHT+IGHTn9Vsf53EhS0k08RNXBinFE+r34EZUnAIJIwQnhcX/n78R6sFoc+BgQGUu3E9YM9cYTMirBzfSY= X-Received: from zombine.c.googlers.com ([fda3:e722:ac3:cc00:68:949d:c0a8:7f3e]) (user=barnas job=sendgmr) by 2002:a05:6902:1001:b0:dcc:79ab:e522 with SMTP id w1-20020a056902100100b00dcc79abe522mr515142ybt.11.1708360368073; Mon, 19 Feb 2024 08:32:48 -0800 (PST) Date: Mon, 19 Feb 2024 16:32:04 +0000 In-Reply-To: <20240219163213.855667-1-barnas@google.com> Mime-Version: 1.0 References: <20240219163213.855667-1-barnas@google.com> X-Mailer: git-send-email 2.44.0.rc0.258.g7320e95886-goog Message-ID: <20240219163213.855667-4-barnas@google.com> Subject: [PATCH 3/3] misc: atsha204a: fix wakeup function From: " =?utf-8?b?TWljaGHFgiBCYXJuYcWb?= " To: u-boot@lists.denx.de Cc: " =?utf-8?b?TWljaGHFgiBCYXJuYcWb?= " , Tom Rini X-Mailman-Approved-At: Mon, 19 Feb 2024 17:35:44 +0100 X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 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.8 at phobos.denx.de X-Virus-Status: Clean The ATSHA204A chip requires SDA line to go low for at least 60us to wake up the chip. Previous implementation did not meet this requirement due to the NAK received on bus and not sending the zeroes. The function to ignore the NAK and send bytes regardless is not supported in the u-boot making it impossible to wake up the chip this way. Instead, the bus speed, if needed, is set to lowest value and the message is sent to the address 0x0. This way, the address of zero makes the SDA line go low for about 80us, meeting the required time to wake up the chip. The zero length packet is not sent by the i2c, so the one byte is sent to the transfer function, but only the address is sent anyway. After sending the zero address, the bus speed is restored to the previous value if it was slowed down to wake up the chip. Signed-off-by: Michał Barnaś --- drivers/misc/atsha204a-i2c.c | 49 ++++++++++++++++++++++++++++++++---- 1 file changed, 44 insertions(+), 5 deletions(-) diff --git a/drivers/misc/atsha204a-i2c.c b/drivers/misc/atsha204a-i2c.c index 29daefb2a5..707daa90bd 100644 --- a/drivers/misc/atsha204a-i2c.c +++ b/drivers/misc/atsha204a-i2c.c @@ -21,7 +21,6 @@ #include #include -#define ATSHA204A_TWLO_US 60 #define ATSHA204A_TWHI_US 2500 #define ATSHA204A_TRANSACTION_TIMEOUT 100000 #define ATSHA204A_TRANSACTION_RETRY 5 @@ -34,6 +33,48 @@ static inline u16 atsha204a_crc16(const u8 *buffer, size_t len) return bitrev16(crc16(0, buffer, len)); } +static int atsha204a_ping_bus(struct udevice *dev) +{ + struct udevice *bus = dev_get_parent(dev); + struct i2c_msg msg; + int speed; + int res; + u8 val = 0; + + speed = dm_i2c_get_bus_speed(bus); + if (speed != I2C_SPEED_STANDARD_RATE) { + int rv; + + rv = dm_i2c_set_bus_speed(bus, I2C_SPEED_STANDARD_RATE); + if (rv) + debug("Couldn't change the I2C bus speed\n"); + } + + /* + * The I2C drivers don't support sending messages when NAK is received. + * This chip requires wake up low signal on SDA for >= 60us. + * To achieve this, we slow the bus to 100kHz and send an empty + * message to address 0. This will hold the SDA line low for the + * required time to wake up the chip. + */ + msg.addr = 0; + msg.flags = I2C_M_STOP; + msg.len = sizeof(val); + msg.buf = &val; + + res = dm_i2c_xfer(dev, &msg, 1); + + if (speed != I2C_SPEED_STANDARD_RATE) { + int rv; + + rv = dm_i2c_set_bus_speed(bus, speed); + if (rv) + debug("Couldn't restore the I2C bus speed\n"); + } + + return res; +} + static int atsha204a_send(struct udevice *dev, const u8 *buf, u8 len) { fdt_addr_t *priv = dev_get_priv(dev); @@ -94,7 +135,6 @@ static int atsha204a_recv_resp(struct udevice *dev, int atsha204a_wakeup(struct udevice *dev) { - u8 req[4]; struct atsha204a_resp resp; int res; @@ -105,10 +145,9 @@ int atsha204a_wakeup(struct udevice *dev) * when the device is idle, asleep or during waking up. * Don't check for error when waking up the device. */ - memset(req, 0, 4); - atsha204a_send(dev, req, 4); + atsha204a_ping_bus(dev); - udelay(ATSHA204A_TWLO_US + ATSHA204A_TWHI_US); + udelay(ATSHA204A_TWHI_US); res = atsha204a_recv_resp(dev, &resp); if (res) {