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


Groups > linux.kernel > #1174437

Re: [PATCH v5 1/7] extcon: arizona: Simplify pdata symantics for micd_dbtime

From Charles Keepax <ckeepax@opensource.wolfsonmicro.com>
Newsgroups linux.kernel
Subject Re: [PATCH v5 1/7] extcon: arizona: Simplify pdata symantics for micd_dbtime
Date 2015-06-30 13:00 +0200
Message-ID <pH1qG-80t-7@gated-at.bofh.it> (permalink)
References <pD7uG-4pU-19@gated-at.bofh.it> <pD7uH-4pU-31@gated-at.bofh.it> <pDOgh-7u6-19@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Sun, Jun 21, 2015 at 11:17:56PM +0900, Chanwoo Choi wrote:
> Hi Charles,
> 
> Looks good go me. But you need the review from MFD maintainer
> before appling it.
> 
> Thanks,
> Chanwoo Choi
> 
> On Sat, Jun 20, 2015 at 1:23 AM, Charles Keepax
> <ckeepax@opensource.wolfsonmicro.com> wrote:
> > Currently micd_dbtime can be set to 0 for default, 1 for 4 measurements,
> > or a greater than 16-bit value for 2 measurements. This patch simplifies
> > the symantics to 2 for 2 measurements, 4 for 4 measurements and any
> > other value for the default.
> >
> > I am not super keen on changing the symantics of the pdata at this stage
> > in the drivers life, but acceptance of the DT binding for this field has
> > been made conditional on this happening.
> >
> > Signed-off-by: Charles Keepax <ckeepax@opensource.wolfsonmicro.com>
> > ---
> > diff --git a/include/linux/mfd/arizona/pdata.h b/include/linux/mfd/arizona/pdata.h
> > index aa5c48b..bd68355 100644
> > --- a/include/linux/mfd/arizona/pdata.h
> > +++ b/include/linux/mfd/arizona/pdata.h
> > @@ -53,6 +53,9 @@
> >
> >  #define ARIZONA_MAX_PDM_SPK 2
> >
> > +#define ARIZONA_MICD_DBTIME_TWO_READINGS 2
> > +#define ARIZONA_MICD_DBTIME_FOUR_READINGS 4
> > +

I am gonna respin and move these into the extcon driver itself,
should allow us to get the patch merged and their use in the
pdata header is probably minimal.

Thanks,
Charles
--
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 | Find similar | Unroll thread


Thread

Re: [PATCH v5 1/7] extcon: arizona: Simplify pdata symantics for  micd_dbtime Charles Keepax <ckeepax@opensource.wolfsonmicro.com> - 2015-06-30 13:00 +0200

csiph-web