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;
Subject: Re: [PATCH v2 1/3] dt-bindings: rtc: add reset-source property
To:     Rob Herring <robh+dt@kernel.org>
Cc:     "open list:REAL TIME CLOCK (RTC) SUBSYSTEM" 
        <linux-rtc@vger.kernel.org>,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        =?UTF-8?Q?Uwe_Kleine-K=c3=b6nig?= <u.kleine-koenig@pengutronix.de>,
        Qiang Zhao <qiang.zhao@nxp.com>,
        Bruno Thomsen <bruno.thomsen@gmail.com>
References: <20201204092752.GE74177@piout.net>
 <20201211215611.24392-1-rasmus.villemoes@prevas.dk>
 <20201211215611.24392-2-rasmus.villemoes@prevas.dk>
 <CAL_JsqJ-5gwycTRQCdX=ZsefEJ=F1GyTjjDW6QB1PBynibFzLg@mail.gmail.com>
From:   Rasmus Villemoes <rasmus.villemoes@prevas.dk>
Message-ID: <b3c05d29-3ed6-d5f5-d1dd-0ddec1f89276@prevas.dk>
Date:   Sat, 12 Dec 2020 00:10:11 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101
 Thunderbird/68.10.0
In-Reply-To: <CAL_JsqJ-5gwycTRQCdX=ZsefEJ=F1GyTjjDW6QB1PBynibFzLg@mail.gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
Received: from [192.168.1.149] (5.186.115.188) by AM5PR0502CA0015.eurprd05.prod.outlook.com (2603:10a6:203:91::25) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3654.12 via Frontend Transport; Fri, 11 Dec 2020 23:10:12 +0000
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-Correlation-Id: 377c082a-d4bf-406d-f743-08d89e29e9d7
X-MS-TrafficTypeDiagnostic: AM9PR10MB4135:
X-Microsoft-Antispam-PRVS: <AM9PR10MB413513795E1D030653C5666393CA0@AM9PR10MB4135.EURPRD10.PROD.OUTLOOK.COM>
X-MS-Oob-TLC-OOBClassifiers: OLM:9508;
X-MS-Exchange-SenderADCheck: 1
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: 1QWOQITLnRAVyy/tEdYV6HLJRqlEXohvAA82cbQfP6Z9dSFCBuUbHrr5kjI885/ytxVnsweurTEgQn71LzlXK5agXeKrWiLZObATs+aOTGG16DUzyK1B1YfX5ZZaic/GW3p3MFGhGHVP9ORwCyuSTksrOzICCFx3Dsb1zfthjUj0/qlr7WCW1UYjzjAC2AocsJn5N4VLwDPYYXK45so55uCHGnvYNnbBeTb2fCpQF+fQe+z0E8ki+PAVBiP29E13npdDwvDNwYwk8fEfs+OfelLHuMeazXySi4KV2gXSn8Va+KVSSJLFiJrdzymHLlHRbPe6zVrB5D6nIE9/bCmZxpCyIpqnpspIDFcjslA3XHbPDJc8r98v9sLoJwi3X6l4VXWBqPeKROyN3F/7PQhthpsNIzBhMCZLeGHxZdrKDOw=
X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM0PR10MB1874.EURPRD10.PROD.OUTLOOK.COM;PTR:;CAT:NONE;SFS:(346002)(396003)(136003)(376002)(366004)(39840400004)(16526019)(53546011)(16576012)(36756003)(54906003)(316002)(4326008)(2616005)(44832011)(956004)(66946007)(8976002)(66556008)(26005)(6486002)(66476007)(5660300002)(2906002)(8936002)(31696002)(31686004)(8676002)(478600001)(86362001)(52116002)(186003)(43740500002)(45980500001);DIR:OUT;SFP:1102;
X-MS-Exchange-AntiSpam-MessageData: =?utf-8?B?ZFRON01CcENSbjduM0cycXVvZlQ5N09ERFZGbzBzWjc1QkRlWkxlS0w4MXFk?=
 =?utf-8?B?MWs0THRRWXlDT0hJUmdKYlNmZWhkdnFwUVFCVnhOaEQ2SHZ3Vi9lbDlkV1gz?=
 =?utf-8?B?TU52alhOTUpvdllSbEtzZ0xUeCtibGhGbXE3Q200OUptNGtrdGRuWFV0Q3hn?=
 =?utf-8?B?dmErZDMwSnBMUDRGVkdTR3gzakhtaWFKMVR3a2laZW90cGxObldVMWo3RlVo?=
 =?utf-8?B?Y0FGcUhscC90WmU5MzVKa0QraU1KTWNyOXd1c0lKWlc4bDdpdHhpWHVVTE5J?=
 =?utf-8?B?WVJHSmYwa2xvdzRLc0hSMzJkSmFPTHIrREh5QnRjeklJVDV0eTZsQkRGaHVS?=
 =?utf-8?B?WU93ZGdDV1B2SXZIYmViL1NRbS9xOUdqUG02dEdwMSsrc2grelpYRlB2Z21q?=
 =?utf-8?B?aVhyMnVIWHJ5TkM5R2JTeW94N000cnZnUjhqTU5DRExxc0srd1JRaFJRTWdF?=
 =?utf-8?B?ZmUyd0F3eUd5L0o3VGFyVkNaYm96TzA5R1RzWkdGSGg0aDEwYi9FUWFMbWFv?=
 =?utf-8?B?aVF4YkE5bEhiZlBFMG82WnFVRjZLejlEV3pKRUx6cWJLSDlIZjNWdVh1WFBx?=
 =?utf-8?B?ck4xeXF2dlV1QldnVnpINjRGT3pFMGtpS3Bpc1NDTTF6ZEcxdWVrNnp3bTZx?=
 =?utf-8?B?b3h5SE1qRVQ3NTBIcXdwME1WS1RqZVo0U1RkR0pXWkdRQ3hnOTR1WEVleCt0?=
 =?utf-8?B?YkpSQ2ptNlltYTlER25XdG1ldzhMbkk2bkRoVG5ZcXlzZ01TYU00RW5jK1VV?=
 =?utf-8?B?ZnFCTDVmSEs1TUVxditYSzRFSlR3Vk9WcllYRDQ2eFRtZSs4MlM3Ull4Ly9B?=
 =?utf-8?B?em52aVhlRnlMeDRsYURIQ3d3SEg5ejM2NzNpdkNjZDFDcE84MndWeDZZNlRw?=
 =?utf-8?B?ZHNUVVYxTHM4SGF1WEdZV2RLQVcwWFNMeFViaWFPU09LSVgyd3h5Y3hiNURL?=
 =?utf-8?B?Y1puODJJa0s2N1RYMGM2V1Y4TWhFVkd0Qzl0TmhNL0YxdnBDN3pzTlRIRFNV?=
 =?utf-8?B?dzV4ZVZjaHJBUFpXMHVId3FXN0dnbnFudGxwUHlhWmZ2OEc3aWMxenExbGFu?=
 =?utf-8?B?SUlwOEJLTjVtWTd6K1ROVDYvbFVhaXZZbzNuU0d1cWJIZll6NUFkNVF6T0dR?=
 =?utf-8?B?S2oxVFM1b0h4T09LVmM2Q042QnQ0bUc5S1d6S2o5cjFBYlY3ODFKeUE5UHRx?=
 =?utf-8?B?V0hJeGlDQkxRSXhGOG9pdXNFeGhDcmV0TCtmeVBjMzZBZkliSGZIN09SanJt?=
 =?utf-8?B?eWYybWRpalU0aEdzY1hvRGhjaXhNa1BZUVB6OW9jNE9XcGR6Sys0NVhEMFl3?=
 =?utf-8?Q?OVf6aVy2KnY8B0Wk1MKd5tep4gCpfu5j9E?=
