[SRU J/K/L][PATCH v3 0/3] UBUNTU: [Config] arm64: Enable PCI_MESON module

[Isaac True]

Hi Tim,

I'm not quite sure I understand what you mean - all of the other
annotation notes are at the end of each config section in the Jammy
kernel, so I was just following the existing format of previous
commits (e.g. https://git.launchpad.net/~canonical-kernel/ubuntu/+source/linux/+git/jammy/commit/debian.master/config/annotations?id=51c27b408c5fdea5bc3076724eae6a8ef2c21ac6
and https://git.launchpad.net/~canonical-kernel/ubuntu/+source/linux/+git/jammy/commit/debian.master/config/annotations?id=a8c8d8e170ad6d448ea26db88bf3e5a279efbd9b).

Cheers,
Isaac

On Mon, 27 Mar 2023 at 15:07, Tim Gardner <tim.gardner at canonical.com> wrote:
>
> On 3/27/23 6:23 AM, Isaac True wrote:
> > BugLink: https://bugs.launchpad.net/bugs/2007745
> >
> > [ Impact ]
> >
> > The PCI driver for the amlogic-based Meson platforms (CONFIG_PCI_MESON)
> > is not enabled as a module in the kernel configuration.
> >
> > [ Test Plan ]
> >
> > Check if pci_meson.ko is compiled and present in the linux-modules
> > package.
> >
> > [ Where problems could occur ]
> >
> > This driver does not enable any dependencies that the kernel does not
> > already have enabled, and it is a stable upstream driver. There
> > shouldn't be any issues that arise from enabling this driver.
> >
> > [ Other Info ]
> >
> > This driver is important for systems like the ODROID HC4, which uses a
> > PCIE-SATA bridge to provide a storage interface. This system is fully
> > supported by the kernel (including a device tree), except for the fact
> > that this PCI driver is disabled.
> >
> > Isaac True (1):
> >    UBUNTU: [Config] arm64: Enable PCI_MESON module
> >
> >   debian.master/config/amd64/config.common.amd64     | 1 +
> >   debian.master/config/annotations                   | 4 +++-
> >   debian.master/config/arm64/config.common.arm64     | 1 +
> >   debian.master/config/armhf/config.common.armhf     | 1 +
> >   debian.master/config/config.common.ubuntu          | 1 -
> >   debian.master/config/ppc64el/config.common.ppc64el | 1 +
> >   debian.master/config/s390x/config.common.s390x     | 1 +
> >   7 files changed, 8 insertions(+), 2 deletions(-)
> >
>
> Usually the note follows the policy change on the next line. Patch 3/3
> is correct in that regard.
> --
> -----------
> Tim Gardner
> Canonical, Inc
>