2021-08-13 14:19:22

by Qian Cai

[permalink] [raw]
Subject: Re: [Internet]Re: [PATCH v5] arm pl011 serial: support multi-irq request



On 8/12/2021 11:32 PM, tombinfan(范兵) wrote:
> hello,
>
> I have sent the modified patch to the following mailboxes. THX
>
> [email protected]
> [email protected]
> [email protected]

Thanks for the pointer. It is good to get some reviews from Arm
people in the first place, so we don't break Arm like this.

Anyway, for anyone who might be watching, here is the new patch,

https://lore.kernel.org/linux-serial/[email protected]/

Unfortunately, I saw Greg mentioned that it was not based on
tty-next, so not something I can apply easily on linux-next here
as well.


2021-08-13 15:07:55

by Greg Kroah-Hartman

[permalink] [raw]
Subject: Re: [Internet]Re: [PATCH v5] arm pl011 serial: support multi-irq request

On Fri, Aug 13, 2021 at 09:42:52AM -0400, Qian Cai wrote:
>
>
> On 8/12/2021 11:32 PM, tombinfan(范兵) wrote:
> > hello,
> >
> > I have sent the modified patch to the following mailboxes. THX
> >
> > [email protected]
> > [email protected]
> > [email protected]
>
> Thanks for the pointer. It is good to get some reviews from Arm
> people in the first place, so we don't break Arm like this.
>
> Anyway, for anyone who might be watching, here is the new patch,
>
> https://lore.kernel.org/linux-serial/[email protected]/
>
> Unfortunately, I saw Greg mentioned that it was not based on
> tty-next, so not something I can apply easily on linux-next here
> as well.

That is because the code in tty-next (and linux-next), looks to already
do exactly what you submitted, right?

So is this working now for everyone in linux-next?

thanks,

greg k-h

2021-08-13 15:13:34

by Robin Murphy

[permalink] [raw]
Subject: Re: [Internet]Re: [PATCH v5] arm pl011 serial: support multi-irq request

On 2021-08-13 16:01, gregkh wrote:
> On Fri, Aug 13, 2021 at 09:42:52AM -0400, Qian Cai wrote:
>>
>>
>> On 8/12/2021 11:32 PM, tombinfan(范兵) wrote:
>>> hello,
>>>
>>> I have sent the modified patch to the following mailboxes. THX
>>>
>>> [email protected]
>>> [email protected]
>>> [email protected]
>>
>> Thanks for the pointer. It is good to get some reviews from Arm
>> people in the first place, so we don't break Arm like this.
>>
>> Anyway, for anyone who might be watching, here is the new patch,
>>
>> https://lore.kernel.org/linux-serial/[email protected]/
>>
>> Unfortunately, I saw Greg mentioned that it was not based on
>> tty-next, so not something I can apply easily on linux-next here
>> as well.
>
> That is because the code in tty-next (and linux-next), looks to already
> do exactly what you submitted, right?
>
> So is this working now for everyone in linux-next?

AFAICS commit b0819465be8b in linux-next still results in
amba_device-specific code being called from sbsa_uart_startup() and
sbsa_uart_shutdown(), which is what blows up.

Robin.

2021-08-13 15:25:45

by Greg Kroah-Hartman

[permalink] [raw]
Subject: Re: [Internet]Re: [PATCH v5] arm pl011 serial: support multi-irq request

On Fri, Aug 13, 2021 at 04:09:50PM +0100, Robin Murphy wrote:
> On 2021-08-13 16:01, gregkh wrote:
> > On Fri, Aug 13, 2021 at 09:42:52AM -0400, Qian Cai wrote:
> > >
> > >
> > > On 8/12/2021 11:32 PM, tombinfan(范兵) wrote:
> > > > hello,
> > > >
> > > > I have sent the modified patch to the following mailboxes. THX
> > > >
> > > > [email protected]
> > > > [email protected]
> > > > [email protected]
> > >
> > > Thanks for the pointer. It is good to get some reviews from Arm
> > > people in the first place, so we don't break Arm like this.
> > >
> > > Anyway, for anyone who might be watching, here is the new patch,
> > >
> > > https://lore.kernel.org/linux-serial/[email protected]/
> > >
> > > Unfortunately, I saw Greg mentioned that it was not based on
> > > tty-next, so not something I can apply easily on linux-next here
> > > as well.
> >
> > That is because the code in tty-next (and linux-next), looks to already
> > do exactly what you submitted, right?
> >
> > So is this working now for everyone in linux-next?
>
> AFAICS commit b0819465be8b in linux-next still results in
> amba_device-specific code being called from sbsa_uart_startup() and
> sbsa_uart_shutdown(), which is what blows up.

Ick, ok, can someone send me a revert with this information in it
please?

thanks,

greg k-h

2021-08-13 15:31:25

by Qian Cai

[permalink] [raw]
Subject: Re: [Internet]Re: [PATCH v5] arm pl011 serial: support multi-irq request



On 8/13/2021 11:14 AM, gregkh wrote:
>> AFAICS commit b0819465be8b in linux-next still results in
>> amba_device-specific code being called from sbsa_uart_startup() and
>> sbsa_uart_shutdown(), which is what blows up.
>
> Ick, ok, can someone send me a revert with this information in it
> please?

I guess Bin won't be around to response in his timezone on Friday, so
I'll send a revert shortly.