Message-ID: <7268b05b-bbb1-ec4d-6a05-f5ccbdfeed90@kontron.de>
Date:   Mon, 13 Feb 2023 10:18:08 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.7.2
Subject: Re: [PATCH 0/7] Enable backup switch mode on RTCs via devicetree
Content-Language: en-US
From:   Frieder Schrempf <frieder.schrempf@kontron.de>
To:     Alexandre Belloni <alexandre.belloni@bootlin.com>,
        Frieder Schrempf <frieder@fris.de>
Cc:     devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-kernel@vger.kernel.org, linux-rtc@vger.kernel.org,
        Alessandro Zummo <a.zummo@towertech.it>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Rob Herring <robh+dt@kernel.org>,
        Sascha Hauer <s.hauer@pengutronix.de>,
        Shawn Guo <shawnguo@kernel.org>
References: <20230201143431.863784-1-frieder@fris.de>
 <Y9qQHpySbrFcBB0P@mail.local>
 <b58f5e31-f13f-6d90-774c-2017ee3de121@kontron.de>
In-Reply-To: <b58f5e31-f13f-6d90-774c-2017ee3de121@kontron.de>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?Wm8xODU1RXRHM2tKeG8vbjN5ZzdBRUY3SnFJK2lBU2piZGQwaGltNXNNSk5Y?=
 =?utf-8?B?WXZDVmZxNzhCeHVNTG9LMVp6UU9ESVFWY3k2cXhVZW5yZDVHTU1HcmthUUtK?=
 =?utf-8?B?NUZta3hnVGhLVlBrd0RuOUtFVnR4S0REME5RQWRRVW9HeHpSRjdmbUtIYkNk?=
 =?utf-8?B?d0R5blcwT0J1SXUvNGJLUjBhaURlc2QrOXFTSFFNbldodFJNQ1grZk5qUnV2?=
 =?utf-8?B?SVM4R0prbVdRNWpOU1hLYjFwTFBNRkIyRVE4QXRFdy9BVVFmRkNOK0VDNTdh?=
 =?utf-8?B?dEQrZ1pZUXNML0p0VEdyZk94V0s0K0lYZ0podmJaWDhiOTNHRkhUdkVmNFlt?=
 =?utf-8?B?SGZYUzFYTDlVRWtJV1Q0SlBKS216MEZMQUF3QzhScGlkMHJpSzVmZEY3NmZo?=
 =?utf-8?B?SVlCaGRlcWMrRWg2bUpSZWxtNGRFK0JzOFZDcExrWTlJQXo0TmxXVTZzNlFp?=
 =?utf-8?B?bVpnL2NSMTFLVGoxL2FDTFAvbFZtSjVzeURVRWZKR2hBNS9kWjJzQmViMnRj?=
 =?utf-8?B?L1F6enRVMmFhdDVoVlYzK05kWE5uVXBmZllMT2paV2F5QndmSGh0eHBDUnhV?=
 =?utf-8?B?d285RHdMOXhiNWxWZExCQlZ0dzUvYk52VER6Yk84K1Z3eHQrT3RiQUlwOHds?=
 =?utf-8?B?L3FrWnlLTmU4M0YxaE1rYVhPVVZuYlZpSVNnc0JSbmFmb3FvZ1pseHhOZmZQ?=
 =?utf-8?B?VXF5c3p2d1JhK3owT2NzOFZtejR3ME4wK0xZNEJJU2RKbmZtNWJQbWtBNm1O?=
 =?utf-8?B?OHEwWDc1NFVQMWxYSnZQNEZEUmRHUjRpbzJGOFczaklER0twNnRMeWxERlBz?=
 =?utf-8?B?Qk9rR0cza1p2MndETUZxTlFmMGZwNkhHaFNDVElXbHVMbDBESzkyWTAxS00y?=
 =?utf-8?B?RmNwb3hLOHIvdjYrQ3Y0Y1BTVVhVS2dGaDgrRDFZZVZCREI0bW1qVTJxa1BH?=
 =?utf-8?B?UDIycGJ2NnVSRHJZRnhDZzE1aHRhYXVmSERHdHVKZWdjMjNhZFBBaTdzQkhE?=
 =?utf-8?B?RXV0VUdvdTJ6ZWNPd2tDRFpxUUpjNTY1QTA4Q0tLVEhSSkdzR1RndlpjOUc0?=
 =?utf-8?B?b28wL1JNUlZGMkxtb09lZDlVczV6YmtOY3lOYUQ5OGhGR3V6V2xtUG02MEJI?=
 =?utf-8?B?TFBOQS9tbEt3eHcxdGkyQ2pLMFRSMURrcTZUQ2lrTk9odFlaV2hRNEpTUTVD?=
 =?utf-8?B?T29TYmxWL242OE5aV2NrUWZpK2hZWGNFM0JDUk0vU1pDVFk3Q2Nua0F3bDBW?=
 =?utf-8?B?SjBqOEgrR09aUFJMZDROVXNRckk3Tit0dU9EdHRMS3ZmdTdKOXcrYmhwWllp?=
 =?utf-8?B?bENWdTk2Z0dtcjBSc2ZRSStNZmVBWHJnOG9JbnduQWtTZEo5alY1NzlSMzVz?=
 =?utf-8?B?dzRweTdVMW1Qa1J1Vk9iWnFjMmtkSTFaVG1CK1BsRTQ0SlVsc3NUMzFYV2RW?=
 =?utf-8?B?QkFsNU93TDNSc2poc2Q4alNxR2FIejNLM0dEK29SOFNFeGpSemFoWEc5QWlF?=
 =?utf-8?B?dUc0ODBhR2ZjWU5kb28zRzFnZkk2MERQNjJybVJWKzI4eVBwbkwzZXlkdEZD?=
 =?utf-8?B?SG1HTkNSZHd1cVFsOGtiYjZyd2llaHFjclVkVEFYOElIS1hhMS9PdFZ1dm5M?=
 =?utf-8?B?eG1VSHlDamhXZURRa1g1N0EvdnE2dm10S3N2NzFBYUFJZDFLSEZyaTQrU3Nq?=
 =?utf-8?B?UWowMVRkZkdBYjNDajhNVzdXNU1lajNEbnV6VWpWTkRmSGIvS0dCNko1UmFj?=
 =?utf-8?B?YUo0dG8zeVBINGU4bVQ5WTVvcldmU1lSY0J2VFFsTFFrSlhweVNyL24yWUZD?=
 =?utf-8?B?RmhyaE1FbWNGT1Rib3o4Z0NnZkk2SmZwMzJTcktzZjlZTzBhNmg3M1BtL2xC?=
 =?utf-8?B?QmtSKzhzTVZKb1p3UTA2UTZnOU1RK2tmekdRZ0gxMUh3bGVnWHZEYmg2VGI3?=
 =?utf-8?B?NEkwTGpWUTZQd0FGeFlzWUJISUI2VlBnQnEyK1JNSURSSjhPaXYwV05MUUJj?=
 =?utf-8?B?NVJLQlY2TjJlSjEyTFFaTzdURnBrREVReWMyL1U3RlU1NCszajYxK1JtVG5X?=
 =?utf-8?B?RGNpeG43WGlJVHZJT0YvR0I1c3JpVGRVSXl1K1lmVVVMUTBXUGEvZ1F0ZnJB?=
 =?utf-8?B?bTVKb3Y4Qit0VzJ2TW9aNG9JaG1sQ1FQOWJyUHJ0OGh4SXdhUDNvYnlhNkYx?=
 =?utf-8?B?Nnc9PQ==?=
