From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrey Smirnov Subject: Re: [PATCH 03/13] RTC: ds1307: Add DS1341 specific power-saving options Date: Wed, 20 Jul 2016 09:11:28 -0700 Message-ID: References: <1465970379-14703-4-git-send-email-andrew.smirnov@gmail.com> <20160619142934.GA8522@rob-hp-laptop> <20160621204904.GA9779@rob-hp-laptop> <20160621210739.GY5809@piout.net> <20160719224728.GP7132@piout.net> <20160720090220.GQ7132@piout.net> <20160720153836.GS7132@piout.net> Reply-To: rtc-linux-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: Sender: rtc-linux-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org In-Reply-To: <20160720153836.GS7132-m++hUPXGwpdeoWH0uzbU5w@public.gmane.org> List-Post: , List-Help: , List-Archive: , List-Unsubscribe: , To: Alexandre Belloni Cc: Rob Herring , rtc-linux-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org, Alessandro Zummo , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org On Wed, Jul 20, 2016 at 8:38 AM, Alexandre Belloni wrote: > On 20/07/2016 at 07:36:55 -0700, Andrey Smirnov wrote : >> On Wed, Jul 20, 2016 at 2:02 AM, Alexandre Belloni >> wrote: >> > On 19/07/2016 at 16:56:56 -0700, Andrey Smirnov wrote : >> >> >> I don't see any value in doing that, could you give me a realistic >> >> >> example of a scenario in which a user would want to spend some of >> >> >> uptime with RTC oscillator fault detection/glitch filtering disabled >> >> >> and then enable it? >> >> >> >> >> > >> >> > Well, the issue is not being dynamic, it is differentiating between >> >> > hardware description and user configuration. Configuration must not be in >> >> > DT. >> >> >> >> Why? And I don't mean in a generic sense, but in this particular case. >> >> What is gained by not having this bit of configuration, whose only >> >> consumer is the driver, in the device tree file? >> >> >> > >> > Because configuration doesn't belong to DT. DT is about hardware >> > description, not configuration. >> >> That doesn't really answer my question. You just re-iterating some >> maxim without explaining what is the point behind applying it. >> > > Well, that is from the device tree specification and how the device tree > maintainers want it... And yet, we have whole subsystems such as "nvmem" and I am sure plenty of other smaller examples where that maxim is being applied very lax if at all. So it seems people can and do make compromises regarding the "no configuration in DT" rule if pros of doing so outweighs the cons. Andrey -- You received this message because you are subscribed to "rtc-linux". Membership options at http://groups.google.com/group/rtc-linux . Please read http://groups.google.com/group/rtc-linux/web/checklist before submitting a driver. --- You received this message because you are subscribed to the Google Groups "rtc-linux" group. To unsubscribe from this group and stop receiving emails from it, send an email to rtc-linux+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org For more options, visit https://groups.google.com/d/optout.