mbox series

[SRU,Jammy,0/1] UBUNTU: [Config] arm64: Enable PCI_MESON module

Message ID 20230324102826.2470397-1-isaac.true@canonical.com
Headers show
Series UBUNTU: [Config] arm64: Enable PCI_MESON module | expand

Message

Isaac True March 24, 2023, 10:28 a.m. UTC
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                   | 2 +-
 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, 6 insertions(+), 2 deletions(-)

Comments

Stefan Bader March 24, 2023, 11:30 a.m. UTC | #1
On 24.03.23 11:28, Isaac True wrote:

The cover email should rather be "[SRU J/K/L]... " since then its clear 
what will follow. You could prepare the whole batch by using 
--start-number on the other kernels format-patch. Then everything can be 
sent in one go.

-Stefan

> 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                   | 2 +-
>   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, 6 insertions(+), 2 deletions(-)
>
Isaac True March 24, 2023, 11:44 a.m. UTC | #2
Thanks Stefan, still getting used to formatting the git emails properly.

Do you want me to resubmit it all?

On Fri, 24 Mar 2023 at 12:31, Stefan Bader <stefan.bader@canonical.com> wrote:
>
> On 24.03.23 11:28, Isaac True wrote:
>
> The cover email should rather be "[SRU J/K/L]... " since then its clear
> what will follow. You could prepare the whole batch by using
> --start-number on the other kernels format-patch. Then everything can be
> sent in one go.
>
> -Stefan
>
> > 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                   | 2 +-
> >   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, 6 insertions(+), 2 deletions(-)
> >
>
>
Stefan Bader March 24, 2023, 1:16 p.m. UTC | #3
On 24.03.23 12:44, Isaac True wrote:
> Thanks Stefan, still getting used to formatting the git emails properly.
> 
> Do you want me to resubmit it all?
> 
> On Fri, 24 Mar 2023 at 12:31, Stefan Bader <stefan.bader@canonical.com> wrote:
>>
>> On 24.03.23 11:28, Isaac True wrote:
>>
>> The cover email should rather be "[SRU J/K/L]... " since then its clear
>> what will follow. You could prepare the whole batch by using
>> --start-number on the other kernels format-patch. Then everything can be
>> sent in one go.
>>
>> -Stefan
>>
>>> 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                   | 2 +-
>>>    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, 6 insertions(+), 2 deletions(-)
>>>
>>
>>

Yeah as explained in other conversations it is better to have the top 
level mention all affected targets.

-Stefan