RE: [PATCH v5 01/13] ASoC: Intel: Add catpt device

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

 



> On Wed, Sep 16, 2020 at 06:24:56PM +0300, Andy Shevchenko wrote:
> > On Tue, Sep 15, 2020 at 06:29:32PM +0200, Cezary Rojewski wrote:
> > > Declare base structures, registers and device routines for the catpt
> > > solution. Catpt deprecates and is a direct replacement for
> > > sound/soc/intel/haswell. Supports Lynxpoint and Wildcat Point both.
> >
> > Few nit-picks below. Overall looks good, FWIW,
> > Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> 
> Actually hold on. See below.
> 
> > > +void catpt_sram_init(struct resource *sram, u32 start, u32 size);
> > > +void catpt_sram_free(struct resource *sram);
> > > +struct resource *
> > > +catpt_request_region(struct resource *root, resource_size_t size);
> 
> These seems dangling declarations that has to be moved to the
> corresponding
> patch. Please, revisit entire series to be sure that:
> 
> - each patch doesn't add any warnings on W=1
> - each patch doesn't have dangling stuff
> - each patch is bisectable for compilation and run-time
> 

TLDR: you want patches:
6/13 ASoC: Intel: catpt: PCM operations
5/13 ASoC: Intel: catpt: Add IPC messages
4/13 ASoC: Intel: catpt: Implement IPC protocol
3/13 ASoC: Intel: catpt: Firmware loading and context restore
2/13 ASoC: Intel: catpt: Define DSP operations
1/13 ASoC: Intel: Add catpt device

squashed. There is no other way to achieve that without combining
all the core-code together. fs and traces can be provided separately,
but not the first 6.

Czarek





[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux