2023-06-20 11:06:30

by Shiji Yang

[permalink] [raw]
Subject: [PATCH 0/2] mips: pci-mt7620: reduce unnecessary error logs

These patches silence some mt7620 PCIe driver error messeges by
removing the useless debugging codes and replacing incorrectly
used 'dev_err()' with 'dev_info()'.

Shiji Yang (2):
mips: pci-mt7620: do not print NFTS register value as error log
mips: pci-mt7620: use dev_info() to log PCIe device detection result

arch/mips/pci/pci-mt7620.c | 7 ++-----
1 file changed, 2 insertions(+), 5 deletions(-)

--
2.30.2



2023-06-20 11:43:06

by Sergio Paracuellos

[permalink] [raw]
Subject: Re: [PATCH 0/2] mips: pci-mt7620: reduce unnecessary error logs

On Tue, Jun 20, 2023 at 12:48 PM Shiji Yang <[email protected]> wrote:
>
> These patches silence some mt7620 PCIe driver error messeges by
> removing the useless debugging codes and replacing incorrectly
> used 'dev_err()' with 'dev_info()'.
>
> Shiji Yang (2):
> mips: pci-mt7620: do not print NFTS register value as error log
> mips: pci-mt7620: use dev_info() to log PCIe device detection result
>
> arch/mips/pci/pci-mt7620.c | 7 ++-----
> 1 file changed, 2 insertions(+), 5 deletions(-)

For the series:
Reviewed-by: Sergio Paracuellos <[email protected]>

I'd really love to have this driver ported to PCI_DRIVERS_GENERIC
moved to 'drivers/pci/controller/pcie-mt7621.c' since they are pretty
similar just changing that mt7620 and mt7628 init the hardware
different and only use one virtual bridge. My problem is that I don't
have and cannot find hardware to test. Do you have interest in helping
me to do this?

Thanks,
Sergio Paracuellos

>
> --
> 2.30.2
>

2023-06-21 01:35:30

by Shiji Yang

[permalink] [raw]
Subject: Re: [PATCH 0/2] mips: pci-mt7620: reduce unnecessary error logs

>On Tue, Jun 20, 2023 at 12:48 PM Shiji Yang <[email protected]> wrote:
>>
>> These patches silence some mt7620 PCIe driver error messeges by
>> removing the useless debugging codes and replacing incorrectly
>> used 'dev_err()' with 'dev_info()'.
>>
>> Shiji Yang (2):
>> mips: pci-mt7620: do not print NFTS register value as error log
>> mips: pci-mt7620: use dev_info() to log PCIe device detection result
>>
>> arch/mips/pci/pci-mt7620.c | 7 ++-----
>> 1 file changed, 2 insertions(+), 5 deletions(-)
>
>For the series:
>Reviewed-by: Sergio Paracuellos <[email protected]>
>
>I'd really love to have this driver ported to PCI_DRIVERS_GENERIC
>moved to 'drivers/pci/controller/pcie-mt7621.c' since they are pretty
>similar just changing that mt7620 and mt7628 init the hardware
>different and only use one virtual bridge. My problem is that I don't
>have and cannot find hardware to test. Do you have interest in helping
>me to do this?

Thanks,
Sergio Paracuellos

Thank you for rewriting these drivers! I am glad to help test them. Please
let me know at any time when they are ready.

Best regards,
Shiji Yang

2023-06-21 07:10:36

by Sergio Paracuellos

[permalink] [raw]
Subject: Re: [PATCH 0/2] mips: pci-mt7620: reduce unnecessary error logs

On Wed, Jun 21, 2023 at 3:04 AM Shiji Yang <[email protected]> wrote:
>
> >On Tue, Jun 20, 2023 at 12:48 PM Shiji Yang <[email protected]> wrote:
> >>
> >> These patches silence some mt7620 PCIe driver error messeges by
> >> removing the useless debugging codes and replacing incorrectly
> >> used 'dev_err()' with 'dev_info()'.
> >>
> >> Shiji Yang (2):
> >> mips: pci-mt7620: do not print NFTS register value as error log
> >> mips: pci-mt7620: use dev_info() to log PCIe device detection result
> >>
> >> arch/mips/pci/pci-mt7620.c | 7 ++-----
> >> 1 file changed, 2 insertions(+), 5 deletions(-)
> >
> >For the series:
> >Reviewed-by: Sergio Paracuellos <[email protected]>
> >
> >I'd really love to have this driver ported to PCI_DRIVERS_GENERIC
> >moved to 'drivers/pci/controller/pcie-mt7621.c' since they are pretty
> >similar just changing that mt7620 and mt7628 init the hardware
> >different and only use one virtual bridge. My problem is that I don't
> >have and cannot find hardware to test. Do you have interest in helping
> >me to do this?
>
> Thanks,
> Sergio Paracuellos
>
> Thank you for rewriting these drivers! I am glad to help test them. Please
> let me know at any time when they are ready.

Thanks! I need to build time for this but I will do it for sure at any
time and let you know.

Best regards,
Sergio Paracuellos
>
> Best regards,
> Shiji Yang

2023-06-21 14:27:57

by Thomas Bogendoerfer

[permalink] [raw]
Subject: Re: [PATCH 0/2] mips: pci-mt7620: reduce unnecessary error logs

On Tue, Jun 20, 2023 at 06:43:21PM +0800, Shiji Yang wrote:
> These patches silence some mt7620 PCIe driver error messeges by
> removing the useless debugging codes and replacing incorrectly
> used 'dev_err()' with 'dev_info()'.
>
> Shiji Yang (2):
> mips: pci-mt7620: do not print NFTS register value as error log
> mips: pci-mt7620: use dev_info() to log PCIe device detection result
>
> arch/mips/pci/pci-mt7620.c | 7 ++-----
> 1 file changed, 2 insertions(+), 5 deletions(-)

series applied to mips-next.

Thomas.

--
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea. [ RFC1925, 2.3 ]