Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 718B7C64ED6 for ; Tue, 28 Feb 2023 21:11:29 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229910AbjB1VL2 (ORCPT ); Tue, 28 Feb 2023 16:11:28 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57516 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229537AbjB1VLV (ORCPT ); Tue, 28 Feb 2023 16:11:21 -0500 X-Greylist: delayed 82 seconds by postgrey-1.37 at lindbergh.monkeyblade.net; Tue, 28 Feb 2023 13:11:17 PST Received: from relay04.th.seeweb.it (relay04.th.seeweb.it [IPv6:2001:4b7a:2000:18::165]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C970C2B294 for ; Tue, 28 Feb 2023 13:11:17 -0800 (PST) Received: from [192.168.2.144] (bband-dyn207.178-40-209.t-com.sk [178.40.209.207]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-256) server-signature RSA-PSS (2048 bits)) (No client certificate requested) by m-r1.th.seeweb.it (Postfix) with ESMTPSA id BE87620166; Tue, 28 Feb 2023 22:11:14 +0100 (CET) Date: Tue, 28 Feb 2023 22:11:09 +0100 From: Martin Botka Subject: Re: [PATCH v8 2/3] mfd: axp20x: Add support for AXP313a PMIC To: Shengyu Qu Cc: martin.botka1@gmail.com, Konrad Dybcio , AngeloGioacchino Del Regno , Marijn Suijten , Jami Kettunen , Paul Bouchara , Jan Trmal , Jernej Skrabec , Samuel Holland , Andre Przywara , Lee Jones , Rob Herring , Krzysztof Kozlowski , Chen-Yu Tsai , Liam Girdwood , Mark Brown , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Message-Id: In-Reply-To: References: <20230120184500.1899814-1-martin.botka@somainline.org> <20230120184500.1899814-3-martin.botka@somainline.org> X-Mailer: geary/43.0 MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii; format=flowed Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Shengyu On Sun, Feb 19 2023 at 01:52:50 AM +08:00:00, Shengyu Qu wrote: > Hi Martin, >> +static const struct regmap_range axp313a_writeable_ranges[] = { >> + regmap_reg_range(AXP313A_ON_INDICATE, AXP313A_IRQ_STATE), > No, according to datasheet, 0x00 reg is read-only. >> +}; >> + >> +static const struct regmap_range axp313a_volatile_ranges[] = { >> + regmap_reg_range(AXP313A_ON_INDICATE, AXP313A_IRQ_STATE), > > As Tsai said, no meaning to set all register to volatile. ack > > Also, please check the datasheet I metioned in part3 reply, seems its > newer than yours. ack > > Best regards, > > Shengyu > Best regards, Martin