[PATCH xenial/zesty 0/2] CONFIG_SATA_HIGHBANK=y

Stefan Bader stefan.bader at canonical.com
Thu Aug 3 15:29:29 UTC 2017


On 03.08.2017 17:04, Seth Forshee wrote:
> On Thu, Aug 03, 2017 at 03:27:27PM +0200, Stefan Bader wrote:
>> On 02.08.2017 17:35, Seth Forshee wrote:
>>> BugLink: http://bugs.launchpad.net/bugs/1703430
>>>
>>> This was changed after trusty from y to m without any justification
>>> given for the change (appears now only in a squashed Ubuntu changes
>>> commit, perhaps was a victim of previous attempts to change built-in
>>> drivers to modules). We recently got a report about this causing issues
>>> booting on Calxeda Highbank systems.
>>>
>>> Since there doesn't seem to be any big downsides to having the driver
>>> built-in, I'm proposing we change it back in xenial and zesty. I've
>>> already pushed this change to artful.
>>>
>>> Thanks,
>>> Seth
>>>
>> Looking at the annotations in Xenial it looks like hinting =m at least for
>> armhf. Not enforced though. But I am wondering whether this needs to be broken
>> up into per-arches defines and get more hinting in annotations.
> 
> It's only available for armhf, as it depends on ARCH_HIGHBANK.
> 
> I didn't bother updating the annotations since it seems to matter little
> for a stable release, though I did do so in artful. I can resend with an
> annotations update if that's desired.

Yes please, the bug number should be there as note so we have a clue what went
on. Also I realize that this change needs some modules.ignore thing added even
though we need to adapt those whenever the abi moves. But at least its a reminder.

-Stefan

> 
> Thanks,
> Seth
> 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20170803/111c0cea/attachment.sig>


More information about the kernel-team mailing list