From: Simon Horman <horms@verge.net.au> To: Bastian Hecht <hechtb-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Cc: linux-sh-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Magnus Damm <magnus.damm-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>, Bastian Hecht <hechtb+renesas-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Subject: Re: [PATCH] i2c: sh_mobile: Don't start transfers when suspending Date: Sat, 16 Feb 2013 02:26:41 +0000 [thread overview] Message-ID: <20130216022641.GD11628@verge.net.au> (raw) In-Reply-To: <1360862257-10807-1-git-send-email-hechtb+renesas-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> On Thu, Feb 14, 2013 at 11:17:37AM -0600, Bastian Hecht wrote: > Currently we provoke i2c timeouts by allowing transfers when > suspending. This patch adds awareness to suspending and fixes these > issues. > The patch follows the style used in i2c-tegra.c - thanks. Hi, at this stage I do not expect this code to go through the renesas tree. However, in order to provide a basis for work on renesas SoCs I have added this patch to the topic/i2c topic branch in the reneas tree on kernel.org and merged it into topic/all+next. In other words, I am not picking this patch up to merge it or add it to linux-next, rather I am storing it for reference. > > Signed-off-by: Bastian Hecht <hechtb+renesas@gmail.com> > --- > drivers/i2c/busses/i2c-sh_mobile.c | 28 ++++++++++++++++++++++++++++ > 1 file changed, 28 insertions(+) > > diff --git a/drivers/i2c/busses/i2c-sh_mobile.c b/drivers/i2c/busses/i2c-sh_mobile.c > index b6e7a83..da28164 100644 > --- a/drivers/i2c/busses/i2c-sh_mobile.c > +++ b/drivers/i2c/busses/i2c-sh_mobile.c > @@ -132,6 +132,7 @@ struct sh_mobile_i2c_data { > struct i2c_msg *msg; > int pos; > int sr; > + bool is_suspended; > }; > > #define IIC_FLAG_HAS_ICIC67 (1 << 0) > @@ -511,6 +512,9 @@ static int sh_mobile_i2c_xfer(struct i2c_adapter *adapter, > u_int8_t val; > int i, k, retry_count; > > + if (pd->is_suspended) > + return -EBUSY; > + > activate_ch(pd); > > /* Process all messages */ > @@ -753,9 +757,33 @@ static int sh_mobile_i2c_runtime_nop(struct device *dev) > return 0; > } > > +static int sh_mobile_i2c_suspend(struct device *dev) > +{ > + struct sh_mobile_i2c_data *pd = dev_get_drvdata(dev); > + > + i2c_lock_adapter(&pd->adap); > + pd->is_suspended = true; > + i2c_unlock_adapter(&pd->adap); > + > + return 0; > +} > + > +static int sh_mobile_i2c_resume(struct device *dev) > +{ > + struct sh_mobile_i2c_data *pd = dev_get_drvdata(dev); > + > + i2c_lock_adapter(&pd->adap); > + pd->is_suspended = false; > + i2c_unlock_adapter(&pd->adap); > + > + return 0; > +} > + > static const struct dev_pm_ops sh_mobile_i2c_dev_pm_ops = { > .runtime_suspend = sh_mobile_i2c_runtime_nop, > .runtime_resume = sh_mobile_i2c_runtime_nop, > + .suspend = sh_mobile_i2c_suspend, > + .resume = sh_mobile_i2c_resume, > }; > > static const struct of_device_id sh_mobile_i2c_dt_ids[] = { > -- > 1.7.9.5 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-sh" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html >
WARNING: multiple messages have this Message-ID (diff)
From: Simon Horman <horms-/R6kz+dDXgpPR4JQBCEnsQ@public.gmane.org> To: Bastian Hecht <hechtb-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Cc: linux-sh-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Magnus Damm <magnus.damm-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>, Bastian Hecht <hechtb+renesas-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Subject: Re: [PATCH] i2c: sh_mobile: Don't start transfers when suspending Date: Sat, 16 Feb 2013 11:26:41 +0900 [thread overview] Message-ID: <20130216022641.GD11628@verge.net.au> (raw) In-Reply-To: <1360862257-10807-1-git-send-email-hechtb+renesas-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> On Thu, Feb 14, 2013 at 11:17:37AM -0600, Bastian Hecht wrote: > Currently we provoke i2c timeouts by allowing transfers when > suspending. This patch adds awareness to suspending and fixes these > issues. > The patch follows the style used in i2c-tegra.c - thanks. Hi, at this stage I do not expect this code to go through the renesas tree. However, in order to provide a basis for work on renesas SoCs I have added this patch to the topic/i2c topic branch in the reneas tree on kernel.org and merged it into topic/all+next. In other words, I am not picking this patch up to merge it or add it to linux-next, rather I am storing it for reference. > > Signed-off-by: Bastian Hecht <hechtb+renesas-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> > --- > drivers/i2c/busses/i2c-sh_mobile.c | 28 ++++++++++++++++++++++++++++ > 1 file changed, 28 insertions(+) > > diff --git a/drivers/i2c/busses/i2c-sh_mobile.c b/drivers/i2c/busses/i2c-sh_mobile.c > index b6e7a83..da28164 100644 > --- a/drivers/i2c/busses/i2c-sh_mobile.c > +++ b/drivers/i2c/busses/i2c-sh_mobile.c > @@ -132,6 +132,7 @@ struct sh_mobile_i2c_data { > struct i2c_msg *msg; > int pos; > int sr; > + bool is_suspended; > }; > > #define IIC_FLAG_HAS_ICIC67 (1 << 0) > @@ -511,6 +512,9 @@ static int sh_mobile_i2c_xfer(struct i2c_adapter *adapter, > u_int8_t val; > int i, k, retry_count; > > + if (pd->is_suspended) > + return -EBUSY; > + > activate_ch(pd); > > /* Process all messages */ > @@ -753,9 +757,33 @@ static int sh_mobile_i2c_runtime_nop(struct device *dev) > return 0; > } > > +static int sh_mobile_i2c_suspend(struct device *dev) > +{ > + struct sh_mobile_i2c_data *pd = dev_get_drvdata(dev); > + > + i2c_lock_adapter(&pd->adap); > + pd->is_suspended = true; > + i2c_unlock_adapter(&pd->adap); > + > + return 0; > +} > + > +static int sh_mobile_i2c_resume(struct device *dev) > +{ > + struct sh_mobile_i2c_data *pd = dev_get_drvdata(dev); > + > + i2c_lock_adapter(&pd->adap); > + pd->is_suspended = false; > + i2c_unlock_adapter(&pd->adap); > + > + return 0; > +} > + > static const struct dev_pm_ops sh_mobile_i2c_dev_pm_ops = { > .runtime_suspend = sh_mobile_i2c_runtime_nop, > .runtime_resume = sh_mobile_i2c_runtime_nop, > + .suspend = sh_mobile_i2c_suspend, > + .resume = sh_mobile_i2c_resume, > }; > > static const struct of_device_id sh_mobile_i2c_dt_ids[] = { > -- > 1.7.9.5 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-sh" in > the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > More majordomo info at http://vger.kernel.org/majordomo-info.html >
next prev parent reply other threads:[~2013-02-16 2:26 UTC|newest] Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top 2013-02-14 16:18 [PATCH] i2c: sh_mobile: Don't start transfers when suspending Bastian Hecht 2013-02-14 17:17 ` Bastian Hecht [not found] ` <1360862257-10807-1-git-send-email-hechtb+renesas-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> 2013-02-16 2:26 ` Simon Horman [this message] 2013-02-16 2:26 ` Simon Horman 2013-02-16 3:54 ` Magnus Damm 2013-02-16 3:54 ` Magnus Damm 2013-02-18 16:03 ` Bastian Hecht 2013-02-18 16:03 ` Bastian Hecht [not found] ` <CABYn4szejurs2gBVOx_+geOxw13r9KvoShfRiM7OOF5wVvDPLA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2013-02-21 9:14 ` Wolfram Sang 2013-02-21 9:14 ` Wolfram Sang 2013-02-21 14:54 ` Bastian Hecht 2013-02-21 14:54 ` Bastian Hecht [not found] ` <CABYn4swuDgTvvyigjnabcg46rcze-DnGhv2Rh7WEFdwkXcHR4Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2013-02-21 14:59 ` Wolfram Sang 2013-02-21 14:59 ` Wolfram Sang 2014-05-21 11:00 ` Wolfram Sang 2014-05-21 11:00 ` Wolfram Sang 2014-06-24 8:14 ` Bastian Hecht 2014-06-24 8:14 ` Bastian Hecht
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20130216022641.GD11628@verge.net.au \ --to=horms@verge.net.au \ --cc=hechtb+renesas-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \ --cc=hechtb-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \ --cc=linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=linux-sh-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=magnus.damm-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.