On Mon, Feb 04, 2019 at 03:36:21PM -0800, Florian Fainelli wrote:
> Hi all,
>
> Based on discussion with Ido and feedback from Jakub there are clearly
> two classes of users that implement SWITCHDEV_ATTR_ID_PORT_PARENT_ID:
>
> - PF/VF drivers which typically only implement return the port's paren
From: Ajay Singh
Avoid use of static variable 'hif_deinit_lock' and move it as part of
wilc struct.
Signed-off-by: Ajay Singh
---
drivers/staging/wilc1000/host_interface.c | 20 +---
drivers/staging/wilc1000/wilc_wfi_netdevice.h | 2 ++
2 files changed, 11 insertions(+), 1
From: Ajay Singh
This series contains pending patches from #9 till #15 from previous
series[1].
Also added 5 new patches to avoid the use of unnecessary static
variables for few leftovers.
[1]. https://patchwork.kernel.org/cover/10794463/
https://patchwork.kernel.org/patch/10794479/
Ajay S
From: Ajay Singh
Make use of cookie information to pass to wpa_s and handle cookie value
received in the cfg80211_ops callbacks.
Signed-off-by: Ajay Singh
---
drivers/staging/wilc1000/host_interface.c | 72 +++
drivers/staging/wilc1000/host_interface.h | 13
From: Ajay Singh
Avoid use of interface name i.e 'wlan0' & 'p2p0' to check the interface
type in dev_state_ev_handler(). Now making use of netdev_ops and iface
type to know interface. Reorder the functions to avoid the forward
declaration after the above changes
Signed-off-by: Ajay Singh
---
d
From: Ajay Singh
Use random number to assign to cookie value.
Signed-off-by: Ajay Singh
---
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c
b/drivers/staging/wilc1000/wilc_w
From: Ajay Singh
Refactor linux_wlan_init_test_config() to use correct endianness for wid
values and remove unnecessary code.
Signed-off-by: Ajay Singh
---
drivers/staging/wilc1000/linux_wlan.c | 200 +---
drivers/staging/wilc1000/wilc_wlan_if.h | 1 -
2 files c
From: Ajay Singh
Avoid use of static variable 'wlan_channel' by moving it inside the wilc
structure.
Signed-off-by: Ajay Singh
---
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 36 ---
drivers/staging/wilc1000/wilc_wfi_netdevice.h | 1 +
2 files changed, 20 inser
From: Ajay Singh
Add constant qualifer for 'p2p_vendor_spec' & 'p2p_oui' static
variable because they are treated like constant values.
Signed-off-by: Ajay Singh
---
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/driver
From: Ajay Singh
Before calling an operation on net_device check if that interface is
available.
Signed-off-by: Ajay Singh
---
drivers/staging/wilc1000/linux_wlan.c | 12
1 file changed, 8 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/wilc1000/linux_wlan.c
b/drivers
From: Ajay Singh
Avoid the use of static variable and move it as part of wilc structure.
Signed-off-by: Ajay Singh
---
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 22 --
drivers/staging/wilc1000/wilc_wfi_netdevice.h | 1 +
2 files changed, 13 insertions(+), 10
From: Ajay Singh
Remove 'add_sta_param' structure as its not used now.
Signed-off-by: Ajay Singh
---
drivers/staging/wilc1000/host_interface.h | 11 ---
1 file changed, 11 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.h
b/drivers/staging/wilc1000/host_interface.h
From: Ajay Singh
Define local variable for 'srcadd' & 'bssid' static variables and use
ether_addr_copy() to copy value into them.
Signed-off-by: Ajay Singh
---
drivers/staging/wilc1000/linux_mon.c | 9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/wilc
From: Ajay Singh
Remove use of 'terminated_handle' variable and set the 'hif_drv' to
NULL once it's free.
Signed-off-by: Ajay Singh
---
drivers/staging/wilc1000/host_interface.c | 15 +++
1 file changed, 7 insertions(+), 8 deletions(-)
diff --git a/drivers/staging/wilc1000/host_in
On 04.02.19 09:53, Mark Brown wrote:
On Mon, Feb 04, 2019 at 09:34:56AM +1100, NeilBrown wrote:
It is extremely common in the kernel for a file to start
// SPDX-License-Identifier.
and to have that immediately followed by a comment lile:
/*
* .
*
Yes, there
On Mon, 4 Feb 2019 15:36:30 -0800, Florian Fainelli wrote:
> netdevsim only supports SWITCHDEV_ATTR_ID_PORT_PARENT_ID, which makes it a
> great candidate to be converted to use the ndo_get_port_parent_id() NDO
> instead of implementing switchdev_port_attr_get().
>
> Signed-off-by: Florian Fainell
On Mon, 4 Feb 2019 15:36:28 -0800, Florian Fainelli wrote:
> NFP only supports SWITCHDEV_ATTR_ID_PORT_PARENT_ID, which makes it a
> great candidate to be converted to use the ndo_get_port_parent_id() NDO
> instead of implementing switchdev_port_attr_get().
>
> Signed-off-by: Florian Fainelli
Ac
Hi Kalle,
On 2/4/2019 7:08 PM, Kalle Valo wrote:
> writes:
>
>> From: Ajay Singh
>>
>> Avoid use of static variable for monitor net_device and move it inside
>> wilc structure.
>>
>> Signed-off-by: Ajay Singh
>
> [...]
>
>> --- a/drivers/staging/wilc1000/linux_mon.c
>> +++ b/drivers/staging/
Sorry this patch isn't working, it's not possible to set BT.709
encoding, working on a fix for v2.
Steve
On 2/3/19 11:47 AM, Steve Longerbeam wrote:
The IC now supports BT.709 Y'CbCr encoding, in addition to existing BT.601
encoding, so allow both, for pipelines that route through the IC.
Re
netdevsim only supports SWITCHDEV_ATTR_ID_PORT_PARENT_ID, which makes it a
great candidate to be converted to use the ndo_get_port_parent_id() NDO
instead of implementing switchdev_port_attr_get().
Signed-off-by: Florian Fainelli
---
drivers/net/netdevsim/netdev.c | 22 ++
1
mlx5e only supports SWITCHDEV_ATTR_ID_PORT_PARENT_ID, which makes it a
great candidate to be converted to use the ndo_get_port_parent_id() NDO
instead of implementing switchdev_port_attr_get().
Signed-off-by: Florian Fainelli
---
.../net/ethernet/mellanox/mlx5/core/en_rep.c | 31 +++
mlxsw implements SWITCHDEV_ATTR_ID_PORT_PARENT_ID and we want to get rid
of switchdev_ops eventually, ease that migration by implementing a
ndo_get_port_parent_id() function which returns what
switchdev_port_attr_get() would do.
Signed-off-by: Florian Fainelli
---
drivers/net/ethernet/rocker/roc
DSA implements SWITCHDEV_ATTR_ID_PORT_PARENT_ID and we want to get rid
of switchdev_ops eventually, ease that migration by implementing a
ndo_get_port_parent_id() function which returns what
switchdev_port_attr_get() would do.
Signed-off-by: Florian Fainelli
---
net/dsa/slave.c | 18
Ocelot only supports SWITCHDEV_ATTR_ID_PORT_PARENT_ID as a valid
switchdev attribute getter, convert it to use ndo_get_port_parent_id()
and get rid of the switchdev_ops::switchdev_port_attr_get altogether.
Signed-off-by: Florian Fainelli
---
drivers/net/ethernet/mscc/ocelot.c | 33 --
Now that we have a dedicated NDO for getting a port's parent ID, get rid
of SWITCHDEV_ATTR_ID_PORT_PARENT_ID and convert all callers to use the
NDO exclusively. This is a preliminary change to getting rid of
switchdev_ops eventually.
Signed-off-by: Florian Fainelli
---
include/net/switchdev.h
ethsw implements SWITCHDEV_ATTR_ID_PORT_PARENT_ID and we want to get rid
of switchdev_ops eventually, ease that migration by implementing a
ndo_get_port_parent_id() function which returns what
switchdev_port_attr_get() would do.
Signed-off-by: Florian Fainelli
---
drivers/staging/fsl-dpaa2/ethsw
Hi all,
Based on discussion with Ido and feedback from Jakub there are clearly
two classes of users that implement SWITCHDEV_ATTR_ID_PORT_PARENT_ID:
- PF/VF drivers which typically only implement return the port's parent
ID, yet have to implement switchdev_port_attr_get() just for that
- Ether
NFP only supports SWITCHDEV_ATTR_ID_PORT_PARENT_ID, which makes it a
great candidate to be converted to use the ndo_get_port_parent_id() NDO
instead of implementing switchdev_port_attr_get().
Signed-off-by: Florian Fainelli
---
.../ethernet/netronome/nfp/nfp_net_common.c | 4 +---
.../net/eth
In preparation for getting rid of switchdev_ops, create a dedicated NDO
operation for getting the port's parent identifier. There are
essentially two classes of drivers that need to implement getting the
port's parent ID which are VF/PF drivers with a built-in switch, and
pure switchdev drivers suc
mlxsw implements SWITCHDEV_ATTR_ID_PORT_PARENT_ID and we want to get rid
of switchdev_ops eventually, ease that migration by implementing a
ndo_get_port_parent_id() function which returns what
switchdev_port_attr_get() would do.
Signed-off-by: Florian Fainelli
---
.../net/ethernet/mellanox/mlxsw
BNXT only supports SWITCHDEV_ATTR_ID_PORT_PARENT_ID, which makes it a
great candidate to be converted to use the ndo_get_port_parent_id() NDO
instead of implementing switchdev_port_attr_get(). The conversion is
straight forward here since the PF and VF code use the same getter.
Signed-off-by: Flor
Liquidio only supports SWITCHDEV_ATTR_ID_PORT_PARENT_ID, which makes it
a great candidate to be converted to use the ndo_get_port_parent_id()
NDO instead of implementing switchdev_port_attr_get().
Signed-off-by: Florian Fainelli
---
.../net/ethernet/cavium/liquidio/lio_main.c | 22
On Mon, Feb 04 2019, Mark Brown wrote:
> On Mon, Feb 04, 2019 at 09:34:56AM +1100, NeilBrown wrote:
>
>> It is extremely common in the kernel for a file to start
>>// SPDX-License-Identifier.
>
>> and to have that immediately followed by a comment lile:
>
>> /*
>> * .
>> *
"psecnetwork" is the address of &psecuritypriv->sec_bss. Since
->sec_bss isn't the first member of the struct that means that
"psecnetwork" can't be NULL.
We can just remove this dead code.
Signed-off-by: Dan Carpenter
---
drivers/staging/rtl8723bs/core/rtw_cmd.c | 8
1 file changed,
Hi Sakari,
[snip]
> > > > > fail_stop_pipeline:
> > > > > @@ -543,6 +546,8 @@ static void ipu3_vb2_stop_streaming(struct
> > > > vb2_queue *vq)
> > > > > dev_err(&imgu->pci_dev->dev,
> > > > > "failed to stop subdev streaming\n");
> > > > >
> > > > > + mut
From: Kimberly Brown Sent: Sunday, February 3, 2019
11:13 PM
>
> Counter values for per-channel interrupts and ring buffer full
> conditions are useful for investigating performance.
>
> Expose counters in sysfs for 2 types of guest to host interrupts:
> 1) Interrupts caused by the channel's ou
I think you sent to wrong address of Alistair on these and other patches.
Corrected address.
On Fri, Feb 01, 2019 at 06:39:03AM +0100, Hugo Lefeuvre wrote:
> simplify handle_vsoc_cond_wait (drivers/staging/android/vsoc.c) using newly
> added wait_event_freezable_hrtimeout helper and remove useles
On Mon, Feb 04, 2019 at 04:25:43AM -0800, Himadri Pandya wrote:
> struct rt_dot11d_info {
> - bool bEnabled;
> + bool b_enabled;
Please think about _what_ you are renaming. Does your new name make any
sense? Why was the "b" there in the first place? Why keep it?
These are not to be ju
From: Colin Ian King
There is a null check on pointer dev which implies it may be null, however
dev can never be null as it is set in rtl8192_usb_probe via the call
to usb_set_intfdata.
Detected by CoverityScan, CID#143078 ("Dereference after null check")
Fixes: 8fc8598e61f6 ("Staging: Added Re
On 03/02/2019 11:31, Dan Carpenter wrote:
> On Sat, Feb 02, 2019 at 10:56:27PM +, Colin King wrote:
>> From: Colin Ian King
>>
>> There is an earlier null check on pointer dev which implies it may be null,
>> however the assignment of pointer pref and the call to free_ieee82011 on
>> a null de
writes:
> From: Ajay Singh
>
> Avoid use of static variable for monitor net_device and move it inside
> wilc structure.
>
> Signed-off-by: Ajay Singh
[...]
> --- a/drivers/staging/wilc1000/linux_mon.c
> +++ b/drivers/staging/wilc1000/linux_mon.c
> @@ -18,8 +18,6 @@ struct wilc_wfi_radiotap_cb
Fix checkpatch warning: Avoid CamelCase by renaming following parameters of
function dot11d_update_country_ie
pTaddr -> p_taddr
CoutryIeLen -> coutry_ie_len
pCoutryIe -> p_coutry_ie
Signed-off-by: Himadri Pandya
---
drivers/staging/rtl8192e/dot11d.c | 14 +++---
d
Fix checkpatch warning: Avoid CameCase by renaming following local
variables of function dot11d_update_country_ie
NumTriples -> num_triples
MaxChnlNum -> max_chnl_num
pTriple -> p_triple
Signed-off-by: Himadri Pandya
---
drivers/staging/rtl8192e/dot11d.c | 30
Fix checkpatch warning: Avoid CamelCase by renaming following functions
cpMacAddr -> cp_mac_addr
Dot11d_Channelmap -> dot11d_channel_map
Dot11d_Reset -> dot11d_reset
Dot11d_UpdateCountryIe -> dot11d_update_country_ie
DOT11D_ScanComplete -> dot11d_scan_complet
Fix checkpatch warning: Avoid CamelCase by renaming following structure
members of struct rtllib_device
pDot11dInfo -> p_dot11d_info
bGlobalDomain -> b_global_domain
IbssStartChnl -> ibss_start_chnl
Signed-off-by: Himadri Pandya
---
drivers/staging/rtl8192e/dot11d.c
Fix checkpatch warning: Avoid CamelCase by renaming following macro
arguments
__pIeeeDev -> __p_ieee_dev
__pTa -> __p_ta
Signed-off-by: Himadri Pandya
---
drivers/staging/rtl8192e/dot11d.h | 30 +++---
1 file changed, 15 insertions(+), 15 deletions(-)
dif
Fix checkpacth warning: Avoid CamelCase by renaming following structure
members of struct chnl_txpow_triple
FirstChnl -> first_chnl
NumChnls -> num_chnls
MaxTxPwrInDbm -> max_tx_pwr_in_dbm
and members for struct rt_dot11d_info
bEnabled -> b_enabled
CountryIe
Hi Greg,
On 2/4/2019 5:09 PM, Greg KH wrote:
> On Mon, Feb 04, 2019 at 12:47:23PM +0300, Dan Carpenter wrote:
>> On Mon, Feb 04, 2019 at 09:42:36AM +, ajay.kat...@microchip.com wrote:
>>>
>>>
>>> On 2/4/2019 1:43 PM, Dan Carpenter wrote:
On Sat, Feb 02, 2019 at 07:17:13PM +, ajay.kat.
Hello Dear
Am a dying woman here in the hospital, i was diagnose as a cancer patient 2
years ago. I am from (Zarqa) Jordan, a business woman dealing with Gold
Exportation. I have a charitable and unfufilment project that am about to
handover to you,
hope to hear from you
Regard
Mrs Mimi Ami
Add power domain index 0 related with mipi-phy to imx7s.
While at it rename pcie power-domain node to remove pgc prefix.
Signed-off-by: Rui Miguel Silva
---
arch/arm/boot/dts/imx7s.dtsi | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/imx7s.dtsi b/arc
Adds MIPI CSI-2 subdev for i.MX7 to connect with sensors with a MIPI
CSI-2 interface.
Signed-off-by: Rui Miguel Silva
---
drivers/staging/media/imx/Makefile |1 +
drivers/staging/media/imx/imx7-mipi-csis.c | 1186
2 files changed, 1187 insertions(+)
create mode
Add rst document to describe the i.MX7 media driver and also a working
example from the Warp7 board usage with a OV2680 sensor.
Signed-off-by: Rui Miguel Silva
Acked-by: Sakari Ailus
---
Documentation/media/v4l-drivers/imx7.rst | 157 ++
Documentation/media/v4l-drivers/inde
Warp7 comes with a Omnivision OV2680 sensor, add the node here to make
complete the camera data path for this system. Add the needed regulator
to the analog voltage supply, the port and endpoints in mipi_csi node
and the pinctrl for the reset gpio.
Signed-off-by: Rui Miguel Silva
---
arch/arm/bo
This patch adds the device tree nodes for csi, video multiplexer and
mipi-csi besides the graph connecting the necessary endpoints to make
the media capture entities to work in imx7 Warp board.
Signed-off-by: Rui Miguel Silva
---
arch/arm/boot/dts/imx7s-warp.dts | 51
Add non vendor bayer formats to the allowed format array.
Signed-off-by: Rui Miguel Silva
Reviewed-by: Philipp Zabel
Acked-by: Sakari Ailus
---
drivers/media/platform/video-mux.c | 20
1 file changed, 20 insertions(+)
diff --git a/drivers/media/platform/video-mux.c
b/dr
Add some i.MX7 related entries to TODO file.
Signed-off-by: Rui Miguel Silva
Acked-by: Sakari Ailus
---
drivers/staging/media/imx/TODO | 9 +
1 file changed, 9 insertions(+)
diff --git a/drivers/staging/media/imx/TODO b/drivers/staging/media/imx/TODO
index aeeb15494a49..6f29b5ca5324 10
Add maintainer entry for the imx7 media csi, mipi csis driver,
dt-bindings and documentation.
Signed-off-by: Rui Miguel Silva
Acked-by: Sakari Ailus
---
MAINTAINERS | 11 +++
1 file changed, 11 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 51029a425dbe..ad267b3dd18b 100644
This add the media entity subdevice and control driver for the i.MX7
CMOS Sensor Interface.
Signed-off-by: Rui Miguel Silva
---
drivers/staging/media/imx/Kconfig |9 +-
drivers/staging/media/imx/Makefile |2 +
drivers/staging/media/imx/imx7-media-csi.c | 1365 +++
Some imx system do not have IPU, so prepare the imx media drivers to
support this kind of devices. Rename the group ids to include an _IPU_
prefix, add a new group id to support systems with only a CSI without
IPU, and also rename the create internal links to make it clear that
only systems with IP
Refactor and move media device initialization code to a new common
module, so it can be used by other devices, this will allow for example
a near to introduce imx7 CSI driver, to use this media device.
Signed-off-by: Rui Miguel Silva
---
drivers/staging/media/imx/Makefile| 1 +
.../
Add bindings documentation for i.MX7 media drivers.
The imx7 MIPI CSI2 and imx7 CMOS Sensor Interface.
Signed-off-by: Rui Miguel Silva
Reviewed-by: Rob Herring
Acked-by: Sakari Ailus
---
.../devicetree/bindings/media/imx7-csi.txt| 45 ++
.../bindings/media/imx7-mipi-csi2.txt
Hi,
This series introduces the Media driver to work with the i.MX7 SoC. it uses the
already existing imx media core drivers but since the i.MX7, contrary to
i.MX5/6, do not have an IPU and because of that some changes in the imx media
core are made along this series to make it support that case.
T
The IOMUXC General Purpose Register has bitfield to control video bus
multiplexer to control the CSI input between the MIPI-CSI2 and parallel
interface. Add that register and mask.
Signed-off-by: Rui Miguel Silva
Reviewed-by: Philipp Zabel
---
arch/arm/boot/dts/imx7s.dtsi | 9 -
1 file
On Mon, Feb 04, 2019 at 12:47:23PM +0300, Dan Carpenter wrote:
> On Mon, Feb 04, 2019 at 09:42:36AM +, ajay.kat...@microchip.com wrote:
> >
> >
> > On 2/4/2019 1:43 PM, Dan Carpenter wrote:
> > > On Sat, Feb 02, 2019 at 07:17:13PM +, ajay.kat...@microchip.com wrote:
> > >> From: Ajay Sing
On Mon, Feb 04, 2019 at 09:42:36AM +, ajay.kat...@microchip.com wrote:
>
>
> On 2/4/2019 1:43 PM, Dan Carpenter wrote:
> > On Sat, Feb 02, 2019 at 07:17:13PM +, ajay.kat...@microchip.com wrote:
> >> From: Ajay Singh
> >>
> >> The value of 'vif_num'(interface count) starts with 0, so modi
On 2/4/2019 1:43 PM, Dan Carpenter wrote:
> On Sat, Feb 02, 2019 at 07:17:13PM +, ajay.kat...@microchip.com wrote:
>> From: Ajay Singh
>>
>> The value of 'vif_num'(interface count) starts with 0, so modified the
>> loop conditions to execute for all interface.
>>
>> Signed-off-by: Ajay Sing
On Sat, Feb 02, 2019 at 10:12:41PM +, Colin Ian King wrote:
> ping?
I seem to have applied this to a wrong branch, it'll now be part of my next
pull request to Mauro.
Thanks!
>
> On 22/12/2018 11:49, Colin King wrote:
> > From: Colin Ian King
> >
> > The comparison css->pipes[pipe].bindex
On Mon, Feb 04, 2019 at 09:34:56AM +1100, NeilBrown wrote:
> It is extremely common in the kernel for a file to start
>// SPDX-License-Identifier.
> and to have that immediately followed by a comment lile:
> /*
> * .
> *
Yes, there was a lot of automated conversion AFAIC
On Sat, Feb 02, 2019 at 07:17:13PM +, ajay.kat...@microchip.com wrote:
> From: Ajay Singh
>
> The value of 'vif_num'(interface count) starts with 0, so modified the
> loop conditions to execute for all interface.
>
> Signed-off-by: Ajay Singh
The right thing to do is to change ->vif_num =
69 matches
Mail list logo