On Tue, 2020-07-28 at 22:14 +0000, Winkler, Tomas wrote:
> > -----Original Message-----
> > From: Gustavo A. R. Silva <gust...@embeddedor.com>
[]
> > I'm using this command:
> >
> > $ scripts/get_maintainer.pl --nokeywords --nogit --nogit-fallback
> >
> > and this is the output for both files:
> >
> > $ scripts/get_maintainer.pl --nokeywords --nogit --nogit-fallback
> > drivers/misc/mei/hdcp/mei_hdcp.c Arnd Bergmann <a...@arndb.de>
> > (supporter:CHAR and MISC DRIVERS) Greg Kroah-Hartman
> > <gre...@linuxfoundation.org> (supporter:CHAR and MISC DRIVERS) linux-
> > ker...@vger.kernel.org (open list) $ scripts/get_maintainer.pl --nokeywords
> > --
> > nogit --nogit-fallback drivers/misc/mei/hdcp/mei_hdcp.h Arnd Bergmann
> > <a...@arndb.de> (supporter:CHAR and MISC DRIVERS) Greg Kroah-Hartman
> > <gre...@linuxfoundation.org> (supporter:CHAR and MISC DRIVERS) linux-
> > ker...@vger.kernel.org (open list)
> > For some reason you don't appear on the list above. Do you see anything
> > wrong with the command I'm running to get the maintainers and lists?
>
> Not sure, it should be caught by drivers/misc/mei/* ?
No, this pattern matches _only_ files in that
particular directory and not any files in any
subdirectory.
> Maybe it is not recursive?
It depends on the pattern.
Patterns with trailing slashes match subdirectories too.
from MAINTAINERS:
F: *Files* and directories wildcard patterns.
A trailing slash includes all files and subdirectory files.
F: drivers/net/ all files in and below drivers/net
F: drivers/net/* all files in drivers/net, but not below
F: */net/* all files in "any top level directory"/net
One pattern per line. Multiple F: lines acceptable.
> Need to check the script,
> frankly I usually check this manually. prob
>
> INTEL MANAGEMENT ENGINE (mei)
> M: Tomas Winkler <tomas.wink...@intel.com>
> L: linux-kernel@vger.kernel.org
> S: Supported
> F: Documentation/driver-api/mei/*
> F: drivers/misc/mei/*
You probably want:
F: drivers/misc/mei/
> F: drivers/watchdog/mei_wdt.c
> F: include/linux/mei_cl_bus.h
> F: include/uapi/linux/mei.h
> F: samples/mei/*
>
>
> > Thanks
> > --
> > Gustavo
> >
> > > > > I really do not appreciate that the code is bypassing driver
> > > > > maintaner review, I think this is a minimum we can ask for, this is
> > > > > not for a first time.
> > > > >
> > > > > This reverts commit c56967d674e361ebe716e66992e3c5332b25ac1f.
> > > > >
> > > > > Cc: Gustavo A. R. Silva <gustavo...@kernel.org>
> > > > > Signed-off-by: Tomas Winkler <tomas.wink...@intel.com>
> > > > > ---
> > > > > drivers/misc/mei/hdcp/mei_hdcp.c | 2 +-
> > > > > drivers/misc/mei/hdcp/mei_hdcp.h | 2 +-
> > > > > 2 files changed, 2 insertions(+), 2 deletions(-)
> > > > >
> > > > > diff --git a/drivers/misc/mei/hdcp/mei_hdcp.c
> > > > > b/drivers/misc/mei/hdcp/mei_hdcp.c
> > > > > index d1d3e025ca0e..e6c3dc595617 100644
> > > > > --- a/drivers/misc/mei/hdcp/mei_hdcp.c
> > > > > +++ b/drivers/misc/mei/hdcp/mei_hdcp.c
> > > > > @@ -572,7 +572,7 @@ static int mei_hdcp_verify_mprime(struct device
> > > > *dev,
> > > > > HDCP_2_2_MPRIME_LEN);
> > > > > drm_hdcp_cpu_to_be24(verify_mprime_in.seq_num_m, data-
> > seq_num_m);
> > > > > memcpy(verify_mprime_in.streams, data->streams,
> > > > > - array_size(data->k, sizeof(*data->streams)));
> > > > > + (data->k * sizeof(struct hdcp2_streamid_type)));
> > > > >
> > > > > verify_mprime_in.k = cpu_to_be16(data->k);
> > > > >
> > > > > diff --git a/drivers/misc/mei/hdcp/mei_hdcp.h
> > > > > b/drivers/misc/mei/hdcp/mei_hdcp.h
> > > > > index 834757f5e072..18ffc773fa18 100644
> > > > > --- a/drivers/misc/mei/hdcp/mei_hdcp.h
> > > > > +++ b/drivers/misc/mei/hdcp/mei_hdcp.h
> > > > > @@ -358,7 +358,7 @@ struct wired_cmd_repeater_auth_stream_req_in
> > {
> > > > > u8
> > > > seq_num_m[HDCP_2_2_SEQ_NUM_LEN];
> > > > > u8 m_prime[HDCP_2_2_MPRIME_LEN];
> > > > > __be16 k;
> > > > > - struct hdcp2_streamid_type streams[];
> > > > > + struct hdcp2_streamid_type streams[1];
> > > > > } __packed;
> > > > >
> > > > > struct wired_cmd_repeater_auth_stream_req_out {
> > > > >
