Re: [PATCH 5/8] fpga: dfl: add dfl bus support to MODULE_DEVICE_TABLE()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Dec 14, 2020 at 05:43:53AM -0800, Tom Rix wrote:
> 
> On 12/10/20 8:58 AM, Moritz Fischer wrote:
> > From: Xu Yilun <yilun.xu@xxxxxxxxx>
> >
> > Device Feature List (DFL) is a linked list of feature headers within the
> > device MMIO space. It is used by FPGA to enumerate multiple sub features
> > within it. Each feature can be uniquely identified by DFL type and
> > feature id, which can be read out from feature headers.
> >
> > A dfl bus helps DFL framework modularize DFL device drivers for
> > different sub features. The dfl bus matches its devices and drivers by
> > DFL type and feature id.
> >
> > This patch adds dfl bus support to MODULE_DEVICE_TABLE() by adding info
> > about struct dfl_device_id in devicetable-offsets.c and add a dfl entry
> > point in file2alias.c.
> >
> > Signed-off-by: Xu Yilun <yilun.xu@xxxxxxxxx>
> > Signed-off-by: Wu Hao <hao.wu@xxxxxxxxx>
> > Signed-off-by: Matthew Gerlach <matthew.gerlach@xxxxxxxxxxxxxxx>
> > Signed-off-by: Russ Weight <russell.h.weight@xxxxxxxxx>
> > Signed-off-by: Moritz Fischer <mdf@xxxxxxxxxx>
> > Acked-by: Wu Hao <hao.wu@xxxxxxxxx>
> > ---
> >  scripts/mod/devicetable-offsets.c |  4 ++++
> >  scripts/mod/file2alias.c          | 13 +++++++++++++
> >  2 files changed, 17 insertions(+)
> >
> > diff --git a/scripts/mod/devicetable-offsets.c b/scripts/mod/devicetable-offsets.c
> > index 27007c18e754..d8350eea6d1a 100644
> > --- a/scripts/mod/devicetable-offsets.c
> > +++ b/scripts/mod/devicetable-offsets.c
> > @@ -243,5 +243,9 @@ int main(void)
> >  	DEVID(mhi_device_id);
> >  	DEVID_FIELD(mhi_device_id, chan);
> 
> addition of auxiliary_device_id causes a trivial conflict.
> 
> Do you want me resolve and resend the set ?

Seems we should wait until 5.11-rc1, I'll rebase and resolve the
conflict then.

Thanks,
Yilun



[Index of Archives]     [LM Sensors]     [Linux Sound]     [ALSA Users]     [ALSA Devel]     [Linux Audio Users]     [Linux Media]     [Kernel]     [Gimp]     [Yosemite News]     [Linux Media]

  Powered by Linux