2023-04-18 09:18:58

by Dmitry Baryshkov

[permalink] [raw]
Subject: Re: [PATCH v2] Bluetooth: btusb: Add WCN6855 devcoredump support

On Tue, 18 Apr 2023 at 12:07, Tim Jiang (QUIC) <[email protected]> wrote:
>
> Hi Dmitry:
>
> -----Original Message-----
> From: Dmitry Baryshkov <[email protected]>
> Sent: Tuesday, April 18, 2023 5:03 PM
> To: Tim Jiang (QUIC) <[email protected]>
> Cc: [email protected]; [email protected]; [email protected]; [email protected]; Balakrishna Godavarthi (QUIC) <[email protected]>; Hemant Gupta (QUIC) <[email protected]>; [email protected]
> Subject: Re: [PATCH v2] Bluetooth: btusb: Add WCN6855 devcoredump support
>
> On Tue, 18 Apr 2023 at 04:02, Tim Jiang (QUIC) <[email protected]> wrote:
> >
> > Hi Dmitry:
> >
> > -----Original Message-----
> > From: Dmitry Baryshkov <[email protected]>
> > Sent: Saturday, April 15, 2023 5:03 AM
> > To: Tim Jiang (QUIC) <[email protected]>
> > Cc: [email protected]; [email protected];
> > [email protected]; [email protected];
> > Balakrishna Godavarthi (QUIC) <[email protected]>; Hemant
> > Gupta (QUIC) <[email protected]>; [email protected]
> > Subject: Re: [PATCH v2] Bluetooth: btusb: Add WCN6855 devcoredump
> > support
> >
> > On Thu, 13 Apr 2023 at 09:44, Tim Jiang <[email protected]> wrote:
> > >
> > > WCN6855 will report memdump via ACL data or HCI event when it get
> > > crashed, so we collect memdump to debug firmware.
> >
> > Is it applicable only to wcn6855 or to some of earlier chips too?
> > [Tim] Also applicable to earlier chips , but currently google only
> > require us to support wcn6855
>
> Since upstream is not a google kernel, please enable this feature for all relevant chipsets.
> [Tim] agreed , but the title of this gerrit unchanged, I raise a new gerrit for other relevant chipsets , is OK?

There is no gerrit here.

Also, is there any chance you can fix your email client to stop
putting old headers at the top of the email?

>
> >
> > >
> > > Signed-off-by: Tim Jiang <[email protected]>
> > > ---
> > > drivers/bluetooth/btusb.c | 222
> > > ++++++++++++++++++++++++++++++++++++++
> > > 1 file changed, 222 insertions(+)
>
>
>
>
> --
> With best wishes
> Dmitry



--
With best wishes
Dmitry


2023-04-18 09:26:05

by Tim Jiang

[permalink] [raw]
Subject: RE: [PATCH v2] Bluetooth: btusb: Add WCN6855 devcoredump support

Hi Dmitry:

> > On Thu, 13 Apr 2023 at 09:44, Tim Jiang <[email protected]> wrote:
> > >
> > > WCN6855 will report memdump via ACL data or HCI event when it get
> > > crashed, so we collect memdump to debug firmware.
> >
> > Is it applicable only to wcn6855 or to some of earlier chips too?
> > [Tim] Also applicable to earlier chips , but currently google only
> > require us to support wcn6855
>
> Since upstream is not a google kernel, please enable this feature for all relevant chipsets.
> [Tim] agreed , but the title of this gerrit unchanged, I raise a new gerrit for other relevant chipsets , is OK?

There is no gerrit here.

Also, is there any chance you can fix your email client to stop putting old headers at the top of the email?
[Tim] sorry for confusion, I mean I will raise another new change for other relevant chipset, is OK ?
and I use office 365 to reply your email which will adding old headers automatically , unless I delete the old headers manually, thank you.

>
> >
> > >
> > > Signed-off-by: Tim Jiang <[email protected]>
> > > ---
> > > drivers/bluetooth/btusb.c | 222
> > > ++++++++++++++++++++++++++++++++++++++
> > > 1 file changed, 222 insertions(+)
>
>
>
>
> --
> With best wishes
> Dmitry



--
With best wishes
Dmitry