Re: [PATCH 02/16] media: i2c: rdacm20: Embedded 'serializer' field

2021-02-21 Thread Laurent Pinchart
Hi Jacopo, Thank you for the patch. On Tue, Feb 16, 2021 at 06:41:32PM +0100, Jacopo Mondi wrote: > There's no reason to allocate dynamically the 'serializer' field in > the driver structure. > > Embed the field and adjust all its users in the driver. > > Signed-off-by: Jacopo Mondi This

Re: [PATCH 02/16] media: i2c: rdacm20: Embedded 'serializer' field

2021-02-17 Thread Kieran Bingham
Hi Jacopo, On 16/02/2021 17:41, Jacopo Mondi wrote: > There's no reason to allocate dynamically the 'serializer' field in > the driver structure. > > Embed the field and adjust all its users in the driver. > > Signed-off-by: Jacopo Mondi > --- > drivers/media/i2c/rdacm20.c | 38

[PATCH 02/16] media: i2c: rdacm20: Embedded 'serializer' field

2021-02-16 Thread Jacopo Mondi
There's no reason to allocate dynamically the 'serializer' field in the driver structure. Embed the field and adjust all its users in the driver. Signed-off-by: Jacopo Mondi --- drivers/media/i2c/rdacm20.c | 38 - 1 file changed, 16 insertions(+), 22