On Tue, 30 Jan 2018 10:15:26 +
"Poul-Henning Kamp" wrote:
>
> In message <20180130105812.d1de09e0d4ac88934d776...@bidouilliste.com>,
> Emmanuel
> Vadot writes:
> >On Tue, 30 Jan 2018 09:47:57 +
>
> > I don't think that we want to load it by default as it would mean that
> >som
In message <20180130105812.d1de09e0d4ac88934d776...@bidouilliste.com>, Emmanuel
Vadot writes:
>On Tue, 30 Jan 2018 09:47:57 +
> I don't think that we want to load it by default as it would mean that
>some GPIO are now unusable as they are pwm pins now.
So the essential architectural
On Tue, 30 Jan 2018 09:47:57 +
"Poul-Henning Kamp" wrote:
>
> In message <20180130094251.ga85...@bluezbox.com>, Oleksandr Tymoshenko writes:
> >Poul-Henning Kamp (p...@phk.freebsd.dk) wrote:
>
> >> But only if you add pwm.dtbo to config.txt and reboot, because
> >> I see nothing ind
In message <20180130094251.ga85...@bluezbox.com>, Oleksandr Tymoshenko writes:
>Poul-Henning Kamp (p...@phk.freebsd.dk) wrote:
>> But only if you add pwm.dtbo to config.txt and reboot, because
>> I see nothing indicating that we will put it there in the
>> release images we (may start to)
Poul-Henning Kamp (p...@phk.freebsd.dk) wrote:
>
> In message <20180130025048.ga76...@bluezbox.com>, Oleksandr Tymoshenko writes:
>
> >> You seem to have skipped the bit about "without reboot" ?
> >
> >No I haven't. config.txt is parsed by VC firmware on boot time
> >and pwm.dtbo is appli
In message <20180130025048.ga76...@bluezbox.com>, Oleksandr Tymoshenko writes:
>> You seem to have skipped the bit about "without reboot" ?
>
>No I haven't. config.txt is parsed by VC firmware on boot time
>and pwm.dtbo is applied even before boot loader kicks in. When
>kernel starts pwm
Poul-Henning Kamp (p...@phk.freebsd.dk) wrote:
>
> In message <20180129132736.ga66...@bluezbox.com>, Oleksandr Tymoshenko writes:
>
> >> If I pick a RPI[23] out of the box, download a FreeBSD image,
> >> put the card in and play around, I should be able to put a
> >> LED in a breadboard a
In message <20180129132736.ga66...@bluezbox.com>, Oleksandr Tymoshenko writes:
>> If I pick a RPI[23] out of the box, download a FreeBSD image,
>> put the card in and play around, I should be able to put a
>> LED in a breadboard and configure a line for PWM or attach
>> an I2C or SPI devi
Poul-Henning Kamp (p...@phk.freebsd.dk) wrote:
>
> In message <20180129063950.ga59...@bluezbox.com>, Oleksandr Tymoshenko writes:
>
> >PWM drivers: https://reviews.freebsd.org/D14104
>
> I'll test this in my next timeslot.
>
> >We do not include these
> >overlays in official snapshots a
In message <20180129063950.ga59...@bluezbox.com>, Oleksandr Tymoshenko writes:
>PWM drivers: https://reviews.freebsd.org/D14104
I'll test this in my next timeslot.
>We do not include these
>overlays in official snapshots and I think this should be fixed.
That's one of the details I ran
In message
, Warner
Losh writes:
>That's tricky for a lot of reasons, but the tl;dr version is that
>embedded is different because you don't just enable a device, but all the
>dependencies which, unlike x86, are often 'non-linear' and possibly
>non-orthogonal to other devices (eg, you m
On Sun, Jan 28, 2018 at 11:39 PM, Oleksandr Tymoshenko
wrote:
>
> Ignoring value of ofw_bus_status_okay(dev) in probe method is
> wrong for number of reasons and pinctl and clocks were brought up
> as an example why "status" property is more than just
> attach/dont-attach flag. So there is strictl
Poul-Henning Kamp (p...@phk.freebsd.dk) wrote:
>
> In message <8d8ae9d10058fd72ce3ec467181c9...@megadrive.org>, Emmanuel Vadot
> writes:
>
> > Sometimes it makes sense to reboot.
>
> Yes, *sometimes* it does.
>
> But *always* demanding reboot makes no sense ever.
>
> > Reference pla
In message
, Warner
Losh writes:
>--001a1144c1d25e374c0563dc9147
>No. Such hacks are actively getting in the way. My WIP already breaks PHK's
>hacks (unintentionally, but discovered in hindsight) because they are
>outside the mainstream. I have 0 interest in preserving short-term hacks
In message <8d8ae9d10058fd72ce3ec467181c9...@megadrive.org>, Emmanuel Vadot
writes:
> Sometimes it makes sense to reboot.
Yes, *sometimes* it does.
But *always* demanding reboot makes no sense ever.
> Reference platform doesn't make much sense in the embedded world.
A reference pla
On Sun, Jan 28, 2018 at 10:34 AM, Emmanuel Vadot
wrote:
> On 2018-01-28 00:14, Poul-Henning Kamp wrote:
>
>>
>> In message <20180127210801.37b8001125dd0a2c92372...@bidouilliste.com>,
>> Emmanuel Vadot writes:
>>
>> - We have a commiter that commited something for his own need: he
>>> wan
On Sun, Jan 28, 2018 at 2:01 PM, Adrian Chadd
wrote:
> [snip]
>
> And this is the root bit that's missing - dynamic pinmux/pinctrl stuff
> at runtime.
>
Well, you gotta walk before you can run. We don't even have static pinmx on
rpi. Though someone else is working on that.
> Warner's already s
[snip]
And this is the root bit that's missing - dynamic pinmux/pinctrl stuff
at runtime.
Warner's already said he's WIP'ing it and phk seems like a good test
case for kicking those tyres, so it sounds like those tyres are about
to be kicked.
In the meantime, hacks allow people to make some prog
On 2018-01-28 00:14, Poul-Henning Kamp wrote:
In message <20180127210801.37b8001125dd0a2c92372...@bidouilliste.com>,
Emmanuel Vadot writes:
- We have a commiter that commited something for his own need: he
wanted to use the pwm on his rpi, coded a driver (this part is good)
but feel th
On Sat, 27 Jan 2018 13:32:22 -0800 (PST)
"Rodney W. Grimes" wrote:
> > On Sat, 27 Jan 2018 12:13:57 -0800
> > Adrian Chadd wrote:
> >
> > > Hi,
> > >
> > > Find the middle ground. Don't dissuade the developer too much.
> >
> > This is what happened two years ago when I started hacking on the
On Sat, 2018-01-27 at 13:32 -0800, Rodney W. Grimes wrote:
> >
> > On Sat, 27 Jan 2018 12:13:57 -0800
> > Adrian Chadd wrote:
> >
> > >
> > > Hi,
> > >
> > > Find the middle ground. Don't dissuade the developer too much.
> > This is what happened two years ago when I started hacking on the
>
In message
, Warner
Losh writes:
>> The moment the correct infrastructure appears on the RPi platform,
>> if it ever does, I will change my driver to use that infrastructure.
>
>The root of the problem is that we have the right infrastructure...
>Everywhere except RPi. It needs a proper
On Sat, Jan 27, 2018 at 4:14 PM, Poul-Henning Kamp
wrote:
>
> The moment the correct infrastructure appears on the RPi platform,
> if it ever does, I will change my driver to use that infrastructure.
>
The root of the problem is that we have the right infrastructure...
Everywhere except RPi. It n
In message <20180127210801.37b8001125dd0a2c92372...@bidouilliste.com>, Emmanuel
Vadot writes:
> - We have a commiter that commited something for his own need: he
>wanted to use the pwm on his rpi, coded a driver (this part is good)
>but feel that the standard we were using was crap and c
On Jan 27, 2018 2:32 PM, "Rodney W. Grimes"
wrote:
> On Sat, 27 Jan 2018 12:13:57 -0800
> Adrian Chadd wrote:
>
> > Hi,
> >
> > Find the middle ground. Don't dissuade the developer too much.
>
> This is what happened two years ago when I started hacking on the
> allwinner SoCs :
>
> - I asked
> On Sat, 27 Jan 2018 12:13:57 -0800
> Adrian Chadd wrote:
>
> > Hi,
> >
> > Find the middle ground. Don't dissuade the developer too much.
>
> This is what happened two years ago when I started hacking on the
> allwinner SoCs :
>
> - I asked what should be done for bringing a new board
> -
Adrian Chadd (adrian.ch...@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" co
On Sat, 27 Jan 2018 12:13:57 -0800
Adrian Chadd wrote:
> Hi,
>
> Find the middle ground. Don't dissuade the developer too much.
This is what happened two years ago when I started hacking on the
allwinner SoCs :
- I asked what should be done for bringing a new board
- andrew@ told me that we
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
So, almost a week after this commit, let's recap.
- phk@ commit an hack to allow the pwm driver on rpi to always attach
even if it's not enabled in the dts.
- I say to him that it's a really bad move and that he should use dtb
overlays (which thanks to kevans@ are in a really better state now)
In message
, Warner
Losh writes:
>--089e08222b4ce7f8a4056372a0d5
>I'm actually thinking the right thing is to remove the checks everywhere.
>
>We then create a couple of new simplebus methods:
+1
--
Poul-Henning Kamp | UNIX since Zilog Zeus 3.20
p...@freebsd.org | TCP/
On Mon, Jan 22, 2018 at 4:54 PM, John Baldwin wrote:
> On Monday, January 22, 2018 11:20:40 PM Poul-Henning Kamp wrote:
> >
> > In message gmail.com>
> > , Warner Losh writes:
> >
> > >I think even bootverbose it's insane. Or I don't understand what the
> real
> > >ask is.
> >
> > What
On Monday, January 22, 2018 11:20:40 PM Poul-Henning Kamp wrote:
>
> In message
>
> , Warner Losh writes:
>
> >I think even bootverbose it's insane. Or I don't understand what the real
> >ask is.
>
> What I think would be a good idea, is when a driver is kldloaded
> the user will be to
In message
, Warner Losh writes:
>I think even bootverbose it's insane. Or I don't understand what the real
>ask is.
What I think would be a good idea, is when a driver is kldloaded
the user will be told if there were hardware present which it could
drive (ie: probe() matches), but for
On Mon, Jan 22, 2018 at 4:05 PM, Poul-Henning Kamp
wrote:
>
> In message f4gz7+kpk...@mail.gmail.com>
> , Warner Losh writes:
>
> >This represents a partial understanding of what's required. It's base
> level
> >only. Don't mistake it for full understanding of what needs to be done.
>
>
On Mon, Jan 22, 2018 at 7:07 AM, Poul-Henning Kamp
wrote:
>
> In message <20180122145117.08173be547f5dd6fef296...@bidouilliste.com>,
> Emmanuel
> Vadot writes:
>
> > Using the same logic as before one could have a script starting some
> >pwm stuff (or simply using /etc/sysctl.conf)
> >
In message
, Warner Losh writes:
>This represents a partial understanding of what's required. It's base level
>only. Don't mistake it for full understanding of what needs to be done.
I would love not to, but learning "what needs to be done" have so
far eluded me, apart from some people
On Mon, Jan 22, 2018 at 1:41 PM, Ravi Pokala wrote:
> >> I wonder if it would be too noisy if we allowed ->probe() to run on
> disabled devices and reported something like:
> >>
> >> "foobar0: Matched disabled device <...>"
> >>
> >> (without calling ->attach())
> >>
> >> That way you w
On Mon, Jan 22, 2018 at 1:24 PM, Poul-Henning Kamp
wrote:
>
> In message d...@mail.gmail.com>
> , Warner Losh writes:
>
> >> That also removes a couple of boilerplate lines from all FDT device
> >> drivers.
> >
> >That would be useful, but the status bit is so much more than on/off.
>
>
age under bootverbose would be
reasonable.
-Ravi (rpokala@)
-Original Message-
From: on behalf of Warner Losh
Date: 2018-01-22, Monday at 11:29
To: Poul-Henning Kamp
Cc: John Baldwin , Emmanuel Vadot ,
src-committers , ,
Subject: Re: svn commit: r328257 - in head/sys: arm/broadcom/
In message
, Warner Losh writes:
>> That also removes a couple of boilerplate lines from all FDT device
>> drivers.
>
>That would be useful, but the status bit is so much more than on/off.
The boilerplate code in all the ->probe() functions don't seem to think so ?
--
Poul-Henning Kam
On Mon, Jan 22, 2018 at 1:27 PM, Warner Losh wrote:
> On Mon, Jan 22, 2018 at 12:12 PM, Ian Lepore wrote:
>
>> On Mon, 2018-01-22 at 10:57 -0800, John Baldwin wrote:
>> > On Monday, January 22, 2018 03:30:03 PM Emmanuel Vadot wrote:
>> > >
>> > > On Mon, 22 Jan 2018 14:07:30 +
>> > > "Poul-He
On Mon, Jan 22, 2018 at 12:24 PM, Poul-Henning Kamp
wrote:
>
> In message <52374125.ogxafgl...@ralph.baldwin.cx>, John Baldwin writes:
>
> >Create the corresponding device_t but device_disable() it when there
> >is a disabled property.
>
> That also removes a couple of boilerplate lines
On Mon, Jan 22, 2018 at 12:12 PM, Ian Lepore wrote:
> On Mon, 2018-01-22 at 10:57 -0800, John Baldwin wrote:
> > On Monday, January 22, 2018 03:30:03 PM Emmanuel Vadot wrote:
> > >
> > > On Mon, 22 Jan 2018 14:07:30 +
> > > "Poul-Henning Kamp" wrote:
> > >
> > > >
> > > >
> > > > In
In message <52374125.ogxafgl...@ralph.baldwin.cx>, John Baldwin writes:
>Create the corresponding device_t but device_disable() it when there
>is a disabled property.
That also removes a couple of boilerplate lines from all FDT device
drivers.
>A user can then use 'devctl enable ' to en
On Mon, 2018-01-22 at 10:57 -0800, John Baldwin wrote:
> On Monday, January 22, 2018 03:30:03 PM Emmanuel Vadot wrote:
> >
> > On Mon, 22 Jan 2018 14:07:30 +
> > "Poul-Henning Kamp" wrote:
> >
> > >
> > >
> > > In message <20180122145117.08173be547f5dd6fef296732@bidouilliste.
> > >
On Monday, January 22, 2018 03:30:03 PM Emmanuel Vadot wrote:
> On Mon, 22 Jan 2018 14:07:30 +
> "Poul-Henning Kamp" wrote:
>
> >
> > In message <20180122145117.08173be547f5dd6fef296...@bidouilliste.com>,
> > Emmanuel
> > Vadot writes:
> >
> > > Using the same logic as before one
On Mon, 22 Jan 2018 15:52:41 +
"Poul-Henning Kamp" wrote:
>
> In message <20180122153003.664e1613bbf70ab49c5c1...@bidouilliste.com>,
> Emmanuel
> Vadot writes:
>
> >[...] there is no way to
> >know what the user will do so every node not used by the SBC must be
> >disabled.
>
> [
In message <20180122153003.664e1613bbf70ab49c5c1...@bidouilliste.com>, Emmanuel
Vadot writes:
>[...] there is no way to
>know what the user will do so every node not used by the SBC must be
>disabled.
[...] until the user explicitly tells us, for instance with the command:
kldl
On Mon, 22 Jan 2018 14:07:30 +
"Poul-Henning Kamp" wrote:
>
> In message <20180122145117.08173be547f5dd6fef296...@bidouilliste.com>,
> Emmanuel
> Vadot writes:
>
> > Using the same logic as before one could have a script starting some
> >pwm stuff (or simply using /etc/sysctl.conf
In message <20180122145117.08173be547f5dd6fef296...@bidouilliste.com>, Emmanuel
Vadot writes:
> Using the same logic as before one could have a script starting some
>pwm stuff (or simply using /etc/sysctl.conf)
> Also this is not how DT is suppose to work, if the status ==
>'disabled' no
On Mon, 22 Jan 2018 13:38:47 +
"Poul-Henning Kamp" wrote:
>
> In message <20180122135905.203e19ce94510854777df...@bidouilliste.com>,
> Emmanuel
> Vadot writes:
>
> >> +#if 0
> >> + // XXX: default state is disabled in RPI3 DTB, assume for now
> >> + // XXX: that people want the
In message <20180122135905.203e19ce94510854777df...@bidouilliste.com>, Emmanuel
Vadot writes:
>> +#if 0
>> +// XXX: default state is disabled in RPI3 DTB, assume for now
>> +// XXX: that people want the PWM to work if the KLD this module.
>> if (!ofw_bus_status_okay(dev))
>>
On Mon, 22 Jan 2018 07:10:30 + (UTC)
Poul-Henning Kamp wrote:
> Author: phk
> Date: Mon Jan 22 07:10:30 2018
> New Revision: 328257
> URL: https://svnweb.freebsd.org/changeset/base/328257
>
> Log:
> Add a skeleton Clock Manager for RPi2/3, and use that from pwm
> instead of frobbing the
In message <20180122094629.f292f9f239174f0865b04...@bidouilliste.com>, Emmanuel
Vadot writes:
>> + * Copyright (C) 2013-2015 Daisuke Aoyama
>
> Was is based on some file ?
> Also someone will probably complain about SDPX tag here :)
Yes, I grabbed the blurp from a nearby file and forgo
On Mon, 22 Jan 2018 07:10:30 + (UTC)
Poul-Henning Kamp wrote:
> Author: phk
> Date: Mon Jan 22 07:10:30 2018
> New Revision: 328257
> URL: https://svnweb.freebsd.org/changeset/base/328257
>
> Log:
> Add a skeleton Clock Manager for RPi2/3, and use that from pwm
> instead of frobbing the
Author: phk
Date: Mon Jan 22 07:10:30 2018
New Revision: 328257
URL: https://svnweb.freebsd.org/changeset/base/328257
Log:
Add a skeleton Clock Manager for RPi2/3, and use that from pwm
instead of frobbing the registers directly.
As a hack the bcm2835_pwm kmod presently ignores the 'statu
57 matches
Mail list logo
