> 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