X-OriginatorOrg: prevas.dk
X-MS-Exchange-CrossTenant-AuthSource: AM0PR10MB1874.EURPRD10.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Dec 2020 23:10:12.5848
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: d350cf71-778d-4780-88f5-071a4cb1ed61
X-MS-Exchange-CrossTenant-Network-Message-Id: 377c082a-d4bf-406d-f743-08d89e29e9d7
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: +KnbMUnTHCMZRompVyS+NwGbia1DdZutQlFvVZJ48949xwbctsha+LLHoXrsyExjeQC4d2oLCVm+4bBMwRSwjXIR32gdZ41M4u5dBlTwXow=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM9PR10MB4135
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 11/12/2020 23.30, Rob Herring wrote:
> On Fri, Dec 11, 2020 at 3:56 PM Rasmus Villemoes
> <rasmus.villemoes@prevas.dk> wrote:
>>
>> Some RTCs, e.g. the pcf2127, can be used as a hardware watchdog. But
>> if the reset pin is not actually wired up, the driver exposes a
>> watchdog device that doesn't actually work.
>>
>> Provide a standard binding that can be used to indicate that a given
>> RTC can perform a reset of the machine, similar to wakeup-source.
> 
> Why not use the watchdog 'timeout-sec' property?

Wouldn't that be overloading that property? AFAIU, that is used to ask
the kernel to program an initial timeout value into the watchdog device.
But what if one doesn't want to start the watchdog device at kernel
boot, but just indicate that the RTC has that capability?

It's quite possible that if it can act as a watchdog device (and
has-watchdog was also suggested), one would also want timeout-sec and
other watchdog bindings to apply. But that can be added later, by those
who actually want that.

For now, I'd really like to get my board booting again (or rather, not
get reset by the real watchdog just because the pcf2127 driver now
exposes something as /dev/wathdog0, pushing the real one to
/dev/wathcdog1 which doesn't get pinged from userspace).

Rasmus