Re: [meta-ti] [master][PATCH v2] meta-ti-bsp: Graphics recipe overhaul

[Denys Dmytriyenko <denis@denix.org>]

On Wed, Dec 14, 2022 at 02:26:07PM -0600, Randolph Sapp via lists.yoctoproject.org wrote:
> On Wed, Dec 14 2022 at 12:47:29 -06:00:00, Andrew Davis <afd@ti.com>
> wrote:
> >On 12/9/22 4:18 PM, Sapp, Randolph wrote:
> >>On Wed, Dec 7 2022 at 12:48:00 PM -0600, Andrew Davis
> >><afd@ti.com <mailto:afd@ti.com>> wrote:
> >>>Since the UM driver now RDEPENDS on the KM, could we also drop this
> >>>last line everywhere?
> >>
> >>Nope, we still need the one preferred provider line to indicate
> >>that rogue graphics are present. Now this could be replaced by a
> >>rogue-graphics machine feature instead if that's preferable, but
> >>something will need to signal the mesa switch.
> >>
> >
> >New machine feature sounds like the better route since it changes
> >the content of a package.
> >Using PREFERRED_PROVIDER for that doesn't seem right.
> 
> Fair enough, v4 will roll back to that method.

Can you please explain why existing machine feature cannot be re-used and a 
new one is needed here? Do you plan to use both side-by-side?


> On Wed, Dec 14 2022 at 12:47:29 -06:00:00, Andrew Davis <afd@ti.com>
> wrote:
> >
> >>On Wed, Dec 7 2022 at 12:48:00 PM -0600, Andrew Davis
> >><afd@ti.com <mailto:afd@ti.com>> wrote:
> >>>Bash? Are we sure we need that?
> >>
> >>Yes. The System V init script still needs it. Now could we look
> >>at removing that init script? Sure, so long as the device probe
> >>still works. What is actually taking care of that right now?
> >>It's not udev is it?
> >>
> >
> >The module/device probe is automatic now. At least for SGX that
> >script exists only
> >to run the single command "pvrsrvctl --start --no-module" which
> >loads some microcode.
> >We can have yocto build that service for us to run the command,
> >IIRC it will make
> >it into a systemd or sysv service for us based on what kinda
> >distro we are building.
> 
> If that's true rogue can drop it entirely. Updating to do that now.
> 
> On Wed, Dec 14 2022 at 12:47:29 -06:00:00, Andrew Davis <afd@ti.com>
> wrote:
> >
> >>On Wed, Dec 7 2022 at 12:48:00 PM -0600, Andrew Davis
> >><afd@ti.com <mailto:afd@ti.com>> wrote:
> >>>"ldflags already-stripped" should be the only skips we still need.
> >>
> >>Dev-so is required because of some of the symbolic links that
> >>are shipped. I'm fine dropping arch unless that breaks
> >>something.
> >>
> >>
> >
> >Ah, think that is /usr/lib/dri/pvr_dri.so, odd name, but yeah that
> >skip can
> >stay then. Lets try to drop "arch" at least.
> 
> Just found out the MIPS/RISC firmware bin trips the arch check so
> that has to stay.
> 
> If you don't mind, take a look at the v3 revision. I'm experimenting
> with using the binary_package class there. Not really sure how I
> feel about it, but it's at least less lines & steps than this
> revision.