Tomasz, On Thu, May 16, 2013 at 2:27 PM, Tomasz Figa <tomasz.figa@xxxxxxxxx> 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-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
 |