Re: [PATCH v17 0/5] ZII RAVE platform driver

Andrey Smirnov
2018-01-02T18:39:34Z

On Tue, Jan 2, 2018 at 7:17 AM, Lee Jones <lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> wrote:
> On Wed, 20 Dec 2017, Andrey Smirnov wrote:
>
>> Everyone:
>>
>> This patch series is v17 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.
>>
>> NOTE:
>>
>> * This driver dependends on crc_ccitt_false(), added by
>> 2da9378d531f8cc6670c7497f20d936b706ab80b in 'linux-next', the patch
>> was pulled in by Andrew Morton and is currently avaiting users, so
>> this series might have to go in through Andrew's tree
>
> Hmm... well that's annoying! I just attempted to merge this set, but
> early build tests fail due to a dependency already merged into -next.
>
> ../drivers/mfd/rave-sp.c:227:25: error:
> implicit declaration of function ‘crc_ccitt_false’
> [-Werror=implicit-function-declaration]
>
> We need to figure out if either of the following are true:
>
> - Patch [0] can be dropped from Andrew's tree
> - ... and I can take it via the MFD tree instead

FWIW, it seems to me that the path above should be doable (and might
be simpler?). Let me know if any action needs to be taken on my part.

Thanks,
Andrey Smirnov