X-OriginatorOrg: kontron.de
X-MS-Exchange-CrossTenant-Network-Message-Id: 9f50f52c-f0dc-4d2e-079d-08db0da3395d
X-MS-Exchange-CrossTenant-AuthSource: PA4PR10MB5681.EURPRD10.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Feb 2023 09:18:09.8004
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 8c9d3c97-3fd9-41c8-a2b1-646f3942daf1
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: a9A7oUSNbrWIWmgR5NgxXRk5kLdydEdjS74aGmImEK3nQUVB2r150ni+em5pHOoMW9N7F+zPdhN+cXutWssGbLBwXu0hfILq9K7ibcQsczA=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: GV1PR10MB6075
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Alexandre,

On 01.02.23 17:26, Frieder Schrempf wrote:
> On 01.02.23 17:15, Alexandre Belloni wrote:
>> Hello,
>>
>> You can't do that, this breaks an important use case and it is the
>> reason why I didn't use device tree in the beginning. What is wrong with
>> setting BSM from userspace? You will anyway have to set the time and
>> date from userspace for it to be saved.
> 
> Ok, I was already afraid there is something I missed. Can you give a
> short explanation of what use case this would break?
> 
> There is nothing wrong with setting BSM from userspace. It's just the
> fact that users expect BSM to be enabled in any case as there is a
> battery on the board. It is much more effort to ensure that production,
> user, etc. are aware of an extra step required than to let the kernel
> deal with it behind the scenes.

Would you mind elaborating on your argument that this would break stuff?
I currently don't see how an additional optional devicetree property
would break anything.

Thanks
Frieder