Re: 8250-men-mcb: fix signed/unsigned confusion

Re: 8250-men-mcb: fix signed/unsigned confusion

[Pavel Machek]

[-- Attachment #1: Type: text/plain, Size: 1096 bytes --]

On Fri 2019-11-15 23:09:42, Greg KH wrote:
> On Fri, Nov 15, 2019 at 01:52:34PM +0100, Pavel Machek wrote:
> > Hi!
> > 
> > Commit 5210a3a722543fc25b8830d2236dcbe2c8178468 fixes part of
> > signed/unsigned confusion, but not all of it.
> > 
> > 8250-men-mcb: fix error checking when get_num_ports returns -ENODEV
> > Upstream commit f50b6805dbb993152025ec04dea094c40cc93a0c
> > 
> > Fix function returning -ENODEV to signed prototype, and make error
> > check consistent between two functions.
> > 
> > Signed-off-by: Pavel Machek <pavel@denx.de>
> > 
> 
> Hi,
> 
> This is the friendly patch-bot of Greg Kroah-Hartman.  You have sent him
> a patch that has triggered this response.  He used to manually
> respond

Not sure what is supposed to be friendly about this bot.

Anyway, it would be nice if authors of
5210a3a722543fc25b8830d2236dcbe2c8178468 picked this up. I have no way
to test it.

					 	     	   Pavel

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

Re: 8250-men-mcb: fix signed/unsigned confusion

[Greg KH]

On Fri, Nov 15, 2019 at 10:13:51PM +0100, Pavel Machek wrote:
> On Fri 2019-11-15 23:09:42, Greg KH wrote:
> > On Fri, Nov 15, 2019 at 01:52:34PM +0100, Pavel Machek wrote:
> > > Hi!
> > > 
> > > Commit 5210a3a722543fc25b8830d2236dcbe2c8178468 fixes part of
> > > signed/unsigned confusion, but not all of it.
> > > 
> > > 8250-men-mcb: fix error checking when get_num_ports returns -ENODEV
> > > Upstream commit f50b6805dbb993152025ec04dea094c40cc93a0c
> > > 
> > > Fix function returning -ENODEV to signed prototype, and make error
> > > check consistent between two functions.
> > > 
> > > Signed-off-by: Pavel Machek <pavel@denx.de>
> > > 
> > 
> > Hi,
> > 
> > This is the friendly patch-bot of Greg Kroah-Hartman.  You have sent him
> > a patch that has triggered this response.  He used to manually
> > respond
> 
> Not sure what is supposed to be friendly about this bot.

How was it not?  Did it not say exactly what was wrong with the patch
and how to change it to make it acceptable?

> Anyway, it would be nice if authors of
> 5210a3a722543fc25b8830d2236dcbe2c8178468 picked this up. I have no way
> to test it.

As the bot said, this patch is not acceptable as-is, sorry.  There's no
way it can be 'picked up' in this format.

thanks,

greg k-h