Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 27 Jan 2018 13:12:22 -0800
From:      Oleksandr Tymoshenko <gonzo@bluezbox.com>
To:        Adrian Chadd <adrian.chadd@gmail.com>
Cc:        Emmanuel Vadot <manu@bidouilliste.com>, Poul-Henning Kamp <phk@phk.freebsd.dk>, Warner Losh <imp@bsdimp.com>, John Baldwin <jhb@freebsd.org>, Ravi Pokala <rpokala@mac.com>, src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r328257 - in head/sys: arm/broadcom/bcm2835 dts/arm modules
Message-ID:  <20180127211222.GA36907@bluezbox.com>
In-Reply-To: <CAJ-VmonVwK4pkyxGqfwaNwQ89GdYAaamjBYZHxbjTcuZcEqDsw@mail.gmail.com>
References:  <201801220710.w0M7AUm9091853@repo.freebsd.org> <CANCZdfpq2QoG4EAj0VW2FF=4VXRv-qQKFfJTJerWH9YOwVoVBA@mail.gmail.com> <90451.1516663240@critter.freebsd.dk> <2987003.eeGRFBb6N8@ralph.baldwin.cx> <CANCZdfrh0NHq7cbkq_genEdzo%2BB3G4TTAcEzpgh11sr%2B82e9aw@mail.gmail.com> <93949.1516733748@critter.freebsd.dk> <20180127210801.37b8001125dd0a2c92372f98@bidouilliste.com> <CAJ-VmonVwK4pkyxGqfwaNwQ89GdYAaamjBYZHxbjTcuZcEqDsw@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
Adrian Chadd (adrian.chadd@gmail.com) wrote:
> Hi,
> 
> Find the middle ground. Don't dissuade the developer too much.
> 
> Here's an example:
> 
> Make the driver follow DTS, allow a tunable/kenv check for it to
> override whether it needs to be in the DTS or not (the "keep phk happy
> for now" compromise) and have it default to obeying the device tree.
> 
> That way phk is kept happy and the defaults are the same as normal-ish
> ARM /and/ you have a springboard to experiment with extending FDT
> overlays at runtime for people who wish to do so.

This doesn't sound right. I'm 100% with manu@ on this: driver needs to
be modified to follow DTS. A lot of work was spent to get FreeBSD FDT
support consistent over multiple platforms and to make sure it conforms
standard. When it comes to FDT support Pi port is inferior to the
majority of platforms that have been added later. I bear a fair share of
responsibility for this mess and it makes me appreciate all the effort
spent on correcting the situation even more.

Adding changes that violate standard and undermine all that work for
the convenience of a single developer is plain wrong. If current
framework is not good enough the way to go is to improve it, not hack
around its limitations

-- 
gonzo



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20180127211222.GA36907>