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>