netdev - RE: [PATCH V2 1/2] can: flexcan: disable runtime PM if register flexcandev failed

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Fri, 3 Jan 2020 07:37:37 +0000
From:   Joakim Zhang <qiangqing.zhang@....com>
To:     Sean Nyekjaer <sean@...nix.com>,
        "mkl@...gutronix.de" <mkl@...gutronix.de>,
        "linux-can@...r.kernel.org" <linux-can@...r.kernel.org>
CC:     dl-linux-imx <linux-imx@....com>,
        "netdev@...r.kernel.org" <netdev@...r.kernel.org>
Subject: RE: [PATCH V2 1/2] can: flexcan: disable runtime PM if register
 flexcandev failed


> -----Original Message-----
> From: Joakim Zhang <qiangqing.zhang@....com>
> Sent: 2019年12月19日 13:00
> To: Sean Nyekjaer <sean@...nix.com>; mkl@...gutronix.de;
> linux-can@...r.kernel.org
> Cc: dl-linux-imx <linux-imx@....com>; netdev@...r.kernel.org
> Subject: RE: [PATCH V2 1/2] can: flexcan: disable runtime PM if register
> flexcandev failed
> 
> 
> > -----Original Message-----
> > From: Sean Nyekjaer <sean@...nix.com>
> > Sent: 2019年12月18日 22:04
> > To: Joakim Zhang <qiangqing.zhang@....com>; mkl@...gutronix.de;
> > linux-can@...r.kernel.org
> > Cc: dl-linux-imx <linux-imx@....com>; netdev@...r.kernel.org
> > Subject: Re: [PATCH V2 1/2] can: flexcan: disable runtime PM if
> > register flexcandev failed
> >
> >
> >
> > On 17/12/2019 07.36, Joakim Zhang wrote:
> > >
> > > Hi Sean,
> > >
> > > Have you found time to test this patch set? Thanks :-)
> > >
> > > Best Regards,
> > > Joakim Zhang
> > >
> >
> > Hi Joakim
> >
> > Sorry for the delay :)
> >
> > I have tested this patchset and found no issues...
> Thanks a lot for your test, Sean :-)
> 
> > Just a heads up when adding "ChangeLog:" do it under the "---" and
> > above the diff. That way the ChangeLog doesn't end up in the commit msg...
> I will keep in mind, pay attention to it next time.
> 
> Best Regards,
> Joakim Zhang
> > /Sean
> >

Hi Marc,

How about this patch set? Sean has already test it.

And when you plan to send Flexcan FD related patched to mainline, since it is pending on linux-can-next/flexcan branch for a long time. Could it go into v5.6 kernel? Thanks a lot :-)

Marc, Sean, you are all CAN experts. A question confused me for a long time, is that why CAN must need a transceiver to convert digital signal to differential signal? Why CAN cannot communicate with digital signal?
Could you give me a simple explanation? Thanks.

Best Regards,
Joakim Zhang