On Tue, 16 Aug 2022, "Murthy, Arun R" <[email protected]> wrote: >> -----Original Message----- >> From: Nikula, Jani <[email protected]> >> Sent: Friday, August 12, 2022 12:33 PM >> To: Murthy, Arun R <[email protected]>; intel- >> [email protected] >> Cc: De Marchi, Lucas <[email protected]> >> Subject: RE: [Intel-gfx] [PATCH 12/39] drm/i915: move and split audio under >> display.audio and display.funcs >> >> On Fri, 12 Aug 2022, "Murthy, Arun R" <[email protected]> wrote: >> >> -----Original Message----- >> >> From: Intel-gfx <[email protected]> On Behalf >> >> Of Jani Nikula >> >> Sent: Thursday, August 11, 2022 8:37 PM >> >> To: [email protected] >> >> Cc: Nikula, Jani <[email protected]>; De Marchi, Lucas >> >> <[email protected]> >> >> Subject: [Intel-gfx] [PATCH 12/39] drm/i915: move and split audio >> >> under display.audio and display.funcs >> >> >> >> Move display related members under drm_i915_private display sub- >> struct. >> >> >> >> Split audio funcs to display.funcs to follow the same pattern as all >> >> the other display functions. >> >> >> > Audio is a feature as such so wouldn't intel_audio struct stand parallel to >> intel_display? >> >> For i915, audio doesn't exist other than as a display feature. Display is a >> higher level split here, parallel to gt/gem. >> > Will leave it to you, since you have started this huger series containing the > cleanup and understand you also have some more cleanup in queue. > My small suggestion is not to mix audio with display and let this audi reside > in i915_priv and let it leave in parallel similar to gt/gem.
No, I'm going to stick with this hierarchy. Audio is part of display here. BR, Jani. > > Thanks and Regards, > Arun R Murthy > -------------------- -- Jani Nikula, Intel Open Source Graphics Center
