Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <31b0a8b1-d027-d47a-e10d-614c09211734@kontron.de>
Date:   Wed, 22 Mar 2023 14:14:50 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.9.0
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>
 <7268b05b-bbb1-ec4d-6a05-f5ccbdfeed90@kontron.de>
 <da83fdd0-9eae-09ce-ff6f-47cc012f9628@kontron.de>
In-Reply-To: <da83fdd0-9eae-09ce-ff6f-47cc012f9628@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?dnZhRzJ0blZncW9LVUNud1dZOCsrNGtYYWJiZE1ncWFBTXNWcHRKWXJXVGFP?=
 =?utf-8?B?eFFWL0lOZ0VncWJKeHB6L1FSMmdqZjdmOGlXdU9YdFdQUkJWOTB6WkpsdFpi?=
 =?utf-8?B?dkRudHc0Smx5a2VhVFphRmx0NmxCa09YSW1nTXpWQXNLL09wMUNzcVNZRmJO?=
 =?utf-8?B?WDZ5M1ZsWElnZkxOM01iQkI1VlBCaGhqMUdwaHBRT0ZheFVtbUJlWlpnR2Q5?=
 =?utf-8?B?M2lxNnlMNWZDKy9CeWVHN0t3L1I2RlQvSGxrbFI5QlZvLzZWNkJVK1IwMnVz?=
 =?utf-8?B?d0NwMklSSG51S0hicWozNXNNTVIwdWVKRWFWZDRhTi9TZGFuQlFPamNQMXhn?=
 =?utf-8?B?RzZOVmZmS1BmbTB2WkdxVTc1MEtVRjlBTlJ3U1dFREN2czduM2U1SGoxekhH?=
 =?utf-8?B?Wmp2VnpuUVYyR2FuYnlVNDhMOExXNkVOZ2tMMjRadGNqOGF2RFd0eWpBYTRp?=
 =?utf-8?B?U3Qyc3hHd1ZHdzVZaHBxODFWaGJ4cXlpMDhRYlZMSWJjRXpOdHhmcmhJM1pH?=
 =?utf-8?B?UCtCdXZCWUhicXNtVng0djhwTElhb3UrWmlxKzVWNWp1UDZMQ0RGcTNxQ0k5?=
 =?utf-8?B?em1nWG5MVC9aWHk1OVJ0ZDg4eFplMTZSWTJuaDVHRDI1T2IxYitwaEVINFFL?=
 =?utf-8?B?SHQ0M0JlVXdqMmVCSmYxbUVtb1JMTHpvaUN0bGgzMXdtOFU5bjZtem8wT1lR?=
 =?utf-8?B?b2dNcEtQTDUwakZDQXkrTXpiUTBqVjlqWmZFbUxkN3pkU2VPaXpibWs1Z1JV?=
 =?utf-8?B?WlN0aWVSVlpqa1lOeWNIVTJyUHlHWUpralRyMm8yTnpSUk9JZzd0L1czWFFL?=
 =?utf-8?B?SHN0b1J4aXNGbzZsZEVFM1NSUEdEVjZ1Y3Npd3lqOEs1L2p6ai8wYU96dFdr?=
 =?utf-8?B?K05KbngzWUxVNnBuanpGRERoQlJkTzNOZXBWRU03c0N4VFF6YU9DakRJN1By?=
 =?utf-8?B?ZUljNmZEMHppU1lsR29NUDlFeUhwUnRReTFSSllFd0tCcyt1ZS9aTWJFUjhI?=
 =?utf-8?B?d1k4L0xEMkd5bDJIQmoyUGthMTIwRjE2bU54Ti9jcG5iSHplTzkwcmV3V24v?=
 =?utf-8?B?OFNLODl4NlZiUnNPNmdEYjgyQXdpVGhMbWlpTEpudUZmamtpUHJnL3pyVTQ2?=
 =?utf-8?B?bDdmQ1M3V1dEbG5IVlp2dXZENFBzWGpGVVdBcUhDVDN0ZHUxZ2Y5b0VOYUlH?=
 =?utf-8?B?dnFxK1g4WHgrMUVHdTVLc2IzbUI1N2E0ZTZ1OEw4TzNrckdOdUNEVDBHRVdG?=
 =?utf-8?B?QjB4Ym13dHJvNEM3cUxtZjlxK1c5UVFZRXVDNFBTSVlEU1ZVOEp2U0hiSGtz?=
 =?utf-8?B?aklnajdPL1dnZ25iekdCTlY3UUtpN3czVWtGa1UxSHlscU9tTVN1WjZvUW9E?=
 =?utf-8?B?SllwazFqNFAzdzhhSWs2a2JDWTNJUFNuZEljbTN6UDNhZWRwT0tld0pQa29W?=
 =?utf-8?B?cytid2R6V1F5VEwxZFBnQlRWZVE5M1pjYXR1WUVQaWprb2o2RzBVWXlVWFJD?=
 =?utf-8?B?NElVS1EyTlNpQWR1NFhwRTF1ajNtdVpldWlJWjYxVXVWTHhoY0RtWU55QzN6?=
 =?utf-8?B?K09vUHJMK3ZSYmxzRWpERDBuRklTMHZtcVFUbklrQ1ZIWXhvWUFhc1dQQTVS?=
 =?utf-8?B?K05YVXRkWTJzNEo3ZnJKNG1lUUNqcWhOM2tEN0xNajE1MVdGcktFeGdQMnVM?=
 =?utf-8?B?NmR1STFqZDA4WnFuMXhpQXBWbGxITWhseVRFdVNUcGdzcUtvWlV5U0pMRUto?=
 =?utf-8?B?M25lcUx5OXpqU0RDaTJiSXV4RnZ3a1F2V0t4aDJGZ2h2ZTJmUklic2d4UmxY?=
 =?utf-8?B?WHZaWW53TTFkb2FROGVnUThIM2FiREszdE14bzJsOGpsQVFYVjVSUnRLZkxt?=
 =?utf-8?B?eE9aZ0NYbUkvRkZMVUpQQ3orWHoxa2FJQ3hZZmwwSDRYb1Fnb3dYV1FZV3hR?=
 =?utf-8?B?eGZZZnBSdVJjWmdOcStlQkFpZk5HZ2NtWm1DN3lqZW5FL0ozZ2FOSEh3VWJH?=
 =?utf-8?B?aDZhOU9kVkYzVHM4NmNCaTY5ay9KaGxtN0RBL1ppOUwrWnMwYmZQS1hHUkNI?=
 =?utf-8?B?N3pta055WHFHcmNGbmhnK2kwQTBXWWhlakRDVzI1NDIyMVpPNVZWSmRuOElV?=
 =?utf-8?B?TTRNdVRxck5DM3laN3EyTWhURC9NYzE5MXdqRVJPU2xzTWg3dWpFMk4yMEdI?=
 =?utf-8?Q?K30gmseLbzpPNhlc0c2ZNDGfkg+c3ajFIgV3wzR9r4oI?=
X-OriginatorOrg: kontron.de
X-MS-Exchange-CrossTenant-Network-Message-Id: b399645c-2299-4e4d-897f-08db2ad76c3a
X-MS-Exchange-CrossTenant-AuthSource: PA4PR10MB5681.EURPRD10.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Mar 2023 13:14:52.6628
 (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: WPHjknMUALQ/Uo9Im6L4BLkuh7tJHnFfqlZhufQ+Ad7fCDJq+/0OYSTeiAcIjS4OLpe3fV32gvIQJwoxtxm2JGlO3ZRFjCHDrszSjUARWsE=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DU0PR10MB7997
X-Spam-Status: No, score=-0.0 required=5.0 tests=DKIM_SIGNED,DKIM_VALID,
        NICE_REPLY_A,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,
        SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Alexandre,

On 06.03.23 14:27, Frieder Schrempf wrote:
> On 13.02.23 10:18, Frieder Schrempf wrote:
>> 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.
> 
> Ping!?

It seems like you decided to ignore me for whatever reasons there are.
I'm sure we can sort it out in some way if you would respond, please.

Thanks
Frieder