Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Kris Bahnsen via buildroot <buildroot@buildroot.org>
To: Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
	Kris Bahnsen via buildroot <buildroot@buildroot.org>
Cc: Giulio Benetti <giulio.benetti@benettiengineering.com>
Subject: Re: [Buildroot] [PATCH] package/wilc-driver: bump version to 2024.04
Date: Mon, 15 Jul 2024 13:11:23 -0700	[thread overview]
Message-ID: <bc23a512b541ddfa69d5d7de1d4d2cf2d5c01a1d.camel@embeddedTS.com> (raw)
In-Reply-To: <20240715212442.367a9df9@windsurf>

On Mon, 2024-07-15 at 21:24 +0200, Thomas Petazzoni wrote:
> On Mon, 15 Jul 2024 11:25:13 -0700
> Kris Bahnsen via buildroot <buildroot@buildroot.org> wrote:
> 
> > Be advised that this does NOT support "up to" Linux 6.6. The latest
> > updates from Microchip drop any backwards compatibility support,
> > which
> > is why we have not pushed this up to Buildroot yet.
> > 
> > I think this only will support kernels 6.2~6.6 and currently do not
> > believe it will compile against upstream Linux 6.9 (I tested it a
> > while
> > ago but don't remember the outcome as it wasn't important).
> > 
> > As I mentioned in my previous email responding to build failures,
> > we're
> > working on a solution to be able to handle backwards compatibility
> > with
> > 'backports' but are first catching up on other tasks.
> 
> So what can we do in the mean time? Support a few versions of the
> driver, and let the user chose?

At this time the driver that is in Buildroot, with the patches
submitted by Giulio, I believe will compile up to Linux 6.4. Most of
the testing we did on the latest driver updates was for Linux 6.6
(latest LTS), which did not cleanly compile with the existing patches
from Giulio which is why we pulled in the latest updates from
Microchip.

This patch could be modified to support the older linux4microchip-
2021.10-1 tag on Linux <= 6.4, and linux4microchip-2024.04 tag for
Linux == 6.6 But I'm not sure the best way to go about that other than
adding a Config 'choice' to select between the two, but that would put
the burden on the user to select the correct option.

But the issue also remains that it currently does not compile against
6.9 (I just confirmed this via a test) which I am not greenlit to put
time in to patching the driver solely for 6.9 at this time.

We are working on upstreaming platform support for many of our
products, but with how fiddly the WILC3000 driver is, we're going to
focus on that first so I hope to have a real solution sooner rather
than later.

> 
> Thomas


_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

      reply	other threads:[~2024-07-15 20:11 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-07-15 18:02 [Buildroot] [PATCH] package/wilc-driver: bump version to 2024.04 Giulio Benetti
2024-07-15 18:25 ` Kris Bahnsen via buildroot
2024-07-15 18:54   ` Giulio Benetti
2024-07-15 19:24   ` Thomas Petazzoni via buildroot
2024-07-15 20:11     ` Kris Bahnsen via buildroot [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=bc23a512b541ddfa69d5d7de1d4d2cf2d5c01a1d.camel@embeddedTS.com \
    --to=buildroot@buildroot.org \
    --cc=giulio.benetti@benettiengineering.com \
    --cc=kris@embeddedTS.com \
    --cc=thomas.petazzoni@bootlin.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox