Received: by 2002:a05:6a10:9848:0:0:0:0 with SMTP id x8csp1021967pxf; Thu, 8 Apr 2021 20:16:49 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxFdDrtkAbS185Qyltzol5QRFirjIAlBfcP8K4Q7nVXISwQGrFJpg/aQyZU4L/2tsVHL/x7 X-Received: by 2002:a05:6402:1004:: with SMTP id c4mr15423007edu.364.1617938208890; Thu, 08 Apr 2021 20:16:48 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1617938208; cv=none; d=google.com; s=arc-20160816; b=KCbCOR+6mU6JrAGOQgr6QZNLVyk06NhRvfUJm7Yj5KD4jiJs5oxwl8xXiu23CiaZZc AGF9V/z5ElWSmTwXNAunI0bFNMcrPqWD2IFS5nhUEQkLNjurd6scRideZOd4pqGlJ7KW OjNwwQMuDM3HzUTb33aIXg2TH+8woZWQDOO17tYXMmByRqsSUm76UIYlLZ1SK8C/d5Cs xNz1NNP6ZDwHSpkz3lzKN1QUSoRl2FkSCLIcJvc180XtlfDSxkdd9y8WWzg4QxQ4dSUu ZU9zKseN6/HV+av3ccH7PS7cxrZLFIhUhMaabdU1bYdlw1M9sYMscMBxusIILNBnUyxH joQg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=54Nec5iwCSagU+D8h6pLpuy7QchFo0Lir6iLHH9sio0=; b=ZYoQjluiCl30LD/3ihs2cZJwDMkbrTIR2gOq0P6c5Na+ucsbAh96boAbswExbzF9yV kkRfqGdnn2uQZBvk5Rna5jdv9tv+odBX0VGVqL+VkyWvPUkCe71XlVBM8pG8E8wpuNpJ aUMKtjSSh0y4qghyNnjxy0mdxsm1rpwAagq/C2dti8LryXiyu11MScWkN2dLVKddXTsU iyVBKwf5S3t8R2jJMsILGwBj1tQfR0bueOrDkYJC92YXaOB7DaQk/J1Lky8348h2OSOk 3XJfOYNQethSkUtYaLyFYyOmeGrx6tK8eQHkk/0o9KJJy9ARD31w+kBloTVRdRpGfS0J YbIg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=q0YxXX5M; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id m13si1114668edc.258.2021.04.08.20.16.25; Thu, 08 Apr 2021 20:16:48 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=q0YxXX5M; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233315AbhDIDPN (ORCPT + 99 others); Thu, 8 Apr 2021 23:15:13 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49930 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233318AbhDIDPI (ORCPT ); Thu, 8 Apr 2021 23:15:08 -0400 Received: from mail-qt1-x833.google.com (mail-qt1-x833.google.com [IPv6:2607:f8b0:4864:20::833]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C4156C061760; Thu, 8 Apr 2021 20:14:56 -0700 (PDT) Received: by mail-qt1-x833.google.com with SMTP id y12so3206393qtx.11; Thu, 08 Apr 2021 20:14:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=54Nec5iwCSagU+D8h6pLpuy7QchFo0Lir6iLHH9sio0=; b=q0YxXX5Mt9eefEsjfjHNXXCbx5zzfeuQhsbj6KdGLsHTVEMQ1vzeTb602XCLDQBvCi oV+bl/D39Coaf6l4vGAHPUmlHMbUuQh1KJ4hEpTmP2EmOsRfywm8N8rUHARHAffaTWqB EgYwxt2WrpbtH4N40Q2uziJFDuRreV2nalTOHK4JtKho1VbN4siTWBDjFO+MdTI4H90J qIF+JI1dUns6UinCTSvu5fAQ/xfjKJv1SGnLzWA3FiAfOrbARxe2HMLxLcXS9ZgPup9y kLKGOAahb4XAkVFT0NovL0v0Y2uS/pJV21BG/VpxWoxgQydGHciVjmlbfSw0oSEHw6lQ BpLg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=54Nec5iwCSagU+D8h6pLpuy7QchFo0Lir6iLHH9sio0=; b=el7n8crxmCwFqVWtSKXYJz3h95GFWI+agCS/xKol8xKLDasYGKSy2mHoJoTbr4Yp3W YXePNIxudVDjbEVzCTZthkhM/iKLqd8UU0pruJLdUhDNRfbIcWVTtcP2L39PXNX+KQsw nOz0byTsXfNcEe/1orOt413jUPlSNWGkksU/Rr3pTRBGc0VSSpSW/2WhFWcTA49+w89K F5ZAUOTRDvRTHVkXZCpYGurSdZTVqyYpw7frwM4AK7gwOZp6H/Btv3zQpoB5AM4cVuSS 8X6aK7fl07ieqP6DZTr+EHIldAL3pXrMh87Wu3UuNUOsSPHMKqUVIuIfPnsHCEW+CKCk XLPA== X-Gm-Message-State: AOAM532fu46Y9WOCDpIsWwofeN0eIs6SIGs91Jkk7PvK0iVGisnpqQSr ZiCEyhQBM8VBZjzQQvW+gMua+7lC+RPW8eV5 X-Received: by 2002:ac8:768a:: with SMTP id g10mr4280577qtr.245.1617938096065; Thu, 08 Apr 2021 20:14:56 -0700 (PDT) Received: from LuizSampaio-PC.localdomain ([2804:14d:5c21:af45:3b27:576c:7dde:37f1]) by smtp.gmail.com with ESMTPSA id x1sm1003167qtr.97.2021.04.08.20.14.53 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 08 Apr 2021 20:14:55 -0700 (PDT) From: Luiz Sampaio To: zbr@ioremap.net Cc: corbet@lwn.net, rikard.falkeborn@gmail.com, gregkh@linuxfoundation.org, linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, Luiz Sampaio Subject: [PATCH v6 4/6] w1: ds2438: fixing bug that would always get page0 Date: Fri, 9 Apr 2021 00:15:31 -0300 Message-Id: <20210409031533.442123-5-sampaio.ime@gmail.com> X-Mailer: git-send-email 2.30.1 In-Reply-To: <20210409031533.442123-1-sampaio.ime@gmail.com> References: <20210409030942.441830-1-sampaio.ime@gmail.com> <20210409031533.442123-1-sampaio.ime@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The purpose of the w1_ds2438_get_page function is to get the register values at the page passed as the pageno parameter. However, the page0 was hardcoded, such that the function always returned the page0 contents. Fixed so that the function can retrieve any page. Signed-off-by: Luiz Sampaio --- drivers/w1/slaves/w1_ds2438.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/w1/slaves/w1_ds2438.c b/drivers/w1/slaves/w1_ds2438.c index ccb06b8c2d78..ef6217ecb1cb 100644 --- a/drivers/w1/slaves/w1_ds2438.c +++ b/drivers/w1/slaves/w1_ds2438.c @@ -62,13 +62,13 @@ static int w1_ds2438_get_page(struct w1_slave *sl, int pageno, u8 *buf) if (w1_reset_select_slave(sl)) continue; w1_buf[0] = W1_DS2438_RECALL_MEMORY; - w1_buf[1] = 0x00; + w1_buf[1] = (u8)pageno; w1_write_block(sl->master, w1_buf, 2); if (w1_reset_select_slave(sl)) continue; w1_buf[0] = W1_DS2438_READ_SCRATCH; - w1_buf[1] = 0x00; + w1_buf[1] = (u8)pageno; w1_write_block(sl->master, w1_buf, 2); count = w1_read_block(sl->master, buf, DS2438_PAGE_SIZE + 1); -- 2.30.1