Re: [PATCH 5.15 96/98] hwmon: (occ) Remove sequence numbering and checksum calculation

From: Greg Kroah-Hartman
Date: Wed Jul 06 2022 - 03:03:01 EST


On Wed, Jul 06, 2022 at 06:43:47AM +0000, Joel Stanley wrote:
> On Tue, 5 Jul 2022 at 12:14, Greg Kroah-Hartman
> <gregkh@xxxxxxxxxxxxxxxxxxx> wrote:
> >
> > From: Eddie James <eajames@xxxxxxxxxxxxx>
> >
> > [ Upstream commit 908dbf0242e21dd95c69a1b0935814cd1abfc134 ]
> >
> > Checksumming of the request and sequence numbering is now done in the
> > OCC interface driver in order to keep unique sequence numbers. So
> > remove those in the hwmon driver. Also, add the command length to the
> > send_cmd function pointer, since the checksum must be placed in the
> > last two bytes of the command. The submit interface must receive the
> > exact size of the command - previously it could be rounded to the
> > nearest 8 bytes with no consequence.
> >
> > Signed-off-by: Eddie James <eajames@xxxxxxxxxxxxx>
> > Acked-by: Guenter Roeck <linux@xxxxxxxxxxxx>
> > Link: https://lore.kernel.org/r/20210721190231.117185-3-eajames@xxxxxxxxxxxxx
> > Signed-off-by: Joel Stanley <joel@xxxxxxxxx>
> > Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx>
>
> If this patch is being backported then we must also backport:
>
> 62f79f3d0eb9 ("fsi: occ: Force sequence numbering per OCC")
>
> I was only cc'd on this one so I assume that means 62f79f3d0eb9 is not
> already in the queue.

It was not, so I have now added it, thanks.

greg k-h