RE: [PATCH net v3 4/4] rtase: Add defines for hardware version id
From: Justin Lai
Date: Tue Nov 19 2024 - 02:23:51 EST
>
> On Mon, Nov 18, 2024 at 12:08:28PM +0800, Justin Lai wrote:
> > Add defines for hardware version id.
> >
> > Signed-off-by: Justin Lai <justinlai0215@xxxxxxxxxxx>
> > ---
> > drivers/net/ethernet/realtek/rtase/rtase.h | 5 ++++-
> > drivers/net/ethernet/realtek/rtase/rtase_main.c | 12 ++++++------
> > 2 files changed, 10 insertions(+), 7 deletions(-)
> >
>
> The patch is addressed to the "net" tree, but "Fixes" tag is missing.
> Also, the patch does not look like a bugfix it's rather an improvement of coding
> style with no functional changes. That's why I doubt it should go to the "net"
> tree.
>
> Thanks,
> Michal
Hi Michal,
This patch introduces multiple defines for hardware version IDs, rather
than addressing any issues related to the function logic, which is why it
does not include a "Fixes:" tag. The reason for isolating this change in a
separate patch is to maintain a "single patch, single purpose" approach.
Additionally, these defines will be used in other patches within the same
patch set, which is why they are included in this patch set as a whole.
Justin