Re: [PATCH 02/11] media: vsp1: use kernel __packed for structures

2018-03-13 Thread Kieran Bingham
Hi David, Just for reference here, I've posted a v2 of this patch now titled: [PATCH v2 02/11] media: vsp1: Remove packed attributes from aligned structures which removes the attributes instead of modifying them. Thanks for the pointers! Regards Kieran On 13/03/18 15:03, Kieran Bingham

Re: [PATCH 02/11] media: vsp1: use kernel __packed for structures

2018-03-13 Thread Kieran Bingham
Hi David, Just for reference here, I've posted a v2 of this patch now titled: [PATCH v2 02/11] media: vsp1: Remove packed attributes from aligned structures which removes the attributes instead of modifying them. Thanks for the pointers! Regards Kieran On 13/03/18 15:03, Kieran Bingham

Re: [PATCH 02/11] media: vsp1: use kernel __packed for structures

2018-03-13 Thread Kieran Bingham
Hi David, On 13/03/18 13:38, David Laight wrote: > From: Kieran Bingham [mailto:kieran.bingham+rene...@ideasonboard.com] >> On 13/03/18 11:20, David Laight wrote: >>> From: Kieran Bingham Sent: 09 March 2018 22:04 The kernel provides a __packed definition to abstract away from the

Re: [PATCH 02/11] media: vsp1: use kernel __packed for structures

2018-03-13 Thread Kieran Bingham
Hi David, On 13/03/18 13:38, David Laight wrote: > From: Kieran Bingham [mailto:kieran.bingham+rene...@ideasonboard.com] >> On 13/03/18 11:20, David Laight wrote: >>> From: Kieran Bingham Sent: 09 March 2018 22:04 The kernel provides a __packed definition to abstract away from the

RE: [PATCH 02/11] media: vsp1: use kernel __packed for structures

2018-03-13 Thread David Laight
From: Kieran Bingham [mailto:kieran.bingham+rene...@ideasonboard.com] > On 13/03/18 11:20, David Laight wrote: > > From: Kieran Bingham > >> Sent: 09 March 2018 22:04 > >> The kernel provides a __packed definition to abstract away from the > >> compiler specific attributes tag. > >> > >> Convert

RE: [PATCH 02/11] media: vsp1: use kernel __packed for structures

2018-03-13 Thread David Laight
From: Kieran Bingham [mailto:kieran.bingham+rene...@ideasonboard.com] > On 13/03/18 11:20, David Laight wrote: > > From: Kieran Bingham > >> Sent: 09 March 2018 22:04 > >> The kernel provides a __packed definition to abstract away from the > >> compiler specific attributes tag. > >> > >> Convert

Re: [PATCH 02/11] media: vsp1: use kernel __packed for structures

2018-03-13 Thread Kieran Bingham
Hi David, On 13/03/18 11:20, David Laight wrote: > From: Kieran Bingham >> Sent: 09 March 2018 22:04 >> The kernel provides a __packed definition to abstract away from the >> compiler specific attributes tag. >> >> Convert all packed structures in VSP1 to use it. >> >> Signed-off-by: Kieran

Re: [PATCH 02/11] media: vsp1: use kernel __packed for structures

2018-03-13 Thread Kieran Bingham
Hi David, On 13/03/18 11:20, David Laight wrote: > From: Kieran Bingham >> Sent: 09 March 2018 22:04 >> The kernel provides a __packed definition to abstract away from the >> compiler specific attributes tag. >> >> Convert all packed structures in VSP1 to use it. >> >> Signed-off-by: Kieran

RE: [PATCH 02/11] media: vsp1: use kernel __packed for structures

2018-03-13 Thread David Laight
From: Kieran Bingham > Sent: 09 March 2018 22:04 > The kernel provides a __packed definition to abstract away from the > compiler specific attributes tag. > > Convert all packed structures in VSP1 to use it. > > Signed-off-by: Kieran Bingham > --- >

RE: [PATCH 02/11] media: vsp1: use kernel __packed for structures

2018-03-13 Thread David Laight
From: Kieran Bingham > Sent: 09 March 2018 22:04 > The kernel provides a __packed definition to abstract away from the > compiler specific attributes tag. > > Convert all packed structures in VSP1 to use it. > > Signed-off-by: Kieran Bingham > --- > drivers/media/platform/vsp1/vsp1_dl.c | 6