Date: Sat, 27 Jan 2018 12:13:57 -0800 From: Adrian Chadd <adrian.chadd@gmail.com> To: Emmanuel Vadot <manu@bidouilliste.com> Cc: 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: <CAJ-VmonVwK4pkyxGqfwaNwQ89GdYAaamjBYZHxbjTcuZcEqDsw@mail.gmail.com> In-Reply-To: <20180127210801.37b8001125dd0a2c92372f98@bidouilliste.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>
next in thread | previous in thread | raw e-mail | index | archive | help
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. (I personally hate having to edit the dts/recompile/reboot for every test hardware change; it makes breadboarding things up kinda hilariously annoying.) -adrian
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAJ-VmonVwK4pkyxGqfwaNwQ89GdYAaamjBYZHxbjTcuZcEqDsw>