Re: [PATCH RESEND] net: phy: harmonize phy_id{,_mask} data type

2017-11-30 Thread David Miller
From: Richard Leitner Date: Mon, 27 Nov 2017 08:16:45 +0100 > From: Richard Leitner > > Previously phy_id was u32 and phy_id_mask was unsigned int. As the > phy_id_mask defines the important bits of the phy_id (and is therefore > the same size)

Re: [PATCH RESEND] net: phy: harmonize phy_id{,_mask} data type

2017-11-30 Thread David Miller
From: Richard Leitner Date: Mon, 27 Nov 2017 08:16:45 +0100 > From: Richard Leitner > > Previously phy_id was u32 and phy_id_mask was unsigned int. As the > phy_id_mask defines the important bits of the phy_id (and is therefore > the same size) these two variables should be the same data type.

Re: [PATCH RESEND] net: phy: harmonize phy_id{,_mask} data type

2017-11-27 Thread Richard Leitner
On 11/27/2017 02:50 PM, Andrew Lunn wrote: > On Mon, Nov 27, 2017 at 08:16:45AM +0100, Richard Leitner wrote: >> From: Richard Leitner >> >> Previously phy_id was u32 and phy_id_mask was unsigned int. As the >> phy_id_mask defines the important bits of the phy_id

Re: [PATCH RESEND] net: phy: harmonize phy_id{,_mask} data type

2017-11-27 Thread Richard Leitner
On 11/27/2017 02:50 PM, Andrew Lunn wrote: > On Mon, Nov 27, 2017 at 08:16:45AM +0100, Richard Leitner wrote: >> From: Richard Leitner >> >> Previously phy_id was u32 and phy_id_mask was unsigned int. As the >> phy_id_mask defines the important bits of the phy_id (and is therefore >> the same

Re: [PATCH RESEND] net: phy: harmonize phy_id{,_mask} data type

2017-11-27 Thread Andrew Lunn
On Mon, Nov 27, 2017 at 08:16:45AM +0100, Richard Leitner wrote: > From: Richard Leitner > > Previously phy_id was u32 and phy_id_mask was unsigned int. As the > phy_id_mask defines the important bits of the phy_id (and is therefore > the same size) these two

Re: [PATCH RESEND] net: phy: harmonize phy_id{,_mask} data type

2017-11-27 Thread Andrew Lunn
On Mon, Nov 27, 2017 at 08:16:45AM +0100, Richard Leitner wrote: > From: Richard Leitner > > Previously phy_id was u32 and phy_id_mask was unsigned int. As the > phy_id_mask defines the important bits of the phy_id (and is therefore > the same size) these two variables should be the same data

[PATCH RESEND] net: phy: harmonize phy_id{,_mask} data type

2017-11-26 Thread Richard Leitner
From: Richard Leitner Previously phy_id was u32 and phy_id_mask was unsigned int. As the phy_id_mask defines the important bits of the phy_id (and is therefore the same size) these two variables should be the same data type. Signed-off-by: Richard Leitner

[PATCH RESEND] net: phy: harmonize phy_id{,_mask} data type

2017-11-26 Thread Richard Leitner
From: Richard Leitner Previously phy_id was u32 and phy_id_mask was unsigned int. As the phy_id_mask defines the important bits of the phy_id (and is therefore the same size) these two variables should be the same data type. Signed-off-by: Richard Leitner Reviewed-by: Florian Fainelli