* Russell King - ARM Linux [181207 19:27]:
> You mentioned that edge mode didn't work as well as level mode on
> duovero smsc controller, I think this may help to solve the same
> issue but for edge IRQs - we need a mask_ack_irq function to avoid
> acking while the edge interrupt is masked. Let
* Tony Lindgren [181207 18:14]:
> Hi,
>
> * Russell King - ARM Linux [181207 18:01]:
> > Hi Tony,
> >
> > You know most of what's been going on from IRC, but here's the patch
> > which gets me:
> >
> > 1) working interrupts for netw
Hi,
* Russell King - ARM Linux [181207 18:01]:
> Hi Tony,
>
> You know most of what's been going on from IRC, but here's the patch
> which gets me:
>
> 1) working interrupts for networking
> 2) solves the stuck-wakeup problem
>
> It also contains some of the debug bits I added.
This is
* Russell King - ARM Linux [181206 18:08]:
> reverted, the problem is still there. Revert:
>
> ec0daae685b2 ("gpio: omap: Add level wakeup handling for omap4 based SoCs")
>
> on top, and networking returns to normal. So it appears to be this
> last commit causing the issue.
>
> With that and
Hi,
* Russell King - ARM Linux [181206 13:23]:
> It looks very much like a receive problem - in that the board is not
> always aware of a packet having been received until it attempts to
> transmit (eg, in the case of TFTP, when it re-sends the ACK after a
> receive timeout, it _then_ notices
* Grygorii Strashko [180830 17:08]:
> On 08/29/2018 07:47 PM, Tony Lindgren wrote:
> > In general, it seems cpsw is just an interconnect instance
> > (L4_FAST) with a control module (CPSW_WR) and a pile of
> > independent other modules. That's described nicely in
> > a
* Grygorii Strashko [180830 00:12]:
> Hi Tony,
>
> On 08/29/2018 10:00 AM, Tony Lindgren wrote:
> > The current cpsw usage for cpsw-phy-sel is undocumented but is used for
> > all the boards using cpsw. And cpsw-phy-sel is not really a child of
> > the cpsw device, it
also
update the binding documentation accordingly.
Cc: devicet...@vger.kernel.org
Cc: Andrew Lunn
Cc: Grygorii Strashko
Cc: Ivan Khoronzhuk
Cc: Mark Rutland
Cc: Murali Karicheri
Cc: Rob Herring
Signed-off-by: Tony Lindgren
---
drivers/net/ethernet/ti/cpsw-phy-sel.c | 9 ++---
1 file
instead of a child device for it. That way we can
properly describe the hardware in dts files for things like genpd.
Cc: devicet...@vger.kernel.org
Cc: Andrew Lunn
Cc: Grygorii Strashko
Cc: Ivan Khoronzhuk
Cc: Mark Rutland
Cc: Murali Karicheri
Cc: Rob Herring
Signed-off-by: Tony Lindgren
* Tony Lindgren [180808 13:52]:
> * Andrew Lunn [180808 12:02]:
> >
> > Do you need to handle EPROBE_DEFER here? The phandle points to a
> > device which has not yet been loaded? I'm not sure exactly where it
> > will be returned, maybe it is bus_find_device(), b
* Andrew Lunn [180808 12:02]:
> > +++ b/Documentation/devicetree/bindings/net/cpsw.txt
> > @@ -19,6 +19,9 @@ Required properties:
> > - slaves : Specifies number for slaves
> > - active_slave : Specifies the slave to use for time stamping,
> > ethtool
missing the related binding documentation, so let's also
update the binding documentation accordingly.
Cc: devicet...@vger.kernel.org
Cc: Grygorii Strashko
Cc: Ivan Khoronzhuk
Cc: Mark Rutland
Cc: Murali Karicheri
Cc: Rob Herring
Signed-off-by: Tony Lindgren
---
Documentation/devicetree
* Adam Ford [170828 13:33]:
> On Mon, Aug 28, 2017 at 1:54 PM, Grygorii Strashko
> wrote:
> > Cc: Sekhar
> >
> > On 08/28/2017 10:32 AM, Adam Ford wrote:
> >>
> >> The davinvi_emac MAC address seems to attempt a call to
> >> ti_cm_get_macid in
3 at 1493259036.381161765
> event index 3 at 1493259037.381731909
Cool :)
Acked-by: Tony Lindgren <t...@atomide.com>
* Colin Ian King [170605 10:58]:
> Hi Tony,
>
> While running static analysis on linux-next, CoverityScan picked up a
> NULL pointer deference on ddata->pins when calling pinctrl_lookup_state:
>
> 466ddata->pins = devm_pinctrl_get(ddata->dev);
>
>1.
* Arnd Bergmann <a...@arndb.de> [170428 08:06]:
> Tony Lindgren reports a kernel oops that resulted from my compile-time
> fix on the default config. This shows two problems:
>
> a) configurations that did not already enable PTP_1588_CLOCK will
>now miss the cpts dri
* Arnd Bergmann <a...@arndb.de> [170424 11:14]:
> On Mon, Apr 24, 2017 at 7:44 PM, Tony Lindgren <t...@atomide.com> wrote:
> > * Arnd Bergmann <a...@arndb.de> [170424 10:38]:
> >> On Mon, Apr 24, 2017 at 6:51 PM, Tony Lindgren <t...@atomide.com> wrote:
* Arnd Bergmann <a...@arndb.de> [170424 10:38]:
> On Mon, Apr 24, 2017 at 6:51 PM, Tony Lindgren <t...@atomide.com> wrote:
> > Hi,
> >
> > Looks like commit 07fef3623407 ("cpsw/netcp: cpts depends on posix_timers")
> > in mainline started tr
Hi,
Looks like commit 07fef3623407 ("cpsw/netcp: cpts depends on posix_timers")
in mainline started triggering the following oops at least on j5eco-evm.
Adding CONFIG_PTP_1588_CLOCK to .config solves it, but the oops hints
something is wrong with the dependencies.. CONFIG_TI_CPTS defaults to N
* Roger Quadros [170330 05:37]:
> Hi Tony & Dave,
>
> On 13/03/17 15:42, Roger Quadros wrote:
> > Hi,
> >
> > This series adds ethernet support to am335x-icev2 board.
> >
> > The ethernet PHYs on the board need an explicit GPIO reset pulse
> > to ensure they bootstrap to the
* Bjørn Mork <bj...@mork.no> [170319 10:23]:
> Tony Lindgren <t...@atomide.com> writes:
> > And the v3.8 kernel also has drivers/usb/serial/mdm6600.c:
> >
> > +static const struct usb_device_id mdm6600_id_table[] = {
> > + { USB_DEVICE_AND_INTERFAC
* Tony Lindgren <t...@atomide.com> [170319 10:05]:
> * Bjørn Mork <bj...@mork.no> [170319 09:33]:
> > This is a bit unusual, so I'd like to verify that it is correct. Do you
> > happen to have a "lsusb -v" or /sys/kernel/debug/usb/devices dump for
> > th
* Bjørn Mork <bj...@mork.no> [170319 09:33]:
> Tony Lindgren <t...@atomide.com> writes:
>
> > This gets qmicli working with the MDM6600 modem.
> >
> > Cc: Bjørn Mork <bj...@mork.no>
> > Reviewed-by: Sebastian Reichel <s...@kernel.org>
> &g
This gets qmicli working with the MDM6600 modem.
Cc: Bjørn Mork <bj...@mork.no>
Reviewed-by: Sebastian Reichel <s...@kernel.org>
Tested-by: Sebastian Reichel <s...@kernel.org>
Signed-off-by: Tony Lindgren <t...@atomide.com>
---
drivers/net/usb/qmi_wwan.c | 4
1 fi
* Kalle Valo <kv...@codeaurora.org> [170130 22:36]:
> Tony Lindgren <t...@atomide.com> writes:
>
> > * Pavel Machek <pa...@ucw.cz> [170127 11:41]:
> >> On Fri 2017-01-27 17:23:07, Kalle Valo wrote:
> >> > Pali Rohár <pali.ro...@gmail.com>
* Pavel Machek [170127 11:41]:
> On Fri 2017-01-27 17:23:07, Kalle Valo wrote:
> > Pali Rohár writes:
> >
> > > On Friday 27 January 2017 14:26:22 Kalle Valo wrote:
> > >> Pali Rohár writes:
> > >>
> > >> > 2) It was already tested
* Grygorii Strashko [170112 09:15]:
> According to errata i880 description the speed of Ethernet port 1 on AM572x
> SoCs rev 1.1 shuld be limited to 10/100Mbps, because RGMII2 Switching
> Characteristics are not compatible with 1000 Mbps operation [1].
> The issue is
* Grygorii Strashko <grygorii.stras...@ti.com> [170109 11:26]:
>
>
> On 01/06/2017 03:54 PM, Tony Lindgren wrote:
> > * Grygorii Strashko <grygorii.stras...@ti.com> [170106 12:56]:
> > > TI DRA72-EVM Rev C has two DP83867 ethernet phys which support IRQ
* Grygorii Strashko [170106 12:56]:
> TI DRA72-EVM Rev C has two DP83867 ethernet phys which support IRQ
> generation in case of phy/link status changes. The INT/PWDN lines from both
> DP83867 phys are wired to DRA7 gpio6.16, so reflect the same in DT.
Hmm not seeing
* Kalle Valo <kv...@codeaurora.org> [161220 09:12]:
> Tony Lindgren <t...@atomide.com> writes:
>
> > * Kalle Valo <kv...@codeaurora.org> [161220 03:47]:
> >> Arend Van Spriel <arend.vanspr...@broadcom.com> writes:
> >>
> >> >
* Kalle Valo [161220 03:47]:
> Arend Van Spriel writes:
>
> > On 18-12-2016 13:09, Pali Rohár wrote:
> >
> >> File wl1251-nvs.bin is provided by linux-firmware package and contains
> >> default data which should be overriden by model specific
* Pali Rohár [161126 09:21]:
> On Thursday 24 November 2016 19:46:01 Aaro Koskinen wrote:
> > Hi,
> >
> > On Thu, Nov 24, 2016 at 04:20:45PM +0100, Pali Rohár wrote:
> > > Proprietary, signed and closed bootloader NOLO does not support DT.
> > > So for booting you need to
* Jeroen Hofstee <jhofs...@victronenergy.com> [161028 11:19]:
> Hello Tony,
>
> On 28-10-16 17:52, Tony Lindgren wrote:
> > * Jeroen Hofstee <jhofs...@victronenergy.com> [161028 08:33]:
> > > Commit b6745f6e4e63 ("drivers: net: cpsw: davinci_emac: move
* Jeroen Hofstee [161028 08:33]:
> Commit b6745f6e4e63 ("drivers: net: cpsw: davinci_emac: move reading mac
> id to common file") did not only move the code for an am3517, it also
> added the slave parameter, resulting in an invalid (all zero) mac address
> being
* Jeroen Hofstee [161021 02:31]:
> Aaah, lets wait a sec. I just saw there is another user of this function,
> so above is simply not true
>
> if (of_machine_is_compatible("ti,dra7"))
> return davinci_emac_3517_get_macid(dev, 0x514, slave, mac_addr);
* Jeroen Hofstee <jhofs...@victronenergy.com> [161021 00:37]:
> Hello Tony,
>
> On 21-10-16 08:38, Tony Lindgren wrote:
> > * Jeroen Hofstee <jhofs...@victronenergy.com> [161020 12:57]:
> > > Commit b6745f6e4e63 ("drivers: net: cpsw: davinci_emac: move
* Jeroen Hofstee [161020 12:57]:
> Commit b6745f6e4e63 ("drivers: net: cpsw: davinci_emac: move reading mac
> id to common file") did not only move the code for an am3517, it also
> added the slave parameter, resulting in a invalid (all zero) mac address
> being
device tree.
> This patch makes sure the mac from DT is preserved.
>
> On a am3517 this address is incorrectly read as all zeros, making it
> impossible to set a valid mac address without this patch.
OK, at least I don't have better ideas for fixing this:
Acked-by: Tony Lindgren <t...@atomide.com>
code in the RTC
> subsystem.
>
> The code looks for an additional interrupt - apart from the
> ordinary device interrupt - and in case that is present,
> we register an interrupt handler to respons to this,
> and flag the device and this interrupt as a wakeup.
>
> Cc:
* Linus Walleij [160708 02:10]:
> The SMSC911x have a line out of the chip called "PME",
> Power Management Event. When connected to an asynchronous
> interrupt controller this is able to wake the system up
> from sleep in response to certain network events.
Cool, so
not cause merge conflicts, please feel free
to merge along with the other CPSW driver patches after the
pending comments are dealt with:
Acked-by: Tony Lindgren <t...@atomide.com>
* Al Viro <v...@zeniv.linux.org.uk> [160509 08:41]:
> On Mon, May 09, 2016 at 08:21:38AM -0700, Tony Lindgren wrote:
>
> > Looks like with both patches applied I still also get this eventually:
> >
> > =
> > [ BUG: bad unlo
* Tony Lindgren <t...@atomide.com> [160509 08:15]:
> * Eric Dumazet <eduma...@google.com> [160509 07:16]:
> > On Mon, May 9, 2016 at 12:32 AM, Al Viro <v...@zeniv.linux.org.uk> wrote:
> > > On Sun, May 08, 2016 at 03:16:29PM +0100, Al Viro wrote:
> >
* Eric Dumazet [160509 07:16]:
> On Mon, May 9, 2016 at 12:32 AM, Al Viro wrote:
> > On Sun, May 08, 2016 at 03:16:29PM +0100, Al Viro wrote:
> >
> >> Very strange. We grab that rwsem at the entry into nfs_call_unlink()
> >> and then either release
c mode, but with 2 PHYs using phy-handle properties [1].
>
> Nicolas Chauvet reported testing v2 on an HP t410 (dm8148).
>
> Markus Brunner reported testing v1 on the following [2]:
> - emac0 with phy_id and emac1 with fixed phy
> - emac0 with phy-handle and emac1 with fixed phy
h dedicated linux-omap list.
> >
> > Also add Mugunthan as maintainer and myself as the reviewer.
> >
> > Cc: "David S. Miller" <da...@davemloft.net>
> > Cc: Tony Lindgren <t...@atomide.com>
> > Cc: Mugunthan V N <mugunthan...@ti.com&
* Grygorii Strashko <grygorii.stras...@ti.com> [160420 09:19]:
> On 04/20/2016 05:23 PM, Tony Lindgren wrote:
> > * Grygorii Strashko <grygorii.stras...@ti.com> [160420 04:26]:
> >> Add record for TI Ethernet Switch Driver CPSW/CPDMA/MDIO HW
> >> (am33/am43/a
* Hannes Frederic Sowa <han...@stressinduktion.org> [160420 08:24]:
> Hi,
>
> On 20.04.2016 17:01, Tony Lindgren wrote:
> > Looks like commit fafc4e1ea1a4 ("sock: tigthen lockdep checks for
> > sock_owned_by_user") in next causes a regression at least
; data->slaves.
Missing Signed-off-by?
This gets cpsw boards working in next for me again:
Tested-by: Tony Lindgren <t...@atomide.com>
* Tony Lindgren <t...@atomide.com> [160420 08:02]:
> Hi,
>
> Looks like commit fafc4e1ea1a4 ("sock: tigthen lockdep checks for
> sock_owned_by_user") in next causes a regression at least for
> smsc911x with CONFIG_LOCKDEP. It keeps spamming with the following
>
Hi,
Looks like commit fafc4e1ea1a4 ("sock: tigthen lockdep checks for
sock_owned_by_user") in next causes a regression at least for
smsc911x with CONFIG_LOCKDEP. It keeps spamming with the following
message. Any ideas?
Regards,
Tony
8<
WARNING: CPU: 0 PID: 0 at
* Grygorii Strashko [160420 04:26]:
> Add record for TI Ethernet Switch Driver CPSW/CPDMA/MDIO HW
> (am33/am43/am57/dr7/davinci) to ensure that related patches
> will go through dedicated linux-omap list.
>
> Also add Mugunthan as maintainer and myself as the reviewer.
han...@ti.com>
Signed-off-by: Tony Lindgren <t...@atomide.com>
---
drivers/net/ethernet/ti/cpsw-common.c | 3 +++
1 file changed, 3 insertions(+)
--- a/drivers/net/ethernet/ti/cpsw-common.c
+++ b/drivers/net/ethernet/ti/cpsw-common.c
@@ -78,6 +78,9 @@ static int cpsw_am33xx_cm_get_macid(s
>
Ethernet works for me with this patch:
Tested-by: Tony Lindgren <t...@atomide.com>
> ---
> drivers/net/ethernet/ti/davinci_emac.c | 8 ++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/net/ethernet/ti/davinci_emac.c
> b/drivers/net/
and ethernet becomes functional.
> Verified this on DRA72x with pcf as module and ramdisk.
>
> Signed-off-by: Mugunthan V N <mugunthan...@ti.com>
Acked-by: Tony Lindgren <t...@atomide.com>
> ---
>
> Changes from v2:
> * Used mode-gpios, so that the driver is generic e
* Mugunthan V N [150902 23:05]:
> In DRA72x EVM, by default slave 1 is connected to the onboard
> phy, but slave 2 pins are also muxed with video input module
> which is controlled by pcf857x gpio and currently to select slave
> 0 to connect to phy gpio hogging is used, but
* Mugunthan V N [150901 04:28]:
> --- a/Documentation/devicetree/bindings/net/cpsw.txt
> +++ b/Documentation/devicetree/bindings/net/cpsw.txt
> @@ -26,6 +26,9 @@ Optional properties:
> - dual_emac : Specifies Switch to act as Dual EMAC
> - syscon :
The interrupt handler may not be available when smsc911x probes if the
interrupt handler is a GPIO controller for example. Let's fix that
by adding handling for -EPROBE_DEFER.
Cc: Steve Glendinning steve.glendinn...@shawell.net
Signed-off-by: Tony Lindgren t...@atomide.com
---
drivers/net
* Guenter Roeck li...@roeck-us.net [150826 11:37]:
On 08/26/2015 10:04 AM, Tony Lindgren wrote:
Hi,
* Guenter Roeck li...@roeck-us.net [150817 13:48]:
Commit 0b50dc4fc971 (Convert smsc911x to use ACPI as well as DT) makes
the call to smsc911x_probe_config() unconditional, and no longer
Hi,
* Guenter Roeck li...@roeck-us.net [150817 13:48]:
Commit 0b50dc4fc971 (Convert smsc911x to use ACPI as well as DT) makes
the call to smsc911x_probe_config() unconditional, and no longer fails if
there is no device node. device_get_phy_mode() is called unconditionally,
and if there is no
* Jeremy Linton jeremy.lin...@arm.com [150826 10:35]:
On 08/26/2015 12:04 PM, Tony Lindgren wrote:
* Guenter Roeck li...@roeck-us.net [150817 13:48]:
Commit 0b50dc4fc971 (Convert smsc911x to use ACPI as well as DT) makes
Looks like this change makes at least omap boards using smsc911x
fail
* Guenter Roeck li...@roeck-us.net [150826 10:40]:
Hi Tony,
On 08/26/2015 10:04 AM, Tony Lindgren wrote:
Hi,
* Guenter Roeck li...@roeck-us.net [150817 13:48]:
Commit 0b50dc4fc971 (Convert smsc911x to use ACPI as well as DT) makes
the call to smsc911x_probe_config() unconditional
.
Thanks this fixes smsc911x regression in Linux next for me with FDT:
Tested-by: Tony Lindgren t...@atomide.com
drivers/net/ethernet/smsc/smsc911x.c | 14 --
1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/drivers/net/ethernet/smsc/smsc911x.c
b/drivers/net/ethernet
* Guenter Roeck li...@roeck-us.net [150826 13:58]:
Hi Tony,
On 08/26/2015 01:16 PM, Tony Lindgren wrote:
[ ... ]
We may need two separate patches, one to fix up device_property_read_u32()
to return -ENXIO, and one to fix smsc911x_probe_config() to ignore the error
from
:
Acked-by: Tony Lindgren t...@atomide.com
---
arch/arm/boot/dts/dra7.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/dra7.dtsi b/arch/arm/boot/dts/dra7.dtsi
index 8f1e25b..b4fdd10 100644
--- a/arch/arm/boot/dts/dra7.dtsi
+++ b/arch/arm/boot/dts/dra7
* Mugunthan V N mugunthan...@ti.com [150812 02:56]:
CPSW driver has been updated with compatibles for enabling errata
workarounds. So updating cpsw compatibles.
Signed-off-by: Mugunthan V N mugunthan...@ti.com
This too:
Acked-by: Tony Lindgren t...@atomide.com
---
arch/arm/boot/dts
66 matches
Mail list logo