From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753044AbcBOMtt (ORCPT ); Mon, 15 Feb 2016 07:49:49 -0500 Received: from lists.s-osg.org ([54.187.51.154]:48366 "EHLO lists.s-osg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752632AbcBOMtr (ORCPT ); Mon, 15 Feb 2016 07:49:47 -0500 Subject: Re: [rtc-linux] [PATCH] rtc: s3c: Document required clocks in the DT binding To: Krzysztof Kozlowski , rtc-linux@googlegroups.com, linux-kernel@vger.kernel.org References: <1455038632-14545-1-git-send-email-javier@osg.samsung.com> <56C01FDD.9050204@samsung.com> From: Javier Martinez Canillas Cc: k.kozlowski.k@gmail.com, devicetree@vger.kernel.org, linux-samsung-soc@vger.kernel.org, Alexandre Belloni , Rob Herring , Alessandro Zummo Message-ID: <56C1C95E.7030002@osg.samsung.com> Date: Mon, 15 Feb 2016 09:49:34 -0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.0 MIME-Version: 1.0 In-Reply-To: <56C01FDD.9050204@samsung.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello Krzysztof, Thanks a lot for your feedback. On 02/14/2016 03:34 AM, Krzysztof Kozlowski wrote: > W dniu 10.02.2016 o 02:23, Javier Martinez Canillas pisze: >> The S3C Real Time Clock driver requires the clock and source clock to >> be defined in the device node but that requirement is not documented. > > +Cc: Alessandro Zummo > I didn't add Alessandro to cc deliberately since Alexandre is the one maintaining the RTC subsystem lately and before that, the RTC patches were picked by Andrew Morton since Alessandro didn't even answer the emails. > No, not entirely. The samsung,s3c6410-rtc needs rtc and rtc_src clocks > but rest of compatible devices need only rtc clock. > Right, sorry for missing that. I noticed that this patch has already been picked by Rob Herring (I thought it would be picked by Alexandre though) so I will post a follow up patch explaining that rtc_src is only for the s3c6410-rtc compatible. > Best regards, > Krzysztof > Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America