Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1250132

Re: [PATCH] pinctrl: mvebu: armada-38x: add suspend/resume support

Show all headers | View raw

Hello Marcin,

On Sun, 18 Oct 2015 10:43:42 +0200, Marcin Wojtas wrote:

> Thanks for pointing this. I based on pinctrl-armada-xp.c (it needs a
> fix then, too) and it worked. I must have missed, because I got proper
> registers' number and values in suspend/resume routines. As
> pinctrl-armada-xp.c needs also a small fix and in order not to
> duplicate code, how about a following solution:
> - *mpp_saved_regs and *mpp_base become members of struct mvebu_pinctrl_soc_info

I don't like this. The mvebu_pinctrl_soc_info structure is meant to be
a read-only structure that only describes static information giving
SoC-specific details for pin-muxing. The idea is that in the event
where you had multiple pinctrl in the same system, you would still have
only one instance of mvebu_pinctrl_soc_info.

So, I'd prefer if mpp_saved_regs and mpp_base became members of a new
structure, that gets allocated at ->probe() time, on a per-instance
basis.

> - common mvebu_pinctrl_suspend/resume functions in pinctrl-mvebu.c
> (now there will be two users AXP and A38X)

Not sure how to handle that if the per-instance structure is defined on
a per-SoC basis, but I'm interested in seeing proposals.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
--
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/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] pinctrl: mvebu: armada-38x: add suspend/resume support Marcin Wojtas <mw@semihalf.com> - 2015-10-17 23:40 +0200
  Re: [PATCH] pinctrl: mvebu: armada-38x: add suspend/resume support Russell King - ARM Linux <linux@arm.linux.org.uk> - 2015-10-18 00:40 +0200
    Re: [PATCH] pinctrl: mvebu: armada-38x: add suspend/resume support Marcin Wojtas <mw@semihalf.com> - 2015-10-18 10:50 +0200
      Re: [PATCH] pinctrl: mvebu: armada-38x: add suspend/resume support Thomas Petazzoni <thomas.petazzoni@free-electrons.com> - 2015-10-18 16:10 +0200
        Re: [PATCH] pinctrl: mvebu: armada-38x: add suspend/resume support Marcin Wojtas <mw@semihalf.com> - 2015-10-19 08:10 +0200
          Re: [PATCH] pinctrl: mvebu: armada-38x: add suspend/resume support Thomas Petazzoni <thomas.petazzoni@free-electrons.com> - 2015-10-19 09:30 +0200
            Re: [PATCH] pinctrl: mvebu: armada-38x: add suspend/resume support Marcin Wojtas <mw@semihalf.com> - 2015-10-19 11:30 +0200
              Re: [PATCH] pinctrl: mvebu: armada-38x: add suspend/resume support Marcin Wojtas <mw@semihalf.com> - 2015-10-19 12:20 +0200
              Re: [PATCH] pinctrl: mvebu: armada-38x: add suspend/resume support Russell King - ARM Linux <linux@arm.linux.org.uk> - 2015-10-19 12:50 +0200

csiph-web