lkml.org 
[lkml]   [2018]   [Mar]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: [PATCH 3/3] iio: adc: meson-axg: add saradc driver
    On Mon, Mar 26, 2018 at 10:46 AM, Yixun Lan <yixun.lan@amlogic.com> wrote:
    > From: Xingyu Chen <xingyu.chen@amlogic.com>
    >
    > Add the SAR ADC driver for the Amlogic Meson-AXG SoC.
    >
    > Signed-off-by: Xingyu Chen <xingyu.chen@amlogic.com>
    I suggest changing the subject of this patch to:
    iio: adc: meson-saradc: add support for Meson AXG

    (because "iio: adc: meson-axg" does not point to the "meson-saradc"
    driver and no new driver is added with this patch)

    with that:
    Acked-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>

    > ---
    > drivers/iio/adc/meson_saradc.c | 8 ++++++++
    > 1 file changed, 8 insertions(+)
    >
    > diff --git a/drivers/iio/adc/meson_saradc.c b/drivers/iio/adc/meson_saradc.c
    > index 799ed929ab99..a5d481a2b4ef 100644
    > --- a/drivers/iio/adc/meson_saradc.c
    > +++ b/drivers/iio/adc/meson_saradc.c
    > @@ -935,6 +935,11 @@ static const struct meson_sar_adc_data meson_sar_adc_gxm_data = {
    > .name = "meson-gxm-saradc",
    > };
    >
    > +static const struct meson_sar_adc_data meson_sar_adc_axg_data = {
    > + .param = &meson_sar_adc_gxl_param,
    > + .name = "meson-axg-saradc",
    > +};
    > +
    > static const struct of_device_id meson_sar_adc_of_match[] = {
    > {
    > .compatible = "amlogic,meson8-saradc",
    > @@ -953,6 +958,9 @@ static const struct of_device_id meson_sar_adc_of_match[] = {
    > }, {
    > .compatible = "amlogic,meson-gxm-saradc",
    > .data = &meson_sar_adc_gxm_data,
    > + }, {
    > + .compatible = "amlogic,meson-axg-saradc",
    > + .data = &meson_sar_adc_axg_data,
    > },
    > {},
    > };
    > --
    > 2.15.1
    >

    \
     
     \ /
      Last update: 2018-03-30 13:10    [W:2.454 / U:0.160 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site