Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754369Ab3EPVv4 (ORCPT ); Thu, 16 May 2013 17:51:56 -0400 Received: from mail-lb0-f180.google.com ([209.85.217.180]:46479 "EHLO mail-lb0-f180.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751269Ab3EPVvy (ORCPT ); Thu, 16 May 2013 17:51:54 -0400 MIME-Version: 1.0 In-Reply-To: <2469545.4zh8j9i5RI@flatron> References: <1368724352-10849-1-git-send-email-dianders@chromium.org> <1944811.006SLH6aYj@flatron> <2469545.4zh8j9i5RI@flatron> Date: Thu, 16 May 2013 14:51:53 -0700 X-Google-Sender-Auth: Qy1P3PNa-evwoCk9uUJyMWimRDc Message-ID: Subject: Re: [PATCH 1/2] pinctrl: samsung: fix suspend/resume functionality From: Doug Anderson To: Tomasz Figa Cc: Kukjin Kim , Olof Johansson , Stephen Warren , Thomas Abraham , Linus Walleij , Prathyush K , linux-samsung-soc , "linux-kernel@vger.kernel.org" Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2488 Lines: 56 Tomasz, On Thu, May 16, 2013 at 2:27 PM, Tomasz Figa wrote: > OK. I will be fine to go with your patches, after addressing the comments. > In the end it's good that you posted them, as reviewing them allowed me to > find even better ways of doing some things than I had in mine ;) . Yes. I often find that the best way to review code is to think about how I would implement it myself. Certainly I think we've ended up with something better / less buggy this way. ;) > How all of this works is basically a good question. I couldn't find any > mention about pins switching from power down to normal mode in the > documentation, but maybe there is a small side note somewhere, which I > could miss. > > On S3C6410, for example, there are two modes. State is switched to power > down mode automatically, but can be switched out either automatically on > wake-up (exact timing is unknown to me) or by clearing a special bit, > depending on value of other special bit. > > IMHO this is rather important, so we should find out how it work on other > SoCs and make the code account for it. Agreed that it's important. ...but it's also good not to have tons of complexity when it's not needed. It sounds like S3C6410 could be handled OK by just using the special bits and waiting to take things out of power down mode. ...thinking about it, all SoCs that have power down modes (which you _must_ have if your pinctrl state is lost across a low power) would be slightly broken if they didn't have a bit to switch out of power down mode. Otherwise you're asking for at least some type of glitch because you'll end up in the default state of pins for a little while during resume. That's not to say that there aren't broken SoCs out there and it's entirely possible that people even designed systems around them (knowing that the default state of each pin after wakeup is not harmful to whatever is connected to that pin). If there are any cases like this then they would need the special code like my V1 patch had. Do you know of any SoCs like this that we need to support on kernel 3.10 and higher? I'm planning on going back to the "simpler" code for my next patchset unless I can find a problem with it. -Doug -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/