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=-3.8 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no 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 D293DC10F27 for ; Tue, 10 Mar 2020 00:36:24 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id AEF9D24655 for ; Tue, 10 Mar 2020 00:36:24 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727527AbgCJAgX (ORCPT ); Mon, 9 Mar 2020 20:36:23 -0400 Received: from mail.baikalelectronics.com ([87.245.175.226]:44492 "EHLO mail.baikalelectronics.ru" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727287AbgCJAgX (ORCPT ); Mon, 9 Mar 2020 20:36:23 -0400 Received: from localhost (unknown [127.0.0.1]) by mail.baikalelectronics.ru (Postfix) with ESMTP id 925DC80307C8; Tue, 10 Mar 2020 00:36:19 +0000 (UTC) X-Virus-Scanned: amavisd-new at baikalelectronics.ru Received: from mail.baikalelectronics.ru ([127.0.0.1]) by localhost (mail.baikalelectronics.ru [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id esZUdWVnJi4B; Tue, 10 Mar 2020 03:36:18 +0300 (MSK) Date: Tue, 10 Mar 2020 03:35:28 +0300 From: Sergey Semin To: Lee Jones CC: Alexey Malahov , Maxim Kaurkin , Pavel Parkhomenko , Ramil Zaripov , Ekaterina Skachko , Vadim Vlasov , Thomas Bogendoerfer , Paul Burton , Ralf Baechle , Sebastian Reichel , Rob Herring , Mark Rutland , , , Subject: Re: [PATCH 0/4] syscon: Alter syscon and reboot-mode drivers References: <20200306130356.6ABDD8030703@mail.baikalelectronics.ru> <20200309071233.12D6980307C8@mail.baikalelectronics.ru> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20200309071233.12D6980307C8@mail.baikalelectronics.ru> X-ClientProxiedBy: MAIL.baikal.int (192.168.51.25) To mail (192.168.51.25) Message-Id: <20200310003619.925DC80307C8@mail.baikalelectronics.ru> Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Mar 09, 2020 at 07:13:13AM +0000, Lee Jones wrote: > On Fri, 06 Mar 2020, Sergey.Semin@baikalelectronics.ru wrote: > > > From: Serge Semin > > > > A lot of custom functionality can be tuned by means of a syscon-registers > > placed in Baikal-T1 CCU memory space (though in documentation they are > > called system devices registers). While in most of the case the syscon > > registers usage will be distributed between corresponding subsystem drivers > > (thanks to pretty much coherent design of the registers fields), there are > > some peculiarities we should reflect in the generic syscon code. > > > > First of all seeing there is an map endiannes config acceptable by the syscon > > it would be justified to add the corresponding properties declared in the > > mfd/syscon.yaml bindings file. Don't really know why this hasn't been done > > before. > > > > Then as it's going to be general in the framework of the Baikal-T1 SoC support > > integration into the kernel, we suggest to replace the legacy text-based > > syscon-reboot-mode dts-bindings file with yaml-based one. > > > > Finally seeing the syscon-reboot-mode devices are mostly used in conjuction > > with just syscon-reboot device (in particular Baikal-T1 CCU WDT RCR register is > > preserved during any type of reboots) and for several other reasons (see the > > commit message for details), we suggest to add the 'regmap' property support > > to the syscon-reboot-mode driver. This would eliminate the requirement of > > placing the syscon-reboot-mode dts-node in the syscon sub-nodes area. > > > > This patchset is rebased and tested on the mainline Linux kernel 5.6-rc4: > > commit 98d54f81e36b ("Linux 5.6-rc4"). > > > > Signed-off-by: Serge Semin > > Signed-off-by: Alexey Malahov > > Cc: Maxim Kaurkin > > Cc: Pavel Parkhomenko > > Cc: Ramil Zaripov > > Cc: Ekaterina Skachko > > Cc: Vadim Vlasov > > Cc: Thomas Bogendoerfer > > Cc: Paul Burton > > Cc: Ralf Baechle > > Cc: Lee Jones > > Cc: Sebastian Reichel > > Cc: Rob Herring > > Cc: Mark Rutland > > Cc: devicetree@vger.kernel.org > > Cc: linux-pm@vger.kernel.org > > Cc: linux-kernel@vger.kernel.org > > > > Serge Semin (4): > > dt-bindings: syscon: Add syscon endian properties support > > dt-bindings: power: reset: Replace SYSCON reboot-mode legacy bindings > > with YAML-based one > > dt-bindings: power: reset: Add regmap support to the SYSCON > > reboot-mode bindings > > power: reset: syscon-reboot-mode: Add regmap dts-property support > > > > .../devicetree/bindings/mfd/syscon.yaml | 14 +++++ > > .../power/reset/syscon-reboot-mode.txt | 35 ----------- > > .../power/reset/syscon-reboot-mode.yaml | 58 +++++++++++++++++++ > > drivers/power/reset/syscon-reboot-mode.c | 11 +++- > > 4 files changed, 80 insertions(+), 38 deletions(-) > > delete mode 100644 Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.txt > > create mode 100644 Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.yaml > > Please re-send this patch --threaded. > > As it stands, it's been scattered to the 4 winds throughout my inbox. > Hello Lee, Of course I've sent the patchset with 'thread=true' config set. It appears our corporate email server changes the Message-Id field of messages passing through it. Due to that the emails threading gets to be broken. I'll resubmit the properly structured v2 patchset as soon as our system administrator fixes the problem. Sorry for the inconvenience caused by it. Regards -Sergey > -- > Lee Jones [李琼斯] > Linaro Services Technical Lead > Linaro.org │ Open source software for ARM SoCs > Follow Linaro: Facebook | Twitter | Blog