On Mon, Jun 15, 2020 at 04:45:22PM +0300, Yehezkel Bernat wrote:
> On Mon, Jun 15, 2020 at 4:02 PM Mika Westerberg
> <mika.westerberg@xxxxxxxxxxxxxxx> wrote:
> >
> > diff --git a/include/linux/thunderbolt.h b/include/linux/thunderbolt.h
> > index ff397c0d5c07..5db2b11ab085 100644
> > --- a/include/linux/thunderbolt.h
> > +++ b/include/linux/thunderbolt.h
> > @@ -504,8 +504,6 @@ struct tb_ring {
> > #define RING_FLAG_NO_SUSPEND BIT(0)
> > /* Configure the ring to be in frame mode */
> > #define RING_FLAG_FRAME BIT(1)
> > -/* Enable end-to-end flow control */
> > -#define RING_FLAG_E2E BIT(2)
> >
>
> Isn't it better to keep it (or mark it as reserved) so it'll not cause
> compatibility issues with older versions of the driver or with Windows?
How can you have "older versions of the driver"? All drivers are in the
kernel tree at the same time, you can't ever mix-and-match drivers and
kernels.
And how does Windows come into play here?
thanks,
greg k-h
