* [alsa-devel] [PATCH 1/2] mfd: arizona: Add support for INn_MODE register control
@ 2014-08-09 8:31 ` Inha Song
0 siblings, 0 replies; 8+ messages in thread
From: Inha Song @ 2014-08-09 8:31 UTC (permalink / raw)
To: broonie
Cc: alsa-devel, patches, linux-kernel, lee.jones, ckeepax, lgirdwood,
sameo, devicetree, galak, ijc+devicetree, mark.rutland,
pawel.moll, robh+dt, cw00.choi
Some boards need to set the INn_MODE[1:0] register to change
the input signal patch. This wlf,inmode property is optional.
If present values must be specified by the number of
ARIZONA_MAX_INPUT.
Example:
- wlf,inmode = <2 0 2 0>; /* IN1, IN3 use DMIC */
Signed-off-by: Inha Song <ideal.song@samsung.com>
---
drivers/mfd/arizona-core.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/drivers/mfd/arizona-core.c b/drivers/mfd/arizona-core.c
index 10a0cb9..4b2eb73 100644
--- a/drivers/mfd/arizona-core.c
+++ b/drivers/mfd/arizona-core.c
@@ -560,6 +560,10 @@ static int arizona_of_get_core_pdata(struct arizona *arizona)
ret);
}
+ of_property_read_u32_array(arizona->dev->of_node,
+ "wlf,inmode",
+ arizona->pdata.inmode,
+ ARRAY_SIZE(arizona->pdata.inmode));
return 0;
}
--
2.0.0.390.gcb682f8
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH 1/2] mfd: arizona: Add support for INn_MODE register control
@ 2014-08-09 8:31 ` Inha Song
0 siblings, 0 replies; 8+ messages in thread
From: Inha Song @ 2014-08-09 8:31 UTC (permalink / raw)
To: broonie
Cc: mark.rutland, devicetree, alsa-devel, sameo, pawel.moll,
ijc+devicetree, patches, linux-kernel, lgirdwood, cw00.choi,
robh+dt, galak, ckeepax, lee.jones
Some boards need to set the INn_MODE[1:0] register to change
the input signal patch. This wlf,inmode property is optional.
If present values must be specified by the number of
ARIZONA_MAX_INPUT.
Example:
- wlf,inmode = <2 0 2 0>; /* IN1, IN3 use DMIC */
Signed-off-by: Inha Song <ideal.song@samsung.com>
---
drivers/mfd/arizona-core.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/drivers/mfd/arizona-core.c b/drivers/mfd/arizona-core.c
index 10a0cb9..4b2eb73 100644
--- a/drivers/mfd/arizona-core.c
+++ b/drivers/mfd/arizona-core.c
@@ -560,6 +560,10 @@ static int arizona_of_get_core_pdata(struct arizona *arizona)
ret);
}
+ of_property_read_u32_array(arizona->dev->of_node,
+ "wlf,inmode",
+ arizona->pdata.inmode,
+ ARRAY_SIZE(arizona->pdata.inmode));
return 0;
}
--
2.0.0.390.gcb682f8
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [alsa-devel] [PATCH 1/2] mfd: arizona: Add support for INn_MODE register control
2014-08-09 8:31 ` Inha Song
@ 2014-08-12 8:28 ` Charles Keepax
-1 siblings, 0 replies; 8+ messages in thread
From: Charles Keepax @ 2014-08-12 8:28 UTC (permalink / raw)
To: Inha Song
Cc: broonie, mark.rutland, devicetree, alsa-devel, sameo, pawel.moll,
ijc+devicetree, patches, linux-kernel, lgirdwood, cw00.choi,
robh+dt, galak, lee.jones
On Sat, Aug 09, 2014 at 05:31:38PM +0900, Inha Song wrote:
> Some boards need to set the INn_MODE[1:0] register to change
> the input signal patch. This wlf,inmode property is optional.
> If present values must be specified by the number of
> ARIZONA_MAX_INPUT.
>
> Example:
> - wlf,inmode = <2 0 2 0>; /* IN1, IN3 use DMIC */
>
> Signed-off-by: Inha Song <ideal.song@samsung.com>
> ---
> drivers/mfd/arizona-core.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/drivers/mfd/arizona-core.c b/drivers/mfd/arizona-core.c
> index 10a0cb9..4b2eb73 100644
> --- a/drivers/mfd/arizona-core.c
> +++ b/drivers/mfd/arizona-core.c
> @@ -560,6 +560,10 @@ static int arizona_of_get_core_pdata(struct arizona *arizona)
> ret);
> }
>
> + of_property_read_u32_array(arizona->dev->of_node,
> + "wlf,inmode",
> + arizona->pdata.inmode,
> + ARRAY_SIZE(arizona->pdata.inmode));
Might be better to allow the device tree to be shorter than the
array in pdata, should this get expanded on future devices.
Thanks,
Charles
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [alsa-devel] [PATCH 1/2] mfd: arizona: Add support for INn_MODE register control
@ 2014-08-12 8:28 ` Charles Keepax
0 siblings, 0 replies; 8+ messages in thread
From: Charles Keepax @ 2014-08-12 8:28 UTC (permalink / raw)
To: Inha Song
Cc: broonie-DgEjT+Ai2ygdnm+yROfE0A, mark.rutland-5wv7dgnIgG8,
devicetree-u79uwXL29TY76Z2rM5mHXA,
alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw, sameo-VuQAYsv1563Yd54FQh9/CA,
pawel.moll-5wv7dgnIgG8, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
patches-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
lgirdwood-Re5JQEeQqe8AvxtiuMwx3w,
cw00.choi-Sze3O3UU22JBDgjK7y7TUQ, robh+dt-DgEjT+Ai2ygdnm+yROfE0A,
galak-sgV2jX0FEOL9JmXXK+q4OQ, lee.jones-QSEj5FYQhm4dnm+yROfE0A
On Sat, Aug 09, 2014 at 05:31:38PM +0900, Inha Song wrote:
> Some boards need to set the INn_MODE[1:0] register to change
> the input signal patch. This wlf,inmode property is optional.
> If present values must be specified by the number of
> ARIZONA_MAX_INPUT.
>
> Example:
> - wlf,inmode = <2 0 2 0>; /* IN1, IN3 use DMIC */
>
> Signed-off-by: Inha Song <ideal.song-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
> ---
> drivers/mfd/arizona-core.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/drivers/mfd/arizona-core.c b/drivers/mfd/arizona-core.c
> index 10a0cb9..4b2eb73 100644
> --- a/drivers/mfd/arizona-core.c
> +++ b/drivers/mfd/arizona-core.c
> @@ -560,6 +560,10 @@ static int arizona_of_get_core_pdata(struct arizona *arizona)
> ret);
> }
>
> + of_property_read_u32_array(arizona->dev->of_node,
> + "wlf,inmode",
> + arizona->pdata.inmode,
> + ARRAY_SIZE(arizona->pdata.inmode));
Might be better to allow the device tree to be shorter than the
array in pdata, should this get expanded on future devices.
Thanks,
Charles
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [alsa-devel] [PATCH 1/2] mfd: arizona: Add support for INn_MODE register control
@ 2014-08-13 0:26 ` Inha Song
0 siblings, 0 replies; 8+ messages in thread
From: Inha Song @ 2014-08-13 0:26 UTC (permalink / raw)
To: Charles Keepax
Cc: broonie, mark.rutland, devicetree, alsa-devel, sameo, pawel.moll,
ijc+devicetree, patches, linux-kernel, lgirdwood, cw00.choi,
robh+dt, galak, lee.jones
Hi, Charles.
Thanks for your review.
On Tue, 12 Aug 2014 09:28:11 +0100
Charles Keepax <ckeepax@opensource.wolfsonmicro.com> wrote:
> On Sat, Aug 09, 2014 at 05:31:38PM +0900, Inha Song wrote:
> > Some boards need to set the INn_MODE[1:0] register to change
> > the input signal patch. This wlf,inmode property is optional.
> > If present values must be specified by the number of
> > ARIZONA_MAX_INPUT.
> >
> > Example:
> > - wlf,inmode = <2 0 2 0>; /* IN1, IN3 use DMIC */
> >
> > Signed-off-by: Inha Song <ideal.song@samsung.com>
> > ---
> > drivers/mfd/arizona-core.c | 4 ++++
> > 1 file changed, 4 insertions(+)
> >
> > diff --git a/drivers/mfd/arizona-core.c b/drivers/mfd/arizona-core.c
> > index 10a0cb9..4b2eb73 100644
> > --- a/drivers/mfd/arizona-core.c
> > +++ b/drivers/mfd/arizona-core.c
> > @@ -560,6 +560,10 @@ static int arizona_of_get_core_pdata(struct arizona *arizona)
> > ret);
> > }
> >
> > + of_property_read_u32_array(arizona->dev->of_node,
> > + "wlf,inmode",
> > + arizona->pdata.inmode,
> > + ARRAY_SIZE(arizona->pdata.inmode));
>
> Might be better to allow the device tree to be shorter than the
> array in pdata, should this get expanded on future devices.
>
OK, I will fix in v2 to use "of_property_count_u32_elems" and "of_property_read_u32_index".
Best regards,
Inha Song.
> Thanks,
> Charles
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [alsa-devel] [PATCH 1/2] mfd: arizona: Add support for INn_MODE register control
@ 2014-08-13 0:26 ` Inha Song
0 siblings, 0 replies; 8+ messages in thread
From: Inha Song @ 2014-08-13 0:26 UTC (permalink / raw)
To: Charles Keepax
Cc: broonie-DgEjT+Ai2ygdnm+yROfE0A, mark.rutland-5wv7dgnIgG8,
devicetree-u79uwXL29TY76Z2rM5mHXA,
alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw, sameo-VuQAYsv1563Yd54FQh9/CA,
pawel.moll-5wv7dgnIgG8, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
patches-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
lgirdwood-Re5JQEeQqe8AvxtiuMwx3w,
cw00.choi-Sze3O3UU22JBDgjK7y7TUQ, robh+dt-DgEjT+Ai2ygdnm+yROfE0A,
galak-sgV2jX0FEOL9JmXXK+q4OQ, lee.jones-QSEj5FYQhm4dnm+yROfE0A
Hi, Charles.
Thanks for your review.
On Tue, 12 Aug 2014 09:28:11 +0100
Charles Keepax <ckeepax-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E@public.gmane.org> wrote:
> On Sat, Aug 09, 2014 at 05:31:38PM +0900, Inha Song wrote:
> > Some boards need to set the INn_MODE[1:0] register to change
> > the input signal patch. This wlf,inmode property is optional.
> > If present values must be specified by the number of
> > ARIZONA_MAX_INPUT.
> >
> > Example:
> > - wlf,inmode = <2 0 2 0>; /* IN1, IN3 use DMIC */
> >
> > Signed-off-by: Inha Song <ideal.song-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
> > ---
> > drivers/mfd/arizona-core.c | 4 ++++
> > 1 file changed, 4 insertions(+)
> >
> > diff --git a/drivers/mfd/arizona-core.c b/drivers/mfd/arizona-core.c
> > index 10a0cb9..4b2eb73 100644
> > --- a/drivers/mfd/arizona-core.c
> > +++ b/drivers/mfd/arizona-core.c
> > @@ -560,6 +560,10 @@ static int arizona_of_get_core_pdata(struct arizona *arizona)
> > ret);
> > }
> >
> > + of_property_read_u32_array(arizona->dev->of_node,
> > + "wlf,inmode",
> > + arizona->pdata.inmode,
> > + ARRAY_SIZE(arizona->pdata.inmode));
>
> Might be better to allow the device tree to be shorter than the
> array in pdata, should this get expanded on future devices.
>
OK, I will fix in v2 to use "of_property_count_u32_elems" and "of_property_read_u32_index".
Best regards,
Inha Song.
> Thanks,
> Charles
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 8+ messages in thread
* [alsa-devel] [PATCH v2 1/2] mfd: arizona: Add support for INn_Mode register control
2014-08-09 8:31 ` Inha Song
@ 2014-08-13 11:49 ` Inha Song
-1 siblings, 0 replies; 8+ messages in thread
From: Inha Song @ 2014-08-13 11:49 UTC (permalink / raw)
To: Inha Song
Cc: broonie, alsa-devel, patches, linux-kernel, lee.jones, ckeepax,
lgirdwood, sameo, devicetree, galak, ijc+devicetree,
mark.rutland, pawel.moll, robh+dt, cw00.choi
Some boards need to set the INn_MODE[1:0] register to change
the input signal patch. This wlf,inmode property is optional.
If present values must be specified by the number of
ARIZONA_MAX_INPUT.
Example:
- wlf,inmode = <2 0 2 0>; /* IN1, IN3 use DMIC */
Signed-off-by: Inha Song <ideal.song@samsung.com>
---
drivers/mfd/arizona-core.c | 11 ++++++++++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/drivers/mfd/arizona-core.c b/drivers/mfd/arizona-core.c
index 10a0cb9..2b68ccd 100644
--- a/drivers/mfd/arizona-core.c
+++ b/drivers/mfd/arizona-core.c
@@ -534,7 +534,7 @@ EXPORT_SYMBOL_GPL(arizona_of_get_named_gpio);
static int arizona_of_get_core_pdata(struct arizona *arizona)
{
struct arizona_pdata *pdata = &arizona->pdata;
- int ret, i;
+ int ret, i, proplen;
pdata->reset = arizona_of_get_named_gpio(arizona, "wlf,reset", true);
@@ -560,6 +560,15 @@ static int arizona_of_get_core_pdata(struct arizona *arizona)
ret);
}
+ proplen = of_property_count_u32_elems(arizona->dev->of_node,
+ "wlf,inmode");
+ if (proplen > 0 && proplen <= ARRAY_SIZE(arizona->pdata.inmode)) {
+ for (i = 0; i < proplen; i++)
+ of_property_read_u32_index(arizona->dev->of_node,
+ "wlf,inmode", i,
+ &arizona->pdata.inmode[i]);
+ }
+
return 0;
}
--
2.0.0.390.gcb682f8
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [alsa-devel] [PATCH v2 1/2] mfd: arizona: Add support for INn_Mode register control
@ 2014-08-13 11:49 ` Inha Song
0 siblings, 0 replies; 8+ messages in thread
From: Inha Song @ 2014-08-13 11:49 UTC (permalink / raw)
To: Inha Song
Cc: broonie-DgEjT+Ai2ygdnm+yROfE0A,
alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw,
patches-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
lee.jones-QSEj5FYQhm4dnm+yROfE0A,
ckeepax-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E,
lgirdwood-Re5JQEeQqe8AvxtiuMwx3w, sameo-VuQAYsv1563Yd54FQh9/CA,
devicetree-u79uwXL29TY76Z2rM5mHXA, galak-sgV2jX0FEOL9JmXXK+q4OQ,
ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg, mark.rutland-5wv7dgnIgG8,
pawel.moll-5wv7dgnIgG8, robh+dt-DgEjT+Ai2ygdnm+yROfE0A,
cw00.choi-Sze3O3UU22JBDgjK7y7TUQ
Some boards need to set the INn_MODE[1:0] register to change
the input signal patch. This wlf,inmode property is optional.
If present values must be specified by the number of
ARIZONA_MAX_INPUT.
Example:
- wlf,inmode = <2 0 2 0>; /* IN1, IN3 use DMIC */
Signed-off-by: Inha Song <ideal.song-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
---
drivers/mfd/arizona-core.c | 11 ++++++++++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/drivers/mfd/arizona-core.c b/drivers/mfd/arizona-core.c
index 10a0cb9..2b68ccd 100644
--- a/drivers/mfd/arizona-core.c
+++ b/drivers/mfd/arizona-core.c
@@ -534,7 +534,7 @@ EXPORT_SYMBOL_GPL(arizona_of_get_named_gpio);
static int arizona_of_get_core_pdata(struct arizona *arizona)
{
struct arizona_pdata *pdata = &arizona->pdata;
- int ret, i;
+ int ret, i, proplen;
pdata->reset = arizona_of_get_named_gpio(arizona, "wlf,reset", true);
@@ -560,6 +560,15 @@ static int arizona_of_get_core_pdata(struct arizona *arizona)
ret);
}
+ proplen = of_property_count_u32_elems(arizona->dev->of_node,
+ "wlf,inmode");
+ if (proplen > 0 && proplen <= ARRAY_SIZE(arizona->pdata.inmode)) {
+ for (i = 0; i < proplen; i++)
+ of_property_read_u32_index(arizona->dev->of_node,
+ "wlf,inmode", i,
+ &arizona->pdata.inmode[i]);
+ }
+
return 0;
}
--
2.0.0.390.gcb682f8
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 8+ messages in thread
end of thread, other threads:[~2014-08-13 11:49 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-08-09 8:31 [alsa-devel] [PATCH 1/2] mfd: arizona: Add support for INn_MODE register control Inha Song
2014-08-09 8:31 ` Inha Song
2014-08-12 8:28 ` [alsa-devel] " Charles Keepax
2014-08-12 8:28 ` Charles Keepax
2014-08-13 0:26 ` Inha Song
2014-08-13 0:26 ` Inha Song
2014-08-13 11:49 ` [alsa-devel] [PATCH v2 1/2] mfd: arizona: Add support for INn_Mode " Inha Song
2014-08-13 11:49 ` Inha Song
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.