Groups > linux.kernel > #1698848 > unrolled thread

[PATCH v5 0/2] ZII RAVE platform driver

Started by	Andrey Smirnov <andrew.smirnov@gmail.com>
First post	2017-07-28 16:30 +0200
Last post	2017-07-29 02:10 +0200
Articles	3 — 2 participants

Back to article view | Back to linux.kernel

  [PATCH v5 0/2] ZII RAVE platform driver Andrey Smirnov <andrew.smirnov@gmail.com> - 2017-07-28 16:30 +0200
    Re: [PATCH v5 0/2] ZII RAVE platform driver Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2017-07-29 01:50 +0200
      Re: [PATCH v5 0/2] ZII RAVE platform driver Andrey Smirnov <andrew.smirnov@gmail.com> - 2017-07-29 02:10 +0200

#1698848 — [PATCH v5 0/2] ZII RAVE platform driver

From	Andrey Smirnov <andrew.smirnov@gmail.com>
Date	2017-07-28 16:30 +0200
Subject	[PATCH v5 0/2] ZII RAVE platform driver
Message-ID	<u8er7-4Rs-9@gated-at.bofh.it>

Greg,

I am not sure if you are the right person to send these to, if you are
not, please let me know who would be the appropriate recepient and sorry for the noise.

Thanks!


The orginal conver letter text follows:

Hi everyone,

This patch series is v4 of the driver for supervisory processor found
on RAVE series of devices from ZII. Supervisory processor is a PIC
microcontroller connected to various electrical subsystems on RAVE
devices whose firmware implements protocol to command/qery them.

Changes since [v4]:

    - Replaced usage of DEVICE_ATTR with DEVICE_ATTR_RW

    - Fixed a number of warnings produces by sparse tool

    - Incorporated event more feedback from Andy Shevchenko

    - Collected Reviewed-by from Andy

Changes since [v3]:

    - Re-collected lost Acked-by from Rob

    - Incorporated further feedback from Andy Shevchenko

    - Dropped useless change (stray newline) to drivers/mfd/Makefile

Changes since [v2]:

    - Fixed swapped command codes in rave_sp_common_get_boot_source()
      and rave_sp_common_set_boot_source() revealed by further testing
      of the code

    - Incorporated feedback from Andy Shevchenko

Changes since [v1]:

    - Updated wording in DT-bindings as per Rob's request.

    - Collected Rob's Acked-by for patch 2/2

NOTE:

 * The driver for "zii,rave-sp-watchdog" exists, but I haven't
   submitted it yet, becuase I wanted to make sure that API exposed by
   this MFD is acceptable and doesn't need drastic changes.

 * This driver is dependent on crc_ccitt_false() introduced in
   2da9378d531f8cc6670c7497f20d936b706ab80b in 'linux-next'

Feedback is greatly appreciated!

Thanks,
Andrey Smirnov

[v4] lkml.kernel.org/r/20170725184450.13171-1-andrew.smirnov@gmail.com
[v3] lkml.kernel.org/r/20170724150915.4824-1-andrew.smirnov@gmail.com
[v2] lkml.kernel.org/r/20170718175604.11735-1-andrew.smirnov@gmail.com
[v1] lkml.kernel.org/r/20170710170449.4544-1-andrew.smirnov@gmail.com


Andrey Smirnov (2):
  platform: Add driver for RAVE Supervisory Processor
  dt-bindings: mfd: Add bindings for ZII RAVE devices

 .../devicetree/bindings/mfd/zii,rave-sp.txt        |   39 +
 drivers/platform/Kconfig                           |    2 +
 drivers/platform/Makefile                          |    1 +
 drivers/platform/rave/Kconfig                      |   26 +
 drivers/platform/rave/Makefile                     |    1 +
 drivers/platform/rave/rave-sp.c                    | 1127 ++++++++++++++++++++
 include/linux/rave-sp.h                            |   54 +
 7 files changed, 1250 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/mfd/zii,rave-sp.txt
 create mode 100644 drivers/platform/rave/Kconfig
 create mode 100644 drivers/platform/rave/Makefile
 create mode 100644 drivers/platform/rave/rave-sp.c
 create mode 100644 include/linux/rave-sp.h

-- 
2.13.3

[toc] | [next] | [standalone]

#1699164

From	Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Date	2017-07-29 01:50 +0200
Message-ID	<u8nb3-2d9-1@gated-at.bofh.it>
In reply to	#1698848

On Fri, Jul 28, 2017 at 07:27:02AM -0700, Andrey Smirnov wrote:
> Greg,
> 
> I am not sure if you are the right person to send these to, if you are
> not, please let me know who would be the appropriate recepient and sorry for the noise.

Why not the platform driver maintainer(s)?  Doesn't
scripts/get_maintainer.pl give you an idea for who to send them to?

thanks,

greg k-h

[toc] | [prev] | [next] | [standalone]

#1699183

From	Andrey Smirnov <andrew.smirnov@gmail.com>
Date	2017-07-29 02:10 +0200
Message-ID	<u8nup-2Ax-3@gated-at.bofh.it>
In reply to	#1699164

On Fri, Jul 28, 2017 at 4:30 PM, Greg Kroah-Hartman
<gregkh@linuxfoundation.org> wrote:
> On Fri, Jul 28, 2017 at 07:27:02AM -0700, Andrey Smirnov wrote:
>> Greg,
>>
>> I am not sure if you are the right person to send these to, if you are
>> not, please let me know who would be the appropriate recepient and sorry for the noise.
>
> Why not the platform driver maintainer(s)?  Doesn't
> scripts/get_maintainer.pl give you an idea for who to send them to?

Unfortunately I wasn't able to find anyone in MAINTAINERS and running
scripts/get_maintainer.pl on the patch returns only:

scripts/get_maintainer.pl
v5-0001-platform-Add-driver-for-RAVE-Supervisory-Processo.patch
Andrey Smirnov <andrew.smirnov@gmail.com>
(commit_signer:1/1=100%,authored:1/1=100%,added_lines:2/2=100%,added_lines:1/1=100%,added_lines:26/26=100%,added_lines:1127/1127=100%,added_lines:54/54=100%)
Andy Shevchenko <andy.shevchenko@gmail.com> (commit_signer:1/1=100%)
linux-kernel@vger.kernel.org (open list)

hence me reaching out to you.

Thanks,
Andrey Smirnov

[toc] | [prev] | [standalone]

csiph-web

[PATCH v5 0/2] ZII RAVE platform driver

Contents

#1698848 — [PATCH v5 0/2] ZII RAVE platform driver

#1699164

#1699183