> -----Original Message----- > From: Krzysztof Kozlowski <[email protected]> > Sent: Friday, March 29, 2024 15:47 > To: Usyskin, Alexander <[email protected]>; Miquel Raynal > <[email protected]>; Richard Weinberger <[email protected]>; Vignesh > Raghavendra <[email protected]>; Jani Nikula <[email protected]>; > Joonas Lahtinen <[email protected]>; Vivi, Rodrigo > <[email protected]> > Cc: Lubart, Vitaly <[email protected]>; [email protected]; > intel- > [email protected] > Subject: Re: [PATCH 01/13] spi: add auxiliary device for intel dg spi > > On 28/03/2024 13:22, Alexander Usyskin wrote: > > Add auxilary bus device structure for > > Intel Discrete Graphics SPI device. > > > > Signed-off-by: Alexander Usyskin <[email protected]> > > --- > > include/linux/intel_dg_spi_aux.h | 27 +++++++++++++++++++++++++++ > > There is no user of this header. Please add it in the patch which uses it. > This header describes auxiliary device that bridges between i915 or Xe driver with spi-intel-dg driver. This header included in both drivers; this is the reason this header has its own patch.
> It will also justify why you use this or other subject prefix. Now, it > looks just wrong: it's not a patch for spi. > Can you suggest what subject prefix to use? Like: "auxiliary:"? Something else? > If it was a patch for spi, then: > > Please use scripts/get_maintainers.pl to get a list of necessary people > and lists to CC (and consider --no-git-fallback argument). It might > happen, that command when run on an older kernel, gives you outdated > entries. Therefore please be sure you base your patches on recent Linux > kernel. > Sorry for that, will run this script to pick the fresh maintainers list and push v2 with a new list. > Best regards, > Krzysztof -- Thanks, Sasha
